summaryrefslogtreecommitdiffstats
path: root/Documentation
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2022-11-09 06:03:03 -0500
committerPaolo Bonzini <pbonzini@redhat.com>2023-01-09 05:35:21 -0500
commit74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c (patch)
tree17f25a78256c226319e83d95406e6db7debb9d83 /Documentation
parentb7bfaa761d760e72a969d116517eaa12e404c262 (diff)
downloadlinux-74905e3de8adf0e6b5d7f455dcd32cdec13dfb6c.tar.bz2
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')
0 files changed, 0 insertions, 0 deletions