[PATCH] Fix a typo in radix encodings print

2016-11-04 Thread Balbir Singh
Rename sift to shift Signed-off-by: Balbir Singh --- arch/powerpc/mm/pgtable-radix.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/mm/pgtable-radix.c b/arch/powerpc/mm/pgtable-radix.c index cd3d400..ea7f796 100644 ---

[PATCH] Fix the message in facility unavailable exception

2016-11-04 Thread Balbir Singh
I ran into this during some testing on qemu. The current facility_strings[] are correct when the trap address is 0xf80 (hypervisor facility unavailable). When the trap address is 0xf60, IC (Interruption Cause) a.k.a status in the code is undefined for values 0 and 1. This patch adds a check to

Re: Coding Style: Reverse XMAS tree declarations ?

2016-11-04 Thread Lino Sanfilippo
On 04.11.2016 18:44, Joe Perches wrote: > On Fri, 2016-11-04 at 11:07 -0400, David Miller wrote: >> From: Lino Sanfilippo >> > On 04.11.2016 07:53, Joe Perches wrote: >> >> CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to >> >> shortest >> >> #446: FILE:

Re: Coding Style: Reverse XMAS tree declarations ? (was Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet)

2016-11-04 Thread David VomLehn
On Fri, Nov 04, 2016 at 10:05:15AM -0700, Randy Dunlap wrote: > On 11/03/16 23:53, Joe Perches wrote: > > On Thu, 2016-11-03 at 15:58 -0400, David Miller wrote: > >> From: Madalin Bucur > >> Date: Wed, 2 Nov 2016 22:17:26 +0200 > >> > >>> This introduces the Freescale Data

Re: [PATCH v6] powerpc: Do not make the entire heap executable

2016-11-04 Thread Kees Cook
Hi, Jason just reminded me about this patch. :) Denys, can you resend a v7 with all the Acked/Reviewed/Tested-bys added and send it To: akpm, with everyone else (and lkml) in CC? That should be the easiest way for Andrew to pick it up. Thanks! -Kees On Mon, Oct 24, 2016 at 5:17 PM, Kees Cook

Re: [RFC v2 5/7] powerpc: Rename context.vdso_base to context.vdso

2016-11-04 Thread Will Deacon
[fixing akpm's email address] On Fri, Nov 04, 2016 at 03:58:22PM +1100, Michael Ellerman wrote: > Christopher Covington writes: > > > Checkpoint/Restore In Userspace (CRIU) needs to be able to unmap and remap > > the VDSO to successfully checkpoint and restore applications

Re: [RFC] kexec_file: Add support for purgatory built as PIE

2016-11-04 Thread Thiago Jung Bauermann
Hello Eric, Am Freitag, 4. November 2016, 10:13:39 BRST schrieb Eric W. Biederman: > Baoquan He writes: > > On 11/02/16 at 04:00am, Thiago Jung Bauermann wrote: > >> Hello, > >> > >> The kexec_file code currently builds the purgatory as a partially linked > >> object (using ld

Re: Coding Style: Reverse XMAS tree declarations ?

2016-11-04 Thread Joe Perches
On Fri, 2016-11-04 at 11:07 -0400, David Miller wrote: > From: Lino Sanfilippo > > On 04.11.2016 07:53, Joe Perches wrote: > >> CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to > >> shortest > >> #446: FILE: drivers/net/ethernet/ethoc.c:446: > >> + 

Re: Coding Style: Reverse XMAS tree declarations ? (was Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet)

2016-11-04 Thread Randy Dunlap
On 11/03/16 23:53, Joe Perches wrote: > On Thu, 2016-11-03 at 15:58 -0400, David Miller wrote: >> From: Madalin Bucur >> Date: Wed, 2 Nov 2016 22:17:26 +0200 >> >>> This introduces the Freescale Data Path Acceleration Architecture >>> +static inline size_t

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-04 Thread Sergey Senozhatsky
Hi Paul, On (11/03/16 21:17), Paul Burton wrote: > > [..] > > > + * The device tree stdout-path chosen node property was > > > + * specified so we don't want to enable the first > > > + * registered console just now in order to give the > > > +

Re: [RFC] kexec_file: Add support for purgatory built as PIE

2016-11-04 Thread Thiago Jung Bauermann
Hello Baoquan, Am Freitag, 4. November 2016, 15:38:40 BRST schrieb Baoquan He: > On 11/02/16 at 04:00am, Thiago Jung Bauermann wrote: > > Hello, > > > > The kexec_file code currently builds the purgatory as a partially linked > > object (using ld -r). Is there a particular reason to use that

Re: [RFC] kexec_file: Add support for purgatory built as PIE

2016-11-04 Thread Eric W. Biederman
Baoquan He writes: > On 11/02/16 at 04:00am, Thiago Jung Bauermann wrote: >> Hello, >> >> The kexec_file code currently builds the purgatory as a partially linked >> object >> (using ld -r). Is there a particular reason to use that instead of a >> position >> independent

Re: Coding Style: Reverse XMAS tree declarations ?

2016-11-04 Thread David Miller
From: Lino Sanfilippo Date: Fri, 4 Nov 2016 12:01:17 +0100 > Hi, > > On 04.11.2016 07:53, Joe Perches wrote: >> >> CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to >> shortest >> #446: FILE: drivers/net/ethernet/ethoc.c:446: >> +int size

Re: Coding Style: Reverse XMAS tree declarations ? (was Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet)

2016-11-04 Thread Lino Sanfilippo
Hi, On 04.11.2016 07:53, Joe Perches wrote: CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to shortest #446: FILE: drivers/net/ethernet/ethoc.c:446: + int size = bd.stat >> 16; + struct sk_buff *skb; should not this case be valid?

[PATCH 00/22] mtd: nand: return error code of nand_scan(_ident, _tail) on error

2016-11-04 Thread Masahiro Yamada
nand_scan(), nand_scan_ident(), nand_scan_tail() return an appropriate negative value on error. Most of drivers return the value from them on error, but some of them return the fixed error code -ENXIO (and a few return -ENODEV). This series make those drivers return more precise error code.

[PATCH 08/22] mtd: nand: pasemi: return error code of nand_scan() on error

2016-11-04 Thread Masahiro Yamada
The nand_scan() returns an appropriate error value when it fails. Use it instead of the fixed error code -ENXIO. Signed-off-by: Masahiro Yamada --- drivers/mtd/nand/pasemi_nand.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git

Re: [RFC] kexec_file: Add support for purgatory built as PIE

2016-11-04 Thread Baoquan He
On 11/02/16 at 04:00am, Thiago Jung Bauermann wrote: > Hello, > > The kexec_file code currently builds the purgatory as a partially linked > object > (using ld -r). Is there a particular reason to use that instead of a position > independent executable (PIE)? It's taken as "-r", relocatable

Re: [RESEND] [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-11-04 Thread Uma Krishnan
Frederic/Andrew, Just recently this issue has been reported by system test without any of the two patches you are suspecting - this patch nor the lspci patch. I was hoping the lspci patch from Andrew can possibly solve it. System test CQ is SW370625. The stack reported in that is same, [

Re: [RESEND] [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-11-04 Thread Frederic Barrat
Hi Andrew, Le 04/11/2016 à 07:27, Andrew Donnellan a écrit : On 14/10/16 20:38, Vaibhav Jain wrote: This patch prevents resetting the cxl adapter via sysfs in presence of one or more active cxl_context on it. This protects against an unrecoverable error caused by PSL owning a dirty cache line

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

2016-11-04 Thread Nicholas Piggin
On Fri, 4 Nov 2016 14:34:09 +0530 Gautham R Shenoy wrote: > Hi Nick, > > On Thu, Nov 03, 2016 at 06:02:44PM +1100, Nicholas Piggin wrote: > > On Thu, 3 Nov 2016 02:32:39 -0400 > > "Shreyas B. Prabhu" wrote: > > > > > On Thu, Nov 3, 2016 at 2:17

[PATCH v2] ppc: cpufreq: disable preemption while checking CPU throttling state

2016-11-04 Thread Denis Kirjanov
[ 67.700897] BUG: using smp_processor_id() in preemptible [] code: cat/7343 [ 67.700988] caller is .powernv_cpufreq_throttle_check+0x2c/0x710 [ 67.700998] CPU: 13 PID: 7343 Comm: cat Not tainted 4.8.0-rc5-dirty #1 [ 67.701038] Call Trace: [ 67.701066]

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-04 Thread Paolo Bonzini
On 04/11/2016 06:48, Michael Ellerman wrote: > Currently the kconfig logic for VFIO_IOMMU_SPAPR_TCE and VFIO_SPAPR_EEH > is broken when SPAPR_TCE_IOMMU=n. Leading to: > > warning: (VFIO) selects VFIO_IOMMU_SPAPR_TCE which has unmet direct > dependencies (VFIO && SPAPR_TCE_IOMMU) >

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

2016-11-04 Thread Gautham R Shenoy
Hi Nick, On Thu, Nov 03, 2016 at 06:02:44PM +1100, Nicholas Piggin wrote: > On Thu, 3 Nov 2016 02:32:39 -0400 > "Shreyas B. Prabhu" wrote: > > > On Thu, Nov 3, 2016 at 2:17 AM, Nicholas Piggin wrote: > > > On Thu, 3 Nov 2016 01:56:46 -0400 > > > "Shreyas

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-04 Thread Andreas Schwab
On Nov 03 2016, Paul Burton wrote: > If a device tree specified a preferred device for kernel console output > via the stdout-path or linux,stdout-path chosen node properties there's > no guarantee that it will have specified a device for which we have a > driver. It may

Coding Style: Reverse XMAS tree declarations ? (was Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet)

2016-11-04 Thread Joe Perches
On Thu, 2016-11-03 at 15:58 -0400, David Miller wrote: > From: Madalin Bucur > Date: Wed, 2 Nov 2016 22:17:26 +0200 > > > This introduces the Freescale Data Path Acceleration Architecture > > +static inline size_t bpool_buffer_raw_size(u8 index, u8 cnt) > > +{ > > + u8

Re: [PATCH 2/2] cpufreq: powernv: Use PMSR to verify global and local pstate

2016-11-04 Thread Viresh Kumar
On 04-11-16, 10:57, Akshay Adiga wrote: > As fast_switch may get called in interrupt disable mode, it does not s/in interrupt disable mode/with interrupts disabled s/it does/it may > update the global_pstate_info data structure. Hence the global_pstate_info > has stale data whenever pstate is

Re: [PATCH 1/2] cpufreq: powernv: Adding fast_switch for schedutil

2016-11-04 Thread Viresh Kumar
On 04-11-16, 10:57, Akshay Adiga wrote: > Adding fast_switch which does light weight operation to > set the desired pstate. > > Signed-off-by: Akshay Adiga > --- > drivers/cpufreq/powernv-cpufreq.c | 22 +- > 1 file changed, 21 insertions(+),

Re: [RESEND] [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-11-04 Thread Andrew Donnellan
On 14/10/16 20:38, Vaibhav Jain wrote: This patch prevents resetting the cxl adapter via sysfs in presence of one or more active cxl_context on it. This protects against an unrecoverable error caused by PSL owning a dirty cache line even after reset and host tries to touch the same cache line.

[PATCH] powerpc/oops: fix pr_cont() issues in instruction dump

2016-11-04 Thread Andrew Donnellan
Since the KERN_CONT changes, the current code in show_instructions() prints out a whole bunch of unnecessary newlines. Change occurrences of printk("\n") to pr_cont("\n"). While we're here, change all the other cases of printk(KERN_CONT ...) to pr_cont() as well. Signed-off-by: Andrew Donnellan