[PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-03-11 Thread Thiago Jung Bauermann
When testing DLPAR CPU add/remove on a system under stress, pseries_cpu_die() doesn't wait long enough for a CPU to die: [ 446.983944] cpu 148 (hwid 148) Ready to die... [ 446.984062] cpu 149 (hwid 149) Ready to die... [ 446.993518] cpu 150 (hwid 150) Ready to die... [ 446.993543] Querying DEA

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-03-12 Thread Gautham R Shenoy
Hello Thiago, On Mon, Mar 11, 2019 at 04:35:17PM -0300, Thiago Jung Bauermann wrote: > When testing DLPAR CPU add/remove on a system under stress, > pseries_cpu_die() doesn't wait long enough for a CPU to die: > > [ 446.983944] cpu 148 (hwid 148) Ready to die... > [ 446.984062] cpu 149 (hwid 14

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-03-12 Thread Thiago Jung Bauermann
Gautham R Shenoy writes: >> Signed-off-by: Thiago Jung Bauermann > > Thanks for this version. I have tested the patch and we no longer see > the "Querying DEAD? cpu X (Y) shows 2" message. > > > Tested-and-Reviewed-by: Gautham R. Shenoy Thanks for reviewing and testing the patch! -- Thiago

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-04-10 Thread Thiago Jung Bauermann
Hello, Ping? -- Thiago Jung Bauermann IBM Linux Technology Center Thiago Jung Bauermann writes: > When testing DLPAR CPU add/remove on a system under stress, > pseries_cpu_die() doesn't wait long enough for a CPU to die: > > [ 446.983944] cpu 148 (hwid 148) Ready to die... > [ 446.984062

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-04-15 Thread Nicholas Piggin
Thiago Jung Bauermann's on April 11, 2019 9:08 am: > > Hello, > > Ping? > > -- > Thiago Jung Bauermann > IBM Linux Technology Center > > > Thiago Jung Bauermann writes: > >> When testing DLPAR CPU add/remove on a system under stress, >> pseries_cpu_die() doesn't wait long enough for a CPU t

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-04-17 Thread Thiago Jung Bauermann
Hello Nick, Thank you very much for reviewing this patch! Nicholas Piggin writes: > Thiago Jung Bauermann's on April 11, 2019 9:08 am: >> >> Thiago Jung Bauermann writes: >> >>> diff --git a/arch/powerpc/platforms/pseries/hotplug-cpu.c >>> b/arch/powerpc/platforms/pseries/hotplug-cpu.c >>>

Re: [PATCH v3] powerpc/pseries: Only wait for dying CPU after call to rtas_stop_self()

2019-04-17 Thread Nicholas Piggin
Thiago Jung Bauermann's on April 18, 2019 11:00 am: > > Hello Nick, > > Thank you very much for reviewing this patch! > > Nicholas Piggin writes: > >> Thiago Jung Bauermann's on April 11, 2019 9:08 am: >>> >>> Thiago Jung Bauermann writes: >>> diff --git a/arch/powerpc/platforms/pseries/