diff mbox

[v2] powerpc: Append linux_banner to exception information in xmon.

Message ID 1448419585-5181-1-git-send-email-rashmicy@gmail.com (mailing list archive)
State Accepted
Headers show

Commit Message

Rashmica Gupta Nov. 25, 2015, 2:46 a.m. UTC
Currently if you are in xmon without an oops etc. to view the kernel
version you have to type "d $linux_banner" - not necessarily obvious. As
this is useful information, append to the output of "e" command.

Example output:
$mon> e
cpu 0x1: Vector: 0  at [c0000000f879ba80]
    pc: c000000000081718: sysrq_handle_xmon+0x68/0x80
    lr: c000000000081718: sysrq_handle_xmon+0x68/0x80
    sp: c0000000f879bbe0
   msr: 8000000000009033
  current = 0xc0000000f604d5c0
  paca    = 0xc00000000fdc0480	 softe: 0	 irq_happened: 0x01
    pid   = 2467, comm = bash
Linux version 4.4.0-rc2-00008-gc51af91c3ab3-dirty (rashmica@circle) (gcc
version 5.1.1 20150629 (GCC) ) #45 SMP Wed Nov 25 10:25:12 AEDT 2015

Signed-off-by: Rashmica Gupta <rashmicy@gmail.com>
---
Added example output and word wrapping.

 arch/powerpc/xmon/xmon.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Denis Kirjanov Nov. 25, 2015, 8:15 a.m. UTC | #1
On 11/25/15, Rashmica Gupta <rashmicy@gmail.com> wrote:
> Currently if you are in xmon without an oops etc. to view the kernel
> version you have to type "d $linux_banner" - not necessarily obvious. As
> this is useful information, append to the output of "e" command.
>
> Example output:
> $mon> e
> cpu 0x1: Vector: 0  at [c0000000f879ba80]
>     pc: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     lr: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     sp: c0000000f879bbe0
>    msr: 8000000000009033
>   current = 0xc0000000f604d5c0
>   paca    = 0xc00000000fdc0480	 softe: 0	 irq_happened: 0x01
>     pid   = 2467, comm = bash
> Linux version 4.4.0-rc2-00008-gc51af91c3ab3-dirty (rashmica@circle) (gcc
> version 5.1.1 20150629 (GCC) ) #45 SMP Wed Nov 25 10:25:12 AEDT 2015
>
> Signed-off-by: Rashmica Gupta <rashmicy@gmail.com>
You forgot to update the subject line
> ---
> Added example output and word wrapping.
>
>  arch/powerpc/xmon/xmon.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c
> index df05bd2fca07..6d3d9d031aea 100644
> --- a/arch/powerpc/xmon/xmon.c
> +++ b/arch/powerpc/xmon/xmon.c
> @@ -1528,6 +1528,8 @@ static void excprint(struct pt_regs *fp)
>
>  	if (trap == 0x700)
>  		print_bug_trap(fp);
> +
> +	printf(linux_banner);
>  }
>
>  static void prregs(struct pt_regs *fp)
> --
> 2.5.0
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
Michael Ellerman Nov. 25, 2015, 11:15 a.m. UTC | #2
On 25 November 2015 7:15:46 pm AEDT, Denis Kirjanov <kda@linux-powerpc.org> wrote:
>You forgot to update the subject line

That's OK I will fix it up when I apply it.

cheers
Stewart Smith Dec. 2, 2015, 10:43 p.m. UTC | #3
Rashmica Gupta <rashmicy@gmail.com> writes:
> Currently if you are in xmon without an oops etc. to view the kernel
> version you have to type "d $linux_banner" - not necessarily obvious. As
> this is useful information, append to the output of "e" command.
>
> Example output:
> $mon> e
> cpu 0x1: Vector: 0  at [c0000000f879ba80]
>     pc: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     lr: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     sp: c0000000f879bbe0
>    msr: 8000000000009033
>   current = 0xc0000000f604d5c0
>   paca    = 0xc00000000fdc0480	 softe: 0	 irq_happened: 0x01
>     pid   = 2467, comm = bash
> Linux version 4.4.0-rc2-00008-gc51af91c3ab3-dirty (rashmica@circle) (gcc
> version 5.1.1 20150629 (GCC) ) #45 SMP Wed Nov 25 10:25:12 AEDT 2015
>
> Signed-off-by: Rashmica Gupta <rashmicy@gmail.com>

+10,245,789

Acked-by: Stewart Smith <stewart@linux.vnet.ibm.com>

If I had a dollar for every time this would have helped *this morning*,
I'd at least be buying you enough coffee for it to be problematic.

I'd be keen in it heading to stable@ fixing the actual problem of
"people can't actually include the damn kernel version in their bug
report" - but maybe somebody else can work out if this is something
that's likely to be accepted.
Michael Ellerman Dec. 15, 2015, 11:27 a.m. UTC | #4
On Wed, 2015-25-11 at 02:46:25 UTC, Rashmica Gupta wrote:
> Currently if you are in xmon without an oops etc. to view the kernel
> version you have to type "d $linux_banner" - not necessarily obvious. As
> this is useful information, append to the output of "e" command.
> 
> Example output:
> $mon> e
> cpu 0x1: Vector: 0  at [c0000000f879ba80]
>     pc: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     lr: c000000000081718: sysrq_handle_xmon+0x68/0x80
>     sp: c0000000f879bbe0
>    msr: 8000000000009033
>   current = 0xc0000000f604d5c0
>   paca    = 0xc00000000fdc0480	 softe: 0	 irq_happened: 0x01
>     pid   = 2467, comm = bash
> Linux version 4.4.0-rc2-00008-gc51af91c3ab3-dirty (rashmica@circle) (gcc
> version 5.1.1 20150629 (GCC) ) #45 SMP Wed Nov 25 10:25:12 AEDT 2015
> 
> Signed-off-by: Rashmica Gupta <rashmicy@gmail.com>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/eb925d64604991095b6e9476

cheers
diff mbox

Patch

diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c
index df05bd2fca07..6d3d9d031aea 100644
--- a/arch/powerpc/xmon/xmon.c
+++ b/arch/powerpc/xmon/xmon.c
@@ -1528,6 +1528,8 @@  static void excprint(struct pt_regs *fp)
 
 	if (trap == 0x700)
 		print_bug_trap(fp);
+
+	printf(linux_banner);
 }
 
 static void prregs(struct pt_regs *fp)