Commit d603fd8d authored May 09, 2022 by Yury Norov

KVM: x86: hyper-v: replace bitmap_weight() with hweight64()



kvm_hv_flush_tlb() applies bitmap API to a u64 variable valid_bank_mask.
Since valid_bank_mask has a fixed size, we can use hweight64() and avoid
excessive bloating.

CC: Borislav Petkov <bp@alien8.de>
CC: Dave Hansen <dave.hansen@linux.intel.com>
CC: H. Peter Anvin <hpa@zytor.com>
CC: Ingo Molnar <mingo@redhat.com>
CC: Jim Mattson <jmattson@google.com>
CC: Joerg Roedel <joro@8bytes.org>
CC: Paolo Bonzini <pbonzini@redhat.com>
CC: Sean Christopherson <seanjc@google.com>
CC: Thomas Gleixner <tglx@linutronix.de>
CC: Vitaly Kuznetsov <vkuznets@redhat.com>
CC: Wanpeng Li <wanpengli@tencent.com>
CC: kvm@vger.kernel.org
CC: linux-kernel@vger.kernel.org
CC: x86@kernel.org
Acked-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Signed-off-by: Yury Norov <yury.norov@gmail.com>

parent a7ef9b45

arch/x86/kvm/hyperv.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -1879,7 +1879,7 @@ static u64 kvm_hv_flush_tlb(struct kvm_vcpu vcpu, struct kvm_hv_hcall hc)
		all_cpus = flush_ex.hv_vp_set.format !=
		HV_GENERIC_SET_SPARSE_4K;

		if (hc->var_cnt != bitmap_weight((unsigned long *)&valid_bank_mask, 64))
		if (hc->var_cnt != hweight64(valid_bank_mask))
		return HV_STATUS_INVALID_HYPERCALL_INPUT;

		if (all_cpus)
		@@ -1980,7 +1980,7 @@ static u64 kvm_hv_send_ipi(struct kvm_vcpu vcpu, struct kvm_hv_hcall hc)
		valid_bank_mask = send_ipi_ex.vp_set.valid_bank_mask;
		all_cpus = send_ipi_ex.vp_set.format == HV_GENERIC_SET_ALL;

		if (hc->var_cnt != bitmap_weight((unsigned long *)&valid_bank_mask, 64))
		if (hc->var_cnt != hweight64(valid_bank_mask))
		return HV_STATUS_INVALID_HYPERCALL_INPUT;

		if (all_cpus)