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

Re: [Xen-devel] [PATCH] x86/S3: Restore broken vcpu affinity on resume (v4)



>>> On 27.03.13 at 14:13, Ben Guthro <benjamin.guthro@xxxxxxxxxx> wrote:
> When in SYS_STATE_suspend, and going through the cpu_disable_scheduler
> path, save a copy of the current cpu affinity, and mark a flag to
> restore it later.
> 
> Later, in the resume process, when enabling nonboot cpus restore these
> affinities.
> 
> v2:
> Formatting: Fix hard tabs.
> remove early return in cpu_disable_scheduler() path.
> 
> v3:
> Formatting: Fix remaining errant tab.
> Move restore_vcpu_affinity() to thaw_domains(), eliminating the need to
> promote for_each_cpupool()
> 
> v4:
> Formatting: Fix if statement spacing.
> Eliminate unnecessary if statement in thaw_domains()
> Suppress affinity related logging to XENLOG_DEBUG to reduce noise.
> 
> Signed-off-by: Ben Guthro <benjamin.guthro@xxxxxxxxxx>

Looks good to me now, but needs an ack by Keir (or George,
considering the minor change to xen/common/domain.c is perhaps
acceptable without formal ack).

Thanks, Jan


_______________________________________________
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®.