Re: [PATCH] powerpc: mpc512x: drop bogus and unused psc register bit definitions

2015-09-17 Thread Uwe Kleine-König
Hello, On Tue, Jul 21, 2015 at 09:21:13AM +0200, Uwe Kleine-König wrote: > These were introduced in commit 25ae3a0739c6 ("[POWERPC] mpc512x: Add > MPC512x PSC support to MPC52xx psc driver") and never used. Moreover > according to the datasheet[1] MEMERROR is bit 25 (0x40) and ORERR is > bit 27 (0

[PATCH 0/2] VFIO: Accept IOMMU group (PE) ID

2015-09-17 Thread Gavin Shan
This allows to accept IOMMU group (PE) ID from the parameter from userland when handling EEH operation so that the operation only affects the target IOMMU group (PE). If the IOMMU group (PE) ID in the parameter from userland is invalid, all IOMMU groups (PEs) attached to the specified container are

[PATCH 2/2] drivers/vfio: Support IOMMU group for EEH operations

2015-09-17 Thread Gavin Shan
Currently, EEH module works based on the assumption that every container has only one attached IOMMU group. It's not true any more. So the userland has to specify the IOMMU group (PE) to which the requested EEH operation is applied. This exposes "v2" interface for the userland to specify IOMMU gro

[PATCH 1/2] drivers/vfio: Support EEH API revision

2015-09-17 Thread Gavin Shan
This extends the return value from container's IOCTL command (VFIO_CHECK_EXTENSION + VFIO_EEH) to EEH API revision. Also, extra check is applied to return -ENOTTY if EEH functionality is disabled in vfio_spapr_iommu_eeh_ioctl(). Signed-off-by: Gavin Shan --- drivers/vfio/vfio_iommu_spapr_tce.c |

Re: [PATCH] PowerPC: Move Freescale device tree files into fsl folder

2015-09-17 Thread Scott Wood
On Fri, 2015-09-18 at 12:00 +0800, Jia Hongtao wrote: > It makes no sense that some Freescale device tree files are in fsl > directory while some others not. This patch move Freescale device tree > files into fsl folder. To do that the following two steps are made: > - Move Freescale device tree fi

[PATCH] PowerPC: Move Freescale device tree files into fsl folder

2015-09-17 Thread Jia Hongtao
It makes no sense that some Freescale device tree files are in fsl directory while some others not. This patch move Freescale device tree files into fsl folder. To do that the following two steps are made: - Move Freescale device tree files into fsl folder. - Update the include path in these files

RE: [PATCH] PowerPC: Move Freescale device tree files into fsl folder

2015-09-17 Thread Hongtao Jia
Typo in message. This patch was superseded. Please refer to the new one I just sent. Thanks. --- Best Regards, Hongtao > -Original Message- > From: Jia Hongtao [mailto:hongtao@freescale.com] > Sent: Friday, September 18, 2015 11:52 AM > To: Wood Scott-B07421 > Cc: linuxppc-dev@list

[PATCH] PowerPC: Move Freescale device tree files into fsl folder

2015-09-17 Thread Jia Hongtao
It makes no sense that some Freescale device tree files are in fsl directory while some others not. This patch move Freescale device tree files into fsl folder. To do that the following two steps are made: - Move Freescale device tree files into fsl folder. - Update the include path in these files

Re: [PATCH v9 3/5] qe_common: add qe_muram_ functions to manage muram

2015-09-17 Thread Scott Wood
On Thu, 2015-09-17 at 21:35 -0500, Zhao Qiang-B45475 wrote: > On Mon, 2015-09-18 at 04:28 +0800, Wood Scott-B07421 wrote: > > -Original Message- > > From: Wood Scott-B07421 > > Sent: Friday, September 18, 2015 4:28 AM > > To: Zhao Qiang-B45475 > > Cc: linux-ker...@vger.kernel.org; linuxppc-

RE: [PATCH v9 3/5] qe_common: add qe_muram_ functions to manage muram

2015-09-17 Thread Zhao Qiang
On Mon, 2015-09-18 at 04:28 +0800, Wood Scott-B07421 wrote: > -Original Message- > From: Wood Scott-B07421 > Sent: Friday, September 18, 2015 4:28 AM > To: Zhao Qiang-B45475 > Cc: linux-ker...@vger.kernel.org; linuxppc-dev@lists.ozlabs.org; > lau...@codeaurora.org; Xie Xiaobo-R63061; b...@k

Re: [PATCH v2 07/30] cxlflash: Fix context encode mask width

2015-09-17 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v2 06/30] cxlflash: Fix to avoid sizeof(bool)

2015-09-17 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v2 05/30] cxlflash: Fix data corruption when vLUN used over multiple cards

2015-09-17 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v2 04/30] cxlflash: Fix potential oops following LUN removal

2015-09-17 Thread Brian King
On 09/16/2015 04:27 PM, Matthew R. Ochs wrote: > When a LUN is removed, the sdev that is associated with the LUN > remains intact until its reference count drops to 0. In order > to prevent an sdev from being removed while a context is still > associated with it, obtain an additional reference per-

Re: [PATCH v2 03/30] cxlflash: Fix read capacity timeout

2015-09-17 Thread Brian King
On 09/16/2015 04:26 PM, Matthew R. Ochs wrote: > @@ -296,7 +296,7 @@ static int read_cap16(struct scsi_device *sdev, struct > llun_info *lli) > int rc = 0; > int result = 0; > int retry_cnt = 0; > - u32 tout = (MC_DISCOVERY_TIMEOUT * HZ); > + u32 to = (CMD_TIMEOUT * HZ);

Re: [PATCH v2 02/30] cxlflash: Replace magic numbers with literals

2015-09-17 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v2 01/30] cxlflash: Fix to avoid invalid port_sel value

2015-09-17 Thread Brian King
Reviewed-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH] powerpc: Kconfig: remove BE-only platforms from LE kernel build

2015-09-17 Thread Boqun Feng
Ping ;-) Regards, Boqun On Mon, Sep 07, 2015 at 07:58:00AM +0800, Boqun Feng wrote: > Currently, little endian is only supported on powernv and pseries, > however, Kconfigs still allow us to include other platforms in a LE > kernel, this may result in space wasting or even build error if some > B

Re: [PATCH v9 3/5] qe_common: add qe_muram_ functions to manage muram

2015-09-17 Thread Scott Wood
On Mon, 2015-09-14 at 09:38 +0800, Zhao Qiang wrote: > diff --git a/arch/powerpc/sysdev/qe_lib/qe_common.c > b/arch/powerpc/sysdev/qe_lib/qe_common.c > new file mode 100644 > index 000..1213458 > --- /dev/null > +++ b/arch/powerpc/sysdev/qe_lib/qe_common.c > @@ -0,0 +1,242 @@ > +/* > + * Frees

Re: [PATCH v9 2/5] genalloc:support allocating specific region

2015-09-17 Thread Scott Wood
On Thu, 2015-09-17 at 15:19 -0500, Scott Wood wrote: > On Mon, 2015-09-14 at 09:38 +0800, Zhao Qiang wrote: > > Add new algo for genalloc, it reserve a specific region of > > memory matching the size requirement (no alignment constraint) > > > > Signed-off-by: Zhao Qiang > > --- > > Changes for v

Re: [PATCH v9 2/5] genalloc:support allocating specific region

2015-09-17 Thread Scott Wood
On Mon, 2015-09-14 at 09:38 +0800, Zhao Qiang wrote: > Add new algo for genalloc, it reserve a specific region of > memory matching the size requirement (no alignment constraint) > > Signed-off-by: Zhao Qiang > --- > Changes for v9: > - reserve a specific region, if the return region >

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Segher Boessenkool
On Thu, Sep 17, 2015 at 09:47:39AM +, David Laight wrote: > > It also says "tftp", which is intriguing if nothing else :-) > > Much more likely than the above instruction. > If the address it wass read from is in the dump, look at the entire string. And tell us what network drivers you use, w

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Segher Boessenkool
On Thu, Sep 17, 2015 at 04:38:52PM +0200, Thomas Gleixner wrote: > [2.398263] Instruction dump: > [2.401297] 0100 00037000 f000 0001 0a641e09 > acde4823 > [2.409237] 000f 179a7b00 07de2900 03ef1480 <01f78a40> 0001c200 > 6000 9421fff0 Those are

Re: Time to remove platforms/cell?

2015-09-17 Thread Geoff Levand
On Thu, 2015-09-17 at 20:05 +1000, Michael Ellerman wrote: > Right. I wouldn't bet money that the oprofile code still works, but spufs > definitely would have to stay. oprofile on PS3 needed additional patches [1] that I never merged upstream, and no longer plan to, so oprofile support should be

[PATCH] powerpc: rackmeter: Fix module autoload for OF platform driver

2015-09-17 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt --- Hello, This patch adds the missing MODULE_DEVICE_TABLE() for OF to export that information so modules have the correct aliase

Re: [PATCH v4 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-09-17 Thread Bjorn Helgaas
On Thu, Sep 17, 2015 at 12:17 PM, Marc Zyngier wrote: > On 17/09/15 16:30, Bjorn Helgaas wrote: >> On Fri, Sep 04, 2015 at 05:50:07PM +0100, Marc Zyngier wrote: >>> The pci-host-generic driver parses the linux,pci-probe-only property, >>> and assumes that it will have a boolean parameter. >>> >>>

Re: [PATCH v4 0/4] PCI: arm64/powerpc: Fix parsing of linux, pci-probe-only

2015-09-17 Thread Marc Zyngier
On 17/09/15 16:30, Bjorn Helgaas wrote: > On Fri, Sep 04, 2015 at 05:50:07PM +0100, Marc Zyngier wrote: >> The pci-host-generic driver parses the linux,pci-probe-only property, >> and assumes that it will have a boolean parameter. >> >> Turns out that the Seattle DTS file has a naked "linux,pci-pro

Re: [PATCH v2 09/30] cxlflash: Fix to stop interrupt processing on remove

2015-09-17 Thread Matthew R. Ochs
> On Sep 17, 2015, at 7:38 AM, Tomas Henzl wrote: > > On 16.9.2015 18:53, Matthew R. Ochs wrote: >> Interrupt processing can run in parallel to a remove operation. This >> can lead to a condition where the interrupt handler is processing with >> memory that has been freed. >> >> To avoid process

Re: [PATCH v2 09/30] cxlflash: Fix to stop interrupt processing on remove

2015-09-17 Thread Matthew R. Ochs
> On Sep 17, 2015, at 6:58 AM, David Laight wrote: > > From: Linuxppc-dev Matthew R. Ochs >> Sent: 16 September 2015 22:28 >> Interrupt processing can run in parallel to a remove operation. This >> can lead to a condition where the interrupt handler is processing with >> memory that has been free

Re: [PATCH v4 0/4] PCI: arm64/powerpc: Fix parsing of linux,pci-probe-only

2015-09-17 Thread Bjorn Helgaas
On Fri, Sep 04, 2015 at 05:50:07PM +0100, Marc Zyngier wrote: > The pci-host-generic driver parses the linux,pci-probe-only property, > and assumes that it will have a boolean parameter. > > Turns out that the Seattle DTS file has a naked "linux,pci-probe-only" > property, which leads to the drive

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Thomas Gleixner
On Thu, 17 Sep 2015, Steven Rostedt wrote: > On Thu, 17 Sep 2015 16:38:52 +0200 (CEST) > Thomas Gleixner wrote: > > > On Thu, 17 Sep 2015, Steven Rostedt wrote: > > > > > On Thu, 17 Sep 2015 12:13:15 +0200 (CEST) > > > Thomas Gleixner wrote: > > > > > > > Digging deeper. My assumption that it

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Steven Rostedt
On Thu, 17 Sep 2015 16:38:52 +0200 (CEST) Thomas Gleixner wrote: > On Thu, 17 Sep 2015, Steven Rostedt wrote: > > > On Thu, 17 Sep 2015 12:13:15 +0200 (CEST) > > Thomas Gleixner wrote: > > > > > Digging deeper. My assumption that it's a post powerpc merge failure > > > turned out to be wrong.

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Thomas Gleixner
On Thu, 17 Sep 2015, Steven Rostedt wrote: > On Thu, 17 Sep 2015 12:13:15 +0200 (CEST) > Thomas Gleixner wrote: > > > Digging deeper. My assumption that it's a post powerpc merge failure > > turned out to be wrong. > > Does 4.2 have the problem? No. Neither does 4c92b5bb1422: Merge branch 'p

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Steven Rostedt
On Thu, 17 Sep 2015 12:13:15 +0200 (CEST) Thomas Gleixner wrote: > Digging deeper. My assumption that it's a post powerpc merge failure > turned out to be wrong. Does 4.2 have the problem? > > Some more data points. I see the above splat with > > CONFIG_FUNCTION_TRACER=y > CONFIG_FUNCTION_GRA

Re: [PATCH v2 09/30] cxlflash: Fix to stop interrupt processing on remove

2015-09-17 Thread Tomas Henzl
On 16.9.2015 18:53, Matthew R. Ochs wrote: > Interrupt processing can run in parallel to a remove operation. This > can lead to a condition where the interrupt handler is processing with > memory that has been freed. > > To avoid processing an interrupt while memory may be yanked, check for >

RE: [PATCH v2 09/30] cxlflash: Fix to stop interrupt processing on remove

2015-09-17 Thread David Laight
From: Linuxppc-dev Matthew R. Ochs > Sent: 16 September 2015 22:28 > Interrupt processing can run in parallel to a remove operation. This > can lead to a condition where the interrupt handler is processing with > memory that has been freed. > > To avoid processing an interrupt while memory may be

Re: Time to remove platforms/cell?

2015-09-17 Thread Arnd Bergmann
On Thursday 17 September 2015 12:28:13 Marc Dietrich wrote: > Am Donnerstag, 17. September 2015, 19:45:38 schrieb Michael Ellerman: > > On Thu, 2015-09-17 at 10:43 +0200, Marc Dietrich wrote: > > > Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > > > > [...] > > > > Having sa

Re: Time to remove platforms/cell?

2015-09-17 Thread Arnd Bergmann
On Thursday 17 September 2015 20:05:25 Michael Ellerman wrote: > On Thu, 2015-09-17 at 11:31 +0200, Arnd Bergmann wrote: > > b) Support for IBM blades: > >It is unlikely that there are QS20 blades still around and being used > >at all, but there is very little code specific to them. > > Wa

Re: Time to remove platforms/cell?

2015-09-17 Thread Marc Dietrich
Am Donnerstag, 17. September 2015, 19:45:38 schrieb Michael Ellerman: > On Thu, 2015-09-17 at 10:43 +0200, Marc Dietrich wrote: > > Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > > [...] > > Having said all that, this email was mainly a fishing expedition to see if > anyone

Re: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread Thomas Gleixner
On Wed, 16 Sep 2015, Steven Rostedt wrote: > On Wed, 16 Sep 2015 22:01:06 +0200 (CEST) > Thomas Gleixner wrote: > > So now I have to chase down that one: > > > > [0.230210] ftrace: allocating 16560 entries in 49 pages > > [0.273313] [ cut here ] > > [0.278048]

Re: Time to remove platforms/cell?

2015-09-17 Thread Michael Ellerman
On Thu, 2015-09-17 at 11:31 +0200, Arnd Bergmann wrote: > On Thursday 17 September 2015 10:43:39 Marc Dietrich wrote: > > Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > > > Discuss ... > > > > as long as Geoff still maintains the ps3 port - why? > > > We already removed c

Re: [PATCH] arch-powerpc: Return false instead of -EFAULT

2015-09-17 Thread Peter Senna Tschudin
On Thu, Sep 17, 2015 at 11:47 AM, Michael Ellerman wrote: > On Thu, 2015-09-17 at 11:18 +0200, Peter Senna Tschudin wrote: >> Returning a negative value for a boolean function seem to have the >> undesired effect of returning true. Replace -EINVAL by false in a >> bool-returning function. >> >> Th

RE: [BUG] Revert 0b05e2d671c4 'powerpc/32: cacheable_memcpy becomes memcpy'

2015-09-17 Thread David Laight
From: Segher Boessenkool > Sent: 17 September 2015 04:19 > On Thu, Sep 17, 2015 at 12:50:12PM +1000, Michael Ellerman wrote: > > On Wed, 2015-09-16 at 21:54 -0400, Steven Rostedt wrote: > > > This could be a symptom and not the problem. What the above shows is > > > that ftrace tried to convert the

Re: [PATCH] arch-powerpc: Return false instead of -EFAULT

2015-09-17 Thread Michael Ellerman
On Thu, 2015-09-17 at 11:18 +0200, Peter Senna Tschudin wrote: > Returning a negative value for a boolean function seem to have the > undesired effect of returning true. Replace -EINVAL by false in a > bool-returning function. > > The diff of the .s file before and after the change (using cross >

Re: Time to remove platforms/cell?

2015-09-17 Thread Michael Ellerman
On Thu, 2015-09-17 at 10:43 +0200, Marc Dietrich wrote: > Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > > Discuss ... > > as long as Geoff still maintains the ps3 port - why? Because even if Geoff maintains the ps3 port, there's still a non-zero cost to us for carrying t

Re: Time to remove platforms/cell?

2015-09-17 Thread Arnd Bergmann
On Thursday 17 September 2015 10:43:39 Marc Dietrich wrote: > Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > > Discuss ... > > as long as Geoff still maintains the ps3 port - why? > We already removed celleb a while ago, which was arguably the least commonly used one. Wi

Re: [PATCH v3] powerpc: msi: mark bitmap with kmemleak_not_leak()

2015-09-17 Thread Catalin Marinas
On Wed, Sep 16, 2015 at 10:26:14PM +0300, Denis Kirjanov wrote: > During the MSI bitmap test on boot kmemleak spews the following trace: > > unreferenced object 0xc0016e86c900 (size 64): > comm "swapper/0", pid 1, jiffies 4294893173 (age 518.024s) > hex dump (first 32 bytes): > 0

[PATCH] arch-powerpc: Return false instead of -EFAULT

2015-09-17 Thread Peter Senna Tschudin
Returning a negative value for a boolean function seem to have the undesired effect of returning true. Replace -EINVAL by false in a bool-returning function. The diff of the .s file before and after the change (using cross compilation) starts with: 440,441c440,441 < .L43: < li 3,1 # D.257

Re: Time to remove platforms/cell?

2015-09-17 Thread Marc Dietrich
Am Donnerstag, 17. September 2015, 15:28:47 schrieb Michael Ellerman: > Discuss ... as long as Geoff still maintains the ps3 port - why? Marc signature.asc Description: This is a digitally signed message part. ___ Linuxppc-dev mailing list Linuxppc-de