Re: [PATCH 31/37] misc: Add host side pci driver for pci test function device

2017-01-24 Thread Kishon Vijay Abraham I
Hi, On Tuesday 24 January 2017 09:32 PM, Christoph Hellwig wrote: > On Thu, Jan 12, 2017 at 03:56:20PM +0530, Kishon Vijay Abraham I wrote: >> Add PCI endpoint test driver that can verify base address >> register, legacy interrupt/MSI interrupt and read/write/copy >> buffers between host and devic

Re: [PATCH] powerpc/mm: Fix RECLAIM_DISTANCE

2017-01-24 Thread Gavin Shan
On Wed, Jan 25, 2017 at 09:27:44AM +0530, Balbir Singh wrote: >On Tue, Jan 24, 2017 at 10:32:28AM +1100, Gavin Shan wrote: >> When @node_reclaim_mode ("/proc/sys/vm/zone_reclaim_mode") is enabled, >> the nodes in the specified distance (< RECLAIM_DISTANCE) to the preferred >> one will be checked fo

Re: [bug] stack protector panics on v4.10-rc1+

2017-01-24 Thread Segher Boessenkool
On Wed, Jan 25, 2017 at 09:24:53AM +0530, Balbir Singh wrote: > > So my inclination is to revert the powerpc stack protector code for > > 4.10, and we can try again for 4.11 or 12. > > That makes sense. We then wait for the right gcc version? I guess we also > push for per-task gaurd value as oppo

Re: [PATCH] powerpc/mm: Fix RECLAIM_DISTANCE

2017-01-24 Thread Balbir Singh
On Tue, Jan 24, 2017 at 10:32:28AM +1100, Gavin Shan wrote: > When @node_reclaim_mode ("/proc/sys/vm/zone_reclaim_mode") is enabled, > the nodes in the specified distance (< RECLAIM_DISTANCE) to the preferred > one will be checked for page direct reclaim in the fast path, as below > function call c

Re: [bug] stack protector panics on v4.10-rc1+

2017-01-24 Thread Balbir Singh
On Tue, Jan 24, 2017 at 03:09:40PM +1100, Michael Ellerman wrote: > Michael Ellerman writes: > > > # zgrep STACKPROTECTOR /proc/config.gz > > CONFIG_HAVE_CC_STACKPROTECTOR=y > > CONFIG_CC_STACKPROTECTOR=y > > CONFIG_CC_STACKPROTECTOR_REGULAR=y > > > > I guess I'm just lucky? > > No, I'm just us

Re: [PATCH] powerpc: opal-msglog: Report size of memcons log

2017-01-24 Thread Michael Ellerman
Michael Ellerman writes: > Joel Stanley writes: > >> The OPAL memory console is reported to be size zero, as we do not >> initialise the struct attr with any size information due to the size >> being variable. This leads users to think that the console is empty. > > Hmm OK. That is a general pro

Re: [PATCH] powerpc/powernv: Validate memcons descriptor and output buffer

2017-01-24 Thread Gavin Shan
On Tue, Jan 24, 2017 at 09:21:22PM +1100, Michael Ellerman wrote: >Gavin Shan writes: > >> Currently, it's assumed that memcons and its output buffer are included >> in the linear mapping. It's not true when "mem=384M" is included in >> bootargs. The system runs into kernel crash eventually. >> >>

Re: [PATCH] cxl: drop unused header asm/pnv-pci.h

2017-01-24 Thread Andrew Donnellan
On 24/01/17 20:57, Michael Ellerman wrote: Greg Kurz writes: The kernel API does not use anything from this header file. That's actually hard to prove in general, because pnv-pci.h includes linux/pci.h, linux/pci_hotplug.h, linux/irq.h, misc/cxl-base.h and asm/opal-api.h. At least four of wh

Re: [PATCH] ibmveth: Add a proper check for the availability of the checksum features

2017-01-24 Thread Thomas Falcon
On 01/24/2017 12:28 AM, Thomas Huth wrote: > When using the ibmveth driver in a KVM/QEMU based VM, it currently > always prints out a scary error message like this when it is started: > > ibmveth 7103 (unregistered net_device): unable to change > checksum offload settings. 1 rc=-2 ret_attr=71

[PATCH] mtd/ifc: Fix location of eccstat registers for IFC V1.0

2017-01-24 Thread mark.marshall
From: Mark Marshall The commit 7a654172161c ("mtd/ifc: Add support for IFC controller version 2.0") added support for version 2.0 of the IFC controller. The version 2.0 controller has the ECC status registers at a different location to the previous versions. This broke the code for IFC all versi

Re: [PATCH] ibmveth: Add a proper check for the availability of the checksum features

2017-01-24 Thread David Miller
From: Thomas Huth Date: Tue, 24 Jan 2017 07:28:41 +0100 > When using the ibmveth driver in a KVM/QEMU based VM, it currently > always prints out a scary error message like this when it is started: > > ibmveth 7103 (unregistered net_device): unable to change > checksum offload settings. 1 r

Re: [PATCH V9 0/8] powerpc/devtree: Add support for 2 new DRC properties

2017-01-24 Thread Michael Bringmann
Ping On 12/07/2016 01:13 AM, Michael Bringmann wrote: > Several properties in the DRC device tree format are replaced by > more compact representations to allow, for example, for the encoding > of vast amounts of memory, and or reduced duplication of information > in related data structures. > >

Re: [PATCH v4 2/5] ia64: reuse append_elf_note() and final_note() functions

2017-01-24 Thread Tony Luck
On Tue, Jan 24, 2017 at 10:11 AM, Hari Bathini wrote: > Hello IA64 folks, > > Could you please review this patch..? It looks OK in principal. My lab is in partial disarray at the moment (just got back from a sabbatical) so I can't test build and boot. Have you cross-compiled it (or gotten a suc

Re: [PATCH v4 2/5] ia64: reuse append_elf_note() and final_note() functions

2017-01-24 Thread Hari Bathini
On Tuesday 17 January 2017 10:36 PM, Hari Bathini wrote: On Friday 06 January 2017 07:33 AM, Dave Young wrote: On 01/05/17 at 11:01pm, Hari Bathini wrote: Get rid of multiple definitions of append_elf_note() & final_note() functions. Reuse these functions compiled under CONFIG_CRASH_CORE Al

Re: [PATCH v4 2/5] ia64: reuse append_elf_note() and final_note() functions

2017-01-24 Thread Hari Bathini
On Friday 20 January 2017 11:17 AM, Michael Ellerman wrote: Hari Bathini writes: Get rid of multiple definitions of append_elf_note() & final_note() functions. Reuse these functions compiled under CONFIG_CRASH_CORE Also, define Elf_Word and use it instead of generic u32 or the more specific

Re: [PATCH 3/3] powerpc: bpf: implement in-register swap for 64-bit endian operations

2017-01-24 Thread 'Naveen N. Rao'
On 2017/01/24 04:13PM, David Laight wrote: > From: 'Naveen N. Rao' > > Sent: 23 January 2017 19:22 > > On 2017/01/15 09:00AM, Benjamin Herrenschmidt wrote: > > > On Fri, 2017-01-13 at 23:22 +0530, 'Naveen N. Rao' wrote: > > > > > That rather depends on whether the processor has a store to load > >

[PATCH v4 2/2] modversions: treat symbol CRCs as 32 bit quantities

2017-01-24 Thread Ard Biesheuvel
The modversion symbol CRCs are emitted as ELF symbols, which allows us to easily populate the kcrctab sections by relying on the linker to associate each kcrctab slot with the correct value. This has a couple of downsides: - Given that the CRCs are treated as memory addresses, we waste 4 bytes f

[PATCH v4 1/2] kbuild: modversions: add infrastructure for emitting relative CRCs

2017-01-24 Thread Ard Biesheuvel
This add the kbuild infrastructure that will allow architectures to emit vmlinux symbol CRCs as 32-bit offsets to other locations in the kernel where the actual values are stored. This works around problems with CRCs being mistaken for relocatable symbols on kernels that self relocate at runtime (i

[PATCH v4 0/2] modversions: redefine kcrctab entries as 32-bit values

2017-01-24 Thread Ard Biesheuvel
This v4 is a followup to [0] 'modversions: redefine kcrctab entries as relative CRC pointers', but since relative CRC pointers do not work in modules, and are actually only needed by powerpc with CONFIG_RELOCATABLE=y, I have made it a Kconfig selectable feature instead. Patch #1 introduces the MOD

RE: [PATCH 3/3] powerpc: bpf: implement in-register swap for 64-bit endian operations

2017-01-24 Thread David Laight
From: 'Naveen N. Rao' > Sent: 23 January 2017 19:22 > On 2017/01/15 09:00AM, Benjamin Herrenschmidt wrote: > > On Fri, 2017-01-13 at 23:22 +0530, 'Naveen N. Rao' wrote: > > > > That rather depends on whether the processor has a store to load > > > > forwarder > > > > that will satisfy the read fro

Re: [PATCH 31/37] misc: Add host side pci driver for pci test function device

2017-01-24 Thread Christoph Hellwig
On Thu, Jan 12, 2017 at 03:56:20PM +0530, Kishon Vijay Abraham I wrote: > Add PCI endpoint test driver that can verify base address > register, legacy interrupt/MSI interrupt and read/write/copy > buffers between host and device. The corresponding pci-epf-test > function driver should be used on th

Re: [PATCH] cxl: drop unused header asm/pnv-pci.h

2017-01-24 Thread Greg Kurz
On Tue, 24 Jan 2017 20:57:06 +1100 Michael Ellerman wrote: > Greg Kurz writes: > > > The kernel API does not use anything from this header file. > > That's actually hard to prove in general, because pnv-pci.h includes > linux/pci.h, linux/pci_hotplug.h, linux/irq.h, misc/cxl-base.h and > asm

[PATCH] i2c/pasemi: Remove hardcoded bus numbers on smbus

2017-01-24 Thread Darren Stevens
The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which number bus to attach to, however this fails when something else is probed first, for example an ATI Radeon graphics card will claim 9 or 10 busses, including the ones the pasemi wants. Patch the driver to call i2c_add_adapter ra

Re: [PATCH] mtd/ifc: Fix location of eccstat registers for IFC V1.0

2017-01-24 Thread Boris Brezillon
On Tue, 24 Jan 2017 12:01:22 +0100 wrote: > From: Mark Marshall > > The commit 7a654172161c ("mtd/ifc: Add support for IFC controller > version 2.0") added support for version 2.0 of the IFC controller. > The version 2.0 controller has the ECC status registers at a different > location to the p

Re: BUILD_BUG_ON(!__builtin_constant_p(feature)) breaks bcc trace tool

2017-01-24 Thread Naveen N. Rao
On 2017/01/24 04:36PM, Michael Ellerman wrote: > Anton Blanchard writes: > > We added: > > > > BUILD_BUG_ON(!__builtin_constant_p(feature)) > > > > to cpu_has_feature() and mmu_has_feature() in order to catch usage > > issues (such as cpu_has_feature(cpu_has_feature(X)). Unfortunately LLVM > > is

Re: [PATCH v2] powerpc/pasemi: Remove hardcoded bus numbers on smbus

2017-01-24 Thread Michael Ellerman
Darren Stevens writes: > The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which > number bus to attach to, however this fails when something else is > probed first, for example an ATI Radeon graphics card will claim 9 or > 10 busses, including the ones the pasemi wants. > Patch the

Re: [PATCH] powerpc/powernv: Validate memcons descriptor and output buffer

2017-01-24 Thread Michael Ellerman
Gavin Shan writes: > Currently, it's assumed that memcons and its output buffer are included > in the linear mapping. It's not true when "mem=384M" is included in > bootargs. The system runs into kernel crash eventually. > ># od -x /proc/device-tree/ibm,opal/ibm,opal-memcons >000

Re: [PATCH] cxl: drop unused header asm/pnv-pci.h

2017-01-24 Thread Michael Ellerman
Greg Kurz writes: > The kernel API does not use anything from this header file. That's actually hard to prove in general, because pnv-pci.h includes linux/pci.h, linux/pci_hotplug.h, linux/irq.h, misc/cxl-base.h and asm/opal-api.h. At least four of which aren't included directly by api.c. But I

Re: BUILD_BUG_ON(!__builtin_constant_p(feature)) breaks bcc trace tool

2017-01-24 Thread Arnd Bergmann
On Tue, Jan 24, 2017 at 7:15 AM, Michael Ellerman wrote: > Hmm, in fact it seems because we don't define > CONFIG_ARCH_SUPPORTS_OPTIMIZED_INLINING and CONFIG_OPTIMIZE_INLINING, we > get: > > #define inline inline __attribute__((always_inline)) notrace > > So in fact every inline