diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2022-11-09 12:03:03 +0100 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2023-01-09 11:35:21 +0100 |
commit | 74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c (patch) | |
tree | 17f25a78256c226319e83d95406e6db7debb9d83 /Documentation/virt | |
parent | Linux 6.2-rc3 (diff) | |
download | linux-74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c.tar.xz linux-74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c.zip |
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>
Diffstat (limited to 'Documentation/virt')
0 files changed, 0 insertions, 0 deletions