Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Madhavan Srinivasan
On Friday 06 November 2015 08:28 AM, Sukadev Bhattiprolu wrote: > Peter Zijlstra [pet...@infradead.org] wrote: > | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu > registers > | > and adds offsetof ma

Re: [PATCH V3 0/3] perf/powerpc:Add ability to sample intr machine state in powerpc

2015-11-05 Thread Denis Kirjanov
On 11/6/15, Anju T wrote: > Hi Denis, > > On Wednesday 04 November 2015 02:26 PM, Denis Kirjanov wrote: >> On 11/3/15, Anju T wrote: >>> This short patch series adds the ability to sample the interrupted >>> machine state for each hardware sample. >>> >>> To test this patchset, >>> Eg: >>> >>> $p

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Madhavan Srinivasan
On Thursday 05 November 2015 08:12 PM, Stephane Eranian wrote: > Hi, > > On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan > wrote: >> This patchset extend the perf sample regs infrastructure >> to include architecture specific regs. Current perf_sample_regs_intr >> exports only registers in t

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Madhavan Srinivasan
On Thursday 05 November 2015 06:37 PM, Peter Zijlstra wrote: > On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: >> Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers >> and adds offsetof macro for the same. It extends perf_reg_value() >> to use reg i

Re: [PATCH V3 0/3] perf/powerpc:Add ability to sample intr machine state in powerpc

2015-11-05 Thread Anju T
Hi Denis, On Wednesday 04 November 2015 02:26 PM, Denis Kirjanov wrote: On 11/3/15, Anju T wrote: This short patch series adds the ability to sample the interrupted machine state for each hardware sample. To test this patchset, Eg: $perf record -I ls // record machine state at interrupt $p

[GIT PULL] Please pull powerpc/linux.git powerpc-4.4-1 tag

2015-11-05 Thread Michael Ellerman
Hi Linus, Please pull powerpc updates for 4.4: The following changes since commit 9ffecb10283508260936b96022d4ee43a7798b4c: Linux 4.3-rc3 (2015-09-27 07:50:08 -0400) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git tags/powerpc-4.4-1

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Sukadev Bhattiprolu
Peter Zijlstra [pet...@infradead.org] wrote: | On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: | > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers | > and adds offsetof macro for the same. It extends perf_reg_value() | > to use reg idx to decide o

[PATCH] powerpc/xmon: Add 'dt' command to dump trace buffers

2015-11-05 Thread Michael Ellerman
There is a nice interface for asking ftrace to dump all its tracing buffers. The only down side for use in xmon is that it uses printk. Depending on circumstances printk may not work when in xmon, but it also may, so add a 'dt' command which dumps the ftrace buffers, and add a note to the help to m

Re: [PATCH v7 04/50] powerpc/powernv: Cleanup on pnv_pci_ioda_controller_ops

2015-11-05 Thread Gavin Shan
On Fri, Nov 06, 2015 at 09:28:20AM +1100, Daniel Axtens wrote: >Gavin Shan writes: > >> This cleans up on pnv_pci_ioda_controller_ops struct to use tab >> instead of space indent of statement to avoid complains from >> scripts/checkpatch.pl. No logical changes introduced. > >Oh, that was my code :

[PATCH] powerpc: platform: 85xx: mpc85xx ADS: remove pci exclude

2015-11-05 Thread Daniel Walker
This code was reworked in commit, 905e75c46dba5f3061049277e4eb7110beedba43 This change removed the fsl_add_bridge() which originally was above the addition of the pci_exclude_device function. I think the assumption was that the pci_exclude_device would prevent changes to the bridge PCI config aft

Re: [PATCH v7 10/50] powerpc/powernv: Simplify pnv_ioda_setup_pe_seg()

2015-11-05 Thread Gavin Shan
On Fri, Nov 06, 2015 at 09:56:06AM +1100, Daniel Axtens wrote: >Gavin Shan writes: > >> The original implementation of pnv_ioda_setup_pe_seg() configures >> IO and M32 segments by separate logics, which can be merged by >> by caching @segmap, @seg_size, @win in advance. This shouldn't >> cause any

Re: [PATCH v7 03/50] powerpc/pci: Cleanup on struct pci_controller_ops

2015-11-05 Thread Gavin Shan
On Fri, Nov 06, 2015 at 09:32:57AM +1100, Daniel Axtens wrote: >Gavin Shan writes: > >> Each PHB has one instance of "struct pci_controller_ops", which >> includes various callbacks called by PCI subsystem. In the definition >> of this struct, some callbacks have explicit names for its arguments,

Re: [PATCH v7 02/50] powerpc/pci: Override pcibios_setup_bridge()

2015-11-05 Thread Gavin Shan
On Fri, Nov 06, 2015 at 09:27:42AM +1100, Daniel Axtens wrote: >Gavin Shan writes: > >> >> +void pcibios_setup_bridge(struct pci_bus *bus, unsigned long type) >> +{ >> +struct pci_controller *hose = pci_bus_to_host(bus); >Very much a nit-pick, but I thought we were trying to move towards usi

Re: [PATCH] cxl: use correct operator when writing pcie config space values

2015-11-05 Thread Michael Ellerman
On Fri, 2015-11-06 at 10:05 +1100, Daniel Axtens wrote: > Andrew Donnellan writes: > > > When writing a value to config space, cxl_pcie_write_config() calls > > cxl_pcie_config_info() to obtain a mask and shift value, shifts the new > > value accordingly, then uses the mask to combine the shift

Re: powerpc/fsl-pci: Unify pci/pcie initialization code

2015-11-05 Thread Scott Wood
On Thu, 2015-11-05 at 15:16 -0800, Daniel Walker wrote: > On 11/05/2015 03:12 PM, Scott Wood wrote: > > On Thu, 2015-11-05 at 10:48 -0800, Daniel Walker wrote: > > > Hi, > > > > > > 905e75c46dba5f3061049277e4eb7110beedba43:arch/powerpc/platforms/85xx/mpc85xx_ads.c > > > > > > > > > > > > This c

Re: [PATCH] cxl: use correct operator when writing pcie config space values

2015-11-05 Thread Andrew Donnellan
On 06/11/15 10:05, Daniel Axtens wrote: Given that there are no current users of this function, does this need to go to stable? Does it actually fix a real (as opposed to theoretical) bug? I tagged it for stable on mpe's request - I'm not fussed either way. Andrew -- Andrew Donnellan

Re: [PATCH] cxl: use correct operator when writing pcie config space values

2015-11-05 Thread Michael Neuling
On Fri, 2015-11-06 at 10:05 +1100, Daniel Axtens wrote: > Andrew Donnellan writes: > > > When writing a value to config space, cxl_pcie_write_config() calls > > cxl_pcie_config_info() to obtain a mask and shift value, shifts the > > new > > value accordingly, then uses the mask to combine the shi

Re: powerpc/fsl-pci: Unify pci/pcie initialization code

2015-11-05 Thread Daniel Walker
On 11/05/2015 03:12 PM, Scott Wood wrote: On Thu, 2015-11-05 at 10:48 -0800, Daniel Walker wrote: Hi, 905e75c46dba5f3061049277e4eb7110beedba43:arch/powerpc/platforms/85xx/mpc85xx_ads.c This commit seems to cause problems with PCI devices doing DMA, just with mpc85xx ads. If you do the "fsl_ad

Re: powerpc/fsl-pci: Unify pci/pcie initialization code

2015-11-05 Thread Scott Wood
On Thu, 2015-11-05 at 10:48 -0800, Daniel Walker wrote: > Hi, > > 905e75c46dba5f3061049277e4eb7110beedba43:arch/powerpc/platforms/85xx/mpc85xx_ads.c > > > This commit seems to cause problems with PCI devices doing DMA, just > with mpc85xx ads. If you do the "fsl_add_bridge()" after you setup th

Re: [PATCH] cxl: use correct operator when writing pcie config space values

2015-11-05 Thread Daniel Axtens
Andrew Donnellan writes: > When writing a value to config space, cxl_pcie_write_config() calls > cxl_pcie_config_info() to obtain a mask and shift value, shifts the new > value accordingly, then uses the mask to combine the shifted value with the > existing value at the address as part of a read-

Re: [PATCH v7 10/50] powerpc/powernv: Simplify pnv_ioda_setup_pe_seg()

2015-11-05 Thread Daniel Axtens
Gavin Shan writes: > The original implementation of pnv_ioda_setup_pe_seg() configures > IO and M32 segments by separate logics, which can be merged by > by caching @segmap, @seg_size, @win in advance. This shouldn't > cause any behavioural changes. > > Signed-off-by: Gavin Shan > --- > arch/po

Re: [PATCH v7 03/50] powerpc/pci: Cleanup on struct pci_controller_ops

2015-11-05 Thread Daniel Axtens
Gavin Shan writes: > Each PHB has one instance of "struct pci_controller_ops", which > includes various callbacks called by PCI subsystem. In the definition > of this struct, some callbacks have explicit names for its arguments, > but the left don't have. > > This adds all explicit names of the a

Re: [PATCH v7 04/50] powerpc/powernv: Cleanup on pnv_pci_ioda_controller_ops

2015-11-05 Thread Daniel Axtens
Gavin Shan writes: > This cleans up on pnv_pci_ioda_controller_ops struct to use tab > instead of space indent of statement to avoid complains from > scripts/checkpatch.pl. No logical changes introduced. Oh, that was my code :/ Sorry I missed that, thanks for cleaning it up! Reviewed-by: Daniel

Re: [PATCH v7 02/50] powerpc/pci: Override pcibios_setup_bridge()

2015-11-05 Thread Daniel Axtens
Gavin Shan writes: > > +void pcibios_setup_bridge(struct pci_bus *bus, unsigned long type) > +{ > + struct pci_controller *hose = pci_bus_to_host(bus); Very much a nit-pick, but I thought we were trying to move towards using phb instead of hose in new code? Apart from that this looks good.

Re: [PATCH 1/1 v4] drivers/nvme: default to 4k device page size

2015-11-05 Thread Nishanth Aravamudan
On 05.11.2015 [11:58:39 -0800], Christoph Hellwig wrote: > Looks fine, > > Reviewed-by: Christoph Hellwig > > ... but I doubt we'll ever bother updating it. Most architectures > with arger page sizes also have iommus and would need different settings > for different iommus vs direct mapping for

[PATCH] uapi elf: delete unused macros

2015-11-05 Thread mudongliang
delete unused macros and unify the comment format From e21328f3450e841abf8a312865910806d0858373 Mon Sep 17 00:00:00 2001 From: mudongliang Date: Thu, 5 Nov 2015 22:46:19 +0800 Subject: [PATCH] unify the comment form and delete unused macros Signed-off-by: mudongliang --- arch/powerpc/boot/elf.

[PATCH] uapi elf: delete unused macros

2015-11-05 Thread 慕冬亮
delete unused macros and unify the comment format >From e21328f3450e841abf8a312865910806d0858373 Mon Sep 17 00:00:00 2001 From: mudongliang Date: Thu, 5 Nov 2015 22:46:19 +0800 Subject: [PATCH] unify the comment form and delete unused macros Signed-off-by: mudongliang --- arch/powerpc/boot/elf

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Stephane Eranian
Hi, On Wed, Nov 4, 2015 at 9:46 PM, Madhavan Srinivasan wrote: > > This patchset extend the perf sample regs infrastructure > to include architecture specific regs. Current perf_sample_regs_intr > exports only registers in the pt_regs to perf.data using > PERF_SAMPLE_REGS_INTR sample type. But so

Re: [PATCH 1/1 v4] drivers/nvme: default to 4k device page size

2015-11-05 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig ... but I doubt we'll ever bother updating it. Most architectures with arger page sizes also have iommus and would need different settings for different iommus vs direct mapping for very little gain. There's a reason why we never bothered for RDMA eit

Re: powerpc/fsl-pci: Unify pci/pcie initialization code

2015-11-05 Thread Daniel Walker
Hi, 905e75c46dba5f3061049277e4eb7110beedba43:arch/powerpc/platforms/85xx/mpc85xx_ads.c This commit seems to cause problems with PCI devices doing DMA, just with mpc85xx ads. If you do the "fsl_add_bridge()" after you setup the ppc_md.pci_exclude_device to mpc85xx_exclude_device, then the earl

Re: [PATCH] uapi elf: delete unused macros

2015-11-05 Thread kbuild test robot
Hi mudongliang, [auto build test ERROR on: powerpc/next] [also build test ERROR on: v4.3 next-20151105] url: https://github.com/0day-ci/linux/commits/mudongliang/uapi-elf-delete-unused-macros/20151105-234433 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next

[PATCH 1/1 v4] drivers/nvme: default to 4k device page size

2015-11-05 Thread Nishanth Aravamudan
On 03.11.2015 [13:46:25 +], Keith Busch wrote: > On Tue, Nov 03, 2015 at 05:18:24AM -0800, Christoph Hellwig wrote: > > On Fri, Oct 30, 2015 at 02:35:11PM -0700, Nishanth Aravamudan wrote: > > > diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c > > > index ccc0c1f93daa..a9a5285

Re: [PATCH] uapi elf: delete unused macros

2015-11-05 Thread kbuild test robot
Hi mudongliang, [auto build test ERROR on: powerpc/next] [also build test ERROR on: v4.3 next-20151105] url: https://github.com/0day-ci/linux/commits/mudongliang/uapi-elf-delete-unused-macros/20151105-234433 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next

Re: [PATCH] uapi elf: delete unused macros

2015-11-05 Thread kbuild test robot
Hi mudongliang, [auto build test ERROR on: powerpc/next] [also build test ERROR on: v4.3 next-20151105] url: https://github.com/0day-ci/linux/commits/mudongliang/uapi-elf-delete-unused-macros/20151105-234433 base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next

[PATCH] powerpc/mpc85xx: Update B4 FMan MURAM size

2015-11-05 Thread igal.liberman
From: Igal Liberman FMan V3H has 2 different MURAM sizes: In B4860/4420 the MURAM size is 512KB. In T4240 and T2080 the MURAM size is 384KB. The MURAM size in FMan V3H device tree is 384KB. This patch updates the MURAM size for B4 to 512KB. Based on prior submissions: https://patchwork.

Re: [RFC PATCH 0/3]perf/core: extend perf_reg and perf_sample_regs_intr

2015-11-05 Thread Peter Zijlstra
On Thu, Nov 05, 2015 at 02:16:15AM +0530, Madhavan Srinivasan wrote: > Second patch updates struct arch_misc_reg for arch/powerpc with pmu registers > and adds offsetof macro for the same. It extends perf_reg_value() > to use reg idx to decide on struct to return value from. Why; what's in those r

[PATCH] powerpc: allow cross-compilation of ppc64 kernel

2015-11-05 Thread Laurent Vivier
When I try to cross compile a ppc64 kernel, it generally fails on the VDSO stage. This is true for powerpc64 cross- compiler, but also when I try to build a ppc64le kernel on a ppc64 host. VDSO64L fails: VDSO64L arch/powerpc/kernel/vdso64/vdso64.so.dbg /usr/bin/powerpc64-linux-gnu-ld: arch/powe

[PATCH] powerpc/boot: allow wrapper to work on non-english system

2015-11-05 Thread Laurent Vivier
if the language is not english objdump output is not parsed correctly and format is "". Later, "ld -m $format" fails. This patch adds "LANG=C" to force english output for objdump. Signed-off-by: Laurent Vivier --- arch/powerpc/boot/wrapper | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] powerpc: on crash, kexec'ed kernel needs all CPUs are online

2015-11-05 Thread Stewart Smith
David Gibson writes: >> > opal/powernv: >> > https://github.com/open-power/skiboot/commit/9ee56b5 >> >> Very interesting. Is there a way to have a firmware with the fix ? > > From Laurent's analysis of the crash, I don't think this will be > relevant either, but I'm not sure. It would be very in

Re: [PATCH] powerpc: on crash, kexec'ed kernel needs all CPUs are online

2015-11-05 Thread Hari Bathini
On 11/05/2015 07:02 AM, David Gibson wrote: On Wed, 4 Nov 2015 14:54:51 +0100 Laurent Vivier wrote: On 04/11/2015 13:34, Hari Bathini wrote: On 10/16/2015 12:30 AM, Laurent Vivier wrote: On kexec, all secondary offline CPUs are onlined before starting the new kernel, this is not done in the