[lkml]   [2009]   [Aug]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] sysrq: add debug message to reboot event
    On Wed, 19 Aug 2009 19:22:16 -0700
    "tina.yang" <> wrote:

    > Add debug message to detect keyboard vs non-keyboard triggered sysrq-b events.
    > This is to assist postmortem debugging on complicated computing setup with
    > large number of applications involved where reboot event had occurred, but
    > unclear of its origin.

    I'm still struggling to understand the motivation for the change.
    There are a large number of ways in which a machine can be rebooted,
    all the way down to a triple-fault. So it seems fairly arbitrary to
    add additional information to discriminate between just two of those

    I assume that somewhere in your setup you have a script which does
    `echo b /proc/sysrq-trigger' and it took ages to work out that this was
    happening and you felt that having this code in place would have
    helped you debug that problem, yes?

    If so, I wonder what is the likelihood that someone else will have the
    same problem and will find this change useful.

    Perhaps we should do this for all sysrq events rather than just sysrq-b?

    > --- linux-2.6.18.i686/drivers/char/sysrq.c.orig 2009-08-13 10:55:57.526459000 -0700
    > +++ linux-2.6.18.i686/drivers/char/sysrq.c 2009-08-13 10:58:10.798739000 -0700

    2.6.18 is truly ancient and this patch doesn't apply at all to
    current development kernels.

     \ /
      Last update: 2009-08-21 00:13    [W:0.033 / U:23.732 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site