linux-next: build warning after merge of the powerpc tree

2015-10-19 Thread Stephen Rothwell
Hi all, After merging the powerpc tree, today's linux-next build (powerpc allyesconfig) produced this warning: WARNING: vmlinux.o(.text+0x9367c): Section mismatch in reference from the function .msi_bitmap_alloc() to the function .init.text:.memblock_virt_alloc_try_nid() The function .msi_bitma

Re: [RFC][PATCH 3/3]perf/powerpc :add support for sampling intr machine state

2015-10-19 Thread Madhavan Srinivasan
On Monday 19 October 2015 05:48 PM, Anju T wrote: > From: Anju > > The registers to sample are passed through the sample_regs_intr bitmask. > The name and bit position for each register is defined in asm/perf_regs.h. > This feature can be enabled by using -I option with perf record command. > T

Re: [PATCH 1/3] perf/powerpc:add ability to sample intr machine state in power

2015-10-19 Thread Madhavan Srinivasan
On Monday 19 October 2015 05:48 PM, Anju T wrote: > From: Anju > > The enum definition assigns an 'id' to each register in power. I guess it should be "each register in "struct pt_regs" of arch/powerpc > The order of these values in the enum definition are based on > the corresponding macros in

Re: [PATCH] selfttest/powerpc: Add memory page migration tests

2015-10-19 Thread Anshuman Khandual
On 10/16/2015 05:09 PM, Aneesh Kumar K.V wrote: > Anshuman Khandual writes: > >> On 10/15/2015 09:16 PM, Anshuman Khandual wrote: >>> This adds two tests for memory page migration. One for normal page >>> migration which works for both 4K or 64K base page size kernel and >>> the other one is for

Re: [PATCH] powerpc: Add run_cmd function to boot wrapper

2015-10-19 Thread Michael Ellerman
On Mon, 2015-10-19 at 10:53 -0700, Geoff Levand wrote: > Add a boot wrapper script function run_cmd which will run a shell command > quietly and only print the output if either V=1 or an error occurs. > > Also, run the ps3 dd commands with run_cmd to clean up the build output. > > Signed-off-by:

[PATCH v3 13/17] powerpc/book3e-64: Don't limit paca to 256 MiB

2015-10-19 Thread Scott Wood
This limit only makes sense on book3s, and on book3e it can cause problems with kdump if we don't have any memory under 256 MiB. Signed-off-by: Scott Wood --- v3: Fix book3s build error arch/powerpc/kernel/paca.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/arch/

Re: [PATCH 1/5 v2] dma-mapping: add generic dma_get_page_shift API

2015-10-19 Thread Nishanth Aravamudan
On 15.10.2015 [15:52:19 -0700], Nishanth Aravamudan wrote: > On 14.10.2015 [08:42:51 -0700], Christoph Hellwig wrote: > > Hi Nishanth, > > > > sorry for the late reply. > > > > > > On Power, since it's technically variable, we'd need a function. So are > > > > you suggesting define'ing it to a fu

[PATCH] powerpc: Add run_cmd function to boot wrapper

2015-10-19 Thread Geoff Levand
Add a boot wrapper script function run_cmd which will run a shell command quietly and only print the output if either V=1 or an error occurs. Also, run the ps3 dd commands with run_cmd to clean up the build output. Signed-off-by: Geoff Levand --- Hi Michael, How about something like this? It s

Re: [PATCH 04/17] powerpc: mpic: use IRQCHIP_SKIP_SET_WAKE instead of redundant mpic_irq_set_wake

2015-10-19 Thread Scott Wood
On Mon, 2015-10-19 at 18:35 +0100, Sudeep Holla wrote: > Hi Ben, > > On 23/09/15 05:06, Scott Wood wrote: > > On Mon, 2015-09-21 at 16:47 +0100, Sudeep Holla wrote: > > > mpic_irq_set_wake return -ENXIO for non FSL MPIC and sets > > > IRQF_NO_SUSPEND > > > flag for FSL ones. enable_irq_wake alrea

Re: [PATCH 04/17] powerpc: mpic: use IRQCHIP_SKIP_SET_WAKE instead of redundant mpic_irq_set_wake

2015-10-19 Thread Sudeep Holla
Hi Ben, On 23/09/15 05:06, Scott Wood wrote: On Mon, 2015-09-21 at 16:47 +0100, Sudeep Holla wrote: mpic_irq_set_wake return -ENXIO for non FSL MPIC and sets IRQF_NO_SUSPEND flag for FSL ones. enable_irq_wake already returns -ENXIO if irq_set_wak is not implemented. Also there's no need to set

Re: [PATCH v2] barriers: introduce smp_mb__release_acquire and update documentation

2015-10-19 Thread Peter Zijlstra
On Mon, Oct 19, 2015 at 09:17:18AM +0800, Boqun Feng wrote: > This is confusing me right now. ;-) > > Let's use a simple example for only one primitive, as I understand it, > if we say a primitive A is "fully ordered", we actually mean: > > 1.The memory operations preceding(in program order)

Re: [PATCH] rtas: Validate rtas entry before calling enter_rtas

2015-10-19 Thread Michael Ellerman
On Sat, 2015-10-17 at 10:04 +0530, Vasant Hegde wrote: > On 10/16/2015 11:49 PM, Denis Kirjanov wrote: > > On 10/16/15, Vasant Hegde wrote: > > > On 10/16/2015 04:02 PM, Denis Kirjanov wrote: > > > > On 10/16/15, Vasant Hegde wrote: > > > > > Currently we do not validate rtas entry before calling

Re: [PATCH V4 00/31] powerpc/mm: Update page table format for book3s 64

2015-10-19 Thread Aneesh Kumar K.V
Benjamin Herrenschmidt writes: > On Sat, 2015-10-17 at 15:38 +0530, Aneesh Kumar K.V wrote: >> Hi All, >> >> This patch series attempt to update book3s 64 linux page table format to >> make it more flexible. Our current pte format is very restrictive and we >> overload multiple pte bits. This is

Re: [PATCH] powerpc/prom: Avoid reference to potentially freed memory

2015-10-19 Thread Michael Ellerman
On Fri, 2015-10-16 at 22:09 +0200, Christophe JAILLET wrote: > Le 16/10/2015 12:02, Michael Ellerman a écrit : > > As the kbuild robot detected you have left an extra "}" here. > > > > I don't mind too much if you send patches that aren't compile tested, but > > you > > might save yourself some

Re: [PATCH v2] powerpc/prom: Avoid reference to potentially freed memory

2015-10-19 Thread Michael Ellerman
On Fri, 2015-10-16 at 23:38 +0200, Christophe JAILLET wrote: > of_get_property() is used inside the loop, but then the reference to the > node is dropped before dereferencing the prop pointer, which could by then > point to junk if the node has been freed. > > Instead use of_property_read_u32() t