[PATCH] drm: avoid switching to text console if there is no panic timeout

Dave Airlie airlied at gmail.com
Thu Nov 10 13:29:16 PST 2011


On Thu, Nov 10, 2011 at 9:15 PM, Mandeep Singh Baines <msb at chromium.org> wrote:
> David Rientjes (rientjes at google.com) wrote:
>> On Mon, 17 Oct 2011, David Rientjes wrote:
>>
>> > On Mon, 17 Oct 2011, Mandeep Singh Baines wrote:
>> >
>> > > From: Hugh Dickins <hughd at chromium.org>
>> > >
>> > > Add a check for panic_timeout in the drm_fb_helper_panic() notifier: if
>> > > we're going to reboot immediately, the user will not be able to see the
>> > > messages anyway, and messing with the video mode may display artifacts,
>> > > and certainly get into several layers of complexity (including mutexes and
>> > > memory allocations) which we shall be much safer to avoid.
>> > >
>> > > Signed-off-by: Hugh Dickins <hughd at google.com>
>> > > [ Edited commit message and modified to short-circuit panic_timeout < 0
>> > >   instead of testing panic_timeout >= 0.  -Mandeep ]
>> > > Signed-off-by: Mandeep Singh Baines <msb at chromium.org>
>> > > Cc: Dave Airlie <airlied at redhat.com>
>> > > Cc: Andrew Morton <akpm at linux-foundation.org>
>> > > Cc: dri-devel at lists.freedesktop.org
>> >
>> > Acked-by: David Rientjes <rientjes at google.com>
>> >
>>
>> Dave, where do we stand on this?  I haven't seen it hit Linus' tree and I
>> don't see it in git://people.freedesktop.org/~airlied/linux.

I've just pulled it into my local drm-next, thanks for reminding me.

Dave.


More information about the dri-devel mailing list