Re: [PATCH 2/3] Detect instruction fetch denied and report

2016-11-08 Thread Aneesh Kumar K.V
Balbir Singh writes: > ISA 3 allows for prevention of instruction fetch and execution > of user mode pages. If such an error occurs, SRR1 bit 35 > reports the error. We catch and report the error in do_page_fault() > > Signed-off-by: Balbir Singh >

Re: [PATCH 3/3] Enable storage keys for radix - user mode execution

2016-11-08 Thread Aneesh Kumar K.V
Balbir Singh writes: > ISA 3 defines new encoded access authority that allows instruction > access prevention in privileged mode and allows normal access > to problem state. This patch just enables IAMR (Instruction Authority > Mask Register), enabling AMR would require

Re: [PATCH 2/3] Detect instruction fetch denied and report

2016-11-08 Thread Aneesh Kumar K.V
Balbir Singh writes: > ISA 3 allows for prevention of instruction fetch and execution > of user mode pages. If such an error occurs, SRR1 bit 35 > reports the error. We catch and report the error in do_page_fault() > > Signed-off-by: Balbir Singh >

Re: [v15, 3/7] powerpc/fsl: move mpc85xx.h to include/linux/fsl

2016-11-08 Thread Arnd Bergmann
On Tuesday, November 8, 2016 6:49:51 AM CET Y.B. Lu wrote: > Hi Arnd, > > > > -Original Message- > > From: Arnd Bergmann [mailto:a...@arndb.de] > > Sent: Tuesday, November 08, 2016 5:20 AM > > To: Y.B. Lu > > Cc: linuxppc-dev@lists.ozlabs.org; linux-...@vger.kernel.org; > >

Re: [PATCH 1/3] Setup AMOR in HV mode

2016-11-08 Thread Aneesh Kumar K.V
Balbir Singh writes: > AMOR should be setup in HV mode, we set it up once > and let the generic kernel handle IAMR. This patch is > used to enable storage keys in a following patch as > defined in ISA 3 > > Reported-by: Aneesh Kumar K.V >

Re: compiling master.

2016-11-08 Thread Naveen N. Rao
On 2016/11/08 06:26PM, Michael Ellerman wrote: > Can you guys let me know if Hugh's fix works for you? > > https://patchwork.ozlabs.org/patch/691753/ Yes, that works for me too. - Naveen

Re: [PATCH 3/3] powerpc/pseries: implement nmi ipi with H_SIGNAL_SYS_RESET

2016-11-08 Thread kbuild test robot
Hi Nicholas, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.9-rc4] [cannot apply to next-20161108] [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/Nicholas-Piggin/powerpc

Re: [PATCH v6 4/4] of/fdt: mark hotpluggable memory

2016-11-08 Thread Reza Arbab
On Tue, Nov 08, 2016 at 09:59:26AM +0800, kbuild test robot wrote: All errors (new ones prefixed by >>): drivers/of/fdt.c: In function 'early_init_dt_scan_memory': drivers/of/fdt.c:1064:3: error: implicit declaration of function 'memblock_mark_hotplug' cc1: some warnings being treated as

Re: [PATCH 1/3] powerpc: add NMI IPI infrastructure

2016-11-08 Thread kbuild test robot
Hi Nicholas, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.9-rc4] [cannot apply to next-20161108] [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/Nicholas-Piggin/powerpc

Re: [Linux-ima-devel] [PATCH v6 04/10] ima: maintain memory size needed for serializing the measurement list

2016-11-08 Thread Dmitry Kasatkin
On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann wrote: > From: Mimi Zohar > > In preparation for serializing the binary_runtime_measurements, this patch > maintains the amount of memory required. > > Changelog v5: > - replace

Re: [Linux-ima-devel] [PATCH v6 03/10] ima: permit duplicate measurement list entries

2016-11-08 Thread Dmitry Kasatkin
On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann wrote: > From: Mimi Zohar > > Measurements carried across kexec need to be added to the IMA > measurement list, but should not prevent measurements of the newly > booted kernel from

Re: [Linux-ima-devel] [PATCH v6 04/10] ima: maintain memory size needed for serializing the measurement list

2016-11-08 Thread Mimi Zohar
On Tue, 2016-11-08 at 22:05 +0200, Dmitry Kasatkin wrote: > On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann > wrote: > > From: Mimi Zohar > > > > In preparation for serializing the binary_runtime_measurements, this patch > > maintains

Re: [Linux-ima-devel] [PATCH v6 02/10] ima: on soft reboot, restore the measurement list

2016-11-08 Thread Mimi Zohar
On Tue, 2016-11-08 at 21:46 +0200, Dmitry Kasatkin wrote: > On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann > wrote: > > From: Mimi Zohar > > > > The TPM PCRs are only reset on a hard reboot. In order to validate a > > TPM's quote

Re: [Linux-ima-devel] [PATCH v6 02/10] ima: on soft reboot, restore the measurement list

2016-11-08 Thread Dmitry Kasatkin
On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann wrote: > From: Mimi Zohar > > The TPM PCRs are only reset on a hard reboot. In order to validate a > TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list > of the

Re: [Linux-ima-devel] [PATCH v6 07/10] ima: store the builtin/custom template definitions in a list

2016-11-08 Thread Dmitry Kasatkin
On Fri, Oct 21, 2016 at 5:44 AM, Thiago Jung Bauermann wrote: > From: Mimi Zohar > > The builtin and single custom templates are currently stored in an > array. In preparation for being able to restore a measurement list > containing

Re: [PATCH kernel v4 3/4] vfio/spapr: Reference mm in tce_container

2016-11-08 Thread David Gibson
On Tue, Nov 08, 2016 at 03:25:19PM -0700, Alex Williamson wrote: > On Mon, 24 Oct 2016 17:53:09 +1100 > Alexey Kardashevskiy wrote: > > > In some situations the userspace memory context may live longer than > > the userspace process itself so if we need to do proper memory

Re: [PATCH] powerpc/powernv: Initialise nest mmu

2016-11-08 Thread Balbir Singh
On 14/09/16 15:02, Alistair Popple wrote: > On Mon, 15 Aug 2016 04:51:59 PM Alistair Popple wrote: >> POWER9 contains an off core mmu called the nest mmu (NMMU). This is >> used by other hardware units on the chip to translate virtual >> addresses into real addresses. The unit attempting an

Re: [PATCH 1/2] powerpc/64: Fix checksum folding in csum_tcpudp_nofold and ip_fast_csum_nofold

2016-11-08 Thread Paul Mackerras
On Tue, Nov 08, 2016 at 06:23:30PM +1100, Michael Ellerman wrote: > Paul Mackerras writes: > > > These functions compute an IP checksum by computing a 64-bit sum and > > folding it to 32 bits (the "nofold" in their names refers to folding > > down to 16 bits). However, doing

Re: [PATCH kernel v4 0/4] powerpc/spapr/vfio: Put pages on VFIO container shutdown

2016-11-08 Thread Alex Williamson
On Tue, 08 Nov 2016 18:54:28 +1100 Michael Ellerman wrote: > Alexey Kardashevskiy writes: > > These patches are to fix a bug when pages stay pinned hours > > after QEMU which requested pinning exited. > > > > Please comment. Thanks. > > > > Alexey

Re: [PATCH kernel v4 3/4] vfio/spapr: Reference mm in tce_container

2016-11-08 Thread Alex Williamson
On Mon, 24 Oct 2016 17:53:09 +1100 Alexey Kardashevskiy wrote: > In some situations the userspace memory context may live longer than > the userspace process itself so if we need to do proper memory context > cleanup, we better have tce_container take a reference to mm_struct and

Re: [RFC] fs: add userspace critical mounts event support

2016-11-08 Thread Luis R. Rodriguez
On Wed, Oct 05, 2016 at 09:46:33PM +0200, Luis R. Rodriguez wrote: > On Wed, Oct 05, 2016 at 11:08:06AM -0700, Linus Torvalds wrote: > > On Wed, Oct 5, 2016 at 11:00 AM, Luis R. Rodriguez > > wrote: > > > On Tue, Sep 13, 2016 at 09:38:17PM -0500, Rob Landley wrote: > > > > >

Re: [PATCH V3 1/2] powerpc: Define new ISA v3.00 logical PVR value and PCR register value

2016-11-08 Thread Suraj Jitindar Singh
On Tue, 2016-11-08 at 19:21 +1100, Michael Ellerman wrote: > Suraj Jitindar Singh writes: > > > > > ISA 3.00 adds the logical PVR value 0x0f05, so add a definition > > for > > this. > > > > Define PCR_ARCH_207 to reflect ISA 2.07 compatibility mode in the > >

[v16, 2/7] dt: bindings: move guts devicetree doc out of powerpc directory

2016-11-08 Thread Yangbo Lu
Move guts devicetree doc to Documentation/devicetree/bindings/soc/fsl/ since it's used by not only PowerPC but also ARM. And add a specification for 'little-endian' property. Signed-off-by: Yangbo Lu Acked-by: Rob Herring Acked-by: Scott Wood

[v16, 1/7] ARM64: dts: ls2080a: add device configuration node

2016-11-08 Thread Yangbo Lu
Add the dts node for device configuration unit that provides general purpose configuration and status for the device. Signed-off-by: Yangbo Lu Acked-by: Scott Wood Acked-by: Arnd Bergmann --- Changes for v5: - Added this patch

[v16, 7/7] mmc: sdhci-of-esdhc: fix host version for T4240-R1.0-R2.0

2016-11-08 Thread Yangbo Lu
The eSDHC of T4240-R1.0-R2.0 has incorrect vender version and spec version. Acturally the right version numbers should be VVN=0x13 and SVN = 0x1. This patch adds the GUTS driver support for eSDHC driver to match SoC. And fix host version to avoid that incorrect version numbers break down the ADMA

[v16, 6/7] base: soc: Check for NULL SoC device attributes

2016-11-08 Thread Yangbo Lu
From: Geert Uytterhoeven If soc_device_match() is used to check the value of a specific attribute that is not present for the current SoC, the kernel crashes with a NULL pointer dereference. Fix this by explicitly checking for the absence of a needed property, and

[v16, 0/7] Fix eSDHC host version register bug

2016-11-08 Thread Yangbo Lu
This patchset is used to fix a host version register bug in the T4240-R1.0-R2.0 eSDHC controller. To match the SoC version and revision, 15 previous version patchsets had tried many methods but all of them were rejected by reviewers. Such as - dts compatible method - syscon method

[v16, 3/7] soc: fsl: add GUTS driver for QorIQ platforms

2016-11-08 Thread Yangbo Lu
The global utilities block controls power management, I/O device enabling, power-onreset(POR) configuration monitoring, alternate function selection for multiplexed signals,and clock control. This patch adds a driver to manage and access global utilities block. Initially only reading SVR and

[v16, 4/7] MAINTAINERS: add entry for Freescale SoC drivers

2016-11-08 Thread Yangbo Lu
Add maintainer entry for Freescale SoC drivers including the QE library and the GUTS driver now. Also add maintainer for QE library. Signed-off-by: Yangbo Lu Acked-by: Scott Wood Acked-by: Qiang Zhao Acked-by: Arnd Bergmann

[v16, 5/7] base: soc: introduce soc_device_match() interface

2016-11-08 Thread Yangbo Lu
From: Arnd Bergmann We keep running into cases where device drivers want to know the exact version of the a SoC they are currently running on. In the past, this has usually been done through a vendor specific API that can be called by a driver, or by directly accessing some kind

[PATCH] powerpc/mm: Correct process and partition table max size

2016-11-08 Thread Suraj Jitindar Singh
Version 3.00 of the ISA states that the PATS (partition table size) field of the PTCR (partition table control register) and the PRTS (process table size) field of the partition table entry must both be less than or equal to 24. However the actual size of the partition and process tables is equal

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

2016-11-08 Thread Michael Ellerman
Gautham R Shenoy writes: > On Tue, Nov 08, 2016 at 11:21:23AM +1100, Michael Ellerman wrote: >> Denis Kirjanov writes: >> > On 11/7/16, Michael Ellerman wrote: >> >> Denis Kirjanov writes: >> >>>

Re: [PATCH V3 1/2] powerpc: Define new ISA v3.00 logical PVR value and PCR register value

2016-11-08 Thread Michael Ellerman
Suraj Jitindar Singh writes: > ISA 3.00 adds the logical PVR value 0x0f05, so add a definition for > this. > > Define PCR_ARCH_207 to reflect ISA 2.07 compatibility mode in the processor > compatibility register (PCR). Also define a dummy ISA 3.00 compatibility >

Re: compiling master.

2016-11-08 Thread Denis Kirjanov
On 11/8/16, Michael Ellerman wrote: > Can you guys let me know if Hugh's fix works for you? > > https://patchwork.ozlabs.org/patch/691753/ Ok, the patch works fine for me. > > cheers >

Re: [PATCH] powerpc/pseries: add definitions for new H_SIGNAL_SYS_RESET hcall

2016-11-08 Thread Michael Ellerman
Nicholas Piggin writes: > This has not made its way to a PAPR release yet, but we have an hcall > number assigned. > > H_SIGNAL_SYS_RESET = 0x380 > > Syntax: > hcall(uint64 H_SIGNAL_SYS_RESET, /* generate a system reset NMI on */ >/*

Re: [PATCH] cpufreq, powernv: Disable preemption while checking CPU throttling state throttling state

2016-11-08 Thread Denis Kirjanov
On 11/8/16, Viresh Kumar wrote: > +some more folks from IBM. > > On 08-11-16, 03:35, Denis Kirjanov wrote: >> [ 67.700897] BUG: using smp_processor_id() in preemptible [] >> code: cat/7343 >> [ 67.700988] caller is

[PATCH v2] cpufreq, powernv: Disable preemption while checking CPU throttling state throttling state

2016-11-08 Thread Denis Kirjanov
With preemption turned on we can read incorrect throttling state while being switched to CPU on a different chip. [ 67.700897] BUG: using smp_processor_id() in preemptible [] code: cat/7343 [ 67.700988] caller is .powernv_cpufreq_throttle_check+0x2c/0x710 [ 67.700998]

[PATCH] cpufreq, powernv: Disable preemption while checking CPU throttling state throttling state

2016-11-08 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] cpufreq, powernv: Disable preemption while checking CPU throttling state throttling state

2016-11-08 Thread Viresh Kumar
+some more folks from IBM. On 08-11-16, 03:35, Denis Kirjanov wrote: > [ 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

[PATCH 0/2] powerpc: oops fixes

2016-11-08 Thread Nicholas Piggin
Hi, Couple of small things I've come across. After this, injecting nmi to enter xmon (via qemu) and then recovering works nicely. Thanks, Nick Nicholas Piggin (2): powerpc: fix graceful debugger recovery powerpc: fix second nested oops hang arch/powerpc/kernel/traps.c | 14 --

Re: [PATCHv3 1/8] powerpc/vdso: unify return paths in setup_additional_pages

2016-11-08 Thread Dmitry Safonov
On 11/08/2016 03:10 AM, Michael Ellerman wrote: Hi Dmitry, Thanks for the patches. Dmitry Safonov writes: Impact: cleanup I'm not a fan of these "Impact" lines, especially when they're not correct, ie. this is not a cleanup, a cleanup doesn't change logic. Rename

[PATCH 1/2] powerpc: fix graceful debugger recovery

2016-11-08 Thread Nicholas Piggin
When exiting xmon with 'x' (exit and recover), oops_begin bails out immediately, but die then calls __die() and oops_end(), which cause a lot of bad things to happen. If the debugger was attached then went to graceful recovery, exit from die() immediately. Signed-off-by: Nicholas Piggin

[PATCH 2/2] powerpc: fix second nested oops hang

2016-11-08 Thread Nicholas Piggin
When ending an oops, don't clear die_owner unless the nest count went to zero. This prevents a second nested oops from hanging forever on the die_lock. Signed-off-by: Nicholas Piggin --- arch/powerpc/kernel/traps.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

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

2016-11-08 Thread Akshay Adiga
Adding fast_switch which does light weight operation to set the desired pstate. Both global and local pstates are set to the same desired pstate. Signed-off-by: Akshay Adiga Reviewed-by: Gautham R. Shenoy Acked-by: Viresh Kumar

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

2016-11-08 Thread Akshay Adiga
Thanks gautham for the review. Good point, I have made the macros more generic in the next version as you have mentioned. I will post a separate patch to set pstates using these macros. :) On 11/08/2016 09:10 AM, Gautham R Shenoy wrote: On Mon, Nov 07, 2016 at 01:09:09PM +0530, Akshay Adiga

[PATCH 0/3][RFC] powerpc: NMI IPIs

2016-11-08 Thread Nicholas Piggin
Hi, This is a significant reworking of earlier patches, it should be getting closer to the right shape now, but I'm always open to change anything. Thanks, Nick Nicholas Piggin (3): powerpc: add NMI IPI infrastructure powerpc: add struct smp_ops_t.cause_nmi_ipi operation powerpc/pseries:

[PATCH 1/3] powerpc: add NMI IPI infrastructure

2016-11-08 Thread Nicholas Piggin
Add an NMI IPI system that handles platform differences and concurrency and reentrancy issues. The platform does not have to implement a true non-maskable interrupt. The default is to simply uses the debugger break IPI message. The debugger break users (debugger and crash) have been

[PATCH 2/3] powerpc: add struct smp_ops_t.cause_nmi_ipi operation

2016-11-08 Thread Nicholas Piggin
Allow platforms to provide an NMI IPI, and wire that to the NMI system. --- arch/powerpc/include/asm/smp.h| 5 + arch/powerpc/kernel/smp.c | 21 - arch/powerpc/platforms/85xx/smp.c | 1 + arch/powerpc/platforms/86xx/mpc86xx_smp.c | 1

[PATCH 3/3] powerpc/pseries: implement nmi ipi with H_SIGNAL_SYS_RESET

2016-11-08 Thread Nicholas Piggin
H_SIGNAL_SYS_RESET can provide a hard NMI (it is not recoverable if raised when the target has MSR_RI clear). This patch makes a couple of changes to generic system_reset_exception handler, which should be split out and platforms defining a handler audited. --- arch/powerpc/kernel/traps.c

Re: [PATCHv3 0/8] powerpc/mm: refactor vDSO mapping code

2016-11-08 Thread Dmitry Safonov
On 11/08/2016 02:57 AM, Michael Ellerman wrote: Dmitry Safonov <0x7f454...@gmail.com> writes: 2016-10-27 20:09 GMT+03:00 Dmitry Safonov : ping? There's another series doing some similar changes: http://www.spinics.net/lists/linux-mm/msg115860.html Well, that

[PATCH v3 2/2] cpufreq: powernv: Use PMCR to verify global and local pstate

2016-11-08 Thread Akshay Adiga
As fast_switch() may get called with interrupt disable mode, we cannot hold a mutex to update the global_pstate_info. So currently, fast_switch() does not update the global_pstate_info and it will end up with stale data whenever pstate is updated through fast_switch(). As the gpstate_timer can

revert 05fd007e46296afb (was: [PATCH v3] console: use first console if stdout-path device doesn't appear)

2016-11-08 Thread Sergey Senozhatsky
Hello Andrew, (just in case) please revert 'commit 05fd007e46296afb24 ("console: don't prefer first registered if DT specifies stdout-path")' -ss