[PATCH 1/1] KVM: PPC: Book3S: Fix server always zero from kvmppc_xive_get_xive()

2017-09-25 Thread Sam Bobroff
In KVM's XICS-on-XIVE emulation, kvmppc_xive_get_xive() returns the value of state->guest_server as "server". However, this value is not set by it's counterpart kvmppc_xive_set_xive(). When the guest uses this interface to migrate interrupts away from a CPU that is going offline, it sees all interr

Re: [PATCH v2 2/3] powerpc/64: enhance memcmp() with VMX instruction for long bytes comparision

2017-09-25 Thread Michael Ellerman
Cyril Bur writes: > On Sun, 2017-09-24 at 05:18 +0800, Simon Guo wrote: >> Hi Cyril, >> On Sat, Sep 23, 2017 at 12:06:48AM +1000, Cyril Bur wrote: >> > On Thu, 2017-09-21 at 07:34 +0800, wei.guo.si...@gmail.com wrote: >> > > From: Simon Guo >> > > >> > > This patch add VMX primitives to do memc

Re: [PATCH] KVM: PPC: Book3S PR: only call slbmte for valid SLB entries

2017-09-25 Thread Michael Ellerman
David Gibson writes: > On Fri, Sep 22, 2017 at 11:34:29AM +0200, Greg Kurz wrote: >> Userland passes an array of 64 SLB descriptors to KVM_SET_SREGS, >> some of which are valid (ie, SLB_ESID_V is set) and the rest are >> likely all-zeroes (with QEMU at least). >> >> Each of them is then passed t

Re: [PATCH v3 0/3] Split default display handling out from VGA arbiter

2017-09-25 Thread Daniel Vetter
On Mon, Sep 18, 2017 at 7:49 AM, Daniel Axtens wrote: > Hi all, > > The merge window is well over by now - is there anything else I can do > to get this series to a mergeable state? I'm not particularly across the > rules for drm-misc, so please let me know if there are things I need to > be doing

Re: [PATCH] KVM: PPC: Book3S PR: only call slbmte for valid SLB entries

2017-09-25 Thread David Gibson
On Fri, Sep 22, 2017 at 11:34:29AM +0200, Greg Kurz wrote: > Userland passes an array of 64 SLB descriptors to KVM_SET_SREGS, > some of which are valid (ie, SLB_ESID_V is set) and the rest are > likely all-zeroes (with QEMU at least). > > Each of them is then passed to kvmppc_mmu_book3s_64_slbmte(

Re: [PATCH V2] cxl: Fix memory page not handled

2017-09-25 Thread Andrew Donnellan
On 26/09/17 11:44, Vaibhav Jain wrote: Hi Christophe, A minor nitpick Christophe Lombard writes: + for (dar = (addr & ~(page_size - 1)); dar < (addr + size); dar += page_size) { + if (dar < vma->vm_start || dar > vma->vm_end) { Code comment in mm_types.h for vm_end says

Re: [PATCH V2] cxl: Fix memory page not handled

2017-09-25 Thread Vaibhav Jain
Hi Christophe, A minor nitpick Christophe Lombard writes: > + for (dar = (addr & ~(page_size - 1)); dar < (addr + size); dar += > page_size) { > + if (dar < vma->vm_start || dar > vma->vm_end) { Code comment in mm_types.h for vm_end says it "The first byte after our end address

Re: [PATCH V2] cxl: Fix memory page not handled

2017-09-25 Thread Andrew Donnellan
On 25/09/17 18:58, Christophe Lombard wrote: The in-kernel 'library' API can be called by drivers to help interaction with an IBM XSL on a POWER9 system. The cxllib_handle_fault() API is used to handle memory fault. All memory pages of the specified buffer have to be handled but under certain co

Re: [PATCH v2 2/3] powerpc/64: enhance memcmp() with VMX instruction for long bytes comparision

2017-09-25 Thread Cyril Bur
On Sun, 2017-09-24 at 05:18 +0800, Simon Guo wrote: > Hi Cyril, > On Sat, Sep 23, 2017 at 12:06:48AM +1000, Cyril Bur wrote: > > On Thu, 2017-09-21 at 07:34 +0800, wei.guo.si...@gmail.com wrote: > > > From: Simon Guo > > > > > > This patch add VMX primitives to do memcmp() in case the compare siz

Re: [PATCH v3 0/3] Split default display handling out from VGA arbiter

2017-09-25 Thread Daniel Axtens
Hi Bjorn, > But I think trying to split the "default device" part out from the VGA > arbiter ends up being overkill and making things more complicated > instead of simpler. Fair enough. > Would something like the following work for you as well as the powerpc > case? On powerpc, we already use vg

Re: [PATCH v3 00/20] Speculative page faults

2017-09-25 Thread Andrew Morton
On Mon, 25 Sep 2017 09:27:43 -0700 Alexei Starovoitov wrote: > On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour > wrote: > > Despite the unprovable lockdep warning raised by Sergey, I didn't get any > > feedback on this series. > > > > Is there a chance to get it moved upstream ? > > what is t

[PATCH v8 0/9] Application Data Integrity feature introduced by SPARC M7

2017-09-25 Thread Khalid Aziz
SPARC M7 processor adds additional metadata for memory address space that can be used to secure access to regions of memory. This additional metadata is implemented as a 4-bit tag attached to each cacheline size block of memory. A task can set a tag on any number of such blocks. Access to such bloc

Re: [PATCH] mm: fix RODATA_TEST failure "rodata_test: test data was not read only"

2017-09-25 Thread Segher Boessenkool
On Mon, Sep 25, 2017 at 04:01:55PM +, David Laight wrote: > From: Segher Boessenkool > > The compiler puts this item in .sdata, for 32-bit. There is no .srodata, > > so if it wants to use a small data section, it must use .sdata . > > > > Non-external, non-referenced symbols are not put in .s

Re: [PATCH 6/6] Documentation/admin-guide: single quotes in kernel arguments.

2017-09-25 Thread Michal Suchánek
On Fri, 15 Sep 2017 10:59:00 -0700 Randy Dunlap wrote: > On 09/15/17 10:02, Michal Suchanek wrote: > > Signed-off-by: Michal Suchanek > > --- > > Documentation/admin-guide/kernel-parameters.rst | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/ad

Re: [PATCH 1/6] lib/cmdline.c: Add backslash support to kernel commandline parsing.

2017-09-25 Thread Michal Suchánek
On Fri, 15 Sep 2017 19:28:56 +0200 Michal Suchánek wrote: > On Fri, 15 Sep 2017 18:14:09 +0100 > Al Viro wrote: > > > On Fri, Sep 15, 2017 at 07:02:46PM +0200, Michal Suchanek wrote: > > > > > for (i = 0; args[i]; i++) { > > > - if (isspace(args[i]) && !in_quote) > > > + if

Re: FSL serial 166550 errata broken?

2017-09-25 Thread York Sun
On 09/25/2017 09:55 AM, Joakim Tjernlund wrote: > We got some "broken" boards(mpx8321) where UART RX is held low(BREAK) > There we get a few: >serial8250: too much work for irq18 > and the board freezes. > > Looking inte to driver/CPU there is an errtum(A-004737) w.r.t BREAK handling > and I c

Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers

2017-09-25 Thread Helge Deller
On 25.09.2017 18:29, Luck, Tony wrote: speaking of upstream, any objections if this patch set will go through the printk tree, in one piece? Fine with me too. Helge

FSL serial 166550 errata broken?

2017-09-25 Thread Joakim Tjernlund
We got some "broken" boards(mpx8321) where UART RX is held low(BREAK) There we get a few: serial8250: too much work for irq18 and the board freezes. Looking inte to driver/CPU there is an errtum(A-004737) w.r.t BREAK handling and I can see we are hitting the irq function fsl8250_handle_irq() add

[PATCH v8 7/9] mm: Add address parameter to arch_validate_prot()

2017-09-25 Thread Khalid Aziz
A protection flag may not be valid across entire address space and hence arch_validate_prot() might need the address a protection bit is being set on to ensure it is a valid protection flag. For example, sparc processors support memory corruption detection (as part of ADI feature) flag on memory ad

Re: [PATCH v3 0/3] Split default display handling out from VGA arbiter

2017-09-25 Thread Bjorn Helgaas
On Fri, Sep 01, 2017 at 05:27:41PM +1000, Daniel Axtens wrote: > This patch set: > > - splits the default display handling out from VGA arbiter, into its >own file and behind its own Kconfig option (and gives the functions >better names). > > - adds extra detection of default devices. T

RE: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers

2017-09-25 Thread Luck, Tony
> speaking of upstream, any objections if this patch set will go through > the printk tree, in one piece? Seems to be a better idea than trying to coordinate pulls from three separate "arch/" trees. Fine with me. -Tony

Re: [PATCH v3 00/20] Speculative page faults

2017-09-25 Thread Alexei Starovoitov
On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour wrote: > Despite the unprovable lockdep warning raised by Sergey, I didn't get any > feedback on this series. > > Is there a chance to get it moved upstream ? what is the status ? We're eagerly looking forward for this set to land, since we have se

RE: [PATCH] mm: fix RODATA_TEST failure "rodata_test: test data was not read only"

2017-09-25 Thread David Laight
From: Segher Boessenkool > Sent: 25 September 2017 08:37 > On Sun, Sep 24, 2017 at 12:17:51PM -0700, Kees Cook wrote: > > On Thu, Sep 21, 2017 at 2:37 AM, Christophe Leroy > > wrote: > > > On powerpc, RODATA_TEST fails with message the following messages: > > > > > > [6.199505] Freeing unused

[PATCH] crypto: talitos - fix AEAD for sha224 on non sha224 capable chips

2017-09-25 Thread Christophe Leroy
sha224 AEAD test fails with: [2.803125] talitos ff02.crypto: DEUISR 0x_ [2.808743] talitos ff02.crypto: MDEUISR 0x8010_ [2.814678] talitos ff02.crypto: DESCBUF 0x20731f21_0018 [2.820616] talitos ff02.crypto: DESCBUF 0x0628d64c_001

[PATCH] crypto: talitos - fix setkey to check key weakness

2017-09-25 Thread Christophe Leroy
Crypto manager test report the following failures: [3.061081] alg: skcipher: setkey failed on test 5 for ecb-des-talitos: flags=100 [3.069342] alg: skcipher-ddst: setkey failed on test 5 for ecb-des-talitos: flags=100 [3.077754] alg: skcipher-ddst: setkey failed on test 5 for ecb-des-

Re: [PATCH V2] cxl: Fix memory page not handled

2017-09-25 Thread Frederic Barrat
Le 25/09/2017 à 10:58, Christophe Lombard a écrit : The in-kernel 'library' API can be called by drivers to help interaction with an IBM XSL on a POWER9 system. The cxllib_handle_fault() API is used to handle memory fault. All memory pages of the specified buffer have to be handled but under c

Re: [PATCH v3 04/20] mm: VMA sequence count

2017-09-25 Thread Peter Zijlstra
On Fri, Sep 15, 2017 at 02:38:51PM +0200, Laurent Dufour wrote: > > /* > > * well... answering your question - it seems raw versions of seqcount > > * functions don't call lockdep's lock_acquire/lock_release... > > * > > * but I have never told you that. never. > > */ > > Hum... I'm not

Re: [PATCH v2 3/3] powerpc:selftest update memcmp_64 selftest for VMX implementation

2017-09-25 Thread Simon Guo
Hi David, On Mon, Sep 25, 2017 at 09:30:28AM +, David Laight wrote: > From: wei.guo.si...@gmail.com > > Sent: 21 September 2017 00:35 > > This patch adjust selftest memcmp_64 so that memcmp selftest can be > > compiled successfully. > ... > > #define ITERATIONS 1 > > > > +#define LARGE_SI

Re: [PATCH] powerpc/perf: Fix for core/nest imc call trace on cpuhotplug

2017-09-25 Thread Anju T Sudhakar
Hi mpe, On Thursday 21 September 2017 10:04 AM, Michael Ellerman wrote: Anju T Sudhakar writes: Nest/core pmu units are enabled only when it is used. A reference count is maintained for the events which uses the nest/core pmu units. Currently in *_imc_counters_release function a WARN() is us

RE: [PATCH v2 3/3] powerpc:selftest update memcmp_64 selftest for VMX implementation

2017-09-25 Thread David Laight
From: wei.guo.si...@gmail.com > Sent: 21 September 2017 00:35 > This patch adjust selftest memcmp_64 so that memcmp selftest can be > compiled successfully. ... > #define ITERATIONS 1 > > +#define LARGE_SIZE (5 * 1024) > +#define LARGE_ITERATIONS 1000 ... Measuring performance by doing a lot

Re: [linux-next][EXT4][Oops]kernel panics when running fsfuzzer

2017-09-25 Thread Jan Kara
On Fri 22-09-17 15:46:21, Abdul Haleem wrote: > On Wed, 2017-09-20 at 16:44 +1000, Michael Ellerman wrote: > > Is it reproducible? > > No, bug is not seen once for 3 re-run from yesterday. > > Hope to hit it again in the future CI runs. I was also looking into this but I couldn't figure out what

[PATCH V2] cxl: Fix memory page not handled

2017-09-25 Thread Christophe Lombard
The in-kernel 'library' API can be called by drivers to help interaction with an IBM XSL on a POWER9 system. The cxllib_handle_fault() API is used to handle memory fault. All memory pages of the specified buffer have to be handled but under certain conditions,the last page may not be touched, and

Re: [PATCH] mm: fix RODATA_TEST failure "rodata_test: test data was not read only"

2017-09-25 Thread Segher Boessenkool
On Sun, Sep 24, 2017 at 12:17:51PM -0700, Kees Cook wrote: > On Thu, Sep 21, 2017 at 2:37 AM, Christophe Leroy > wrote: > > On powerpc, RODATA_TEST fails with message the following messages: > > > > [6.199505] Freeing unused kernel memory: 528K > > [6.203935] rodata_test: test data was not

Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers

2017-09-25 Thread Sergey Senozhatsky
On (09/22/17 16:48), Luck, Tony wrote: [..] > Tested patch series on ia64 successfully. > > Tested-by: Tony Luck thanks! > After this goes upstream, you should submit a patch to get rid of > all uses of %pF (70 instances in 35 files) and %pf (63 in 34) > > Perhaps break the patch by top-level