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

2016-11-07 Thread Michael Ellerman
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 Kardashevskiy (4): > powerpc/iommu: Pass mm_struct to init/cleanup helpers > powerpc/iommu: Stop using

Re: compiling master.

2016-11-07 Thread Michael Ellerman
Can you guys let me know if Hugh's fix works for you? https://patchwork.ozlabs.org/patch/691753/ cheers

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

2016-11-07 Thread Michael Ellerman
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 (u32) (s + (s >> 32)) is not > sufficient to fold a 64-bit sum to 32 bits

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

2016-11-07 Thread Y.B. Lu
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; > ulf.hans...@linaro.org; Scott Wood; Mark Rutland; Greg Kroah-Hartman; X.B. > Xie; M.H.

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

2016-11-07 Thread Nicholas Piggin
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 */ /* the threads indicated by target */ int64 target);

Re: 4.9-rc G5 build broken with a 2.23 binutils

2016-11-07 Thread Michael Ellerman
Hugh Dickins writes: > Hi Nick, > > I've not been able to build 4.9-rc kernels for the PowerMac G5, getting > > arch/powerpc/kernel/exceptions-64s.S: > Assembler messages: > arch/powerpc/kernel/exceptions-64s.S:770: Error: operand out of range > (0x8100 is not

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

2016-11-07 Thread Gautham R Shenoy
On Mon, Nov 07, 2016 at 01:09:09PM +0530, Akshay Adiga wrote: > 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 >

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

2016-11-07 Thread David Gibson
On Thu, Nov 03, 2016 at 12:02:28PM +1100, Paul Mackerras wrote: > On Wed, Nov 02, 2016 at 01:44:03PM +1100, Alexey Kardashevskiy wrote: > > On 31/10/16 15:23, David Gibson wrote: > [...] > > > > > > Um.. yeah.. that's not really ok. Prohibiting overlapping > > > registrations on the same

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

2016-11-07 Thread David Gibson
On Mon, Oct 24, 2016 at 05:53:10PM +1100, Alexey Kardashevskiy wrote: > At the moment the userspace tool is expected to request pinning of > the entire guest RAM when VFIO IOMMU SPAPR v2 driver is present. > When the userspace process finishes, all the pinned pages need to > be put; this is done

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

2016-11-07 Thread David Gibson
On Mon, Oct 24, 2016 at 05:53:09PM +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 > use it

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

2016-11-07 Thread Gautham R Shenoy
Hi Michael, 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: > >>> [ 67.700897] BUG: using smp_processor_id()

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

2016-11-07 Thread kbuild test robot
Hi Reza, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.9-rc4 next-20161028] [cannot apply to mmotm/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

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

2016-11-07 Thread Michael Ellerman
Denis Kirjanov writes: > On 11/7/16, Michael Ellerman wrote: >> Denis Kirjanov writes: >>> [ 67.700897] BUG: using smp_processor_id() in preemptible [] >>> code: cat/7343 >>> [ 67.700988] caller is

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

2016-11-07 Thread Michael Ellerman
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 `rc' variable which doesn't seems to mean

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-07 Thread Alex Williamson
On Tue, 08 Nov 2016 10:49:35 +1100 Michael Ellerman wrote: > Alex Williamson writes: > > On Mon, 07 Nov 2016 19:34:42 +1100 > > Michael Ellerman wrote: > >> Paolo Bonzini writes: > >> > On

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

2016-11-07 Thread Michael Ellerman
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 And I don't like all the macro games in 3/8, eg: +#ifndef BITS

Re: [RFC v2 6/7] mm/powerpc: Use generic VDSO remap and unmap functions

2016-11-07 Thread Michael Ellerman
Laurent Dufour writes: > On 04/11/2016 05:59, Michael Ellerman wrote: >> Christopher Covington writes: >> >>> The PowerPC VDSO remap and unmap code was copied to a generic location, >>> only modifying the variable name expected in mm->context

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-07 Thread Michael Ellerman
Alex Williamson writes: > On Mon, 07 Nov 2016 19:34:42 +1100 > Michael Ellerman wrote: >> Paolo Bonzini writes: >> > On 04/11/2016 06:48, Michael Ellerman wrote: >> >> diff --git a/drivers/vfio/Kconfig

[PATCH v6 1/4] powerpc/mm: allow memory hotplug into a memoryless node

2016-11-07 Thread Reza Arbab
Remove the check which prevents us from hotplugging into an empty node. The original commit b226e4621245 ("[PATCH] powerpc: don't add memory to empty node/zone"), states that this was intended to be a temporary measure. It is a workaround for an oops which no longer occurs. Signed-off-by: Reza

[PATCH v6 2/4] mm: remove x86-only restriction of movable_node

2016-11-07 Thread Reza Arbab
In commit c5320926e370 ("mem-hotplug: introduce movable_node boot option"), the memblock allocation direction is changed to bottom-up and then back to top-down like this: 1. memblock_set_bottom_up(true), called by cmdline_parse_movable_node(). 2. memblock_set_bottom_up(false), called by x86's

[PATCH v6 0/4] enable movable nodes on non-x86 configs

2016-11-07 Thread Reza Arbab
This patchset allows more configs to make use of movable nodes. When CONFIG_MOVABLE_NODE is selected, there are two ways to introduce such nodes into the system: 1. Discover movable nodes at boot. Currently this is only possible on x86, but we will enable configs supporting fdt to do the same.

[PATCH v6 3/4] mm: enable CONFIG_MOVABLE_NODE on non-x86 arches

2016-11-07 Thread Reza Arbab
To support movable memory nodes (CONFIG_MOVABLE_NODE), at least one of the following must be true: 1. This config has the capability to identify movable nodes at boot. Right now, only x86 can do this. 2. Our config supports memory hotplug, which means that a movable node can be created by

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

2016-11-07 Thread Reza Arbab
When movable nodes are enabled, any node containing only hotpluggable memory is made movable at boot time. On x86, hotpluggable memory is discovered by parsing the ACPI SRAT, making corresponding calls to memblock_mark_hotplug(). If we introduce a dt property to describe memory as hotpluggable,

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

2016-11-07 Thread Arnd Bergmann
On Monday, October 31, 2016 9:35:33 AM CET Y.B. Lu wrote: > > > > I don't see any of the contents of this header referenced by the soc > > driver any more. I think you can just drop this patch. > > > > [Lu Yangbo-B47093] This header file was included by guts.c. > The guts driver used macro

Re: [PATCH v2 0/7] soc: renesas: Identify SoC and register with the SoC bus

2016-11-07 Thread Krzysztof Kozlowski
On Mon, Nov 07, 2016 at 10:35:31AM +0100, Geert Uytterhoeven wrote: > On Mon, Oct 31, 2016 at 12:30 PM, Geert Uytterhoeven > wrote: > > Some Renesas SoCs may exist in different revisions, providing slightly > > different functionalities (e.g. R-Car H3 ES1.x and ES2.0),

Re: [RFC v2 6/7] mm/powerpc: Use generic VDSO remap and unmap functions

2016-11-07 Thread Laurent Dufour
On 04/11/2016 05:59, Michael Ellerman wrote: > Christopher Covington writes: > >> The PowerPC VDSO remap and unmap code was copied to a generic location, >> only modifying the variable name expected in mm->context (vdso instead of >> vdso_base) to match most other

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-07 Thread Larry Finger
On 11/07/2016 11:21 AM, Paul Burton wrote: Hi Larry, On Monday, 7 November 2016 09:26:37 GMT Larry Finger wrote: A revert was already submitted by Hans de Goede & is being discussed over here: https://marc.info/?l=linux-kernel=147826151427455=2 I am a little surprised that I was not CCd on

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-07 Thread Paul Burton
Hi Larry, On Monday, 7 November 2016 09:26:37 GMT Larry Finger wrote: > > A revert was already submitted by Hans de Goede & is being discussed over > > here: > > > > https://marc.info/?l=linux-kernel=147826151427455=2 > > I am a little surprised that I was not CCd on that thread. Hans started

[PATCH] cxl: Fix coredump generation when cxl_get_fd() is used

2016-11-07 Thread Frederic Barrat
If a process dumps core while owning a cxl file descriptor obtained from an AFU driver (e.g. cxlflash) through the cxl_get_fd() API, the following error occurs: [ 868.027591] Unable to handle kernel paging request for data at address ... [ 868.027778] Faulting instruction address:

RE: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread Madalin-Cristian Bucur
> From: David Miller [mailto:da...@davemloft.net] > Sent: Monday, November 07, 2016 6:40 PM > > From: Madalin-Cristian Bucur > Date: Mon, 7 Nov 2016 16:32:16 + > > >> From: David Miller [mailto:da...@davemloft.net] > >> Sent: Monday, November 07, 2016 5:55 PM > >> >

Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread David Miller
From: Madalin-Cristian Bucur Date: Mon, 7 Nov 2016 16:32:16 + >> -Original Message- >> From: David Miller [mailto:da...@davemloft.net] >> Sent: Monday, November 07, 2016 5:55 PM >> >> From: Madalin-Cristian Bucur >> Date: Mon, 7 Nov

Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread Joakim Tjernlund
On Wed, 2016-11-02 at 22:17 +0200, Madalin Bucur wrote: > This introduces the Freescale Data Path Acceleration Architecture > (DPAA) Ethernet driver (dpaa_eth) that builds upon the DPAA QMan, > BMan, PAMU and FMan drivers to deliver Ethernet connectivity on > the Freescale DPAA QorIQ platforms.

RE: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread Madalin-Cristian Bucur
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Monday, November 07, 2016 5:55 PM > > From: Madalin-Cristian Bucur > Date: Mon, 7 Nov 2016 15:43:26 + > > >> From: David Miller [mailto:da...@davemloft.net] > >> Sent: Thursday,

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-07 Thread Paolo Bonzini
On 07/11/2016 17:25, Alex Williamson wrote: > On Mon, 07 Nov 2016 19:34:42 +1100 > Michael Ellerman wrote: > >> Paolo Bonzini writes: >>> On 04/11/2016 06:48, Michael Ellerman wrote: diff --git a/drivers/vfio/Kconfig b/drivers/vfio/Kconfig

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-07 Thread Alex Williamson
On Mon, 07 Nov 2016 19:34:42 +1100 Michael Ellerman wrote: > Paolo Bonzini writes: > > On 04/11/2016 06:48, Michael Ellerman wrote: > >> diff --git a/drivers/vfio/Kconfig b/drivers/vfio/Kconfig > >> index da6e2ce77495..6b51a4ebed8a 100644 > >> ---

Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread David Miller
From: Madalin-Cristian Bucur Date: Mon, 7 Nov 2016 15:43:26 + >> From: David Miller [mailto:da...@davemloft.net] >> Sent: Thursday, November 03, 2016 9:58 PM >> >> Why? By clearing this, you disallow an important fundamental way to do >> performane testing, via

RE: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-07 Thread Madalin-Cristian Bucur
> From: David Miller [mailto:da...@davemloft.net] > Sent: Thursday, November 03, 2016 9:58 PM > > From: Madalin Bucur > Date: Wed, 2 Nov 2016 22:17:26 +0200 > > > This introduces the Freescale Data Path Acceleration Architecture > > +static inline size_t

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-07 Thread Larry Finger
On 11/07/2016 03:18 AM, Paul Burton wrote: On Monday, 7 November 2016 19:27:32 GMT Michael Ellerman wrote: Paul Burton writes: If a device tree specified a preferred device for kernel console output via the stdout-path or linux,stdout-path chosen node properties

[PATCH V2] mm: move vma_is_anonymous check within pmd_move_must_withdraw

2016-11-07 Thread Aneesh Kumar K.V
Architectures like ppc64 want to use page table deposit/withraw even with huge pmd dax entries. Allow arch to override the vma_is_anonymous check by moving that to pmd_move_must_withdraw function Signed-off-by: Aneesh Kumar K.V --- * Changes from V1: Fix build

Re: [PATCH 1/2] mm: move vma_is_anonymous check within pmd_move_must_withdraw

2016-11-07 Thread Aneesh Kumar K.V
> https://github.com/0day-ci/linux/commits/Aneesh-Kumar-K-V/mm-move-vma_is_anonymous-check-within-pmd_move_must_withdraw/20161107-164033 > base: git://git.cmpxchg.org/linux-mmotm.git master > config: i386-randconfig-x006-201645 (attached as .config) > compiler: gcc-6 (

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

2016-11-07 Thread Dmitry Safonov
2016-10-27 20:09 GMT+03:00 Dmitry Safonov : > Changes since v1, v2: > - use vdso64_pages only under CONFIG_PPC64 (32-bit build fix) > - remove arch_vma_name helper as not needed anymore, > simplify vdso_base pointer initializing in map_vdso() > > Cleanup patches for vDSO

RE: Coding Style: Reverse XMAS tree declarations ?

2016-11-07 Thread David Laight
From: Lino Sanfilippo > Sent: 04 November 2016 20:07 ... > In this case it is IMHO rather the declaration + initialization that makes > "bar" hard to find at one glance, not the use of RXT. You could do something > like > > [longish list of reverse xmas tree identifiers...] > struct

Re: [PATCH] powerpc/mm: Add _PAGE_DEVMAP pte bit for ppc64.

2016-11-07 Thread Denis Kirjanov
On Monday, November 7, 2016, Aneesh Kumar K.V < aneesh.ku...@linux.vnet.ibm.com> wrote: > Signed-off-by: Aneesh Kumar K.V > Hi Aneesh, > --- > NOTE: We need other patches to enable pmem on ppc64. I am waiting > for dax pmd fault handling cleanup

Re: [Patch V6 2/6] irqchip: xilinx: Clean up irqdomain argument and read/write

2016-11-07 Thread Zubair Lutfullah Kakakhel
Hi, On 11/01/2016 11:05 AM, Zubair Lutfullah Kakakhel wrote: Hi, Thanks for the review. On 10/31/2016 07:51 PM, Thomas Gleixner wrote: On Mon, 31 Oct 2016, Zubair Lutfullah Kakakhel wrote: The drivers read/write function handling is a bit quirky. Can you please explain in more detail

Re: [PATCH v2 0/7] soc: renesas: Identify SoC and register with the SoC bus

2016-11-07 Thread Geert Uytterhoeven
On Mon, Oct 31, 2016 at 12:30 PM, Geert Uytterhoeven wrote: > Some Renesas SoCs may exist in different revisions, providing slightly > different functionalities (e.g. R-Car H3 ES1.x and ES2.0), and behavior > (errate and quirks). This needs to be catered for by drivers

Re: [PATCH 1/2] mm: move vma_is_anonymous check within pmd_move_must_withdraw

2016-11-07 Thread kbuild test robot
-check-within-pmd_move_must_withdraw/20161107-164033 base: git://git.cmpxchg.org/linux-mmotm.git master config: i386-randconfig-x006-201645 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=i386

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-07 Thread Paul Burton
On Monday, 7 November 2016 19:27:32 GMT Michael Ellerman wrote: > Paul Burton writes: > > If a device tree specified a preferred device for kernel console output > > via the stdout-path or linux,stdout-path chosen node properties there's > > no guarantee that it will have

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

2016-11-07 Thread Viresh Kumar
On 07-11-16, 13:09, Akshay Adiga wrote: > 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 > --- > Changes from v1 : > - Removed

Re: [PATCH 1/2] mm: move vma_is_anonymous check within pmd_move_must_withdraw

2016-11-07 Thread Hillf Danton
On Monday, November 07, 2016 4:35 PM Aneesh Kumar K.V > > Architectures like ppc64 want to use page table deposit/withraw > even with huge pmd dax entries. Allow arch to override the > vma_is_anonymous check by moving that to pmd_move_must_withdraw > function > > Signed-off-by: Aneesh Kumar K.V

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

2016-11-07 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]

[PATCH] powerpc/mm: Add _PAGE_DEVMAP pte bit for ppc64.

2016-11-07 Thread Aneesh Kumar K.V
Signed-off-by: Aneesh Kumar K.V --- NOTE: We need other patches to enable pmem on ppc64. I am waiting for dax pmd fault handling cleanup to hit upstream before I send the changes. Meanwhile reserve the pte bit. arch/powerpc/include/asm/book3s/64/pgtable.h | 24

[PATCH 2/2] mm: THP page cache support for ppc64

2016-11-07 Thread Aneesh Kumar K.V
Add arch specific callback in the generic THP page cache code that will deposit and withdarw preallocated page table. Archs like ppc64 use this preallocated table to store the hash pte slot information. Testing: kernel build of the patch series on tmpfs mounted with option huge=always The

[PATCH 1/2] mm: move vma_is_anonymous check within pmd_move_must_withdraw

2016-11-07 Thread Aneesh Kumar K.V
Architectures like ppc64 want to use page table deposit/withraw even with huge pmd dax entries. Allow arch to override the vma_is_anonymous check by moving that to pmd_move_must_withdraw function Signed-off-by: Aneesh Kumar K.V --- include/asm-generic/pgtable.h

Re: [PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-07 Thread Michael Ellerman
Paolo Bonzini writes: > On 04/11/2016 06:48, Michael Ellerman wrote: >> diff --git a/drivers/vfio/Kconfig b/drivers/vfio/Kconfig >> index da6e2ce77495..6b51a4ebed8a 100644 >> --- a/drivers/vfio/Kconfig >> +++ b/drivers/vfio/Kconfig >> @@ -6,12 +6,12 @@ config VFIO_IOMMU_TYPE1

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-07 Thread Michael Ellerman
Paul Burton writes: > If a device tree specified a preferred device for kernel console output > via the stdout-path or linux,stdout-path chosen node properties there's > no guarantee that it will have specified a device for which we have a > driver. It may also be the

Re: Coding Style: Reverse XMAS tree declarations ? (was Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet)

2016-11-07 Thread Michael Ellerman
Joe Perches writes: > On Thu, 2016-11-03 at 15:58 -0400, David Miller wrote: >> From: Madalin Bucur >> Date: Wed, 2 Nov 2016 22:17:26 +0200 >> >> > This introduces the Freescale Data Path Acceleration Architecture >> > +static inline size_t

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

2016-11-07 Thread Denis Kirjanov
On 11/7/16, Michael Ellerman wrote: > Denis Kirjanov writes: > >> [ 67.700897] BUG: using smp_processor_id() in preemptible [] >> code: cat/7343 >> [ 67.700988] caller is .powernv_cpufreq_throttle_check+0x2c/0x710 >> [

Re: [RFC v2 5/7] powerpc: Rename context.vdso_base to context.vdso

2016-11-07 Thread Michael Ellerman
Will Deacon writes: > On Fri, Nov 04, 2016 at 03:58:22PM +1100, Michael Ellerman wrote: >> Christopher Covington writes: >> > arch/powerpc/include/asm/book3s/32/mmu-hash.h | 2 +- >> > arch/powerpc/include/asm/book3s/64/mmu.h | 2 +- >> >