KVM: VMX: Remove scratch 'cpu' variable that shadows an identical scratch var
authorPeng Hao <flyingpenghao@gmail.com>
Tue, 22 Feb 2022 10:39:54 +0000 (18:39 +0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 25 Feb 2022 13:20:13 +0000 (08:20 -0500)
 From: Peng Hao <flyingpeng@tencent.com>

 Remove a redundant 'cpu' declaration from inside an if-statement that
 that shadows an identical declaration at function scope.  Both variables
 are used as scratch variables in for_each_*_cpu() loops, thus there's no
 harm in sharing a variable.

Reviewed-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Peng Hao <flyingpeng@tencent.com>
Message-Id: <20220222103954.70062-1-flyingpeng@tencent.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/vmx/vmx.c

index 7ceda4646f131ed023e1ab80faa10c80e4417c6f..5285731fdab949529896bf187cc9835ed631c9d8 100644 (file)
@@ -8129,7 +8129,6 @@ static int __init vmx_init(void)
            ms_hyperv.hints & HV_X64_ENLIGHTENED_VMCS_RECOMMENDED &&
            (ms_hyperv.nested_features & HV_X64_ENLIGHTENED_VMCS_VERSION) >=
            KVM_EVMCS_VERSION) {
-               int cpu;
 
                /* Check that we have assist pages on all online CPUs */
                for_each_online_cpu(cpu) {