Re: [Xen-devel] [PATCH RFC v13 07/20] pvh: Do not allow PVH guests to change paging modes

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 23.09.13 at 18:49, George Dunlap <george.dunlap@xxxxxxxxxxxxx> wrote:
> --- a/xen/arch/x86/hvm/hvm.c
> +++ b/xen/arch/x86/hvm/hvm.c
> @@ -1765,6 +1765,16 @@ int hvm_set_cr0(unsigned long value)
>           (value & (X86_CR0_PE | X86_CR0_PG)) == X86_CR0_PG )
>          goto gpf;
>  
> +    
> +

Please drop these bogus blank lines ...

> +    /* A pvh is not expected to change to real mode. */
> +    if ( is_pvh_vcpu(v)
> +         && (value & (X86_CR0_PE | X86_CR0_PG)) != (X86_CR0_PG | X86_CR0_PE) 
> )
> +    {
> +        printk(XENLOG_G_WARNING
> +               "PVH attempting to turn off PE/PG. CR0:%lx\n", value);
> +        goto gpf;
> +    }

... and perhaps insert one here instead.

>      if ( (value & X86_CR0_PG) && !(old_value & X86_CR0_PG) )
>      {
>          if ( v->arch.hvm_vcpu.guest_efer & EFER_LME )

Jan


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