Re: [Xen-devel] [PATCH v14 06/17] pvh: Disable unneeded features of HVM containers

[Tim Deegan <tim@xxxxxxx>]

At 09:00 +0000 on 07 Nov (1383811221), Jan Beulich wrote:
> >>> On 07.11.13 at 00:54, Tim Deegan <tim@xxxxxxx> wrote:
> >> @@ -528,10 +536,16 @@ int hvm_domain_initialise(struct domain *d)
> >>      if ( rc != 0 )
> >>          goto fail0;
> >>  
> >> +    rc = -ENOMEM;
> >>      d->arch.hvm_domain.params = xzalloc_array(uint64_t, HVM_NR_PARAMS);
> >> +    if ( !d->arch.hvm_domain.params )
> >> +        goto fail1;
> >> +
> >> +    if ( is_pvh_domain(d) )
> >> +        return 0;
> > 
> > Doesn't this skip hvm_init_cacheattr_region_list() and
> > paging_enable(), which are on your list of things to keep for PVH guests?
> 
> No, patch 02 moved this up before the patch context seen
> here.

Ah, grand so.  Acked-by: Tim Deegan <tim@xxxxxxx>

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