Re: [PATCH v4 2/3] powerpc/modules: Don't try to restore r2 after a sibling call

2017-11-15 Thread Naveen N. Rao
Josh Poimboeuf wrote: On Tue, Nov 14, 2017 at 03:59:21PM +0530, Naveen N. Rao wrote: Kamalesh Babulal wrote: > From: Josh Poimboeuf > > When attempting to load a livepatch module, I got the following error: > > module_64: patch_module: Expect noop after relocate, got 3c82 > > The error

Re: linux-next: Signed-off-by missing for commit in the powerpc tree

2017-11-15 Thread Michael Ellerman
Stephen Rothwell writes: > Hi all, > > Commit > > f23ab3efb1b3 ("powerpc: Fix DABR match on hash based systems") > > is missing a Signed-off-by from its committer. Crud. Hoist on my own petard as they say. cheers

Re: [PATCH v3] kernel/module_64.c: Add REL24 relocation support of livepatch symbols

2017-11-15 Thread Michael Ellerman
Kamalesh Babulal writes: > On Monday 13 November 2017 02:08 PM, Balbir Singh wrote: >> On Fri, Nov 10, 2017 at 2:28 PM, Josh Poimboeuf wrote: >>> On Fri, Nov 10, 2017 at 01:06:25PM +1100, Balbir Singh wrote: On Fri, Nov 10, 2017 at 2:19 AM, Josh Poimboeuf wrote: > FWIW, I think i

[PATCH v4 1/2] livepatch: send a fake signal to all blocking tasks

2017-11-15 Thread Miroslav Benes
Live patching consistency model is of LEAVE_PATCHED_SET and SWITCH_THREAD. This means that all tasks in the system have to be marked one by one as safe to call a new patched function. Safe means when a task is not (sleeping) in a set of patched functions. That is, no patched function is on the task

[PATCH v9 0/8] reduce memory consumption for powerpc firmware-assisted capture kernel

2017-11-15 Thread Hari Bathini
I posted the initial version [1] of patchset [2] adding support to enforce additional parameters when firmware-assisted dump capture kernel is active. Michal reposted it with few improvements to parameter processing to make it more robust. He further posted patchset [3] with few more improvements.

[PATCH v9 1/8] lib/cmdline.c: remove quotes symmetrically

2017-11-15 Thread Hari Bathini
From: Michal Suchanek Remove quotes from argument value only if there is qoute on both sides. Signed-off-by: Michal Suchanek --- lib/cmdline.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/cmdline.c b/lib/cmdline.c index 171c19b..6d398a8 100644 --- a/lib/

[PATCH v9 2/8] boot/param: add pointer to current and next argument to unknown parameter callback

2017-11-15 Thread Hari Bathini
From: Michal Suchanek Add pointer to current and next argument to make parameter processing more robust. This can make parameter processing easier and less error prone in cases where the parameters need to be enforced/ignored based on firmware/system state. Signed-off-by: Michal Suchanek Signed

[PATCH v9 3/8] lib/cmdline.c: add backslash support to kernel commandline parsing

2017-11-15 Thread Hari Bathini
From: Michal Suchanek This allows passing quotes in kernel arguments. It is useful for passing nested arguemnts and might be useful if somebody wanted to pass a double quote directly as part of an argument. It is also useful to have quoting grammar more similar to shells and bootloaders. Signed-

[PATCH v9 4/8] Documentation/admin-guide: backslash support in commandline

2017-11-15 Thread Hari Bathini
From: Michal Suchanek Signed-off-by: Michal Suchanek --- Documentation/admin-guide/kernel-parameters.rst |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/admin-guide/kernel-parameters.rst b/Documentation/admin-guide/kernel-parameters.rst index b2598cc..7

[PATCH v9 5/8] lib/cmdline.c: implement single quotes in commandline argument parsing

2017-11-15 Thread Hari Bathini
From: Michal Suchanek This brings the kernel parser about on par with bourne shell, grub, and other tools that chew the arguments before kernel does. This should make it easier to deal with multiple levels of nesting/quoting. With same quoting grammar on each level there is less room for confusi

[PATCH v9 7/8] powerpc/fadump: reduce memory consumption for capture kernel

2017-11-15 Thread Hari Bathini
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 unnecessary subsystems in fadump kernel, that much memory is wasted

[PATCH v9 8/8] powerpc/fadump: update documentation about 'fadump_extra_args=' parameter

2017-11-15 Thread Hari Bathini
With the introduction of 'fadump_extra_args=' parameter to pass additional parameters to fadump (capture) kernel, update documentation about it. Signed-off-by: Hari Bathini Signed-off-by: Michal Suchanek --- Documentation/powerpc/firmware-assisted-dump.txt | 20 +++- 1 file ch

[PATCH v9 6/8] Documentation/admin-guide: single quotes in kernel arguments

2017-11-15 Thread Hari Bathini
From: Michal Suchanek Signed-off-by: Michal Suchanek --- Documentation/admin-guide/kernel-parameters.rst |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Documentation/admin-guide/kernel-parameters.rst b/Documentation/admin-guide/kernel-parameters.rst index 722d3f7..

Re: [PATCH] ibmveth: Kernel crash LSO offload flag toggle

2017-11-15 Thread Bryant G. Ly
On 11/14/17 8:47 PM, Daniel Axtens wrote: > Hi Bryant, > > This looks a bit better, but... > >> The following patch ensures that the bounce_buffer is not null >> prior to using it within skb_copy_from_linear_data. > How would this occur? > > Looking at ibmveth.c, I see bounce_buffer being freed in

Re: [PATCH V2 2/3] pseries/findnodes: Find nodes with memory for memoryless nodes

2017-11-15 Thread Michael Bringmann
Hello: Sorry for the out-of-date description. This entire patch has been removed / eliminated from subsequent patch sets. All changes to correct powerpc memoryless nodes will be confined to powerpc-specific code. Regards, Michael On 10/19/2017 03:56 AM, Michael Ellerman wrote: > Hi Micha

[PATCH 1/3] libnvdimm: Add of_node to region and bus descriptors

2017-11-15 Thread Oliver O'Halloran
We want to be able to cross reference the region and bus devices with the device tree node that they were spawned from. libNVDIMM handles creating the actual devices for these internally, so we need to pass in a pointer to the relevant node in the descriptor. Signed-off-by: Oliver O'Halloran ---

[PATCH 2/3] libnvdimm: Add a device-tree interface

2017-11-15 Thread Oliver O'Halloran
A fairly bare-bones set of device-tree bindings so libnvdimm can be used on powerpc and other device-tree based platforms. Cc: devicet...@vger.kernel.org Signed-off-by: Oliver O'Halloran --- .../devicetree/bindings/nvdimm/nvdimm-bus.txt | 69 +++ MAINTAINERS

[PATCH 3/3] powerpc/powernv: Create platform devs for nvdimm buses

2017-11-15 Thread Oliver O'Halloran
Scan the devicetree for an nvdimm-bus compatible and create a platform device for them. Signed-off-by: Oliver O'Halloran --- arch/powerpc/platforms/powernv/opal.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/platforms/powernv/opal.c b/arch/powerpc/platforms/powernv/opal.c

Re: [PATCH 1/3] libnvdimm: Add of_node to region and bus descriptors

2017-11-15 Thread Dan Williams
On Wed, Nov 15, 2017 at 9:51 AM, Oliver O'Halloran wrote: > We want to be able to cross reference the region and bus devices > with the device tree node that they were spawned from. libNVDIMM > handles creating the actual devices for these internally, so we > need to pass in a pointer to the relev

RESEND [PATCH V3 0/4] powerpc/devtree: Add support for 'ibm, drc-info' property

2017-11-15 Thread Michael Bringmann
Several properties in the DRC device tree format are replaced by more compact representations to allow, for example, for the encoding of vast amounts of memory, and or reduced duplication of information in related data structures. "ibm,drc-info": This property, when present, replaces the following

RESEND [PATCH V3 3/4] hotplug/drc-info: Add code to search ibm,drc-info property

2017-11-15 Thread Michael Bringmann
rpadlpar_core.c: Provide parallel routines to search the older device- tree properties ("ibm,drc-indexes", "ibm,drc-names", "ibm,drc-types" and "ibm,drc-power-domains"), or the new property "ibm,drc-info". The interface to examine the DRC information is changed from a "get" function that returns v

RESEND [PATCH V3 1/4] powerpc/firmware: Add definitions for new drc-info firmware feature

2017-11-15 Thread Michael Bringmann
Firmware Features: Define new bit flag representing the presence of new device tree property "ibm,drc-info". The flag is used to tell the front end processor whether the Linux kernel supports the new property, and by the front end processor to tell the Linux kernel that the new property is present

RESEND [PATCH V3 4/4] powerpc: Enable support for ibm,drc-info devtree property

2017-11-15 Thread Michael Bringmann
prom_init.c: Enable support for new DRC device tree property "ibm,drc-info" in initial handshake between the Linux kernel and the front end processor. Signed-off-by: Michael Bringmann --- arch/powerpc/kernel/prom_init.c |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/kernel/p

RESEND [PATCH V3 2/4] pseries/drc-info: Search DRC properties for CPU indexes

2017-11-15 Thread Michael Bringmann
pseries/drc-info: Provide parallel routines to convert between drc_index and CPU numbers at runtime, using the older device-tree properties ("ibm,drc-indexes", "ibm,drc-names", "ibm,drc-types" and "ibm,drc-power-domains"), or the new property "ibm,drc-info". Signed-off-by: Michael Bringmann --- C

Re: [PATCH 2/2] powerpc/hotplug: Ensure nodes initialized for hotplug

2017-11-15 Thread Michael Bringmann
Hello: See below. On 10/16/2017 07:54 AM, Michael Ellerman wrote: > Michael Bringmann writes: > >> powerpc/hotplug: On systems like PowerPC which allow 'hot-add' of CPU, >> it may occur that the new resources are to be inserted into nodes >> that were not used for memory resources at bootup.

[PATCH v10 07/10] mm: Add address parameter to arch_validate_prot()

2017-11-15 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 0/9] posix_clocks: Prepare syscalls for 64 bit time_t conversion

2017-11-15 Thread Deepa Dinamani
> I had on concern about x32, maybe we should check > for "COMPAT_USE_64BIT_TIME" before zeroing out the tv_nsec > bits. Thanks, I think you are right. I had the check conditional on CONFIG_64BIT_TIME and then removed as I forgot why I added it. :) > Regarding CONFIG_COMPAT_TIME/CONFIG_64BIT_TIME

[PATCH v10 00/10] Application Data Integrity feature introduced by SPARC M7

2017-11-15 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] ibmveth: Kernel crash LSO offload flag toggle

2017-11-15 Thread Benjamin Herrenschmidt
On Wed, 2017-11-15 at 10:45 -0600, Bryant G. Ly wrote: > This patch just closes the window, bad things can still happen. I wanted to > leave it > up to the people who actively develop in ibmveth to close the window, since > introducing > a lock can be expensive in tx. You don't need to instrodu

Re: [PATCH v4 2/3] powerpc/modules: Don't try to restore r2 after a sibling call

2017-11-15 Thread Josh Poimboeuf
On Wed, Nov 15, 2017 at 02:58:33PM +0530, Naveen N. Rao wrote: > > +int instr_is_link_branch(unsigned int instr) > > +{ > > + return (instr_is_branch_iform(instr) || instr_is_branch_bform(instr)) && > > + (instr & BRANCH_SET_LINK); > > +} > > + > > Nitpicking here, but since we're not c

Re: [PATCH 1/3] libnvdimm: Add of_node to region and bus descriptors

2017-11-15 Thread Oliver
On Thu, Nov 16, 2017 at 5:05 AM, Dan Williams wrote: > On Wed, Nov 15, 2017 at 9:51 AM, Oliver O'Halloran wrote: >> We want to be able to cross reference the region and bus devices >> with the device tree node that they were spawned from. libNVDIMM >> handles creating the actual devices for these

Re: [PATCH v2] powerpc/powernv: Add pci_reset_phbs parameter to issue a PHB reset

2017-11-15 Thread Balbir Singh
On Thu, Oct 26, 2017 at 2:27 AM, Guilherme G. Piccoli wrote: > During a kdump kernel boot in PowerPC, we request a reset of the PHBs > to the FW. It makes sense, since if we are booting a kdump kernel it > means we had some trouble before and we cannot rely in the adapters' > health; they could be

Re: [PATCH v2 0/8] powerpc: Support ibm,dynamic-memory-v2 property

2017-11-15 Thread Bharata B Rao
On Fri, Oct 20, 2017 at 6:51 PM, Nathan Fontenot wrote: > This patch set provides a set of updates to de-couple the LMB information > provided in the ibm,dynamic-memory device tree property from the device > tree property format. A part of this patch series introduces a new > device tree property

Re: [PATCH] powerpc/npu: Cleanup MMIO ATSD flushing

2017-11-15 Thread Aneesh Kumar K.V
Balbir Singh writes: + address = start; > + do { > + local_irq_disable(); > + find_linux_pte(mm->pgd, address, &is_thp, &hshift); > + if (!is_thp) > + shift = PAGE_SHIFT; It can still be hugetlb if is_thp is false. > +

[PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-11-15 Thread Christian Zigotzky
Hi All, Just a small patch for the P.A. Semi Nemo board: - diff -rupN a/drivers/pci/probe.c b/drivers/pci/probe.c --- a/drivers/pci/probe.c    2017-11-16 08:18:40.504012010 +0100 +++ b/drivers/pci/probe.c    2017-11-16 08:17:22.044368405 +0100 @@ -2219,6 +2219,8 @@ static int only_one_child