Re: [Xen-devel] [PATCH v5 8/9] xen/x86: Setup PVHv2 Dom0 CPUs

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 08.02.17 at 13:48, <roger.pau@xxxxxxxxxx> wrote:
> On Thu, Jan 26, 2017 at 06:46:17AM -0700, Jan Beulich wrote:
>> >>> On 19.01.17 at 18:29, <roger.pau@xxxxxxxxxx> wrote:
>> > @@ -2124,6 +2175,15 @@ static int __init construct_dom0_pvh(struct domain 
> *d, const module_t *image,
>> >          return rc;
>> >      }
>> >  
>> > +    rc = pvh_setup_cpus(d, entry, start_info);
>> > +    if ( rc )
>> > +    {
>> > +        printk("Failed to setup Dom0 CPUs: %d\n", rc);
>> > +        return rc;
>> > +    }
>> > +
>> > +    clear_bit(_VPF_down, &d->vcpu[0]->pause_flags);
>> 
>> Would you mind moving this into the function (where you then can
>> use just v)?
> 
> Fixed all the above, but I'm not sure about this. In v2 you requested me to
> move the clear_bit outside of pvh_set_cpus:
> 
> https://www.mail-archive.com/xen-devel@xxxxxxxxxxxxx/msg83714.html 

I asked you to move it later, but I did not ask for it to be moved
outside.

Jan


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