Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-27 Thread Juergen Gross
On 27/03/2019 17:29, Jan Beulich wrote: On 18.03.19 at 14:11, wrote: >> --- a/xen/common/cpu.c >> +++ b/xen/common/cpu.c >> @@ -214,7 +214,12 @@ void enable_nonboot_cpus(void) >> printk("Error bringing CPU%d up: %d\n", cpu, error); >> BUG_ON(error == -EBUSY); >>

Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-27 Thread Jan Beulich
>>> On 18.03.19 at 14:11, wrote: > --- a/xen/common/cpu.c > +++ b/xen/common/cpu.c > @@ -214,7 +214,12 @@ void enable_nonboot_cpus(void) > printk("Error bringing CPU%d up: %d\n", cpu, error); > BUG_ON(error == -EBUSY); > } > +else > +__cpumask

Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-27 Thread Dario Faggioli
On Mon, 2019-03-25 at 13:29 +0100, Juergen Gross wrote: > On 25/03/2019 13:21, Dario Faggioli wrote: > > > Reviewed-by: Dario Faggioli > > > > One more (minor) thing... > > > > > /* CPU_REMOVE: CPU was removed. */ > > > -#define CPU_REMOVE (0x0009 | NOTIFY_REVERSE) > > > +#define CPU_REMO

Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-27 Thread George Dunlap
On 3/18/19 1:11 PM, Juergen Gross wrote: > Add a new cpu notifier action CPU_RESUME_FAILED which is called for all > cpus which failed to come up at resume. The calls will be done after > all other cpus are already up. > > Signed-off-by: Juergen Gross Reviewed-by: George Dunlap ___

Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-25 Thread Juergen Gross
On 25/03/2019 13:21, Dario Faggioli wrote: > On Mon, 2019-03-18 at 14:11 +0100, Juergen Gross wrote: >> --- a/xen/include/xen/cpu.h >> +++ b/xen/include/xen/cpu.h >> @@ -32,23 +32,25 @@ void register_cpu_notifier(struct notifier_block >> *nb); >> * (a) A CPU is going down; or (b) CPU_UP_CANCELED

Re: [Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-25 Thread Dario Faggioli
On Mon, 2019-03-18 at 14:11 +0100, Juergen Gross wrote: > --- a/xen/include/xen/cpu.h > +++ b/xen/include/xen/cpu.h > @@ -32,23 +32,25 @@ void register_cpu_notifier(struct notifier_block > *nb); > * (a) A CPU is going down; or (b) CPU_UP_CANCELED > */ > /* CPU_UP_PREPARE: Preparing to bring C

[Xen-devel] [PATCH 3/6] xen: add new cpu notifier action CPU_RESUME_FAILED

2019-03-18 Thread Juergen Gross
Add a new cpu notifier action CPU_RESUME_FAILED which is called for all cpus which failed to come up at resume. The calls will be done after all other cpus are already up. Signed-off-by: Juergen Gross --- xen/common/cpu.c | 5 + xen/include/xen/cpu.h | 20 +++- 2 files