[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC] pvh: clearly specify used parameters in vcpu_guest_context



At 16:32 +0000 on 15 Nov (1384529574), Jan Beulich wrote:
> >>> On 15.11.13 at 16:50, Roger Pau Monne <roger.pau@xxxxxxxxxx> wrote:
> > --- a/xen/arch/x86/domain.c
> > +++ b/xen/arch/x86/domain.c
> > @@ -704,9 +704,11 @@ int arch_set_info_guest(
> >          /* PVH 32bitfixme */
> >          ASSERT(!compat);
> >  
> > -        if ( c(ctrlreg[1]) || c(ldt_base) || c(ldt_ents) ||
> > +        if ( c(ctrlreg[0]) || c(ctrlreg[1]) || c(ctrlreg[2]) ||
> > +             c(ctrlreg[4]) || c(ldt_base) || c(ldt_ents) ||
> 
> I think it should actually be a bug for the guest to request an
> all blank CR0 or CR4. Minimally CR0.PE, CR0.PG, and CR4.PAE
> would seem to be a valid requirement to be set.

I think zero is better.  Guest CRx are explicitly _not_ loaded from 
these fields so making them look like valid CRx values is just confusing.

But even better would be to allow all these fields to be set to any
valid values, and to load the state into the guest vCPU.  I think
that's going to be more sensible once the current restriction that PVH
vCPUs are always in long more goes away.

Tim.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.