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

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 >>

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

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

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 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. > >

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 >

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

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

[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 #

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): >

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.

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

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: [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: > > > > >

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

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

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

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: [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

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 >>

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

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. >>

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

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,

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,

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: [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 @@ > +/* > + *

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

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

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 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 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 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 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

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 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;

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;

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 >

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:

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

[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

[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