Re: [RFC/PATCH-mm] i8042: activate panic blink only in X

[Grant Coady <gcoady.lk@gmail.com>]

On Tue, 5 Sep 2006 01:03:58 +0200, Andreas Mohr <andi@rhlx01.fht-esslingen.de> wrote:

>Hi,
>
>On Tue, Sep 05, 2006 at 08:29:09AM +1000, Grant Coady wrote:
>> If possible, kill the console blank timer too?  (dunno if you have).
>> 
>> Example: Oops screen on 2.4 recently I paged up to where the fault 
>> started, the screen blanker kicked in while I was hand copying info 
>> and wiped previous screens :(  Caused significant delay in working 
>> out what the issue was (slackware-10.2 2.4.33.1 glibc nptl boo-boo).
>> 
>> Is it safe / easy to do on oops/panic?
>
>It should be as easy as e.g. doing a
>
>int system_oopsed __read_mostly = 0;

Can't see where to set this one, from the kernel/panic.c notifier chain?

>if (!system_oopsed)
>    blank();

drivers/char/vt.c:
...
static void blank_screen_t(unsigned long dummy)
{
        if (unlikely(!keventd_up())) {
                mod_timer(&console_timer, jiffies + blankinterval);
                return;
        }
        blank_timer_expired = 1;
        schedule_work(&console_work);
}

Looks like the place to stop the screen blanker, that's as far as I'm game 
to travel.  

Hooking the panic notifier chain looks too scary for this little black duck ;)

Grant.
>
>in the timer handler as opposed to painfully deregistering the whole handler
>in the critical system state after an OOPS.
>A funny side effect is that this way even improves system stability
>after OOPS, since the blanking (which doesn't happen then)
>might bomb, too ;)
>
>Andreas Mohr

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/