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 when running a relocated kern
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 Piggin wrote:
> > > > This patch does a couple of things.
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
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 powernv_add_idle_states, there is a lot of code
> > duplicati
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.
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, ca
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
---
arch/powerpc/lib/sstep.c | 20 --
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 a/arch/powerpc/include/asm/ppc-opcode.h
b/arch/powerpc/include/asm
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 sm
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 {u,s}times
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 utim
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 +++
arch/powerpc/configs/ppc64_defconfig | 3 +++
arch/po
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
>>
>> The offb driver takes
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] emulat
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: operan
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.
> jim@krebstar:~/linux-rc$ make ARCH=powerpc
> CROSS_C
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
---
arc
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 | 10 +-
1 file c
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
---
arch/powerpc/kernel/process.c | 10
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 p
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 a/arch/powerpc/kernel/process.c b/ar
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
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
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
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 t
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 Ellerman
Cc: linuxppc-dev@lists.ozlabs.org
Signe
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
>>
>> Cool; the little-endian build worked fine, but
>
> Yay, than
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 - th
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 | 66 ++
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
---
drivers/net/ethernet/freescale/
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
b/drivers/net/ethernet/freescale/d
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 +
drivers/net/ethernet/freescale/dpaa/dpaa
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 gro
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 +
.../net/ethernet/freescale/d
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 +
.../net/ethernet/freescal
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
+++ b/arch/powerpc/configs/dpaa.config
@@
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
+++ b/arch/powerpc/configs/dpaa.config
@@
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
+++ b/arch/powerpc/configs/dpaa.config
@@
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
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 scaled by frequency. On
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 se
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 memo
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
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 retu
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.
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:
> https://lists.freedesktop.org/
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 a/arch/powerpc/kernel/module_64.c b/arch/powerpc/kerne
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(),
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 --git
a/tools/testing/s
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?
>
> arch/powerpc/kernel/exceptions-64s.S: Ass
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 b
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 coup
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 CPUs of the core will put
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, a
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
to the destination, a
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 l
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 macro
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. Should
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 missing .previouses in f
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 best way to make a P8 do
>> > winkle sleeps?
>>
On Thu, 3 Nov 2016 01:56:46 -0400
"Shreyas B. Prabhu" wrote:
> 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:
> >> >
> >> >
On Thu, Nov 3, 2016 at 2:17 AM, Nicholas Piggin wrote:
> On Thu, 3 Nov 2016 01:56:46 -0400
> "Shreyas B. Prabhu" wrote:
>
>> 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,
On 2016/11/03 03:55PM, Nicholas Piggin wrote:
> 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?
> > Ha
63 matches
Mail list logo
