Re: [PATCH v2 4/5] powerpc/fadump: Make ELF eflags depend on endian

2016-09-08 Thread Mahesh Jagannath Salgaonkar
On 09/06/2016 11:02 AM, Daniel Axtens wrote: > Firmware Assisted Dump is a facility to dump kernel core with assistance > from firmware. As part of this process the kernel ELF version is > stored. > > Currently, fadump.h defines this to 0 if it is not already defined. This > clashes with a define

RE: [PATCH V2 7/7] thermal: qoriq: Add thermal management support

2016-09-08 Thread Troy Jia
> -Original Message- > From: Zhang Rui [mailto:rui.zh...@intel.com] > Sent: Friday, August 19, 2016 8:40 PM > To: Hongtao Jia ; edubez...@gmail.com; > robh...@kernel.org; ga...@codeaurora.org; Scott Wood > ; shawn...@kernel.org > Cc: linux...@vger.kernel.org; devicet...@vger.kernel.org; l

Re: [PATCH v2 1/5] powerpc/cell: drop unused iic_get_irq_host()

2016-09-08 Thread Arnd Bergmann
On Tuesday, September 6, 2016 3:32:39 PM CEST Daniel Axtens wrote: > Sparse checking revealed that it is no longer used. > There is an EXPORT_SYMBOL_GPL, but there's no header that > provides a prototype, so nothing should be using it anyway. > > Remove it. > > Signed-off-by: Daniel Axtens > A

Re: [PATCH v2 4/5] powerpc/fadump: Make ELF eflags depend on endian

2016-09-08 Thread Mahesh Jagannath Salgaonkar
On 09/08/2016 12:30 PM, Mahesh Jagannath Salgaonkar wrote: > On 09/06/2016 11:02 AM, Daniel Axtens wrote: >> Firmware Assisted Dump is a facility to dump kernel core with assistance >> from firmware. As part of this process the kernel ELF version is >> stored. >> >> Currently, fadump.h defines thi

Re: [PATCH] powerpc: Remove all usages of NO_IRQ

2016-09-08 Thread Michael Ellerman
Arnd Bergmann writes: > On Tuesday, September 6, 2016 9:53:24 PM CEST Michael Ellerman wrote: >> drivers/macintosh/macio_asic.c | 4 ++-- >> drivers/macintosh/rack-meter.c | 2 +- >> drivers/macintosh/smu.c | 18 +- >>

Re: [PATCH v2 4/5] powerpc/fadump: Make ELF eflags depend on endian

2016-09-08 Thread Michael Ellerman
Mahesh Jagannath Salgaonkar writes: > On 09/06/2016 11:02 AM, Daniel Axtens wrote: >> diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c >> index 0638b82ce294..457f08e544c6 100644 >> --- a/arch/powerpc/kernel/fadump.c >> +++ b/arch/powerpc/kernel/fadump.c >> @@ -779,7 +779,1

Re: [PATCH] powerpc: Remove all usages of NO_IRQ

2016-09-08 Thread Arnd Bergmann
On Thursday, September 8, 2016 7:16:12 PM CEST Michael Ellerman wrote: > > The rest below I haven't looked at yet, I'm not entirely clear on which > FSL pieces are able to build on arm. > > > drivers/dma/bestcomm/bestcomm.c: if (tsk->irq != NO_IRQ) > > drivers/dma/fsl_raid.c: i

Re: [PATCH v2 4/5] powerpc/fadump: Make ELF eflags depend on endian

2016-09-08 Thread Michael Ellerman
Mahesh Jagannath Salgaonkar writes: > On 09/06/2016 11:02 AM, Daniel Axtens wrote: >> Firmware Assisted Dump is a facility to dump kernel core with assistance >> from firmware. As part of this process the kernel ELF version is >> stored. >> >> Currently, fadump.h defines this to 0 if it is not

Re: [1/3] powerpc/mm: Don't alias user region to other regions below PAGE_OFFSET

2016-09-08 Thread Michael Ellerman
On Fri, 2016-02-09 at 11:47:59 UTC, Paul Mackerras wrote: > In commit c60ac5693c47 ("powerpc: Update kernel VSID range", 2013-03-13) > we lost a check on the region number (the top four bits of the effective > address) for addresses below PAGE_OFFSET. That commit replaced a check > that the top 18

Re: powerpc/32: fix again csum_partial_copy_generic()

2016-09-08 Thread Michael Ellerman
On Fri, 2016-26-08 at 14:45:13 UTC, Christophe Leroy wrote: > commit 7aef4136566b0 ("powerpc32: rewrite csum_partial_copy_generic() > based on copy_tofrom_user()") introduced a bug when destination > address is odd and len is lower than cacheline size. > > In that case the resulting csum value doe

Re: powerpc/powernv: Fix corrupted PE allocation bitmap on releasing PE

2016-09-08 Thread Michael Ellerman
On Tue, 2016-06-09 at 04:17:18 UTC, Gavin Shan wrote: > In pnv_ioda_free_pe(), the PE object (including the associated PE > number) is cleared before resetting the corresponding bit in the > PE allocation bitmap. It means PE#0 is always released to the bitmap > wrongly. > > This fixes above issue

Re: powerpc/powernv: Fix crash on releasing compound PE

2016-09-08 Thread Michael Ellerman
On Tue, 2016-06-09 at 04:16:44 UTC, Gavin Shan wrote: > The compound PE is created to accommodate the devices attached to > one specific PCI bus that consume multiple M64 segments. The compound > PE is made up of one master PE and possibly multiple slave PEs. The > slave PEs should be destroyed whe

Re: powerpc/xics/opal: Fix processor numbers in OPAL ICP

2016-09-08 Thread Michael Ellerman
On Tue, 2016-06-09 at 03:43:45 UTC, Benjamin Herrenschmidt wrote: > When using the OPAL ICP backend we incorrectly pass Linux CPU numbers > rather than HW CPU numbers to OPAL. > > Signed-off-by: Benjamin Herrenschmidt > Fixes: d74361881f0dfe5f9dcac37d1b753a15a2345d8c Applied to powerpc fixes, t

Re: powerpc: Fix build of 64 bit little endian with CONFIG_KEXEC=n

2016-09-08 Thread Michael Ellerman
On Tue, 2016-23-08 at 22:57:39 UTC, Thiago Jung Bauermann wrote: > On ppc64le, builds with CONFIG_KEXEC=n fail with: > > arch/powerpc/platforms/pseries/setup.c: In function > ‘pseries_big_endian_exceptions’: > arch/powerpc/platforms/pseries/setup.c:403:13: error: implicit > declaration of fun

Re: [PATCH v2 3/3] powerpc/mm: Speed up computation of base and actual page size for a HPTE

2016-09-08 Thread Paul Mackerras
On Wed, Sep 07, 2016 at 04:17:09PM +1000, Paul Mackerras wrote: > This replaces a 2-D search through an array with a simple 8-bit table > lookup for determining the actual and/or base page size for a HPT entry. > > The encoding in the second doubleword of the HPTE is designed to encode > the actua

Re: [PATCH v2 3/3] powerpc/mm: Speed up computation of base and actual page size for a HPTE

2016-09-08 Thread Paolo Bonzini
On 08/09/2016 12:08, Paul Mackerras wrote: >> > >> > arch/powerpc/include/asm/book3s/64/mmu-hash.h | 37 >> > arch/powerpc/include/asm/kvm_book3s_64.h | 87 >> > +++ >> > arch/powerpc/include/asm/mmu.h| 1 + >> > arch/powerpc/mm/hash_n

[PATCH -next] scsi: ibmvfc: fix typo in parameter description

2016-09-08 Thread Wei Yongjun
From: Wei Yongjun Fix typo in parameter description. Signed-off-by: Wei Yongjun --- drivers/scsi/ibmvscsi/ibmvfc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c index 6b92169..56b9f45 100644 --- a/drivers/scs

Re: Kexec regression in next-20160906

2016-09-08 Thread Thiago Jung Bauermann
Am Mittwoch, 07 September 2016, 09:08:07 schrieb Russell King - ARM Linux: > Any change to a UAPI header needs to be carefully considered and > questioned as it is always a potential userspace breakage - and in > the kernel, we're supposed to be doing our up-most to avoid > breaking userspace. > >

Re: [PATCH 0/3] OPTPROBES for powerpc

2016-09-08 Thread Masami Hiramatsu
Hi Anju, On Wed, 7 Sep 2016 15:03:09 +0530 Anju T Sudhakar wrote: > This is the patchset of the kprobes jump optimization > (a.k.a OPTPROBES)for powerpc. Kprobe being an inevitable tool > for kernel developers, enhancing the performance of kprobe has > got much importance. > > Currently kprobe

[PATCH] atomic64: No need for CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE

2016-09-08 Thread Vineet Gupta
This came to light when implementing native 64-bit atomics for ARCv2. The atomic64 self-test code uses CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE to check whether atomic64_dec_if_positive() is available. It seems it was needed when not every arch defined it. However as of current code the Kconfig op

Re: [PATCH 2/3] arch/powerpc : optprobes for powerpc core

2016-09-08 Thread Masami Hiramatsu
On Wed, 7 Sep 2016 15:03:11 +0530 Anju T Sudhakar wrote: > Instructions which can be emulated are suppliants for optimization. > Before optimization ensure that the address range between the detour > buffer allocated and the instruction being probed is within ± 32MB. > > Signed-off-by: Anju T S

Re: [PATCH] atomic64: No need for CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE

2016-09-08 Thread Peter Zijlstra
On Thu, Sep 08, 2016 at 09:28:18AM -0700, Vineet Gupta wrote: > This came to light when implementing native 64-bit atomics for ARCv2. > > The atomic64 self-test code uses CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE > to check whether atomic64_dec_if_positive() is available. > It seems it was needed w

Re: [PATCH] atomic64: No need for CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE

2016-09-08 Thread Vineet Gupta
On 09/08/2016 11:19 AM, Peter Zijlstra wrote: > On Thu, Sep 08, 2016 at 09:28:18AM -0700, Vineet Gupta wrote: >> This came to light when implementing native 64-bit atomics for ARCv2. >> >> The atomic64 self-test code uses CONFIG_ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE >> to check whether atomic64_dec_if_

Re: [PATCH v4 0/5] kexec_file: Add buffer hand-over for the next kernel

2016-09-08 Thread Thiago Jung Bauermann
Am Mittwoch, 07 September 2016, 09:19:40 schrieb Eric W. Biederman: > ebied...@xmission.com (Eric W. Biederman) writes: > > Thiago Jung Bauermann writes: > >> Hello, > >> > >> The purpose of this new version of the series is to fix a small issue > >> that I found, which is that the kernel doesn't

Re: [PATCH -next] scsi: ibmvfc: fix typo in parameter description

2016-09-08 Thread Tyrel Datwyler
On 09/08/2016 08:04 AM, Wei Yongjun wrote: > From: Wei Yongjun > > Fix typo in parameter description. > > Signed-off-by: Wei Yongjun Reviewed-by: Tyrel Datwyler

Re: [PATCH v2 1/5] powerpc/cell: drop unused iic_get_irq_host()

2016-09-08 Thread Daniel Axtens
Hi Arnd, > I'm currently experimenting with a patch that adds -Wmissing-declaration to > the normal kernel CFLAGS for everyone, and fixing up the warnings I get with > that on ARM. > > Are you looking at all the powerpc specific code? If we can catch all the > existing warnings for some of the maj

Re: [PATCH 01/13] perf/core: Add perf_arch_regs and mask to perf_regs structure

2016-09-08 Thread Madhavan Srinivasan
On Tuesday 06 September 2016 02:40 PM, Peter Zijlstra wrote: On Tue, Sep 06, 2016 at 09:55:43AM +0530, Madhavan Srinivasan wrote: On Thursday 01 September 2016 12:56 PM, Peter Zijlstra wrote: On Mon, Aug 29, 2016 at 02:30:46AM +0530, Madhavan Srinivasan wrote: It's a perennial request from

Re: [PATCH v2 1/5] powerpc/cell: drop unused iic_get_irq_host()

2016-09-08 Thread Andrew Donnellan
On 09/09/16 10:43, Daniel Axtens wrote: Andrew Donellan was telling me there's a Coccinelle script that looks for things that are both static and EXPORTed - maybe that's worth following up on this point? https://github.com/coccinelle/coccinellery/blob/master/is_static/is_static.cocci -- Andrew

Re: Build regression introduced by 31cdd0c39c7544ced79da53aa0b7e989f3a39582

2016-09-08 Thread Michael Ellerman
Paul Mackerras writes: > On Tue, Sep 06, 2016 at 11:01:02PM +0200, John Paul Adrian Glaubitz wrote: >> Hi Paul! >> >> I'm referring to your change [1]: >> >> "powerpc/xmon: Fix SPR read/write commands and add command to dump SPRs" >> >> which introduced assembly code in the new file >> arch/p

Re: [v11, 5/8] soc: fsl: add GUTS driver for QorIQ platforms

2016-09-08 Thread Scott Wood
On Tue, 2016-09-06 at 16:28 +0800, Yangbo Lu wrote: > The global utilities block controls power management, I/O device > enabling, power-onreset(POR) configuration monitoring, alternate > function selection for multiplexed signals,and clock control. > > This patch adds a driver to manage and acces

[PATCH] powerpc/xmon: Don't use ld on 32-bit

2016-09-08 Thread Michael Ellerman
In commit 31cdd0c39c75 ("powerpc/xmon: Fix SPR read/write commands and add command to dump SPRs") I added two uses of the "ld" instruction in spr_access.S. "ld" is a 64-bit instruction, so shouldn't be used on 32-bit CPUs. Replace it with PPC_LL which is a macro that gives us either "ld" or "lwz"

Re: [PATCH v4 0/5] kexec_file: Add buffer hand-over for the next kernel

2016-09-08 Thread Eric W. Biederman
Thiago Jung Bauermann writes: > Am Mittwoch, 07 September 2016, 09:19:40 schrieb Eric W. Biederman: >> ebied...@xmission.com (Eric W. Biederman) writes: >> > Thiago Jung Bauermann writes: >> >> Hello, >> >> >> >> The purpose of this new version of the series is to fix a small issue >> >> that I