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

Re: [Xen-devel] [PATCH v14 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state



>>> On 01.08.18 at 16:50, <aisaila@xxxxxxxxxxxxxxx> wrote:
> On Ma, 2018-07-31 at 07:08 -0600, Jan Beulich wrote:
>> > 
>> > > 
>> > > > 
>> > > > On 25.07.18 at 14:14, <aisaila@xxxxxxxxxxxxxxx> wrote:
>> > This patch is focused on moving the for loop to the caller so
>> > now we can save info for a single vcpu instance with the save_one
>> > handlers.
>> > 
>> > Signed-off-by: Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>
>> First of all I'd appreciate if this patch was last in the series,
>> after all
>> infrastructure changes have been done.
>> 
> If this patch will be the last one in the series then between the one
> that removes the save functions (and with that the for) and this one
> the code will not run as expected because there will be no for. 

Well, I didn't mean you to re-order things without adjusting the code
accordingly. You will want to transform hvm_save() first, and only as
the final step make hvm_save_one() behave the way you're actually
after. This may (as already indicated in some of the individual
responses) incur changing the splitting between patches.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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