[PATCH 08/19] KVM: SVM: Remove redundant cluster calculation that also creates a shadow

From: Sean Christopherson
Date: Tue Aug 30 2022 - 20:37:49 EST


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