Re: [kernel,v2,1/2] powerpc/iommu: Stop using @current in mm_iommu_xxx

2016-07-22 Thread Nicholas Piggin
On Wed, 20 Jul 2016 14:34:30 +1000 Alexey Kardashevskiy wrote: > static long tce_iommu_register_pages(struct tce_container *container, > @@ -128,10 +129,17 @@ static long tce_iommu_register_pages(struct > tce_container *container, ((vaddr + size) < vaddr)) > return -EINVAL; > >

Re: [PATCH] powerpc/64: implement a slice mask cache

2016-07-22 Thread Balbir Singh
On Fri, Jul 22, 2016 at 10:57:28PM +1000, Nicholas Piggin wrote: > Calculating the slice mask can become a signifcant overhead for > get_unmapped_area. The mask is relatively small and does not change > frequently, so we can cache it in the mm context. > > This saves about 30% kernel time on a 4K

Re: [PATCH v4 00/12] mm: Hardened usercopy

2016-07-22 Thread Laura Abbott
On 07/20/2016 01:26 PM, Kees Cook wrote: Hi, [This is now in my kspp -next tree, though I'd really love to add some additional explicit Tested-bys, Reviewed-bys, or Acked-bys. If you've looked through any part of this or have done any testing, please consider sending an email with your "*-by:" l

Re: [PATCH V5 5/5] powerpc/kvm/stats: Implement existing and add new halt polling vcpu stats

2016-07-22 Thread David Matlack via Linuxppc-dev
On Thu, Jul 21, 2016 at 8:41 PM, Suraj Jitindar Singh wrote: > vcpu stats are used to collect information about a vcpu which can be viewed > in the debugfs. For example halt_attempted_poll and halt_successful_poll > are used to keep track of the number of times the vcpu attempts to and > successfu

Re: [RFC 0/3] extend kexec_file_load system call

2016-07-22 Thread Thiago Jung Bauermann
Am Freitag, 22 Juli 2016, 12:54:28 schrieb Michael Ellerman: > Thiago Jung Bauermann writes: > > So even if not ideal, the solution above is desirable for powerpc. We > > would like to preserve the ability of allowing userspace to pass > > parameters to the OS via the DTB, even if secure boot is e

Re: [PATCH 0/2] cxlflash: Regression patch and updating Maintainers list

2016-07-22 Thread Martin K. Petersen
> "Uma" == Uma Krishnan writes: Uma> First patch in this set fixes a regression that was casued by the Uma> Commit 704c4b0ddc03 ("cxlflash: Shutdown notify support for CXL Uma> Flash cards"), which is currently staged for 4.8 in next/master. Uma> Second patch updates the Maintainers list for

Re: [PATCH v3] of: fix memory leak related to safe_name()

2016-07-22 Thread Rob Herring
On Wed, Jul 20, 2016 at 1:03 AM, Mathieu Malaterre wrote: > On Fri, Jun 24, 2016 at 10:38 PM, Rob Herring wrote: >> On Fri, Jun 17, 2016 at 2:51 AM, Mathieu Malaterre >> wrote: >>> v3 tested here multiple times ! memleak is now gone. >>> >>> Tested-by: Mathieu Malaterre >>> >>> Thanks >>> >>> O

Re: [PATCH v3 02/11] mm: Hardened usercopy

2016-07-22 Thread Josh Poimboeuf
On Thu, Jul 21, 2016 at 11:34:25AM -0700, Kees Cook wrote: > On Wed, Jul 20, 2016 at 11:52 PM, Michael Ellerman > wrote: > > Kees Cook writes: > > > >> diff --git a/mm/usercopy.c b/mm/usercopy.c > >> new file mode 100644 > >> index ..e4bf4e7ccdf6 > >> --- /dev/null > >> +++ b/mm/user

[PATCH] powerpc/eeh: trivial fix to non-conventional PCI address output on EEH log

2016-07-22 Thread Guilherme G. Piccoli
This is a very minor/trivial fix for the output of PCI address on EEH logs. The PCI address on "OF node" field currently is using ":" as a separator for the function, but the usual separator is ".". This patch changes the separator for dot, so the PCI address is printed as usual. No functional cha

Re: [PATCH] cxl: fix sparse warnings

2016-07-22 Thread Matthew R. Ochs
> On Jul 22, 2016, at 4:01 AM, Andrew Donnellan > wrote: > > Make native_irq_wait() static and use NULL rather than 0 to initialise > phb->cfg_data in cxl_pci_vphb_add() to remove sparse warnings. > > Signed-off-by: Andrew Donnellan Reviewed-by: Matthew R. Ochs _

Re: [PATCH 2/2] MAINTAINERS: Update cxlflash maintainers

2016-07-22 Thread Matthew R. Ochs
> On Jul 21, 2016, at 3:44 PM, Uma Krishnan wrote: > > Adding myself as a cxlflash maintainer. > > Signed-off-by: Uma Krishnan Acked-by: Matthew R. Ochs ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/l

Re: [PATCH 1/2] cxlflash: Verify problem state area is mapped before notifying shutdown

2016-07-22 Thread Matthew R. Ochs
> On Jul 21, 2016, at 3:44 PM, Uma Krishnan wrote: > > If an EEH or some other hard error occurs while the > adapter instance was being initialized, on the subsequent > shutdown of the device, the system could crash with: > > [c00f1da03b60] c05eccfc pci_device_shutdown+0x6c/0x100 > [

Re: [for-4.8, 1/2] powerpc/mm: Switch user slb fault handling to translation enabled

2016-07-22 Thread Benjamin Herrenschmidt
On Fri, 2016-07-22 at 22:37 +1000, Nicholas Piggin wrote: > > We also handle fault with proper stack initialized. This enable us > to > > callout to C in fault handling routines. We don't do this for > kernel > > mapping, because of the possibility of taking recursive fault if > > kernel stack in n

[PATCH] powerpc/64: implement a slice mask cache

2016-07-22 Thread Nicholas Piggin
Calculating the slice mask can become a signifcant overhead for get_unmapped_area. The mask is relatively small and does not change frequently, so we can cache it in the mm context. This saves about 30% kernel time on a 4K user address allocation in a microbenchmark. Comments on the approach take

Re: [for-4.8, 1/2] powerpc/mm: Switch user slb fault handling to translation enabled

2016-07-22 Thread Nicholas Piggin
On Wed, 13 Jul 2016 15:10:49 +0530 "Aneesh Kumar K.V" wrote: > We also handle fault with proper stack initialized. This enable us to > callout to C in fault handling routines. We don't do this for kernel > mapping, because of the possibility of taking recursive fault if > kernel stack in not yet

[PATCH] cxl: fix sparse warnings

2016-07-22 Thread Andrew Donnellan
Make native_irq_wait() static and use NULL rather than 0 to initialise phb->cfg_data in cxl_pci_vphb_add() to remove sparse warnings. Signed-off-by: Andrew Donnellan --- drivers/misc/cxl/native.c | 2 +- drivers/misc/cxl/vphb.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --g

Re: [PATCH 05/14] powerpc/pseries: 4GB exception handler offsets

2016-07-22 Thread Nicholas Piggin
On Thu, 21 Jul 2016 14:34:10 + David Laight wrote: > From: Nicholas Piggin > > Sent: 21 July 2016 07:44 > ... > > @@ -739,7 +739,8 @@ kvmppc_skip_Hinterrupt: > > * Ensure that any handlers that get invoked from the exception > > prologs > > * above are below the first 64KB (0x1) of

[RFC] powerpc/64: syscall ABI

2016-07-22 Thread Nicholas Piggin
This adds some documentation for the 64-bit syscall ABI, because we don't seem to have a canonical document anywhere. I have been mostly looking at 64S, so comments about 32-bit and embedded would be welcome too. I have just documented existing practice. The only small issue I have come across is

Re: [PATCH] powernv: Use __printf in pe_level_printk

2016-07-22 Thread kbuild test robot
Hi, [auto build test WARNING on v4.7-rc7] [also build test WARNING on next-20160721] [cannot apply to powerpc/next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Joe-Perches/powernv-Use-__print

[PATCH] powerpc/tm: do not use r13 for tabort_syscall

2016-07-22 Thread Nicholas Piggin
tabort_syscall runs with RI=1, so a nested recoverable machine check will load the paca into r13 and overwrite what we loaded it with, because exceptions returning to privileged mode do not restore r13. This has survived testing with sc instruction inside transaction (bare sc, not glibc syscall be

[PATCH] powerpc/powernv: document cxl dependency on special case in pnv_eeh_reset()

2016-07-22 Thread Andrew Donnellan
pnv_eeh_reset() has special handling for PEs whose primary bus is the root bus or the bus immediately underneath the root port. The cxl bi-modal card support added in b0b5e5918ad1 ("cxl: Add cxl_check_and_switch_mode() API to switch bi-modal cards") relies on this behaviour when hot-resetting the