Re: [v3 0/9] parallelized "struct page" zeroing

2017-05-12 Thread David Miller
From: Pasha Tatashin Date: Fri, 12 May 2017 13:24:52 -0400 > Right now it is larger, but what I suggested is to add a new optimized > routine just for this case, which would do STBI for 64-bytes but > without membar (do membar at the end of memmap_init_zone() and > deferred_init_memmap() > > #de

Re: [v3 0/9] parallelized "struct page" zeroing

2017-05-12 Thread Pasha Tatashin
On 05/12/2017 12:57 PM, David Miller wrote: From: Pasha Tatashin Date: Thu, 11 May 2017 16:59:33 -0400 We should either keep memset() only for deferred struct pages as what I have in my patches. Another option is to add a new function struct_page_clear() which would default to memset() and

Re: [v3 0/9] parallelized "struct page" zeroing

2017-05-12 Thread David Miller
From: Pasha Tatashin Date: Thu, 11 May 2017 16:59:33 -0400 > We should either keep memset() only for deferred struct pages as what > I have in my patches. > > Another option is to add a new function struct_page_clear() which > would default to memset() and to something else on platforms that > d

Re: [v3 0/9] parallelized "struct page" zeroing

2017-05-12 Thread David Miller
From: Pasha Tatashin Date: Thu, 11 May 2017 16:47:05 -0400 > So, moving memset() into __init_single_page() benefits Intel. I am > actually surprised why memset() is so slow on intel when it is called > from memblock. But, hurts SPARC, I guess these membars at the end of > memset() kills the perfo

Re: [PATCH] spin loop primitives for busy waiting

2017-05-12 Thread Nicholas Piggin
On Fri, 12 May 2017 12:58:12 + David Laight wrote: > From: Linus Torvalds > > Sent: 11 May 2017 19:48 > ... > > The one question I have is about "spin_on_cond()": since you > > explicitly document that the "no spinning" case is expected to be the > > default, I really think that the default

Re: [PATCH] spin loop primitives for busy waiting

2017-05-12 Thread Linus Torvalds
On Fri, May 12, 2017 at 5:58 AM, David Laight wrote: > > At least some versions of gcc convert while (cond) do {body} > into if (cond) do {body} while (cond) even when 'cond' > is a non-trivial expression and 'body' is trivial. Afaik pretty much all versions of gcc do that, unless you use -Os (wh

Re: [PATCH v5 2/2] powerpc/fadump: update documentation about 'fadump_append=' parameter

2017-05-12 Thread Michal Suchánek
On Fri, 12 May 2017 15:15:33 +0530 Hari Bathini wrote: > On Thursday 11 May 2017 06:46 PM, Michal Suchánek wrote: > > On Thu, 11 May 2017 02:00:11 +0530 > > Hari Bathini wrote: > > > >> Hello Michal, > >> > >> On Wednesday 10 May 2017 09:31 PM, Michal Suchánek wrote: > >>> Hello, > >>> > >>>

Re: [PATCH 1/2] powerpc/tm: Fix FP and VMX register corruption

2017-05-12 Thread Cyril Bur
On Mon, 2017-05-08 at 17:16 +1000, Michael Neuling wrote: > In this commit: > commit dc3106690b20305c3df06b42456fe386dd632ac9 > Author: Cyril Bur > powerpc: tm: Always use fp_state and vr_state to store live registers > > A section of code was removed that copied the current state to > chec

Re: [PATCH 3/5] powerpc/64: do not link crtsaveres.o in boot

2017-05-12 Thread Nicholas Piggin
drop us a note to > help improve the system] > > url: > https://github.com/0day-ci/linux/commits/Nicholas-Piggin/Register-save-restore-function-build-improvements/20170512-075435 > config: powerpc-defconfig (attached as .config) > compiler: powerpc64-linux-gnu-gcc (Debian

Re: [PATCH 2/2] selftests/powerpc: Test TM and VMX register state

2017-05-12 Thread Cyril Bur
On Mon, 2017-05-08 at 17:16 +1000, Michael Neuling wrote: > Test that the VMX checkpointed register state is maintained when a VMX > unavailable exception is taken during a transaction. > > Thanks to Breno Leitao and > Gustavo Bueno Romero for the original test this > is based heavily on. > Go

Re: [PATCH v2 1/3] powerpc/fadump: avoid duplicates in crash memory ranges

2017-05-12 Thread Michal Suchánek
On Fri, 12 May 2017 00:52:47 +0530 Hari Bathini wrote: > fadump sets up crash memory ranges to be used for creating PT_LOAD > program headers in elfcore header. Memory chunk RMA_START through > boot memory area size is added as the first memory range because > firmware, at the time of crash, move

Re: [PATCH 3/5] powerpc/64: do not link crtsaveres.o in boot

2017-05-12 Thread kbuild test robot
/Nicholas-Piggin/Register-save-restore-function-build-improvements/20170512-075435 config: powerpc-defconfig (attached as .config) compiler: powerpc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O

RE: [PATCH] spin loop primitives for busy waiting

2017-05-12 Thread David Laight
From: Linus Torvalds > Sent: 11 May 2017 19:48 ... > The one question I have is about "spin_on_cond()": since you > explicitly document that the "no spinning" case is expected to be the > default, I really think that the default implementation should be > along the lines if > > #define spin_on_c

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-12 Thread Paul Clarke
On 05/11/2017 10:46 PM, Michael Ellerman wrote: > Paul Clarke writes: >> On 05/11/2017 06:24 AM, Nicholas Piggin wrote: >>> Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU >>> feature discovery, and fall back to the "cputable" based version. >> >> This boat has already sailed,

Re: [PATCH 2/3] powerpc/32: tlbie provide L operand explicitly

2017-05-12 Thread Nick Piggin
On 12 May 2017 at 13:35, Michael Ellerman wrote: > Nicholas Piggin writes: > > > The single-operand form of tlbie used to be accepted as the second > > operand (L) being implicitly 0. Newer binutils reject this. > > > > Change remaining single-op tlbie instructions to have explicit 0 > > second

[GIT PULL] Please pull powerpc/linux.git powerpc-4.12-2 tag

2017-05-12 Thread Michael Ellerman
Hi Linus, Please pull the second batch of powerpc updates for 4.12. The change to the Linux page table geometry was delayed for more testing with 16G pages, and there's the new CPU features stuff which just needed one more polish before going in. Plus a few changes from Scott which came in a bit

Re: [PATCH v5 2/2] powerpc/fadump: update documentation about 'fadump_append=' parameter

2017-05-12 Thread Hari Bathini
On Thursday 11 May 2017 06:46 PM, Michal Suchánek wrote: On Thu, 11 May 2017 02:00:11 +0530 Hari Bathini wrote: Hello Michal, On Wednesday 10 May 2017 09:31 PM, Michal Suchánek wrote: Hello, On Wed, 03 May 2017 23:52:52 +0530 Hari Bathini wrote: With the introduction of 'fadump_appen

Re: [PATCH] powernv:idle: Set NAPSTATELOST after recovering paca on P9 DD1

2017-05-12 Thread Nicholas Piggin
On Fri, 12 May 2017 14:52:06 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > commit 17ed4c8f81da ("powerpc/powernv: Recover correct PACA on wakeup > from a stop on P9 DD1") promises to set the NAPSTATELOST bit in paca > after recovering the correct paca for the thread waking up

[PATCH] powernv:idle: Set NAPSTATELOST after recovering paca on P9 DD1

2017-05-12 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" commit 17ed4c8f81da ("powerpc/powernv: Recover correct PACA on wakeup from a stop on P9 DD1") promises to set the NAPSTATELOST bit in paca after recovering the correct paca for the thread waking up from stop1 on DD1, so that the GPRs can be correctly restored on the stop

Re: [PATCH 1/3] powerpc/powernv: Fix CPU_HOTPLUG=n idle.c compile error

2017-05-12 Thread Gautham R Shenoy
Hi Nick, On Fri, May 12, 2017 at 01:15:20AM +1000, Nicholas Piggin wrote: > Fixes: a7cd88da97 ("powerpc/powernv: Move CPU-Offline idle state invocation > from smp.c to idle.c") > Cc: Gautham R. Shenoy > Signed-off-by: Nicholas Piggin > --- > arch/powerpc/platforms/powernv/idle.c | 2 ++ > 1 fil

Re: [PATCH] powerpc/modules: If mprofile-kernel is enabled add it to vermagic

2017-05-12 Thread Jessica Yu
+++ Michael Ellerman [10/05/17 16:57 +1000]: On powerpc we can build the kernel with two different ABIs for mcount(), which is used by ftrace. Kernels built with one ABI do not know how to load modules built with the other ABI. The new style ABI is called "mprofile-kernel", for want of a better n