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

[George Dunlap <George.Dunlap@xxxxxxxxxxxxx>]

On Wed, Mar 27, 2013 at 1:13 PM, 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>

I'm not super-familiar with the save/restore paths; but it looks like
a reasonable change to me:

Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>

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