KVM: nSVM: clarify recalc_intercepts() wrt CR8
authorPaolo Bonzini <pbonzini@redhat.com>
Wed, 9 Nov 2022 11:03:03 +0000 (06:03 -0500)
committerPaolo Bonzini <pbonzini@redhat.com>
Mon, 9 Jan 2023 10:35:21 +0000 (05:35 -0500)
commit74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c
tree17f25a78256c226319e83d95406e6db7debb9d83
parentb7bfaa761d760e72a969d116517eaa12e404c262
KVM: nSVM: clarify recalc_intercepts() wrt CR8

The mysterious comment "We only want the cr8 intercept bits of L1"
dates back to basically the introduction of nested SVM, back when
the handling of "less typical" hypervisors was very haphazard.
With the development of kvm-unit-tests for interrupt handling,
the same code grew another vmcb_clr_intercept for the interrupt
window (VINTR) vmexit, this time with a comment that is at least
decent.

It turns out however that the same comment applies to the CR8 write
intercept, which is also a "recheck if an interrupt should be
injected" intercept.  The CR8 read intercept instead has not
been used by KVM for 14 years (commit 649d68643ebf, "KVM: SVM:
sync TPR value to V_TPR field in the VMCB"), so do not bother
clearing it and let one comment describe both CR8 write and VINTR
handling.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/svm/nested.c