On Wed, 2022-03-09 at 14:00 +0100, Paolo Bonzini wrote: > On 3/1/22 15:36, Maxim Levitsky wrote: > > +void svm_copy_lbrs(struct vmcb *from_vmcb, struct vmcb *to_vmcb) > > +{ > > + to_vmcb->save.dbgctl = from_vmcb->save.dbgctl; > > + to_vmcb->save.br_from = from_vmcb->save.br_from; > > + to_vmcb->save.br_to = from_vmcb->save.br_to; > > + to_vmcb->save.last_excp_from = from_vmcb->save.last_excp_from; > > + to_vmcb->save.last_excp_to = from_vmcb->save.last_excp_to; > > + > > + vmcb_mark_dirty(to_vmcb, VMCB_LBR); > > +} > > + > > I think "struct vmcb *to_vmcb, struct vmcb *from_vmcb" is more common > (e.g. svm_copy_vmrun_state, svm_copy_vmloadsave_state). > > Paolo > Will do. Best regards, Maxim Levitsky