Re: [tip:smp/hotplug] cpu/hotplug: Prevent alloc/free of irq descriptors during CPU up/down (again)

[Boris Ostrovsky]

On 08/11/2016 10:44 AM, Sebastian Andrzej Siewior wrote:
> On 2016-08-11 10:30:57 [-0400], Boris Ostrovsky wrote:
>> So you must have converted Xen code already then? Can you post those
>> patches (not all 1285, just Xen-related, assuming there are no
>> dependencies) and I'll see if I can add them to this series?
> 
> This is patch 77. I *hope* the only clash (if any) will be in
> cpuhotplug.h. I did not yet retest things in terms of apply & compile.
> I plan to do so within this + next week. If you want to test it right
> away, please do so :)
> 
> From: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> Date: Wed, 4 Nov 2015 21:06:00 +0100
> Subject: [PATCH] xen: events: fifo: Convert to hotplug state machine
> 
> Install the callbacks via the state machine. In theory we could remove the
> upper call evtchn_fifo_alloc_control_block() and have callback invoked for
> all online CPUs but this does not reflect the currect code.
> 
> Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> Cc: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> Cc: David Vrabel <david.vrabel@xxxxxxxxxx>
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>
> ---
>  drivers/xen/events/events_fifo.c |   33 +++++++++++----------------------
>  include/linux/cpuhotplug.h       |    1 +
>  2 files changed, 12 insertions(+), 22 deletions(-)


I was actually looking for arch/x86/xen/enlighten.c patch but if you
don't have it I can do it myself.


> --- a/include/linux/cpuhotplug.h
> +++ b/include/linux/cpuhotplug.h
> @@ -22,6 +22,7 @@ enum cpuhp_state {
>  	CPUHP_SMPCFD_PREPARE,
>  	CPUHP_RELAY_PREPARE,
>  	CPUHP_SLAB_PREPARE,
> +	CPUHP_XEN_EV_PREPEARE,

Misspelled PREPARE.

-boris