Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On Thu, Nov 3, 2016 at 1:21 AM, Nicholas Piggin wrote: > On Wed, 2 Nov 2016 14:15:48 +0530 > Gautham R Shenoy wrote: > >> Hi Nick, >> >> On Wed, Nov 02, 2016 at 07:36:24PM +1100, Nicholas Piggin wrote: >> > >> > Okay, I'll work with that. What's the

[PATCH v2] powerpc: EX_TABLE macro for exception tables

From: Nicholas Piggin This macro is taken from s390, and allows more flexibility in changing exception table format. mpe: Put it in ppc_asm.h and only define one version using stringinfy_in_c(). Add some empty definitions and headers to keep the selftests happy. Add some

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

On Thursday 03 November 2016 02:34 AM, Anton Blanchard wrote: > Hi Ravi, > >> emulate_step() uses a number of underlying kernel functions that were >> initially not enabled for LE. This has been rectified since. So, fix >> emulate_step() for LE for the corresponding instructions. > Thanks.

[PATCH 1/4] powerpc/64: Make exception table clearer in __copy_tofrom_user_base

This aims to make the generation of exception table entries for the loads and stores in __copy_tofrom_user_base clearer and easier to verify. Instead of having a series of local labels on the loads and stores, with a series of corresponding labels later for the exception handlers, we now use

[PATCH 4/4] powerpc/64: Copy as much as possible in __copy_tofrom_user

In __copy_tofrom_user, if we encounter an exception on a store, we stop copying and return the number of bytes not copied. However, if the store is wider than one byte and is to an unaligned address, it is possible that the store operand overlaps a page boundary and the exception occurred on the

[PATCH 3/4] selftests/powerpc/64: Test exception cases in copy_tofrom_user

From: Michael Ellerman This adds a set of test cases to test the behaviour of copy_tofrom_user when exceptions are encountered accessing the source or destination. Currently, copy_tofrom_user does not always copy as many bytes as possible when an exception occurs on a store

[PATCH 2/4] selftests/powerpc/64: Test all paths through copy routines

The hand-coded assembler 64-bit copy routines include feature sections that select one code path or another depending on which CPU we are executing on. The self-tests for these copy routines end up testing just one path. This adds a mechanism for selecting any desired code path at compile time,

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On Wed, 2 Nov 2016 14:15:48 +0530 Gautham R Shenoy wrote: > Hi Nick, > > On Wed, Nov 02, 2016 at 07:36:24PM +1100, Nicholas Piggin wrote: > > > > Okay, I'll work with that. What's the best way to make a P8 do > > winkle sleeps? > > From the userspace, offlining the

[PATCH 2/2] powerpc/64: Use optimized checksum routines on little-endian

Currently we have optimized hand-coded assembly checksum routines for big-endian 64-bit systems, but for little-endian we use the generic C routines. This modifies the optimized routines to work for little-endian. With this, we no longer need to enable CONFIG_GENERIC_CSUM. This also fixes a

[PATCH 1/2] powerpc/64: Fix checksum folding in csum_tcpudp_nofold and ip_fast_csum_nofold

These functions compute an IP checksum by computing a 64-bit sum and folding it to 32 bits (the "nofold" in their names refers to folding down to 16 bits). However, doing (u32) (s + (s >> 32)) is not sufficient to fold a 64-bit sum to 32 bits correctly. The addition can produce a carry out from

Re: compiling master.

On Wed, 2 Nov 2016 13:49:39 +0300 Denis Kirjanov wrote: > Hi guys, > > compiling ppc head with bunch of asm errors on power8 box (gcc version 4.8.3 ) > checked commit log but found nothing special. Looks like it's asm issue? > Has anybody seen that? > >

[PATCH 2/2] selftests/powerpc: Fail load_unaligned_zeropad on miscompare

If the result returned by load_unaligned_zeropad() doesn't match what we expect we should fail the test! Signed-off-by: Michael Ellerman --- tools/testing/selftests/powerpc/primitives/load_unaligned_zeropad.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

[PATCH 1/2] selftests/powerpc: Abort load_unaligned_zeropad on unhandled SEGV

If the load unaligned zeropad test takes a SEGV which can't be handled, we increment segv_error, print the offending NIP and then return without taking any further action. In almost all cases this means we'll just take the SEGV again, and loop eternally spamming the console. Instead just abort(),

[PATCH] powerpc/module: Add support for R_PPC64_REL32 relocations

We haven't seen these before, but the soon to be merged relative exception tables support causes them to be generated. Signed-off-by: Michael Ellerman --- arch/powerpc/kernel/module_64.c | 5 + 1 file changed, 5 insertions(+) diff --git

Re: [PATCH] Fix loading of module radeonfb on PowerMac

On Wed, 2016-11-02 at 11:28 +0100, Mathieu Malaterre wrote: > > This has been working great so far for `radeon.ko`, so I assumed I > could shamelessly copy the behavior over to `radeonfb.ko`. > > The original reference I found was from Benjamin Herrenschmidt here: >

Re: [PATCH] powerpc/configs: Turn on PPC crypto implementations in the server defconfigs

On Wed, 2 Nov 2016 21:06:17 +1100 Michael Ellerman wrote: > These are the PPC optimised versions of various crypto algorithms, so we > should turn them on by default to get test coverage. Looks good, thanks for that.

Re: [PATCH 1/4] cputime/powerpc: remove cputime_last_delta global variable

On Mon, Oct 31, 2016 at 01:36:26PM +0100, Stanislaw Gruszka wrote: > Since commit: > > cf9efce0ce313 ("powerpc: Account time using timebase rather than PURR") > > cputime_last_delta is not initialized to other value than 0, hence it's > not used except zero check and cputime_to_scaled() just

Re: [PATCH 2/4] cputime/powerpc: remove cputime_to_scaled()

On Mon, Oct 31, 2016 at 01:36:27PM +0100, Stanislaw Gruszka wrote: > Currently cputime_to_scaled() just return it's argument on > all implementations, we don't need to call this function. > > Signed-off-by: Stanislaw Gruszka Looks OK to me. Reviewed-by: Paul Mackerras

Re: [PATCH kernel v4 4/4] powerpc/mm/iommu, vfio/spapr: Put pages on VFIO container shutdown

On Wed, Nov 02, 2016 at 01:44:03PM +1100, Alexey Kardashevskiy wrote: > On 31/10/16 15:23, David Gibson wrote: [...] > > > > Um.. yeah.. that's not really ok. Prohibiting overlapping > > registrations on the same container is reasonable enough. Having a > > container not be able to register

Re: [PATCH net-next] ibmveth: v1 calculate correct gso_size and set gso_type

On 10/27/2016 10:26 AM, Eric Dumazet wrote: > On Wed, 2016-10-26 at 11:09 +1100, Jon Maxwell wrote: >> We recently encountered a bug where a few customers using ibmveth on the >> same LPAR hit an issue where a TCP session hung when large receive was >> enabled. Closer analysis revealed that the

Re: [PATCH 3/4] cputime/powerpc/s390: make scaled cputime arch specific

On Wed, 2 Nov 2016 10:38:20 +0100 Stanislaw Gruszka wrote: > On Wed, Nov 02, 2016 at 10:11:22AM +0100, Christian Borntraeger wrote: > > On 10/31/2016 01:36 PM, Stanislaw Gruszka wrote: > > > Only s390 and powerpc have hardware facilities allowing to measure > > > cputimes

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

Hi Ravi, > emulate_step() uses a number of underlying kernel functions that were > initially not enabled for LE. This has been rectified since. So, fix > emulate_step() for LE for the corresponding instructions. Thanks. Should this be queued up for stable? Anton > Reported-by: Anton Blanchard

[PATCH net-next v6 09/10] arch/powerpc: Enable FSL_FMAN

Signed-off-by: Madalin Bucur --- arch/powerpc/configs/dpaa.config | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/configs/dpaa.config b/arch/powerpc/configs/dpaa.config index f124ee1..9ad9bc0 100644 --- a/arch/powerpc/configs/dpaa.config +++

[PATCH net-next v6 10/10] arch/powerpc: Enable dpaa_eth

Signed-off-by: Madalin Bucur --- arch/powerpc/configs/dpaa.config | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/configs/dpaa.config b/arch/powerpc/configs/dpaa.config index 9ad9bc0..2fe76f5 100644 --- a/arch/powerpc/configs/dpaa.config +++

[PATCH net-next v6 08/10] arch/powerpc: Enable FSL_PAMU

Signed-off-by: Madalin Bucur --- arch/powerpc/configs/dpaa.config | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/configs/dpaa.config b/arch/powerpc/configs/dpaa.config index efa99c0..f124ee1 100644 --- a/arch/powerpc/configs/dpaa.config +++

[PATCH net-next v6 07/10] dpaa_eth: add trace points

Add trace points on the hot processing path. Signed-off-by: Ruxandra Ioana Radulescu --- drivers/net/ethernet/freescale/dpaa/Makefile | 1 + drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 15 +++ drivers/net/ethernet/freescale/dpaa/dpaa_eth.h | 1

[PATCH net-next v6 06/10] dpaa_eth: add sysfs exports

Export Frame Queue and Buffer Pool IDs through sysfs. Signed-off-by: Madalin Bucur --- drivers/net/ethernet/freescale/dpaa/Makefile | 2 +- drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 4 + drivers/net/ethernet/freescale/dpaa/dpaa_eth.h | 4 +

[PATCH net-next v6 05/10] dpaa_eth: add ethtool statistics

Add a series of counters to be exported through ethtool: - add detailed counters for reception errors; - add detailed counters for QMan enqueue reject events; - count the number of fragmented skbs received from the stack; - count all frames received on the Tx confirmation path; - add congestion

[PATCH net-next v6 04/10] dpaa_eth: add ethtool functionality

Add support for basic ethtool operations. Signed-off-by: Madalin Bucur --- drivers/net/ethernet/freescale/dpaa/Makefile | 2 +- drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 2 + drivers/net/ethernet/freescale/dpaa/dpaa_eth.h | 3 +

[PATCH net-next v6 03/10] dpaa_eth: add option to use one buffer pool set

Signed-off-by: Madalin Bucur --- drivers/net/ethernet/freescale/dpaa/Kconfig| 6 ++ drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 23 +++ 2 files changed, 29 insertions(+) diff --git a/drivers/net/ethernet/freescale/dpaa/Kconfig

[PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

This introduces the Freescale Data Path Acceleration Architecture (DPAA) Ethernet driver (dpaa_eth) that builds upon the DPAA QMan, BMan, PAMU and FMan drivers to deliver Ethernet connectivity on the Freescale DPAA QorIQ platforms. Signed-off-by: Madalin Bucur ---

[PATCH net-next v6 01/10] devres: add devm_alloc_percpu()

Introduce managed counterparts for alloc_percpu() and free_percpu(). Add devm_alloc_percpu() and devm_free_percpu() into the managed interfaces list. Signed-off-by: Madalin Bucur --- Documentation/driver-model/devres.txt | 4 +++ drivers/base/devres.c |

[PATCH net-next v6 00/10] dpaa_eth: Add the QorIQ DPAA Ethernet driver

This patch series adds the Ethernet driver for the Freescale QorIQ Data Path Acceleration Architecture (DPAA). This version includes changes following the feedback received on previous versions from Eric Dumazet, Bob Cochran, Joe Perches, Paul Bolle, Joakim Tjernlund, Scott Wood, David Miller -

Re: [PATCH] cxl: Fix error handling

On Wed, Nov 2, 2016 at 4:12 AM, Michael Ellerman wrote: > Jim Davis writes: > >> On Sun, Oct 30, 2016 at 10:37 PM, Michael Ellerman >> wrote: >>> More here: >>> >>> https://github.com/linuxppc/linux/wiki/Building-powerpc-kernels >>

[mm PATCH v2 18/26] arch/powerpc: Add option to skip DMA sync as a part of mapping

This change allows us to pass DMA_ATTR_SKIP_CPU_SYNC which allows us to avoid invoking cache line invalidation if the driver will just handle it via a sync_for_cpu or sync_for_device call. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael

Re: [PATCH] cxl: Fix memory allocation failure test

Le 30/10/2016 à 20:35, Christophe JAILLET a écrit : 'cxl_context_alloc()' does not return an error pointer. It is just a shortcut for a call to 'kzalloc' with 'sizeof(struct cxl_context)' as the size parameter. So its return value should be compared with NULL. While fixing it, simplify a bit

Re: [PATCH] cxl: Fix error handling

Le 30/10/2016 à 22:40, Christophe JAILLET a écrit : 'cxl_dev_context_init()' returns an error pointer in case of error, not NULL. So test it with IS_ERR. Signed-off-by: Christophe JAILLET --- Thanks for the 3 patches! Acked-by: Frederic Barrat

Re: [PATCH] cxl: Fix error handling

Le 30/10/2016 à 22:34, Christophe JAILLET a écrit : 'cxl_dev_context_init()' returns an error pointer in case of error, not NULL. So test it with IS_ERR. Signed-off-by: Christophe JAILLET Acked-by: Frederic Barrat

Re: [PATCH] Fix loading of module radeonfb on PowerMac

On Wed, Nov 02, 2016 at 10:28:55AM +0200, Tomi Valkeinen wrote: > On 08/10/16 15:09, Mathieu Malaterre wrote: > > When the linux kernel is build with (typical kernel ship with Debian > > installer): > > > > CONFIG_FB_OF=y > > CONFIG_VT_HW_CONSOLE_BINDING=y > > CONFIG_FB_RADEON=m > > > > The offb

[PATCH 5/5] powerpc/oops: Fix remaining pr_cont() issues

Now that we print SOFTE first, it's clear that the rest of these lines need to be continuations, so use pr_cont(). Signed-off-by: Michael Ellerman --- arch/powerpc/kernel/process.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git

[PATCH 4/5] powerpc/oops: Move printing of SOFTE before CFAR

Now that KERN_CONT is required to do continuation lines properly, our oops output is messed up. But as the code is currently written we can't actually use pr_cont() correctly, because some of the output may or may not be a continuation line, depending on what was printed previously. So move the

[PATCH 3/5] powerpc/oops: Fix printing of GPRs since KERN_CONT changes

Since the KERN_CONT changes our print out of the GPRs is messed up: GPR04: 0001 c0155810 0001 GPR08: 0007 Fix it for now by using pr_cont(). Signed-off-by: Michael Ellerman ---

[PATCH 2/5] powerpc/oops: Fix missing pr_cont()s in print_msr_bits() et. al.

Since the KERN_CONT changes these are being horribly split across lines, for example: MSR: 80009033 < SF,EE ,ME,IR ,DR,RI ,LE> So fix it by using pr_cont() where appropriate. Signed-off-by: Michael Ellerman --- arch/powerpc/kernel/process.c |

[PATCH 1/5] powerpc/oops: Fix missing KERN_CONT in show_stack()

Previously we got away with printing the stack trace in multiple pieces and it usually looked right. But since commit 4bcc595ccd80 ("printk: reinstate KERN_CONT for printing continuation lines"), KERN_CONT is now required when printing continuation lines. Signed-off-by: Michael Ellerman

Re: [PATCH] cxl: Fix error handling

Jim Davis writes: > On Sun, Oct 30, 2016 at 10:37 PM, Michael Ellerman > wrote: >> More here: >> >> https://github.com/linuxppc/linux/wiki/Building-powerpc-kernels > > Cool; the little-endian build worked fine, but Yay, thanks for trying. >

compiling master.

Hi guys, compiling ppc head with bunch of asm errors on power8 box (gcc version 4.8.3 ) checked commit log but found nothing special. Looks like it's asm issue? Has anybody seen that? arch/powerpc/kernel/exceptions-64s.S: Assembler messages: arch/powerpc/kernel/exceptions-64s.S:421: Error:

Re: [PATCH 3/3] powerpc: emulate_step test for load/store instructions

On 2016/11/02 02:23PM, Ravi Bangoria wrote: > Add new selftest that test emulate_step for Normal, Floating Point, > Vector and Vector Scalar - load/store instructions. Test should run > at boot time if CONFIG_KPROBES_SANITY_TEST and CONFIG_PPC64 is set. > > Sample log: > > [0.762063]

Re: [PATCH] Fix loading of module radeonfb on PowerMac

Tomi, On Wed, Nov 2, 2016 at 9:28 AM, Tomi Valkeinen wrote: > On 08/10/16 15:09, Mathieu Malaterre wrote: >> When the linux kernel is build with (typical kernel ship with Debian >> installer): >> >> CONFIG_FB_OF=y >> CONFIG_VT_HW_CONSOLE_BINDING=y >> CONFIG_FB_RADEON=m >>

[PATCH] powerpc/configs: Turn on PPC crypto implementations in the server defconfigs

These are the PPC optimised versions of various crypto algorithms, so we should turn them on by default to get test coverage. Suggested-by: Nicholas Piggin Signed-off-by: Michael Ellerman --- arch/powerpc/configs/powernv_defconfig | 3 +++

Re: [PATCH 3/4] cputime/powerpc/s390: make scaled cputime arch specific

On Wed, Nov 02, 2016 at 10:11:22AM +0100, Christian Borntraeger wrote: > On 10/31/2016 01:36 PM, Stanislaw Gruszka wrote: > > Only s390 and powerpc have hardware facilities allowing to measure > > cputimes scaled by frequency. On all other architectures > > utimescaled/stimescaled are equal to

Re: [PATCH 3/4] cputime/powerpc/s390: make scaled cputime arch specific

On 10/31/2016 01:36 PM, Stanislaw Gruszka wrote: > Only s390 and powerpc have hardware facilities allowing to measure > cputimes scaled by frequency. On all other architectures > utimescaled/stimescaled are equal to utime/stime (however they are > accounted separately). > > Patch remove

[PATCH 3/3] powerpc: emulate_step test for load/store instructions

Add new selftest that test emulate_step for Normal, Floating Point, Vector and Vector Scalar - load/store instructions. Test should run at boot time if CONFIG_KPROBES_SANITY_TEST and CONFIG_PPC64 is set. Sample log: [0.762063] emulate_step smoke test: start. [0.762219] emulate_step

[PATCH 2/3] powerpc: Add encoding for couple of load/store instructions

These encodings will be used in subsequent patch that test emulate_step for load/store instructions. Signed-off-by: Ravi Bangoria --- arch/powerpc/include/asm/ppc-opcode.h | 7 +++ 1 file changed, 7 insertions(+) diff --git

[PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

emulate_step() uses a number of underlying kernel functions that were initially not enabled for LE. This has been rectified since. So, fix emulate_step() for LE for the corresponding instructions. Reported-by: Anton Blanchard Signed-off-by: Ravi Bangoria

[PATCH 0/3] powerpc: Emulation support for load/store instructions on LE

emulate_step is the basic infrastructure which is used by number of other kernel infrastructures like kprobe, hw-breakpoint(data breakpoint) etc. In case of kprobe, enabling emulation of load/store instructions will speedup the execution of probed instruction. In case of kernel-space breakpoint,

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

Hi Nick, On Wed, Nov 02, 2016 at 07:36:24PM +1100, Nicholas Piggin wrote: > > Okay, I'll work with that. What's the best way to make a P8 do > winkle sleeps? >From the userspace, offlining the CPUs of the core will put them to winkle. > > Thanks, > Nick > -- Thanks and Regards gautham.

Re: [PATCH v2 2/3] cpuidle:powernv: Add helper function to populate powernv idle states.

Hi Oliver, Thanks for reviewing the patch! On Tue, Nov 01, 2016 at 07:32:58PM +1100, Oliver O'Halloran wrote: > On Thu, Oct 27, 2016 at 7:35 PM, Gautham R. Shenoy > wrote: > > From: "Gautham R. Shenoy" > > > > In the current code for

Re: [PATCH] Fix loading of module radeonfb on PowerMac

On 08/10/16 15:09, Mathieu Malaterre wrote: > When the linux kernel is build with (typical kernel ship with Debian > installer): > > CONFIG_FB_OF=y > CONFIG_VT_HW_CONSOLE_BINDING=y > CONFIG_FB_RADEON=m > > The offb driver takes precedence over module radeonfb. It is then > impossible to load the

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On Wed, 2 Nov 2016 13:54:43 +0530 Mahesh J Salgaonkar wrote: > On 2016-11-02 17:57:01 Wed, Nicholas Piggin wrote: > > On Wed, 2 Nov 2016 11:34:59 +0530 > > Mahesh Jagannath Salgaonkar wrote: > > > > > On 10/13/2016 07:47 AM, Nicholas

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On 2016-11-02 17:57:01 Wed, Nicholas Piggin wrote: > On Wed, 2 Nov 2016 11:34:59 +0530 > Mahesh Jagannath Salgaonkar wrote: > > > On 10/13/2016 07:47 AM, Nicholas Piggin wrote: > > > This patch does a couple of things. First of all, powernv immediately > > > explodes

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On Wed, 2 Nov 2016 11:34:59 +0530 Mahesh Jagannath Salgaonkar wrote: > On 10/13/2016 07:47 AM, Nicholas Piggin wrote: > > This patch does a couple of things. First of all, powernv immediately > > explodes when running a relocated kernel, because the system reset > >

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

On 10/13/2016 07:47 AM, Nicholas Piggin wrote: > This patch does a couple of things. First of all, powernv immediately > explodes when running a relocated kernel, because the system reset > exception for handling sleeps does not do correct relocated branches. > > Secondly, the sleep handling code

[RFC] kexec_file: Add support for purgatory built as PIE

Hello, The kexec_file code currently builds the purgatory as a partially linked object (using ld -r). Is there a particular reason to use that instead of a position independent executable (PIE)? I found a discussion from 2013 in the archives but from what I understood it was about the