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 <anisinha@xxxxxxxxxx> --- 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.