Re: [PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-09 Thread Michael Ellerman
Nicholas Piggin writes: > On Fri, 7 Jul 2017 20:34:16 +0530 > Gautham R Shenoy wrote: >> On Fri, Jul 07, 2017 at 01:16:09AM +1000, Nicholas Piggin wrote: >> > >> > Speaking of which... core_idle_state and thread_sibling_pacas are >> > allocated with kmalloc_node... What happens if we take an SLB

Re: [PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-08 Thread Nicholas Piggin
On Fri, 7 Jul 2017 20:34:16 +0530 Gautham R Shenoy wrote: > On Fri, Jul 07, 2017 at 01:16:09AM +1000, Nicholas Piggin wrote: > > On Wed, 5 Jul 2017 22:08:15 +0530 > > "Gautham R. Shenoy" wrote: > > > > > From: "Gautham R. Shenoy" > > > > > > On POWER9 DD1, in order to get around a hardware

Re: [PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-07 Thread Gautham R Shenoy
On Fri, Jul 07, 2017 at 01:16:09AM +1000, Nicholas Piggin wrote: > On Wed, 5 Jul 2017 22:08:15 +0530 > "Gautham R. Shenoy" wrote: > > > From: "Gautham R. Shenoy" > > > > On POWER9 DD1, in order to get around a hardware issue, we store in > > every CPU thread's paca the paca pointers of all its

Re: [PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:15 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > On POWER9 DD1, in order to get around a hardware issue, we store in > every CPU thread's paca the paca pointers of all its siblings. > > Move this code into pnv_alloc_idle_core_states() soon after the

[PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-05 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" On POWER9 DD1, in order to get around a hardware issue, we store in every CPU thread's paca the paca pointers of all its siblings. Move this code into pnv_alloc_idle_core_states() soon after the space for saving the sibling pacas is allocated. Signed-off-by: Gautham R.