Re: mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Jeremy Kerr
Hi Stephen, >> Do you know why mailman would be re-writing From: there? It's confusing >> patchwork, as multiple mails are now coming from that address. > > Yep, Anton posts from samba.org. They publish a DMARC policy that > breaks mailing lists. (╯°□°)╯︵ ┻━━┻ This also breaks git-am:

Re: [PATCH 2/2] powerpc/pagetable: Add option to dump kernel hashpagetable

2016-03-29 Thread Balbir Singh
On 22/03/16 09:04, Rashmica Gupta wrote: > Useful to be able to dump the kernel hash page table to check > which pages are hashed along with their sizes and other details. > > Add a debugfs file to check the page tables. To use this the PPC_PTDUMP > config option must be selected. > >

Re: [PATCH 1/2] powerpc/pagetable: Add option to dump the linux pagetables

2016-03-29 Thread Balbir Singh
On 22/03/16 09:04, Rashmica Gupta wrote: > Useful to be able to dump the kernels page tables to check permissions > and memory types - derived from arm64's implementation. > > Add a debugfs file to check the page tables. To use this the PPC_PTDUMP > config option must be selected. > >

Re: mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Michael Ellerman
On Tue, 2016-03-29 at 21:06 +1100, Stephen Rothwell wrote: > Hi Jeremy, > > On Tue, 29 Mar 2016 16:56:58 +0800 Jeremy Kerr wrote: > > > > From the mail that Anton has sent to linuxppc-dev: > > > > > From: Anton Blanchard via Linuxppc-dev > > >

Re: [PATCH] perf jit: genelf makes assumptions about endian

2016-03-29 Thread Michael Ellerman
On Tue, 2016-03-29 at 17:59 +1100, Anton Blanchard wrote: > Commit 9b07e27f88b9 ("perf inject: Add jitdump mmap injection support") > incorrectly assumed that PowerPC is big endian only. > > Simplify things by consolidating the define of GEN_ELF_ENDIAN and checking > for __BYTE_ORDER ==

Re: [PATCH 65/65] powerpc/mm/radix: Cputable update for radix

2016-03-29 Thread Michael Neuling
On Sun, 2016-03-27 at 13:54 +0530, Aneesh Kumar K.V wrote: > This patch move the existing p9 hash to a different PVR and add > radix feature with p9 PVR. That implies we will not be able to > runtime select P9 hash. With P9 Radix we need to do > > * set UPRT = 0 in cpu setup > * set different TLB

Re: [PATCH v2] powerpc: mm: fixup preempt undefflow with huge pages

2016-03-29 Thread Michael Ellerman
On Tue, 2016-03-29 at 15:40 +0200, Sebastian Andrzej Siewior wrote: > On 2016-03-10 01:04:24 [+0530], Aneesh Kumar K.V wrote: > > Sebastian Andrzej Siewior writes: > > *ping* > http://patchwork.ozlabs.org/patch/593943/ *pong* The merge window just closed, I'm still

Re: [v2] ppc64/book3s: fix branching to out of line handlers in relocation kernel

2016-03-29 Thread Michael Ellerman
On Tue, 2016-29-03 at 18:34:37 UTC, Hari Bathini wrote: > diff --git a/arch/powerpc/kernel/exceptions-64s.S > b/arch/powerpc/kernel/exceptions-64s.S > index 7716ceb..e598580 100644 > --- a/arch/powerpc/kernel/exceptions-64s.S > +++ b/arch/powerpc/kernel/exceptions-64s.S > @@ -764,8 +764,8 @@

Re: [v2,2/2] cxl: Configure the PSL for two CAPI ports on POWER8NVL

2016-03-29 Thread Michael Ellerman
On Wed, 2016-16-03 at 16:28:56 UTC, Philippe Bergheaud wrote: > The POWER8NVL chip has two CAPI ports. Configure the PSL to route > data to the port corresponding to the CAPP unit. > > diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c > index 0c6c17a1..924ba63 100644 > ---

Re: [PATCH] Remove kretprobe_trampoline_holder.

2016-03-29 Thread Michael Ellerman
On Tue, 2016-03-29 at 15:34 -0300, Thiago Jung Bauermann wrote: > Am Dienstag, 29 März 2016, 10:45:57 schrieb Michael Ellerman: > > On Mon, 2016-03-28 at 17:29 -0300, Thiago Jung Bauermann wrote: > > > If I do s/_do_fork/._do_fork/ in kprobe_ftrace.tc then all ftrace kprobe > > > tests pass: > >

Re: [PATCH] Remove kretprobe_trampoline_holder.

2016-03-29 Thread Thiago Jung Bauermann
Am Dienstag, 29 März 2016, 14:31:34 schrieb Michael Ellerman: > On Mon, 2016-03-28 at 17:06 -0300, Thiago Jung Bauermann wrote: > > With this patch, all vmlinux symbols match /proc/kallsyms and the > > testcase passes. > > Have you tested this on an LE system? No, I was focusing on ppc64 BE. >

Re: [PATCH] cxl: fix setting of _PAGE_USER bit when handling page faults

2016-03-29 Thread Matthew R. Ochs
> On Mar 17, 2016, at 11:01 PM, Andrew Donnellan > wrote: > > When handling page faults, cxl_handle_page_fault() checks whether the page > should be accessible by userspace and have its _PAGE_USER access bit set. > _PAGE_USER should be set if the context's kernel

Re: [PATCH V2 2/2] pseries/eeh: Handle RTAS delay requests in configure_bridge

2016-03-29 Thread Russell Currey
On Tue, 2016-03-29 at 20:49 +1100, Gavin Shan wrote: > On Tue, Mar 29, 2016 at 12:51:51PM +1000, Russell Currey wrote: > > > > In the configure_pe and configure_bridge RTAS calls, the spec states > > that values of 9900-9905 can be returned, indicating that software > > should delay for 10^x

Re: [PATCH V2 2/2] pseries/eeh: Handle RTAS delay requests in configure_bridge

2016-03-29 Thread Russell Currey
On Tue, 2016-03-29 at 08:51 -0700, Tyrel Datwyler wrote: > On 03/28/2016 07:51 PM, Russell Currey wrote: > > + /* > > +  * RTAS can return a delay value of up to 10^5 > > milliseconds > > +  * (RTAS_EXTENDED_DELAY_MAX), which is too long.  Only > > respect > > +

Re: [PATCH 2/3] swiotlb: prefix dma_to_phys and phys_to_dma functions

2016-03-29 Thread Arnd Bergmann
On Monday 28 March 2016 14:29:29 Konrad Rzeszutek Wilk wrote: > > diff --git a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c > > index ada00c3..8c0f66b 100644 > > --- a/arch/arm64/mm/dma-mapping.c > > +++ b/arch/arm64/mm/dma-mapping.c > > @@ -29,6 +29,14 @@ > > > > #include > > > >

[PATCH v2] ppc64/book3s: fix branching to out of line handlers in relocation kernel

2016-03-29 Thread Hari Bathini
Some of the interrupt vectors on 64-bit POWER server processors are only 32 bytes long (8 instructions), which is not enough for the full first-level interrupt handler. For these we need to branch to an out- of-line (OOL) handler. But when we are running a relocatable kernel, interrupt vectors

Re: [PATCH] Remove kretprobe_trampoline_holder.

2016-03-29 Thread Thiago Jung Bauermann
Am Dienstag, 29 März 2016, 10:45:57 schrieb Michael Ellerman: > On Mon, 2016-03-28 at 17:29 -0300, Thiago Jung Bauermann wrote: > > If I do s/_do_fork/._do_fork/ in kprobe_ftrace.tc then all ftrace kprobe > > tests pass: > > OK. We fixed that in 'perf probe', but not if you're using the sysfs

Re: ppc64/book3s: copy interrupts till __end_handlers marker instead of __end_interrupts

2016-03-29 Thread Hari Bathini
On 03/29/2016 03:47 PM, Michael Ellerman wrote: Hi Hari, You win the "Best Change Log of the Year" award. Some comments below ... On Mon, 2016-28-03 at 11:23:22 UTC, Hari Bathini wrote: Some of the interrupt vectors on 64-bit POWER server processors are only 32 bytes long (8

Re: [PATCH V2 2/2] pseries/eeh: Handle RTAS delay requests in configure_bridge

2016-03-29 Thread Tyrel Datwyler
On 03/28/2016 07:51 PM, Russell Currey wrote: > In the configure_pe and configure_bridge RTAS calls, the spec states > that values of 9900-9905 can be returned, indicating that software > should delay for 10^x (where x is the last digit, i.e. 990x) > milliseconds and attempt the call again.

Re: [PATCH v2] powerpc: mm: fixup preempt undefflow with huge pages

2016-03-29 Thread Sebastian Andrzej Siewior
On 2016-03-10 01:04:24 [+0530], Aneesh Kumar K.V wrote: > Sebastian Andrzej Siewior writes: *ping* http://patchwork.ozlabs.org/patch/593943/ > > [ text/plain ] > > hugepd_free() used __get_cpu_var() once. Nothing ensured that the code > > accessing the variable did not

Re: [PATCH 2/3] swiotlb: prefix dma_to_phys and phys_to_dma functions

2016-03-29 Thread Stefano Stabellini
On Mon, 28 Mar 2016, Konrad Rzeszutek Wilk wrote: > On Fri, Mar 18, 2016 at 11:00 AM, Sinan Kaya wrote: > > On 3/18/2016 8:12 AM, Robin Murphy wrote: > >> Since we know for sure that swiotlb_to_phys is a no-op on arm64, it might > >> be cleaner to simply not reference it at

Re: [PATCH 2/3] swiotlb: prefix dma_to_phys and phys_to_dma functions

2016-03-29 Thread Sinan Kaya
On 3/29/2016 8:44 AM, Stefano Stabellini wrote: > Could you please explain what is the problem that you are trying to > solve? In other words, what is the issue with assuming that physical > address = dma address (and the current dma_to_phys and phys_to_dma > static inlines) if no arm64 platforms

Re: ppc64/book3s: copy interrupts till __end_handlers marker instead of __end_interrupts

2016-03-29 Thread Michael Ellerman
Hi Hari, You win the "Best Change Log of the Year" award. Some comments below ... On Mon, 2016-28-03 at 11:23:22 UTC, Hari Bathini wrote: > Some of the interrupt vectors on 64-bit POWER server processors are > only 32 bytes long (8 instructions), which is not enough for the full > first-level

Re: mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Stephen Rothwell
Hi Jeremy, On Tue, 29 Mar 2016 16:56:58 +0800 Jeremy Kerr wrote: > > From the mail that Anton has sent to linuxppc-dev: > > > From: Anton Blanchard via Linuxppc-dev > > Reply-To: Anton Blanchard > > Do you know why mailman

Re: [PATCH V2 2/2] pseries/eeh: Handle RTAS delay requests in configure_bridge

2016-03-29 Thread Gavin Shan
On Tue, Mar 29, 2016 at 12:51:51PM +1000, Russell Currey wrote: >In the configure_pe and configure_bridge RTAS calls, the spec states >that values of 9900-9905 can be returned, indicating that software >should delay for 10^x (where x is the last digit, i.e. 990x) >milliseconds and attempt the call

Re: [PATCH V2 1/2] pseries/eeh: Refactor the configure bridge RTAS tokens

2016-03-29 Thread Gavin Shan
On Tue, Mar 29, 2016 at 03:53:19PM +1000, Russell Currey wrote: >On Tue, 2016-03-29 at 16:26 +1100, Gavin Shan wrote: >> On Tue, Mar 29, 2016 at 12:51:50PM +1000, Russell Currey wrote: > >> >/* >> > * Necessary sanity check. We needn't check "get-config-addr-info" >> > @@ -93,8 +98,7 @@

mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Jeremy Kerr
Hi Stephen, From the mail that Anton has sent to linuxppc-dev: > From: Anton Blanchard via Linuxppc-dev > Reply-To: Anton Blanchard Do you know why mailman would be re-writing From: there? It's confusing patchwork, as multiple mails are now

[PATCH] perf jit: genelf makes assumptions about endian

2016-03-29 Thread Anton Blanchard via Linuxppc-dev
Commit 9b07e27f88b9 ("perf inject: Add jitdump mmap injection support") incorrectly assumed that PowerPC is big endian only. Simplify things by consolidating the define of GEN_ELF_ENDIAN and checking for __BYTE_ORDER == __BIG_ENDIAN. The PowerPC checks were also incorrect, they do not match what

[PATCH] fsl/qe_ic: move qe_ic to drivers/irqchip

2016-03-29 Thread Zhao Qiang
qe_ic was put in drivers/soc/fsl/qe, but it should be in drivers/irqchip. Signed-off-by: Zhao Qiang --- drivers/irqchip/Makefile| 1 + drivers/{soc/fsl/qe => irqchip}/qe_ic.c | 0 drivers/{soc/fsl/qe => irqchip}/qe_ic.h | 0 drivers/soc/fsl/qe/Makefile