Skip to content

Commit f00ccde

Browse files
yosrym93sean-jc
authored andcommitted
KVM: nSVM: Drop redundant/wrong comment in nested_vmcb02_prepare_save()
The comment above DR6 and DR7 initializations is redundant, because the entire function follows the same pattern of only initializing the fields in vmcb02 if the vmcb12 changed or the fields are dirty, which handles the first execution case. Also, the comment refers to new_vmcb12 as new_vmcs12. Just drop the comment. No functional change intended. Signed-off-by: Yosry Ahmed <yosry.ahmed@linux.dev> Link: https://patch.msgid.link/20260113172807.2178526-1-yosry.ahmed@linux.dev Signed-off-by: Sean Christopherson <seanjc@google.com>
1 parent 6955513 commit f00ccde

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

arch/x86/kvm/svm/nested.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -681,7 +681,6 @@ static void nested_vmcb02_prepare_save(struct vcpu_svm *svm, struct vmcb *vmcb12
681681
vmcb02->save.rsp = vmcb12->save.rsp;
682682
vmcb02->save.rip = vmcb12->save.rip;
683683

684-
/* These bits will be set properly on the first execution when new_vmc12 is true */
685684
if (unlikely(new_vmcb12 || vmcb_is_dirty(vmcb12, VMCB_DR))) {
686685
vmcb02->save.dr7 = svm->nested.save.dr7 | DR7_FIXED_1;
687686
svm->vcpu.arch.dr6 = svm->nested.save.dr6 | DR6_ACTIVE_LOW;

0 commit comments

Comments
 (0)