On 3/9/24 13:34, Ani Sinha wrote:
kvm_arch_init() enables a lot of vm capabilities. Refactor them into separate
smaller functions. Energy MSR related operations also moved to its own
function. There should be no functional impact.

Signed-off-by: Ani Sinha <anisi...@redhat.com>
---
  target/i386/kvm/kvm.c | 337 ++++++++++++++++++++++++++----------------
  1 file changed, 211 insertions(+), 126 deletions(-)

Nice cleanup, however it would be easier to review as a series
of dumb "extract FOO function" patches instead of a single one.
Up to the maintainer.

Reply via email to