Drop the redundant "cluster" calculation and its horrific shadowing in the x2avic logical mode path. The "cluster" that is declared at an outer scope is derived using the exact same calculation and has performed the left-shift. No functional change intended. Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> --- arch/x86/kvm/svm/avic.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/x86/kvm/svm/avic.c b/arch/x86/kvm/svm/avic.c index 14f567550a1e..8c9cad96008e 100644 --- a/arch/x86/kvm/svm/avic.c +++ b/arch/x86/kvm/svm/avic.c @@ -410,10 +410,9 @@ static int avic_kick_target_vcpus_fast(struct kvm *kvm, struct kvm_lapic *source * For x2APIC logical mode, cannot leverage the index. * Instead, calculate physical ID from logical ID in ICRH. */ - int cluster = (icrh & 0xffff0000) >> 16; int apic = ffs(bitmap) - 1; - l1_physical_id = (cluster << 4) + apic; + l1_physical_id = cluster + apic; } } -- 2.37.2.672.g94769d06f0-goog
