Skip to content

Commit 8eada24

Browse files
kaihuangsean-jc
authored andcommitted
KVM: x86: Fix a comment inside __kvm_set_or_clear_apicv_inhibit()
Change svm_vcpu_run() to vcpu_enter_guest() in the comment of __kvm_set_or_clear_apicv_inhibit() to make it reflect the fact. When one thread updates VM's APICv state due to updating the APICv inhibit reasons, it kicks off all vCPUs and makes them wait until the new reason has been updated and can be seen by all vCPUs. There was one WARN() to make sure VM's APICv state is consistent with vCPU's APICv state in the svm_vcpu_run(). Commit ee49a89 ("KVM: x86: Move SVM's APICv sanity check to common x86") moved that WARN() to x86 common code vcpu_enter_guest() due to the logic is not unique to SVM, and added comments to both __kvm_set_or_clear_apicv_inhibit() and vcpu_enter_guest() to explain this. However, although the comment in __kvm_set_or_clear_apicv_inhibit() mentioned the WARN(), it seems forgot to reflect that the WARN() had been moved to x86 common, i.e., it still mentioned the svm_vcpu_run() but not vcpu_enter_guest(). Fix it. Note after the change the first line that contains vcpu_enter_guest() exceeds 80 characters, but leave it as is to make the diff clean. Fixes: ee49a89 ("KVM: x86: Move SVM's APICv sanity check to common x86") Signed-off-by: Kai Huang <kai.huang@intel.com> Link: https://lore.kernel.org/r/e462e7001b8668649347f879c66597d3327dbac2.1728383775.git.kai.huang@intel.com Signed-off-by: Sean Christopherson <seanjc@google.com>
1 parent e9e1cb4 commit 8eada24

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

arch/x86/kvm/x86.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10606,11 +10606,11 @@ void __kvm_set_or_clear_apicv_inhibit(struct kvm *kvm,
1060610606
if (!!old != !!new) {
1060710607
/*
1060810608
* Kick all vCPUs before setting apicv_inhibit_reasons to avoid
10609-
* false positives in the sanity check WARN in svm_vcpu_run().
10609+
* false positives in the sanity check WARN in vcpu_enter_guest().
1061010610
* This task will wait for all vCPUs to ack the kick IRQ before
1061110611
* updating apicv_inhibit_reasons, and all other vCPUs will
1061210612
* block on acquiring apicv_update_lock so that vCPUs can't
10613-
* redo svm_vcpu_run() without seeing the new inhibit state.
10613+
* redo vcpu_enter_guest() without seeing the new inhibit state.
1061410614
*
1061510615
* Note, holding apicv_update_lock and taking it in the read
1061610616
* side (handling the request) also prevents other vCPUs from

0 commit comments

Comments
 (0)