Re: [PATCH] sched/topology: Use Identity node only if required

2018-08-10 Thread Srikar Dronamraju
* Peter Zijlstra [2018-08-08 09:58:41]: > On Wed, Aug 08, 2018 at 12:39:31PM +0530, Srikar Dronamraju wrote: > > With Commit 051f3ca02e46 ("sched/topology: Introduce NUMA identity node > > sched domain") scheduler introduces an extra numa level. However that > > leads to > > > > - numa

[PATCH 1/2] sched/topology: Set correct numa topology type

2018-08-10 Thread Srikar Dronamraju
With commit 051f3ca02e46 ("sched/topology: Introduce NUMA identity node sched domain") scheduler introduces an new numa level. However this leads to numa topology on 2 node systems no more marked as NUMA_DIRECT. After this commit, it gets reported as NUMA_BACKPLANE. This is because

Re: [PATCH v3] powerpc/topology: Check at boot for topology updates

2018-08-10 Thread Michael Bringmann
On 08/10/2018 08:21 AM, Srikar Dronamraju wrote: > * Michael Ellerman [2018-08-10 21:42:28]: > >> Srikar Dronamraju writes: >>> diff --git a/arch/powerpc/include/asm/topology.h >>> b/arch/powerpc/include/asm/topology.h >>> index 16b077801a5f..70f2d2285ba7 100644 >>> ---

Re: [PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-10 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Fri, Aug 10, 2018 at 11:29:43AM +0200, Christophe LEROY wrote: > > > Le 03/08/2018 à 13:31, Murilo Opsfelder Araujo a écrit : > > Hi, everyone. > > > > I'd like to thank you all that contributed to refining and making this > > series better. I did appreciate. > > > > Thank

[PATCH 2/2] sched/topology: Expose numa_mask set/clear functions to arch

2018-08-10 Thread Srikar Dronamraju
With commit 051f3ca02e46 ("sched/topology: Introduce NUMA identity node sched domain") scheduler introduces an new numa level. However on shared lpars like powerpc, this extra sched domain creation can lead to repeated rcu stalls, sometimes even causing unresponsive systems on boot. On such

Re: [RFC PATCH 3/3] cpuidle/powernv: Conditionally save-restore sprs using opal

2018-08-10 Thread Nicholas Piggin
On Wed, 8 Aug 2018 21:11:16 +0530 Gautham R Shenoy wrote: > Hello Nicholas, > > On Fri, Aug 03, 2018 at 12:05:47AM +1000, Nicholas Piggin wrote: > > On Thu, 2 Aug 2018 10:21:32 +0530 > > Akshay Adiga wrote: > > > > > From: Abhishek Goel > > > > > > If a state has "opal-supported" compat

Re: [PATCH v7 7/9] powerpc/pseries: Dump the SLB contents on SLB MCE errors.

2018-08-10 Thread Nicholas Piggin
On Tue, 07 Aug 2018 19:47:39 +0530 Mahesh J Salgaonkar wrote: > From: Mahesh Salgaonkar > > If we get a machine check exceptions due to SLB errors then dump the > current SLB contents which will be very much helpful in debugging the > root cause of SLB errors. Introduce an exclusive buffer per

Pull request: scottwood/linux.git next

2018-08-10 Thread Scott Wood
This contains an 8xx compilation fix, and a dpaa device tree fix. The following changes since commit 77b5f703dcc859915f0f20d92bc538e4a99ef149: powerpc/powernv/opal: Use standard interrupts property when available (2018-08-08 00:32:38 +1000) are available in the Git repository at:

Re: [PATCH] powerpc/powernv: Add support for NPU2 relaxed-ordering mode

2018-08-10 Thread Alistair Popple
On Friday, 10 August 2018 5:07:56 PM AEST Michael Ellerman wrote: > Reza Arbab writes: > > From: Alistair Popple > > > > Some device drivers support out of order access to GPU memory. This does > > not affect the CPU view of memory but it does affect the GPU view, so it > > should only be

Re: [PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-10 Thread Christophe LEROY
Le 03/08/2018 à 13:31, Murilo Opsfelder Araujo a écrit : Hi, everyone. I'd like to thank you all that contributed to refining and making this series better. I did appreciate. Thank you! You are welcome. It seems that nested segfaults don't print very well. The code line is split in

[PATCH v2 1/2] powerpc/64s: move machine check SLB flushing to mm/slb.c

2018-08-10 Thread Nicholas Piggin
The machine check code that flushes and restores bolted segments in real mode belongs in mm/slb.c. This will also be used by pseries machine check and idle code in future changes. Signed-off-by: Nicholas Piggin Since v1: - Restore the test for slb_shadow (mpe) ---

Re: [PATCH] powerpc/mm/tlbflush: update the mmu_gather page size while iterating address range

2018-08-10 Thread Michael Ellerman
"Aneesh Kumar K.V" writes: > This patch makes sure we update the mmu_gather page size even if we are > requesting for a fullmm flush. This avoids triggering VM_WARN_ON in code > paths like __tlb_remove_page_size that explicitly check for removing range > page > size to be same as mmu gather

[PATCH v2 2/2] powerpc/64s: reimplement book3s idle code in C

2018-08-10 Thread Nicholas Piggin
Reimplement Book3S idle code in C, moving POWER7/8/9 implementation speific HV idle code to the powernv platform code. Book3S assembly stubs are kept in common code and used only to save the stack frame and non-volatile GPRs before executing architected idle instructions, and restoring the stack

[PATCH v3] powerpc/topology: Check at boot for topology updates

2018-08-10 Thread Srikar Dronamraju
On a shared lpar, Phyp will not update the cpu associativity at boot time. Just after the boot system does recognize itself as a shared lpar and trigger a request for correct cpu associativity. But by then the scheduler would have already created/destroyed its sched domains. This causes - Broken

Re: [PATCH] powerpc/powernv/idle: Fix build error

2018-08-10 Thread Michael Ellerman
"Aneesh Kumar K.V" writes: > Fix the below build error using strlcpy instead of strncpy > > In function 'pnv_parse_cpuidle_dt', > inlined from 'pnv_init_idle_states' at > arch/powerpc/platforms/powernv/idle.c:840:7, > inlined from '__machine_initcall_powernv_pnv_init_idle_states' at >

Re: [PATCH] powerpc/powernv: Add support for NPU2 relaxed-ordering mode

2018-08-10 Thread Michael Ellerman
Reza Arbab writes: > From: Alistair Popple > > Some device drivers support out of order access to GPU memory. This does > not affect the CPU view of memory but it does affect the GPU view, so it > should only be enabled once the GPU driver has requested it. Add APIs > allowing a driver to do so.

Re: [PATCH v2 2/2] powerpc/64s: reimplement book3s idle code in C

2018-08-10 Thread Nicholas Piggin
On Fri, 10 Aug 2018 16:42:49 +1000 Nicholas Piggin wrote: > Reimplement Book3S idle code in C, moving POWER7/8/9 implementation > speific HV idle code to the powernv platform code. > > Book3S assembly stubs are kept in common code and used only to save > the stack frame and non-volatile GPRs

Re: [PATCH v7 8/9] powerpc/mce: Add sysctl control for recovery action on MCE.

2018-08-10 Thread Nicholas Piggin
On Thu, 9 Aug 2018 12:26:46 +0200 Michal Suchánek wrote: > On Thu, 9 Aug 2018 18:33:33 +1000 > Nicholas Piggin wrote: > > > On Thu, 9 Aug 2018 13:39:45 +0530 > > Ananth N Mavinakayanahalli wrote: > > > > > On Thu, Aug 09, 2018 at 06:02:53PM +1000, Nicholas Piggin wrote: > > > > On Thu,

Re: [PATCH v3] powerpc/topology: Check at boot for topology updates

2018-08-10 Thread Srikar Dronamraju
* Michael Ellerman [2018-08-10 21:42:28]: > Srikar Dronamraju writes: > > diff --git a/arch/powerpc/include/asm/topology.h > > b/arch/powerpc/include/asm/topology.h > > index 16b077801a5f..70f2d2285ba7 100644 > > --- a/arch/powerpc/include/asm/topology.h > > +++

[PATCH] powerpc/uaccess: Enable get_user(u64, *p) on 32-bit

2018-08-10 Thread Michael Ellerman
Currently if you build a 32-bit powerpc kernel and use get_user() to load a u64 value it will fail to build with eg: kernel/rseq.o: In function `rseq_get_rseq_cs': kernel/rseq.c:123: undefined reference to `__get_user_bad' This is hitting the check in __get_user_size() that makes sure the

Re: [PATCH v7 4/9] powerpc/pseries: Define MCE error event section.

2018-08-10 Thread Mahesh Jagannath Salgaonkar
On 08/08/2018 08:12 PM, Michael Ellerman wrote: > Hi Mahesh, > > A few nitpicks. > > Mahesh J Salgaonkar writes: >> From: Mahesh Salgaonkar >> >> On pseries, the machine check error details are part of RTAS extended >> event log passed under Machine check exception section. This patch adds >>

Re: [PATCH v7 7/9] powerpc/pseries: Dump the SLB contents on SLB MCE errors.

2018-08-10 Thread Mahesh Jagannath Salgaonkar
On 08/09/2018 06:35 AM, Michael Ellerman wrote: > Mahesh J Salgaonkar writes: > >> diff --git a/arch/powerpc/include/asm/paca.h >> b/arch/powerpc/include/asm/paca.h >> index 7f22929ce915..233d25ff6f64 100644 >> --- a/arch/powerpc/include/asm/paca.h >> +++ b/arch/powerpc/include/asm/paca.h >> @@

Re: [PATCH v7 5/9] powerpc/pseries: flush SLB contents on SLB MCE errors.

2018-08-10 Thread Mahesh Jagannath Salgaonkar
On 08/08/2018 02:34 PM, Nicholas Piggin wrote: > On Tue, 07 Aug 2018 19:47:14 +0530 > Mahesh J Salgaonkar wrote: > >> From: Mahesh Salgaonkar >> >> On pseries, as of today system crashes if we get a machine check >> exceptions due to SLB errors. These are soft errors and can be fixed by >>

Re: [PATCH v7 7/9] powerpc/pseries: Dump the SLB contents on SLB MCE errors.

2018-08-10 Thread Mahesh Jagannath Salgaonkar
On 08/10/2018 04:02 PM, Mahesh Jagannath Salgaonkar wrote: > On 08/09/2018 06:35 AM, Michael Ellerman wrote: >> Mahesh J Salgaonkar writes: >> >>> diff --git a/arch/powerpc/include/asm/paca.h >>> b/arch/powerpc/include/asm/paca.h >>> index 7f22929ce915..233d25ff6f64 100644 >>> ---

Re: [PATCH v7 8/9] powerpc/mce: Add sysctl control for recovery action on MCE.

2018-08-10 Thread Michael Ellerman
Michal Suchánek writes: > On Wed, 8 Aug 2018 21:07:11 +0530 > "Aneesh Kumar K.V" wrote: >> On 08/08/2018 08:26 PM, Michael Ellerman wrote: >> > Mahesh J Salgaonkar writes: >> >> From: Mahesh Salgaonkar >> >> >> >> Introduce recovery action for recovered memory errors (MCEs). >> >> There are

Re: [PATCH v3 2/2] powerpc/pseries: Wait for completion of hotplug events during PRRN handling

2018-08-10 Thread Michael Ellerman
Hi Nathan, Nathan Fontenot writes: > On 08/08/2018 10:29 AM, John Allen wrote: >> While handling PRRN events, the time to handle the actual hotplug events >> dwarfs the time it takes to perform the device tree updates and queue the >> hotplug events. In the case that PRRN events are being queued

Re: [PATCH v3 1/2] powerpc/pseries: Avoid blocking rtas polling handling multiple PRRN events

2018-08-10 Thread Michael Ellerman
John Allen writes: > When a PRRN event is being handled and another PRRN event comes in, the > second event will block rtas polling waiting on the first to complete, > preventing any further rtas events from being handled. This can be > especially problematic in case that PRRN events are

Re: [PATCH v7 5/9] powerpc/pseries: flush SLB contents on SLB MCE errors.

2018-08-10 Thread Mahesh Jagannath Salgaonkar
On 08/07/2018 10:24 PM, Michal Suchánek wrote: > Hello, > > > On Tue, 07 Aug 2018 19:47:14 +0530 > "Mahesh J Salgaonkar" wrote: > >> From: Mahesh Salgaonkar >> >> On pseries, as of today system crashes if we get a machine check >> exceptions due to SLB errors. These are soft errors and can be

Re: [PATCH v3] powerpc/topology: Check at boot for topology updates

2018-08-10 Thread Michael Ellerman
Hi Srikar, Thanks for debugging this. Srikar Dronamraju writes: > diff --git a/arch/powerpc/include/asm/topology.h > b/arch/powerpc/include/asm/topology.h > index 16b077801a5f..70f2d2285ba7 100644 > --- a/arch/powerpc/include/asm/topology.h > +++ b/arch/powerpc/include/asm/topology.h > @@