[PATCH v2] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-07-18 Thread Murilo Opsfelder Araujo
://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo <mopsfel...@gmail.com> --- Changes from v1: - Rebased on top of next-20170718. include/linux/vfio.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/vfio.h b/include/linux/vfio.h

[PATCH v2] drivers/watchdog/Kconfig: Update CONFIG_WATCHDOG_RTAS dependencies

2017-05-29 Thread Murilo Opsfelder Araujo
ed! This was identified during a randconfig build where CONFIG_WATCHDOG_RTAS=m and CONFIG_PPC_RTAS was not set. Logs are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982152/ This patch fixes the issue by updating CONFIG_WATCHDOG_RTAS to depend on just CONFIG_PPC_RTAS, removing COMPI

[PATCH v2] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-06-07 Thread Murilo Opsfelder Araujo
://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo <mopsfel...@gmail.com> --- v1..v2: - Make use of IS_ENABLED() macro because CONFIG_VFIO_SPAPR_EEH is a tristate option (fix http://www.spinics.net/lists/kvm/msg151032.html). include/linux/vfio.h | 4 ++-- 1 file c

[PATCH] Update MAINTAINERS

2017-06-01 Thread Murilo Opsfelder Araujo
drivers/watchdog/wdrtas.c is of insterest of linuxppc maintainers. Signed-off-by: Murilo Opsfelder Araujo <mopsfel...@gmail.com> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9609ca6..3f05927 100644 --- a/MAINTAINERS +++ b/MAINT

[PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-06-06 Thread Murilo Opsfelder Araujo
, the symbol that controls whether vfio_spapr_eeh.c is built, which is where the non-empty versions of these functions are. This issue was found during a randconfig build. Logs are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo <mops

Re: [PATCH V2 4/4] powerpc/mm/hash64: Increase the VA range

2018-02-26 Thread Murilo Opsfelder Araujo
On 02/26/2018 11:08 AM, Aneesh Kumar K.V wrote: > --- > arch/powerpc/include/asm/book3s/64/hash-64k.h | 2 +- > arch/powerpc/include/asm/processor.h | 9 - > arch/powerpc/mm/init_64.c | 6 -- > arch/powerpc/mm/pgtable_64.c | 5 - > 4

Re: [PATCH v3 5/9] powerpc/traps: Print signal name for unhandled signals

2018-08-01 Thread Murilo Opsfelder Araujo
On Wed, Aug 01, 2018 at 12:03:50AM -0700, Joe Perches wrote: > On Wed, 2018-08-01 at 08:37 +0200, Christophe LEROY wrote: > > Le 31/07/2018 à 16:50, Murilo Opsfelder Araujo a écrit : > > > This adds a human-readable name in the unhandled signal message. > > > Before this

Re: [PATCH v3 5/9] powerpc/traps: Print signal name for unhandled signals

2018-08-01 Thread Murilo Opsfelder Araujo
Hi, Segher. On Wed, Aug 01, 2018 at 02:49:03AM -0500, Segher Boessenkool wrote: > On Wed, Aug 01, 2018 at 12:03:50AM -0700, Joe Perches wrote: > > On Wed, 2018-08-01 at 08:37 +0200, Christophe LEROY wrote: > > > Le 31/07/2018 à 16:50, Murilo Opsfelder Araujo a écrit : >

Re: [PATCH v3 9/9] powerpc/traps: Add line prefix in show_instructions()

2018-08-01 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Aug 01, 2018 at 08:41:15AM +0200, Christophe LEROY wrote: > > > Le 31/07/2018 à 16:50, Murilo Opsfelder Araujo a écrit : > > Remove "Instruction dump:" line by adding a prefix to display current->comm > > and current->

Re: [PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-02 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Thu, Aug 02, 2018 at 07:26:20AM +0200, Christophe LEROY wrote: > > > Le 01/08/2018 à 23:33, Murilo Opsfelder Araujo a écrit : > > show_user_instructions() is a slightly modified version of > > show_instructions() that allows users

Re: [PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-03 Thread Murilo Opsfelder Araujo
Hi, everyone. I'd like to thank you all that contributed to refining and making this series better. I did appreciate. Thank you! Cheers Murilo

Re: [PATCH v2 04/10] powerpc/traps: Use REG_FMT in show_signal_msg()

2018-07-30 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Fri, Jul 27, 2018 at 06:40:23PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Simplify the message format by using REG_FMT as the register format. This > > avoids having two different formats and avoids checking for MSR_64BIT

Re: [PATCH v2 04/10] powerpc/traps: Use REG_FMT in show_signal_msg()

2018-07-30 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Mon, Jul 30, 2018 at 06:30:47PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Hi, Christophe. > > > > On Fri, Jul 27, 2018 at 06:40:23PM +0200, LEROY Christophe wrote: > > > Murilo Opsfelder Araujo a écrit : > >

[PATCH v2 03/10] powerpc/reg: Add REG_FMT definition

2018-07-27 Thread Murilo Opsfelder Araujo
Make REG definition, in arch/powerpc/kernel/process.c, generic enough by renaming it to REG_FMT and placing it in arch/powerpc/include/asm/reg.h to be used elsewhere. Replace occurrences of REG by REG_FMT in arch/powerpc/kernel/process.c. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc

[PATCH v2 09/10] powerpc/traps: Show instructions on exceptions

2018-07-27 Thread Murilo Opsfelder Araujo
f93f0020 e93f0020 39400048 <9949> 3920 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index e55ee639d010..3beca17ac1b1

[PATCH v2 02/10] powerpc/traps: Return early in show_signal_msg()

2018-07-27 Thread Murilo Opsfelder Araujo
were coupled together in one single `if` statement. Splitting out the rate limit check outside show_signal_msg() makes it easier to the caller decide if it wants to respect a printk rate limit or not. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 21

[PATCH v2 06/10] powerpc/traps: Print signal name for unhandled signals

2018-07-27 Thread Murilo Opsfelder Araujo
: pandafault[6352]: segfault (11) at 00013a2a09f8 nip 00013a2a086c lr 7fffb63e5100 code 2 in pandafault[13a2a+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 43 + 1 file changed, 39 insertions(+), 4 deletions

[PATCH v2 10/10] powerpc/traps: Add line prefix in show_instructions()

2018-07-27 Thread Murilo Opsfelder Araujo
ffc1 7c3f0b78 3d22fffe pandafault[10850]: code: 392988d0 f93f0020 e93f0020 39400048 <9949> 3920 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/process.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/ker

[PATCH v2 05/10] powerpc/traps: Print VMA for unhandled signals

2018-07-27 Thread Murilo Opsfelder Araujo
this patch, a page fault looks like: pandafault[6303]: unhandled signal 11 at 17d0 nip 161c lr 7fff93c55100 code 2 in pandafault[1000+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion

[PATCH v2 08/10] powerpc: Add stacktrace.h header

2018-07-27 Thread Murilo Opsfelder Araujo
Move show_instructions() declaration to arch/powerpc/include/asm/stacktrace.h and include asm/stracktrace.h in arch/powerpc/kernel/process.c, which contains the implementation. This allows show_instructions() to be called on, for example, show_signal_msg(). Signed-off-by: Murilo Opsfelder Araujo

[PATCH v2 00/10] powerpc: Modernize unhandled signals message

2018-07-27 Thread Murilo Opsfelder Araujo
asm/stacktrace.h - show_instructions(): prefixed lines with current->comm and current->pid Cheers! Murilo Opsfelder Araujo (10): powerpc/traps: Print unhandled signals in a separate function powerpc/traps: Return early in show_signal_msg() powerpc/reg: Add REG_FMT definition powerpc/traps: Use REG_

[PATCH v2 01/10] powerpc/traps: Print unhandled signals in a separate function

2018-07-27 Thread Murilo Opsfelder Araujo
Isolate the logic of printing unhandled signals out of _exception_pkey(). No functional change, only code rearrangement. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/arch

[PATCH v2 04/10] powerpc/traps: Use REG_FMT in show_signal_msg()

2018-07-27 Thread Murilo Opsfelder Araujo
Simplify the message format by using REG_FMT as the register format. This avoids having two different formats and avoids checking for MSR_64BIT. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git

[PATCH v2 07/10] powerpc: Do not call __kernel_text_address() in show_instructions()

2018-07-27 Thread Murilo Opsfelder Araujo
Modify show_instructions() not to call __kernel_text_address(), allowing userspace instruction dump. probe_kernel_address(), which returns -EFAULT if something goes wrong, is still being called. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/process.c | 3 +-- 1 file changed, 1

[PATCH v3 1/9] powerpc/traps: Print unhandled signals in a separate function

2018-07-31 Thread Murilo Opsfelder Araujo
Isolate the logic of printing unhandled signals out of _exception_pkey(). No functional change, only code rearrangement. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/arch

[PATCH v3 4/9] powerpc/traps: Print VMA for unhandled signals

2018-07-31 Thread Murilo Opsfelder Araujo
like: pandafault[6303]: unhandled signal 11 at 17d0 nip 161c lr 7fff93c55100 code 2 in pandafault[1000+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/kernel

[PATCH v3 0/9] powerpc: Modernize unhandled signals message

2018-07-31 Thread Murilo Opsfelder Araujo
400048 <9949> 3920 7d234b78 383f0050 Link to v2: https://lore.kernel.org/lkml/20180727145811.12334-1-muri...@linux.ibm.com/ v2..v3: - Dropped patch 3 - Updated patch 4 to use %lx Cheers! Murilo Opsfelder Araujo (9): powerpc/traps: Print unhandled signals in a separate fu

[PATCH v3 3/9] powerpc/traps: Use %lx format in show_signal_msg()

2018-07-31 Thread Murilo Opsfelder Araujo
this patch: pandafault[4732]: unhandled signal 11 at 1718 nip 1574 lr 7fff86697a6c code 2 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc

[PATCH v3 8/9] powerpc/traps: Show instructions on exceptions

2018-07-31 Thread Murilo Opsfelder Araujo
20 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index e71f12bca146..b27f3f71d745 100644 --- a/arch/powerpc/kernel/traps.c +++ b/arch/p

[PATCH v3 2/9] powerpc/traps: Return early in show_signal_msg()

2018-07-31 Thread Murilo Opsfelder Araujo
were coupled together in one single `if` statement. Splitting out the rate limit check outside show_signal_msg() makes it easier to the caller decide if it wants to respect a printk rate limit or not. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 21

[PATCH v3 9/9] powerpc/traps: Add line prefix in show_instructions()

2018-07-31 Thread Murilo Opsfelder Araujo
88d0 f93f0020 e93f0020 39400048 <9949> 3920 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/process.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index e7879

[PATCH v3 7/9] powerpc: Add stacktrace.h header

2018-07-31 Thread Murilo Opsfelder Araujo
Move show_instructions() declaration to arch/powerpc/include/asm/stacktrace.h and include asm/stracktrace.h in arch/powerpc/kernel/process.c, which contains the implementation. This allows show_instructions() to be called on, for example, show_signal_msg(). Signed-off-by: Murilo Opsfelder Araujo

[PATCH v3 5/9] powerpc/traps: Print signal name for unhandled signals

2018-07-31 Thread Murilo Opsfelder Araujo
]: segfault (11) at 13a2a09f8 nip 13a2a086c lr 7fffb63e5100 code 2 in pandafault[13a2a+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 39 +++-- 1 file changed, 37 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel

[PATCH v3 6/9] powerpc: Do not call __kernel_text_address() in show_instructions()

2018-07-31 Thread Murilo Opsfelder Araujo
Modify show_instructions() not to call __kernel_text_address(), allowing userspace instruction dump. probe_kernel_address(), which returns -EFAULT if something goes wrong, is still being called. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/process.c | 3 +-- 1 file changed, 1

[PATCH v4 0/6] powerpc: Modernize unhandled signals message

2018-08-01 Thread Murilo Opsfelder Araujo
e with a tiny function that returns a literal string for each signal number Cheers! Murilo Opsfelder Araujo (6): powerpc/traps: Print unhandled signals in a separate function powerpc/traps: Use an explicit ratelimit state for show_signal_msg() powerpc/traps: Use %lx format in show_signal_msg()

[PATCH v4 3/6] powerpc/traps: Use %lx format in show_signal_msg()

2018-08-01 Thread Murilo Opsfelder Araujo
this patch: pandafault[4732]: unhandled signal 11 at 1718 nip 1574 lr 7fff86697a6c code 2 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc

[PATCH v4 2/6] powerpc/traps: Use an explicit ratelimit state for show_signal_msg()

2018-08-01 Thread Murilo Opsfelder Araujo
Replace printk_ratelimited() by printk() and a default rate limit burst to limit displaying unhandled signals messages. This will allow us to call print_vma_addr() in a future patch, which does not work with printk_ratelimited(). Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel

[PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-01 Thread Murilo Opsfelder Araujo
s originator, allowing messages to be interleaved in the logs. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/include/asm/stacktrace.h | 13 + arch/powerpc/kernel/process.c | 40 +++ 2 files changed, 53 insertions(+) create mode 100644 arch/powerpc

[PATCH v4 4/6] powerpc/traps: Print VMA for unhandled signals

2018-08-01 Thread Murilo Opsfelder Araujo
like: pandafault[6303]: segfault 11 at 17d0 nip 161c lr 7fff93c55100 code 2 in pandafault[1000+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/arch/powerpc

[PATCH v4 1/6] powerpc/traps: Print unhandled signals in a separate function

2018-08-01 Thread Murilo Opsfelder Araujo
Isolate the logic of printing unhandled signals out of _exception_pkey(). No functional change, only code rearrangement. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/arch

[PATCH v4 6/6] powerpc/traps: Show instructions on exceptions

2018-08-01 Thread Murilo Opsfelder Araujo
e93f0020 39400048 <9949> 3920 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index bcefbb1ee771..8494b0ff4904 100644 --- a/arch/p

Re: [PATCH v4 5/6] powerpc: Add show_user_instructions()

2018-08-10 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Fri, Aug 10, 2018 at 11:29:43AM +0200, Christophe LEROY wrote: > > > Le 03/08/2018 à 13:31, Murilo Opsfelder Araujo a écrit : > > Hi, everyone. > > > > I'd like to thank you all that contributed to refining and making this > > series better. I

Re: [PATCH] powerpc/traps: Avoid rate limit messages from show unhandled signals

2018-08-17 Thread Murilo Opsfelder Araujo
e. > > Signed-off-by: Michael Ellerman Nice catch. Thanks for patching it. Reviewed-by: Murilo Opsfelder Araujo > --- > arch/powerpc/kernel/traps.c | 13 ++--- > 1 file changed, 6 insertions(+), 7 deletions(-) > > diff --git a/arch/powerpc/kernel/traps.

Re: [PATCH 1/2] powerpc/process: fix nested output in show_user_instructions()

2018-08-17 Thread Murilo Opsfelder Araujo
Perhaps adding an output sample of how messages would look like after your patch could be an enhancement to the commit message. Otherwise: Reviewed-by: Murilo Opsfelder Araujo > This patch fixes it by preparing complete lines in a buffer and > printing it at once. > > Fixes:

Re: [PATCH 2/2] powerpc/process: Constify the number of insns printed by show instructions functions.

2018-08-17 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Tue, Aug 14, 2018 at 08:59:20AM +, Christophe Leroy wrote: > instructions_to_print var is assigned value 16 and there is no > way to change it. > > This patch replaces it by a constant. > > Signed-off-by: Christophe Leroy Reviewed-by: Muril

[PATCH v2] powerpc/prom_init: remove linux,stdout-package property

2018-07-18 Thread Murilo Opsfelder Araujo
This property was added in 2004 and the only use of it, which was already inside `#if 0`, was removed a month later. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/prom_init.c | 2 -- 1 file changed, 2 deletions(-) v1..v2: - Move github.com links to this section so they can

Re: [PATCH] powerpc/prom_init: remove linux,stdout-package property

2018-07-18 Thread Murilo Opsfelder Araujo
On Wed, Jul 18, 2018 at 07:37:37PM +1000, Michael Ellerman wrote: > Hi Murilo, > > Thanks for the patch. > > Murilo Opsfelder Araujo writes: > > This property was added in 2004 by > > > > > > https://github.com/mpe/linux-fullhistory/commit/6

[PATCH 2/7] powerpc/traps: Return early in show_signal_msg()

2018-07-24 Thread Murilo Opsfelder Araujo
Modify logic of show_signal_msg() to return early, if possible. Replace printk_ratelimited() by printk() and a default rate limit burst to limit displaying unhandled signals messages. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 21 +++-- 1 file

[PATCH 6/7] powerpc/traps: Print signal name for unhandled signals

2018-07-24 Thread Murilo Opsfelder Araujo
this patch, a page fault looks like: Jul 11 18:14:48 localhost kernel: pandafault[6352]: segfault (11) at 00013a2a09f8 nip 00013a2a086c lr 7fffb63e5100 code 2 in pandafault[13a2a+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 43

[PATCH 5/7] powerpc/traps: Print VMA for unhandled signals

2018-07-24 Thread Murilo Opsfelder Araujo
7fff8d185100 code 2 After this patch, a page fault looks like: Jul 11 16:04:11 localhost kernel: pandafault[6303]: unhandled signal 11 at 17d0 nip 161c lr 7fff93c55100 code 2 in pandafault[1000+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc

[PATCH 3/7] powerpc/reg: Add REG_FMT definition

2018-07-24 Thread Murilo Opsfelder Araujo
Make REG definition, in arch/powerpc/kernel/process.c, generic enough by renaming it to REG_FMT and placing it in arch/powerpc/include/asm/reg.h to be used elsewhere. Replace occurrences of REG by REG_FMT in arch/powerpc/kernel/process.c. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc

[PATCH 7/7] powerpc/traps: Show instructions on exceptions

2018-07-24 Thread Murilo Opsfelder Araujo
20 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/include/asm/stacktrace.h | 7 +++ arch/powerpc/kernel/process.c | 6 +++--- arch/powerpc/kernel/traps.c | 3 +++ 3 files changed, 13 insertions(+), 3 deletions(-) create mode 100644 arch/powerpc/i

[PATCH 0/7] powerpc: Modernize unhandled signals message

2018-07-24 Thread Murilo Opsfelder Araujo
0 39400048 <9949> 3920 7d234b78 383f0040 Cheers Murilo Murilo Opsfelder Araujo (7): powerpc/traps: Print unhandled signals in a separate function powerpc/traps: Return early in show_signal_msg() powerpc/reg: Add REG_FMT definition powerpc/traps: Use REG_FMT in show_signal_msg()

[PATCH 1/7] powerpc/traps: Print unhandled signals in a separate function

2018-07-24 Thread Murilo Opsfelder Araujo
Isolate the logic of printing unhandled signals out of _exception_pkey(). No functional change, only code rearrangement. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/arch

[PATCH 4/7] powerpc/traps: Use REG_FMT in show_signal_msg()

2018-07-24 Thread Murilo Opsfelder Araujo
Simplify the message format by using REG_FMT as the register format. This avoids having two different formats and avoids checking for MSR_64BIT. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git

Re: [PATCH 0/7] powerpc: Modernize unhandled signals message

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Mikey. On Wed, Jul 25, 2018 at 05:00:21PM +1000, Michael Neuling wrote: > On Tue, 2018-07-24 at 16:27 -0300, Murilo Opsfelder Araujo wrote: > > Hi, everyone. > > > > This series was inspired by the need to modernize and display more > > informative messa

Re: [PATCH 6/7] powerpc/traps: Print signal name for unhandled signals

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Gustavo. On Wed, Jul 25, 2018 at 12:19:00PM -0300, Gustavo Romero wrote: > Hi Murilo, > > LGTM. > > Just a comment: > > On 07/24/2018 04:27 PM, Murilo Opsfelder Araujo wrote: > > This adds a human-readable name in the unhandled signal message. > > &g

Re: [PATCH 6/7] powerpc/traps: Print signal name for unhandled signals

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 05:49:27PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > This adds a human-readable name in the unhandled signal message. > > > > Before this patch, a page fault looked like: > > > >

Re: [PATCH 2/7] powerpc/traps: Return early in show_signal_msg()

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 05:42:28PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Modify logic of show_signal_msg() to return early, if possible. Replace > > printk_ratelimited() by printk() and a default rate limit burst to li

Re: [PATCH 7/7] powerpc/traps: Show instructions on exceptions

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 06:01:34PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Move show_instructions() declaration to > > arch/powerpc/include/asm/stacktrace.h > > and include asm/stracktrace.h in arch/powerpc/kernel/proce

Re: [PATCH v3 1/2] powerpc: Detect the presence of big-cores via "ibm, thread-groups"

2018-07-08 Thread Murilo Opsfelder Araujo
On Fri, Jul 06, 2018 at 02:35:48PM +0530, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > On IBM POWER9, the device tree exposes a property array identifed by > "ibm,thread-groups" which will indicate which groups of threads share a > particular set of resources. > > As of today we

[PATCH] powerpc/prom_init: remove linux,stdout-package property

2018-07-12 Thread Murilo Opsfelder Araujo
/1fbe5a6d90f6cd4ea610737ef488719d1a875de7 Fixes: https://github.com/linuxppc/linux/issues/125 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/prom_init.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/kernel/prom_init.c b/arch/powerpc/kernel/prom_init.c index 5425dd3d6a9f..c45fb463c9e5

Re: [v2 PATCH 1/2] powerpc: Detect the presence of big-cores via "ibm, thread-groups"

2018-07-03 Thread Murilo Opsfelder Araujo
On Tue, Jul 03, 2018 at 04:33:50PM +0530, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > On IBM POWER9, the device tree exposes a property array identifed by > "ibm,thread-groups" which will indicate which groups of threads share a > particular set of resources. > > As of today we

Re: [v2 PATCH 2/2] powerpc: Enable CPU_FTR_ASYM_SMT for interleaved big-cores

2018-07-03 Thread Murilo Opsfelder Araujo
On Tue, Jul 03, 2018 at 04:33:51PM +0530, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" > > A pair of IBM POWER9 SMT4 cores can be fused together to form a big-core > with 8 SMT threads. This can be discovered via the "ibm,thread-groups" > CPU property in the device tree which will

Re: [v2 PATCH 2/2] powerpc: Enable CPU_FTR_ASYM_SMT for interleaved big-cores

2018-07-04 Thread Murilo Opsfelder Araujo
On Wed, Jul 04, 2018 at 01:45:05PM +0530, Gautham R Shenoy wrote: > Hi Murilo, > > Thanks for the review. > > On Tue, Jul 03, 2018 at 02:53:46PM -0300, Murilo Opsfelder Araujo wrote: > [..snip..] > > > > -/* Initialize CPU <=> thread mapping/

Re: [PATCH v3 1/5] rfi-flush: Move the logic to avoid a redo into the debugfs code

2018-03-15 Thread Murilo Opsfelder Araujo
On 03/14/2018 07:40 PM, Mauricio Faria de Oliveira wrote: > From: Michael Ellerman > > rfi_flush_enable() includes a check to see if we're already > enabled (or disabled), and in that case does nothing. > > But that means calling setup_rfi_flush() a 2nd time doesn't

Re: [PATCH] powerpc: Don't print kernel instructions in show_user_instructions()

2018-10-05 Thread Murilo Opsfelder Araujo
code 1 > bad-bctr[2969]: Bad NIP, not dumping instructions. > > Fixes: 88b0fe175735 ("powerpc: Add show_user_instructions()") > Signed-off-by: Michael Ellerman Reviewed-by: Murilo Opsfelder Araujo Thank you all! > --- > arch/powerpc/kernel/process.c | 10 ++ &

Re: [PATCH v3 1/3] powerpc/process: fix casting and missing header

2018-09-21 Thread Murilo Opsfelder Araujo
; 'show_user_instructions' was not declared. Should it be static? > > Fixes: 7b051f665c32d ("powerpc: Use probe_kernel_address in > show_instructions") > Fixes: 88b0fe1757359 ("powerpc: Add show_user_instructions()") > Signed-off-by: Christophe Leroy Smoke test pas

Re: [PATCH v3 3/3] powerpc/process: Constify the number of insns printed by show instructions functions.

2018-09-21 Thread Murilo Opsfelder Araujo
On Fri, Sep 07, 2018 at 01:47:33PM +, Christophe Leroy wrote: > instructions_to_print var is assigned value 16 and there is no > way to change it. > > This patch replaces it by a constant. > > Signed-off-by: Christophe Leroy Reviewed-by: Murilo Opsfelder Araujo >

Re: [PATCH v3 2/3] powerpc/process: fix interleaved output in show_user_instructions()

2018-09-21 Thread Murilo Opsfelder Araujo
ines in a buffer and > printing it at once. > > Fixes: 88b0fe1757359 ("powerpc: Add show_user_instructions()") > Cc: Murilo Opsfelder Araujo > Signed-off-by: Christophe Leroy Reviewed-by: Murilo Opsfelder Araujo > --- > v3: no change > v2: Using seq_buf a