Re: [PATCH v2 3/8] mm/memory_hotplug: arch_remove_memory() and __remove_pages() with CONFIG_MEMORY_HOTPLUG

2019-05-13 Thread David Hildenbrand
On 13.05.19 09:48, David Hildenbrand wrote: > On 07.05.19 23:02, Dan Williams wrote: >> On Tue, May 7, 2019 at 11:38 AM David Hildenbrand wrote: >>> >>> Let's prepare for better error handling while adding memory by allowing >>> to use arch_remove_memory() and __remove_pages() even if >>>

Re: [PATCH kernel 2/2] powerpc/pseries/dma: Enable swiotlb

2019-05-13 Thread Alexey Kardashevskiy
On 11/05/2019 08:41, Thiago Jung Bauermann wrote: > > Hello Alexey, > > Thanks! > > I have similar changes in my "Secure Virtual Machine Enablement" > patches, which I am currently preparing for posting again real soon now. > > This is the last version: > >

Re: [PATCH kernel 1/2] powerpc/pseries/dma: Allow swiotlb

2019-05-13 Thread Alexey Kardashevskiy
On 11/05/2019 08:36, Thiago Jung Bauermann wrote: > > Alexey Kardashevskiy writes: > >> The commit 8617a5c5bc00 ("powerpc/dma: handle iommu bypass in >> dma_iommu_ops") merged direct DMA ops into the IOMMU DMA ops allowing >> SWIOTLB as well but only for mapping; the unmapping and bouncing

Re: [RFC PATCH] powerpc/64/ftrace: mprofile-kernel patch out mflr

2019-05-13 Thread Naveen N. Rao
Michael Ellerman wrote: Nicholas Piggin writes: The new mprofile-kernel mcount sequence is mflr r0 bl_mcount Dynamic ftrace patches the branch instruction with a noop, but leaves the mflr. mflr is executed by the branch unit that can only execute one per cycle on POWER9 and shared

Re: [PATCH 03/16] lib,treewide: add new match_string() helper/macro

2019-05-13 Thread Ardelean, Alexandru
On Fri, 2019-05-10 at 17:34 +0300, andriy.shevche...@linux.intel.com wrote: > [External] > > > On Fri, May 10, 2019 at 09:15:27AM +, Ardelean, Alexandru wrote: > > On Wed, 2019-05-08 at 16:22 +0300, Alexandru Ardelean wrote: > > > On Wed, 2019-05-08 at 15:18 +0200, Greg KH wrote: > > > > On

Re: [PATCH 1/2] perf ioctl: Add check for the sample_period value

2019-05-13 Thread Peter Zijlstra
On Sat, May 11, 2019 at 08:12:16AM +0530, Ravi Bangoria wrote: > Add a check for sample_period value sent from userspace. Negative > value does not make sense. And in powerpc arch code this could cause > a recursive PMI leading to a hang (reported when running perf-fuzzer). > > Signed-off-by:

Re: [PATCH v10 2/2] powerpc/64s: KVM update for reimplement book3s idle code in C

2019-05-13 Thread Paul Mackerras
On Sun, Apr 28, 2019 at 09:45:15PM +1000, Nicholas Piggin wrote: > This is the KVM update to the new idle code. A few improvements: > > - Idle sleepers now always return to caller rather than branch out > to KVM first. > - This allows optimisations like very fast return to caller when no >

[PATCH v2] powerpc: Fix compile issue with force DAWR

2019-05-13 Thread Michael Neuling
If you compile with KVM but without CONFIG_HAVE_HW_BREAKPOINT you fail at linking with: arch/powerpc/kvm/book3s_hv_rmhandlers.o:(.text+0x708): undefined reference to `dawr_force_enable' This was caused by commit c1fe190c0672 ("powerpc: Add force enable of DAWR on P9 option"). This puts more

Re: [PATCH] powerpc: Document xive=off option

2019-05-13 Thread Cédric Le Goater
On 5/13/19 7:39 AM, Michael Neuling wrote: > commit 243e25112d06 ("powerpc/xive: Native exploitation of the XIVE > interrupt controller") added an option to turn off Linux native XIVE > usage via the xive=off kernel command line option. > > This documents this option. > > Signed-off-by: Michael

Re: [PATCH v2 3/8] mm/memory_hotplug: arch_remove_memory() and __remove_pages() with CONFIG_MEMORY_HOTPLUG

2019-05-13 Thread David Hildenbrand
On 07.05.19 23:02, Dan Williams wrote: > On Tue, May 7, 2019 at 11:38 AM David Hildenbrand wrote: >> >> Let's prepare for better error handling while adding memory by allowing >> to use arch_remove_memory() and __remove_pages() even if >> CONFIG_MEMORY_HOTREMOVE is not set.

[PATCH v3 1/6] soc/fsl/qe: qe.c: drop useless static qualifier

2019-05-13 Thread Rasmus Villemoes
The local variable snum_init has no reason to have static storage duration. Reviewed-by: Christophe Leroy Reviewed-by: Qiang Zhao Signed-off-by: Rasmus Villemoes --- drivers/soc/fsl/qe/qe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/fsl/qe/qe.c

Re: [PATCH v3 4/6] dt-bindings: soc/fsl: qe: document new fsl,qe-snums binding

2019-05-13 Thread Joakim Tjernlund
On Mon, 2019-05-13 at 11:14 +, Rasmus Villemoes wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > Reading table 4-30, and its footnotes, of the QUICC

Re: [PATCH] crypto: vmx - fix copy-paste error in CTR mode

2019-05-13 Thread Michael Ellerman
Herbert Xu writes: > On Mon, May 06, 2019 at 08:53:17AM -0700, Eric Biggers wrote: >> >> Any progress on this? Someone just reported this again here: >> https://bugzilla.kernel.org/show_bug.cgi?id=203515 > > Guys if I don't get a fix for this soon I'll have to disable CTR > in vmx. No objection

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Petr Mladek
On Mon 2019-05-13 12:13:20, Andy Shevchenko wrote: > On Mon, May 13, 2019 at 08:52:41AM +, David Laight wrote: > > From: christophe leroy > > > Sent: 10 May 2019 18:35 > > > Le 10/05/2019 à 18:24, Steven Rostedt a écrit : > > > > On Fri, 10 May 2019 10:42:13 +0200 > > > > Petr Mladek wrote: >

RE: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread David Laight
From: christophe leroy > Sent: 10 May 2019 18:35 > Le 10/05/2019 à 18:24, Steven Rostedt a écrit : > > On Fri, 10 May 2019 10:42:13 +0200 > > Petr Mladek wrote: > > > >> static const char *check_pointer_msg(const void *ptr) > >> { > >> - char byte; > >> - > >>if (!ptr) > >>

Re: [PATCH 0/1] Forced-wakeup for stop lite states on Powernv

2019-05-13 Thread Abhishek
On 05/08/2019 10:29 AM, Nicholas Piggin wrote: Abhishek Goel's on April 22, 2019 4:32 pm: Currently, the cpuidle governors determine what idle state a idling CPU should enter into based on heuristics that depend on the idle history on that CPU. Given that no predictive heuristic is perfect,

[PATCH v3 4/6] dt-bindings: soc/fsl: qe: document new fsl,qe-snums binding

2019-05-13 Thread Rasmus Villemoes
Reading table 4-30, and its footnotes, of the QUICC Engine Block Reference Manual shows that the set of snum _values_ is not necessarily just a function of the _number_ of snums, as given in the fsl,qe-num-snums property. As an alternative, to make it easier to add support for other variants of

[PATCH] powerpc/boot: fix broken way to pass CONFIG options

2019-05-13 Thread Masahiro Yamada
Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures with -j 1") was also wrong. Check-in source files never ever depend on build artifacts. The correct dependency is: $(obj)/serial.o: $(obj)/autoconf.h

Re: [PATCH v2] powerpc: Fix compile issue with force DAWR

2019-05-13 Thread Christophe Leroy
Le 13/05/2019 à 09:17, Michael Neuling a écrit : If you compile with KVM but without CONFIG_HAVE_HW_BREAKPOINT you fail at linking with: arch/powerpc/kvm/book3s_hv_rmhandlers.o:(.text+0x708): undefined reference to `dawr_force_enable' This was caused by commit c1fe190c0672 ("powerpc: Add

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Andy Shevchenko
On Mon, May 13, 2019 at 08:52:41AM +, David Laight wrote: > From: christophe leroy > > Sent: 10 May 2019 18:35 > > Le 10/05/2019 à 18:24, Steven Rostedt a écrit : > > > On Fri, 10 May 2019 10:42:13 +0200 > > > Petr Mladek wrote: > > >> -if (probe_kernel_address(ptr, byte)) > > >> +

[PATCH v3 5/6] soc/fsl/qe: qe.c: support fsl,qe-snums property

2019-05-13 Thread Rasmus Villemoes
Add driver support for the newly introduced fsl,qe-snums property. Conveniently, of_property_read_variable_u8_array does exactly what we need: If the property fsl,qe-snums is found (and has an allowed size), the array of values get copied to snums, and the return value is the number of snums - we

Re: [PATCH RESEND] powerpc: add simd.h implementation specific to PowerPC

2019-05-13 Thread Michael Ellerman
Shawn Landden writes: > It is safe to do SIMD in an interrupt on PowerPC. No it's not sorry :) > Only disable when there is no SIMD available > (and this is a static branch). > > Tested and works with the WireGuard (Zinc) patch I wrote that needs this. > Also improves performance of the crypto

[PATCH v3 6/6] soc/fsl/qe: qe.c: fold qe_get_num_of_snums into qe_snums_init

2019-05-13 Thread Rasmus Villemoes
The comment "No QE ever has fewer than 28 SNUMs" is false; e.g. the MPC8309 has 14. The code path returning -EINVAL is also a recipe for instant disaster, since the caller (qe_snums_init) uncritically assigns the return value to the unsigned qe_num_of_snum, and would thus proceed to attempt to

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Petr Mladek
On Fri 2019-05-10 12:40:58, Steven Rostedt wrote: > On Fri, 10 May 2019 18:32:58 +0200 > Martin Schwidefsky wrote: > > > On Fri, 10 May 2019 12:24:01 -0400 > > Steven Rostedt wrote: > > > > > On Fri, 10 May 2019 10:42:13 +0200 > > > Petr Mladek wrote: > > > > > > > static const char

[Bug 203571] Allow use of SIMD in interrupts on PowerPC

2019-05-13 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203571 --- Comment #2 from Shawn Landden (sland...@gmail.com) --- X86 manages to allow SIMD in interrupts through some very careful code in arch/x86/kernel/fpu/core.c (starting with irq_fpu_usable()) PowerPC can do the same. -- You are receiving this

Re: [PATCH 1/2] perf ioctl: Add check for the sample_period value

2019-05-13 Thread Peter Zijlstra
On Mon, May 13, 2019 at 09:42:13AM +0200, Peter Zijlstra wrote: > On Sat, May 11, 2019 at 08:12:16AM +0530, Ravi Bangoria wrote: > > Add a check for sample_period value sent from userspace. Negative > > value does not make sense. And in powerpc arch code this could cause > > a recursive PMI

Re: [PATCH 1/2] perf ioctl: Add check for the sample_period value

2019-05-13 Thread Ravi Bangoria
On 5/13/19 2:26 PM, Peter Zijlstra wrote: > On Mon, May 13, 2019 at 09:42:13AM +0200, Peter Zijlstra wrote: >> On Sat, May 11, 2019 at 08:12:16AM +0530, Ravi Bangoria wrote: >>> Add a check for sample_period value sent from userspace. Negative >>> value does not make sense. And in powerpc arch

[PATCH v3 2/6] soc/fsl/qe: qe.c: reduce static memory footprint by 1.7K

2019-05-13 Thread Rasmus Villemoes
The current array of struct qe_snum use 256*4 bytes for just keeping track of the free/used state of each index, and the struct layout means there's another 768 bytes of padding. If we just unzip that structure, the array of snum values just use 256 bytes, while the free/inuse state can be tracked

[PATCH v3 3/6] soc/fsl/qe: qe.c: introduce qe_get_device_node helper

2019-05-13 Thread Rasmus Villemoes
The 'try of_find_compatible_node(NULL, NULL, "fsl,qe"), fall back to of_find_node_by_type(NULL, "qe")' pattern is repeated five times. Factor it into a common helper. Reviewed-by: Christophe Leroy Reviewed-by: Qiang Zhao Signed-off-by: Rasmus Villemoes --- drivers/soc/fsl/qe/qe.c | 71

[PATCH 1/2] powerpc/lib: fix redundant inclusion of quad.o

2019-05-13 Thread Christophe Leroy
quad.o is only for PPC64, and already included in obj64-y, so it doesn't have to be in obj-y Fixes: 31bfdb036f12 ("powerpc: Use instruction emulation infrastructure to handle alignment faults") Signed-off-by: Christophe Leroy --- arch/powerpc/lib/Makefile | 2 +- 1 file changed, 1

[PATCH 2/2] powerpc/lib: only build ldstfp.o when CONFIG_PPC_FPU is set

2019-05-13 Thread Christophe Leroy
The entire code in ldstfp.o is enclosed into #ifdef CONFIG_PPC_FPU, so there is no point in building it when this config is not selected. Fixes: cd64d1697cf0 ("powerpc: mtmsrd not defined") Signed-off-by: Christophe Leroy --- arch/powerpc/lib/Makefile | 3 ++- arch/powerpc/lib/ldstfp.S | 4

[PATCH v3 0/6] soc/fsl/qe: cleanups and new DT binding

2019-05-13 Thread Rasmus Villemoes
This small series consists of some small cleanups and simplifications of the QUICC engine driver, and introduces a new DT binding that makes it much easier to support other variants of the QUICC engine IP block that appears in the wild: There's no reason to expect in general that the number of

Re: [RFC PATCH] powerpc/64/ftrace: mprofile-kernel patch out mflr

2019-05-13 Thread Michael Ellerman
"Naveen N. Rao" writes: > Michael Ellerman wrote: >> Nicholas Piggin writes: >>> The new mprofile-kernel mcount sequence is >>> >>> mflr r0 >>> bl_mcount >>> >>> Dynamic ftrace patches the branch instruction with a noop, but leaves >>> the mflr. mflr is executed by the branch

Re: [PATCH, RFC] byteorder: sanity check toolchain vs kernel endianess

2019-05-13 Thread Michael Ellerman
Dmitry Vyukov writes: > From: Arnd Bergmann > Date: Sat, May 11, 2019 at 2:51 AM > To: Dmitry Vyukov > Cc: Nick Kossifidis, Christoph Hellwig, Linus Torvalds, Andrew Morton, > linux-arch, Linux Kernel Mailing List, linuxppc-dev > >> On Fri, May 10, 2019 at 6:53 AM Dmitry Vyukov wrote: >> > > >>

Re: [PATCH, RFC] byteorder: sanity check toolchain vs kernel endianess

2019-05-13 Thread Christoph Hellwig
On Mon, May 13, 2019 at 01:50:19PM +0200, Dmitry Vyukov wrote: > > We did have some bugs in the past (~1-2 y/ago) but AFAIK they are all > > fixed now. These days I build most of my kernels with a bi-endian 64-bit > > toolchain, and switching endian without running `make clean` also works. > >

Re: [PATCH v1 4/8] soc/fsl/qbman: Use index when accessing device tree properties

2019-05-13 Thread Roy Pledge
On 5/13/2019 12:40 PM, Joakim Tjernlund wrote: > On Mon, 2019-05-13 at 16:09 +, Roy Pledge wrote: >> The index value should be passed to the of_parse_phandle() >> function to ensure the correct property is read. > Is this a bug fix? Maybe for stable too? > > Jocke Yes this could go to stable.

[Bug 203597] New: kernel 4.9.175 fails to boot on a PowerMac G4 3,6 at early stage

2019-05-13 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203597 Bug ID: 203597 Summary: kernel 4.9.175 fails to boot on a PowerMac G4 3,6 at early stage Product: Platform Specific/Hardware Version: 2.5 Kernel Version: 4.9.175

[Bug 203597] kernel 4.9.175 fails to boot on a PowerMac G4 3,6 at early stage

2019-05-13 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203597 --- Comment #1 from Erhard F. (erhar...@mailbox.org) --- Created attachment 282745 --> https://bugzilla.kernel.org/attachment.cgi?id=282745=edit bisect.log -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [PATCH v3 4/6] dt-bindings: soc/fsl: qe: document new fsl,qe-snums binding

2019-05-13 Thread Rob Herring
On Mon, 13 May 2019 11:14:58 +, Rasmus Villemoes wrote: > Reading table 4-30, and its footnotes, of the QUICC Engine Block > Reference Manual shows that the set of snum _values_ is not > necessarily just a function of the _number_ of snums, as given in the > fsl,qe-num-snums property. > > As

Re: [PATCH] powerpc/powernv/npu: Fix reference leak

2019-05-13 Thread Greg Kurz
Michael, Any comments on this patch ? Should I repost with a shorter comment as suggested by Alexey ? Cheers, -- Greg On Mon, 29 Apr 2019 12:36:59 +0200 Greg Kurz wrote: > On Mon, 29 Apr 2019 16:01:29 +1000 > Alexey Kardashevskiy wrote: > > > On 20/04/2019 01:34, Greg Kurz wrote: > > >

Re: [PATCH v6 1/1] iommu: enhance IOMMU dma mode build options

2019-05-13 Thread Leizhen (ThunderTown)
On 2019/5/8 17:42, John Garry wrote: > On 18/04/2019 14:57, Zhen Lei wrote: >> First, add build option IOMMU_DEFAULT_{LAZY|STRICT}, so that we have the >> opportunity to set {lazy|strict} mode as default at build time. Then put >> the three config options in an choice, make people can only

[PATCH v1 1/8] soc/fsl/qbman: Rework QBMan private memory setup

2019-05-13 Thread Roy Pledge
Rework QBMan private memory setup so that the areas are not zeroed if the device was previously initialized If the QMan private memory was already initialized skip the PFDR initialization. Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/bman_ccsr.c | 26 --

[PATCH v1 5/8] soc/fsl/qbman: Fix drain_mr_fqni()

2019-05-13 Thread Roy Pledge
The drain_mr_fqni() function may be called fron uninterruptable context so convert the msleep() to an mdelay(). Also ensure that the valid bit is updated while polling. Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/qman.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

[PATCH v2] powerpc/boot: pass CONFIG options in a simpler and more robust way

2019-05-13 Thread Masahiro Yamada
Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures with -j 1") was also wrong. The correct dependency is: $(obj)/serial.o: $(obj)/autoconf.h However, I do not see the reason why we need to copy

[PATCH v1 2/8] soc/fsl/qbman: Cleanup buffer pools if BMan was initialized prior to bootup

2019-05-13 Thread Roy Pledge
Clean the BMan buffer pools if the device had been initialized previously. This will ensure a consistent state if the kernel was soft restarted (kexec for example) Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/bman.c| 17 + drivers/soc/fsl/qbman/bman_ccsr.c | 10

Re: [PATCH 2/8] powerpc/pseries: Do not save the previous DTL mask value

2019-05-13 Thread Nathan Lynch
"Naveen N. Rao" writes: > When CONFIG_VIRT_CPU_ACCOUNTING_NATIVE is enabled, we always initialize > DTL enable mask to DTL_LOG_PREEMPT (0x2). There are no other places > where the mask is changed. As such, when reading the DTL log buffer > through debugfs, there is no need to save and restore

Re: [PATCH] powerpc/boot: fix broken way to pass CONFIG options

2019-05-13 Thread Masahiro Yamada
On Mon, May 13, 2019 at 9:33 PM Masahiro Yamada wrote: > > Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") > was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures > with -j 1") was also wrong. > > Check-in source files never ever depend on build artifacts.

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Steven Rostedt
On Mon, 13 May 2019 14:42:20 +0200 Petr Mladek wrote: > > The "(null)" is good enough by itself and already an established > > practice.. > > (efault) made more sense with the probe_kernel_read() that > checked wide range of addresses. Well, I still think that > it makes sense to distinguish

Re: [PATCH 1/8] powerpc/pseries: Use macros for referring to the DTL enable mask

2019-05-13 Thread Nathan Lynch
"Naveen N. Rao" writes: > Introduce macros to encode the DTL enable mask fields and use those > instead of hardcoding numbers. This is a good cleanup on its own. Acked-by: Nathan Lynch

[PATCH v1 4/8] soc/fsl/qbman: Use index when accessing device tree properties

2019-05-13 Thread Roy Pledge
The index value should be passed to the of_parse_phandle() function to ensure the correct property is read. Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/dpaa_sys.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/fsl/qbman/dpaa_sys.c

Re: [PATCH v1 4/8] soc/fsl/qbman: Use index when accessing device tree properties

2019-05-13 Thread Joakim Tjernlund
On Mon, 2019-05-13 at 16:09 +, Roy Pledge wrote: > > The index value should be passed to the of_parse_phandle() > function to ensure the correct property is read. Is this a bug fix? Maybe for stable too? Jocke > > Signed-off-by: Roy Pledge > --- > drivers/soc/fsl/qbman/dpaa_sys.c | 2

Re: [PATCH] powerpc/boot: fix broken way to pass CONFIG options

2019-05-13 Thread Oliver
On Mon, May 13, 2019 at 11:56 PM Masahiro Yamada wrote: > > On Mon, May 13, 2019 at 9:33 PM Masahiro Yamada > wrote: > > > > Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") > > was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures > > with -j 1") was also

[PATCH v1 0/8] soc/fsl/qbman: Enable Kexec for DPAA1 devices

2019-05-13 Thread Roy Pledge
Most DPAA1 devices do not support a soft reset which is an issue if Kexec starts a new kernel. This patch series allows Kexec to function by detecting that the QBMan device was previously initialized. The patches fix some issues with device cleanup as well as ensuring that the location of the

[PATCH v1 7/8] soc/fsl/qbman: Fixup qman_shutdown_fq()

2019-05-13 Thread Roy Pledge
When shutting down a FQ on a dedicated channel only the SW portal associated with that channel can dequeue from it. Make sure the correct portal is use. Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/qman.c | 53 +++- 1 file changed, 42

[PATCH v1 8/8] soc/fsl/qbman: Update device tree with reserved memory

2019-05-13 Thread Roy Pledge
When using the reserved memory node in the device tree there are two options - dynamic or static. If a dynamic allocation was selected (where the kernel selects the address for the allocation) convert it to a static allocation by inserting the reg property. This will ensure the same memory is

Re: [PATCH] powerpc/boot: fix broken way to pass CONFIG options

2019-05-13 Thread Oliver
On Mon, May 13, 2019 at 9:23 PM Masahiro Yamada wrote: > > Commit 5e9dcb6188a4 ("powerpc/boot: Expose Kconfig symbols to wrapper") > was wrong, but commit e41b93a6be57 ("powerpc/boot: Fix build failures > with -j 1") was also wrong. > > Check-in source files never ever depend on build artifacts.

[PATCH v1 3/8] soc/fsl/qbman: Cleanup QMan queues if device was already initialized

2019-05-13 Thread Roy Pledge
If the QMan device was previously initialized make sure all the frame queues are out of service once all the portals are probed. This handles the case where the kernel is restarted without the SoC being reset (kexec for example) Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/qman.c

[PATCH v1 6/8] soc/fsl/qbman: Disable interrupts during portal recovery

2019-05-13 Thread Roy Pledge
Disable the QBMan interrupts during recovery. Signed-off-by: Roy Pledge --- drivers/soc/fsl/qbman/qman.c | 22 +++--- drivers/soc/fsl/qbman/qman_ccsr.c | 1 + drivers/soc/fsl/qbman/qman_priv.h | 1 + 3 files changed, 21 insertions(+), 3 deletions(-) diff --git

[PATCH 2/2] [PowerPC] Allow use of SIMD in interrupts from kernel code

2019-05-13 Thread Shawn Landden
This second patch is separate because it could be wrong, like I am not sure about how kernel thread migration works, and it is even allowing simd in preemptible kernel code. Signed-off-by: Shawn Landden --- arch/powerpc/include/asm/simd.h | 8 + arch/powerpc/include/asm/switch_to.h |

Re: [RFC PATCH] mm/nvdimm: Fix kernel crash on devm_mremap_pages_release

2019-05-13 Thread Anshuman Khandual
On 05/14/2019 08:23 AM, Aneesh Kumar K.V wrote: > When we initialize the namespace, if we support altmap, we don't initialize > all the > backing struct page where as while releasing the namespace we look at some of > these uninitilized struct page. This results in a kernel crash as below. Yes

Re: [PATCH] mm/nvdimm: Use correct alignment when looking at first pfn from a region

2019-05-13 Thread Aneesh Kumar K.V
On 5/14/19 9:59 AM, Dan Williams wrote: On Mon, May 13, 2019 at 7:55 PM Aneesh Kumar K.V wrote: We already add the start_pad to the resource->start but fails to section align the start. This make sure with altmap we compute the right first pfn when start_pad is zero and we are doing an align

[Bug 203597] kernel 4.9.175 fails to boot on a PowerMac G4 3,6 at early stage

2019-05-13 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203597 Christophe Leroy (christophe.le...@c-s.fr) changed: What|Removed |Added CC|

Re: [Bug 203597] New: kernel 4.9.175 fails to boot on a PowerMac G4 3,6 at early stage

2019-05-13 Thread Christophe Leroy
Hi Greg, Could you please apply b45ba4a51cde29b2939365ef0c07ad34c8321789 to 4.9 since 51c3c62b58b357e8d35e4cc32f7b4ec907426fe3 was applied allthought marked for #4.13+ Thanks Christophe Le 13/05/2019 à 22:18, bugzilla-dae...@bugzilla.kernel.org a écrit :

Re: Kernel OOPS followed by a panic on next20190507 with 4K page size

2019-05-13 Thread Aneesh Kumar K.V
On 5/8/19 4:30 PM, Sachin Sant wrote: While running LTP tests (specifically futex_wake04) against next-20199597 build with 4K page size on a POWER8 LPAR following crash is observed. [ 4233.214876] BUG: Kernel NULL pointer dereference at 0x001c [ 4233.214898] Faulting instruction address:

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Sergey Senozhatsky
On (05/13/19 14:42), Petr Mladek wrote: > > The "(null)" is good enough by itself and already an established > > practice.. > > (efault) made more sense with the probe_kernel_read() that > checked wide range of addresses. Well, I still think that > it makes sense to distinguish a pure NULL. And

[v2 2/2] [PowerPC] Allow use of SIMD in interrupts from kernel code

2019-05-13 Thread Shawn Landden
This second patch is separate because it could be wrong, like I am not sure about how kernel thread migration works, and it is even allowing simd in preemptible kernel code. Signed-off-by: Shawn Landden --- arch/powerpc/include/asm/switch_to.h | 15 ++- arch/powerpc/kernel/process.c

[v2 1/2] [PowerPC] Add simd.h implementation

2019-05-13 Thread Shawn Landden
Based off the x86 one. WireGuard really wants to be able to do SIMD in interrupts, so it can accelerate its in-bound path. Signed-off-by: Shawn Landden --- arch/powerpc/include/asm/simd.h | 10 ++ arch/powerpc/kernel/process.c | 30 ++ 2 files changed, 40

Re: [RFC PATCH] mm/nvdimm: Fix kernel crash on devm_mremap_pages_release

2019-05-13 Thread Aneesh Kumar K.V
On 5/14/19 9:45 AM, Dan Williams wrote: [ add Keith who was looking at something similar ] On Mon, May 13, 2019 at 7:54 PM Aneesh Kumar K.V wrote: When we initialize the namespace, if we support altmap, we don't initialize all the backing struct page where as while releasing the namespace

Re: [PATCH v1 4/8] soc/fsl/qbman: Use index when accessing device tree properties

2019-05-13 Thread Joakim Tjernlund
On Mon, 2019-05-13 at 17:40 +, Roy Pledge wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On 5/13/2019 12:40 PM, Joakim Tjernlund wrote: > > On Mon,

[PATCH 1/2] [PowerPC] Add simd.h implementation

2019-05-13 Thread Shawn Landden
Based off the x86 one. WireGuard really wants to be able to do SIMD in interrupts, so it can accelerate its in-bound path. Signed-off-by: Shawn Landden --- arch/powerpc/include/asm/simd.h | 13 + arch/powerpc/kernel/process.c | 30 ++ 2 files changed,

[PATCH] mm/nvdimm: Use correct alignment when looking at first pfn from a region

2019-05-13 Thread Aneesh Kumar K.V
We already add the start_pad to the resource->start but fails to section align the start. This make sure with altmap we compute the right first pfn when start_pad is zero and we are doing an align down of start address. Signed-off-by: Aneesh Kumar K.V --- kernel/memremap.c | 4 ++-- 1 file

[PATCH] mm/nvdimm: Pick the right alignment default when creating dax devices

2019-05-13 Thread Aneesh Kumar K.V
Allow arch to provide the supported alignments and use hugepage alignment only if we support hugepage. Right now we depend on compile time configs whereas this patch switch this to runtime discovery. Architectures like ppc64 can have THP enabled in code, but then can have hugepage size disabled

[RFC PATCH] mm/nvdimm: Fix kernel crash on devm_mremap_pages_release

2019-05-13 Thread Aneesh Kumar K.V
When we initialize the namespace, if we support altmap, we don't initialize all the backing struct page where as while releasing the namespace we look at some of these uninitilized struct page. This results in a kernel crash as below. kernel BUG at include/linux/mm.h:1034! cpu 0x2: Vector: 700

Re: [RFC PATCH] mm/nvdimm: Fix kernel crash on devm_mremap_pages_release

2019-05-13 Thread Dan Williams
[ add Keith who was looking at something similar ] On Mon, May 13, 2019 at 7:54 PM Aneesh Kumar K.V wrote: > > When we initialize the namespace, if we support altmap, we don't initialize > all the > backing struct page where as while releasing the namespace we look at some of > these

[PATCH] powerpc/mm: Handle page table allocation failures

2019-05-13 Thread Aneesh Kumar K.V
This fix the below crash that arise due to not handling page table allocation failures while allocating hugetlb page table. BUG: Kernel NULL pointer dereference at 0x001c Faulting instruction address: 0xc1d1e58c Oops: Kernel access of bad area, sig: 11 [#1] LE PAGE_SIZE=4K

Re: [PATCH] vsprintf: Do not break early boot with probing addresses

2019-05-13 Thread Sergey Senozhatsky
On (05/14/19 11:07), Sergey Senozhatsky wrote: > How about this: > > if ptr < PAGE_SIZE -> "(null)" No, this is totally stupid. Forget about it. Sorry. > if IS_ERR_VALUE(ptr)-> "(fault)" But Steven's "(fault)" is nice. -ss

Re: [PATCH] mm/nvdimm: Use correct #defines instead of opencoding

2019-05-13 Thread Dan Williams
On Mon, May 13, 2019 at 7:56 PM Aneesh Kumar K.V wrote: > > The nfpn related change is needed to fix the kernel message > > "number of pfns truncated from 2617344 to 163584" > > The change makes sure the nfpns stored in the superblock is right value. > > Signed-off-by: Aneesh Kumar K.V > --- >

Re: [PATCH] mm/nvdimm: Use correct #defines instead of opencoding

2019-05-13 Thread Aneesh Kumar K.V
On 5/14/19 9:28 AM, Dan Williams wrote: On Mon, May 13, 2019 at 7:56 PM Aneesh Kumar K.V wrote: The nfpn related change is needed to fix the kernel message "number of pfns truncated from 2617344 to 163584" The change makes sure the nfpns stored in the superblock is right value.

Re: [PATCH] mm/nvdimm: Use correct alignment when looking at first pfn from a region

2019-05-13 Thread Dan Williams
On Mon, May 13, 2019 at 7:55 PM Aneesh Kumar K.V wrote: > > We already add the start_pad to the resource->start but fails to section > align the start. This make sure with altmap we compute the right first > pfn when start_pad is zero and we are doing an align down of start address. > >

[PATCH] mm/nvdimm: Use correct #defines instead of opencoding

2019-05-13 Thread Aneesh Kumar K.V
The nfpn related change is needed to fix the kernel message "number of pfns truncated from 2617344 to 163584" The change makes sure the nfpns stored in the superblock is right value. Signed-off-by: Aneesh Kumar K.V --- drivers/nvdimm/pfn_devs.c| 6 +++--- drivers/nvdimm/region_devs.c | 8

Re: [PATCH] mm/nvdimm: Use correct #defines instead of opencoding

2019-05-13 Thread Dan Williams
On Mon, May 13, 2019 at 9:05 PM Aneesh Kumar K.V wrote: > > On 5/14/19 9:28 AM, Dan Williams wrote: > > On Mon, May 13, 2019 at 7:56 PM Aneesh Kumar K.V > > wrote: > >> > >> The nfpn related change is needed to fix the kernel message > >> > >> "number of pfns truncated from 2617344 to 163584" >

Re: [PATCH v2] powerpc: Fix compile issue with force DAWR

2019-05-13 Thread Michael Neuling
On Mon, 2019-05-13 at 11:08 +0200, Christophe Leroy wrote: > > Le 13/05/2019 à 09:17, Michael Neuling a écrit : > > If you compile with KVM but without CONFIG_HAVE_HW_BREAKPOINT you fail > > at linking with: > >arch/powerpc/kvm/book3s_hv_rmhandlers.o:(.text+0x708): undefined > > reference to

Re: [PATCH] mm/nvdimm: Use correct #defines instead of opencoding

2019-05-13 Thread Aneesh Kumar K.V
On 5/14/19 9:42 AM, Dan Williams wrote: On Mon, May 13, 2019 at 9:05 PM Aneesh Kumar K.V wrote: On 5/14/19 9:28 AM, Dan Williams wrote: On Mon, May 13, 2019 at 7:56 PM Aneesh Kumar K.V wrote: The nfpn related change is needed to fix the kernel message "number of pfns truncated from

Re: [PATCH 1/2] [PowerPC] Add simd.h implementation

2019-05-13 Thread Benjamin Herrenschmidt
On Mon, 2019-05-13 at 22:44 -0300, Shawn Landden wrote: > + > +/* > + * Were we in user mode when we were > + * interrupted? > + * > + * Doing kernel_altivec/vsx_begin/end() is ok if we are running > + * in an interrupt context from user mode - we'll just > + * save the FPU state as required. > +

Re: [PATCH v2] powerpc: Fix compile issue with force DAWR

2019-05-13 Thread Christophe Leroy
Le 14/05/2019 à 06:47, Michael Neuling a écrit : On Mon, 2019-05-13 at 11:08 +0200, Christophe Leroy wrote: Le 13/05/2019 à 09:17, Michael Neuling a écrit : If you compile with KVM but without CONFIG_HAVE_HW_BREAKPOINT you fail at linking with: