On Mon, Oct 18, 2021 at 09:16:30AM -0400, Tamas K Lengyel wrote: > On Mon, Oct 18, 2021 at 4:26 AM Jan Beulich <[email protected]> wrote: > > > On 25.08.2021 11:31, Jan Beulich wrote: > > > We should not rely on guests to not use altp2m after reporting failure > > > of HVMOP_altp2m_set_domain_state to them. Set "active" back to false in > > > this case. > > > > > > Signed-off-by: Jan Beulich <[email protected]> > > > > Because of their relationship, I'm also Cc-ing the vm-event maintainers / > > reviewers: Maybe one of you would care to review this one? > > > > Jan > > > > > --- a/xen/arch/x86/hvm/hvm.c > > > +++ b/xen/arch/x86/hvm/hvm.c > > > @@ -4609,6 +4609,8 @@ static int do_altp2m_op( > > > if ( ostate ) > > > p2m_flush_altp2m(d); > > > } > > > + else if ( rc ) > > > + d->arch.altp2m_active = false; > > > > > > domain_unpause_except_self(d); > > > break; > > > Reviewed-by: Tamas K Lengyel <[email protected]>
Reviewed-by: Roger Pau Monné <[email protected]> Thanks, Roger.
