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

2017-09-27 Thread Segher Boessenkool
On Wed, Sep 27, 2017 at 01:38:09PM +1000, Michael Ellerman wrote: > Segher Boessenkool writes: > > A lot of userspace code uses V*X, more and more with newer CPUs and newer > > compiler versions. If you already paid the price for using vector > > registers you do not need to again :-) > > True,

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

2017-09-27 Thread David Laight
From: Segher Boessenkool > Sent: 27 September 2017 10:28 ... > You also need nasty code to deal with the start and end of strings, with > conditional branches and whatnot, which quickly overwhelms the benefit > of using vector registers at all. This tradeoff also changes with newer > ISA versions.

[PATCH 2/2] powerpc/strict_rwx: fixup region splitting

2017-09-27 Thread Balbir Singh
We were aggressive with splitting regions and missed the case when _stext and __init_begin completely overlap addr and addr+mapping. This patch fixes that case and allows us to keep the largest possible mapping through the overlap. The patch also simplies the check into a function Fixes: 7614ff3

[PATCH 1/2] powerpc/strict_rwx: quirks for STRICT_RWX patches

2017-09-27 Thread Balbir Singh
This patch disables STRICT_RWX for power9 DD1 machines where due to some limitations with the way we do tlb updates, we clear the TLB entry of the text that's doing the update to kernel text and that does lead to a crash. Fixes: 7614ff3 ("powerpc/mm/radix: Implement STRICT_RWX/mark_rodata_ro() for

Re: [2/2] powerpc/powernv: Rework EEH initialization on powernv

2017-09-27 Thread Michael Ellerman
On Thu, 2017-09-07 at 06:35:44 UTC, Benjamin Herrenschmidt wrote: > Remove the post_init callback which is only used > by powernv, we can just call it explicitly from > the powernv code. > > This partially kills the ability to "disable" eeh at > runtime via debugfs as this was calling that same >

Re: [1/2] powerpc: Add workaround for P9 vector CI load issue

2017-09-27 Thread Michael Ellerman
On Fri, 2017-09-15 at 05:25:48 UTC, Michael Neuling wrote: > POWER9 DD2.1 and earlier has an issue where some cache inhibited > vector load will return bad data. The workaround is two part, one > firmware/microcode part triggers HMI interrupts when hitting such > loads, the other part is this patch

Re: FSL serial 166550 errata broken?

2017-09-27 Thread Joakim Tjernlund
On Mon, 2017-09-25 at 17:26 +, York Sun wrote: > 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 drive

Re: [PATCH v8 1/6] powerpc/fadump: reduce memory consumption for capture kernel

2017-09-27 Thread Hari Bathini
On Tuesday 12 September 2017 09:31 PM, Michal Suchanek wrote: With fadump (dump capture) kernel booting like a regular kernel, it needs almost the same amount of memory to boot as the production kernel, which is unwarranted for a dump capture kernel. But with no option to disable some of the un

Re: [PATCH v8 1/6] powerpc/fadump: reduce memory consumption for capture kernel

2017-09-27 Thread Hari Bathini
On Tuesday 12 September 2017 09:31 PM, Michal Suchanek wrote: With fadump (dump capture) kernel booting like a regular kernel, it needs almost the same amount of memory to boot as the production kernel, which is unwarranted for a dump capture kernel. But with no option to disable some of the un

Re: [PATCH v4 for 4.14 1/3] membarrier: Provide register expedited private command

2017-09-27 Thread Nicholas Piggin
On Tue, 26 Sep 2017 20:43:28 + (UTC) Mathieu Desnoyers wrote: > - On Sep 26, 2017, at 1:51 PM, Mathieu Desnoyers > mathieu.desnoy...@efficios.com wrote: > > > Provide a new command allowing processes to register their intent to use > > the private expedited command. > > > > I missed

[Part1 PATCH v5 09/17] resource: Provide resource struct in resource walk callback

2017-09-27 Thread Brijesh Singh
From: Tom Lendacky In prep for a new function that will need additional resource information during the resource walk, update the resource walk callback to pass the resource structure. Since the current callback start and end arguments are pulled from the resource structure, the callback functio

Re: FSL serial 166550 errata broken?

2017-09-27 Thread York Sun
On 09/27/2017 04:03 AM, Joakim Tjernlund wrote: > On Mon, 2017-09-25 at 17:26 +, York Sun wrote: >> 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 >>> a

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

2017-09-27 Thread Sergey Senozhatsky
On (09/27/17 16:26), Michael Ellerman wrote: [..] > > Tested-by: Santosh Sivaraj > > Thanks Santosh. > > I also gave it a quick spin. I'll give you an ack for the powerpc changes. > > Acked-by: Michael Ellerman (powerpc) > > > Thanks for cleaning this up Sergey. thanks! -ss

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

2017-09-27 Thread David Gibson
On Tue, Sep 26, 2017 at 04:47:04PM +1000, Sam Bobroff wrote: > 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

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

2017-09-27 Thread Simon Guo
Hi Michael, On Wed, Sep 27, 2017 at 01:38:09PM +1000, Michael Ellerman wrote: > Segher Boessenkool writes: > > > On Tue, Sep 26, 2017 at 03:34:36PM +1000, Michael Ellerman wrote: > >> Cyril Bur writes: > >> > This was written for userspace which doesn't have to explicitly enable > >> > VMX in or

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

2017-09-27 Thread Simon Guo
On Wed, Sep 27, 2017 at 09:43:44AM +, David Laight wrote: > From: Segher Boessenkool > > Sent: 27 September 2017 10:28 > ... > > You also need nasty code to deal with the start and end of strings, with > > conditional branches and whatnot, which quickly overwhelms the benefit > > of using vecto

[PATCH kernel v2] vfio/spapr: Add cond_resched() for huge updates

2017-09-27 Thread Alexey Kardashevskiy
Clearing very big IOMMU tables can trigger soft lockups. This adds cond_resched() for every second spend in the loop. Signed-off-by: Alexey Kardashevskiy --- The testcase is POWER9 box with 264GB guest, 4 VFIO devices from independent IOMMU groups, 64K IOMMU pages. This configuration produces 43