[PATCH] fbcon: warn on invalid cursor blink intervals

Chintakuntla, Radha Radha.Chintakuntla at cavium.com
Sun Jun 12 05:05:27 UTC 2016


> -----Original Message-----
> From: Henrique de Moraes Holschuh [mailto:hmh at hmh.eng.br]
> Sent: Saturday, May 28, 2016 4:49 AM
> To: Scot Doyle <lkml14 at scotdoyle.com>
> Cc: Tomi Valkeinen <tomi.valkeinen at ti.com>; Jean-Christophe Plagniol-
> Villard <plagnioj at jcrosoft.com>; Greg Kroah-Hartman
> <gregkh at linuxfoundation.org>; Jeremy Kerr <jk at ozlabs.org>; Ming Lei
> <ming.lei at canonical.com>; Daney, David <David.Daney at cavium.com>;
> Dann Frazier <dann.frazier at canonical.com>; Peter Hurley
> <peter at hurleysoftware.com>; Pavel Machek <pavel at ucw.cz>; Jonathan Liu
> <net147 at gmail.com>; Alistair Popple <alistair at popple.id.au>; Jean-Philippe
> Brucker <jean-philippe.brucker at arm.com>; Chintakuntla, Radha
> <Radha.Chintakuntla at cavium.com>; Jiri Slaby <jslaby at suse.com>; David
> Airlie <airlied at linux.ie>; David Daney <ddaney.cavm at gmail.com>; dri-
> devel at lists.freedesktop.org; linux-fbdev at vger.kernel.org; Linux Kernel
> Mailing List <linux-kernel at vger.kernel.org>; stable
> <stable at vger.kernel.org>
> Subject: Re: [PATCH] fbcon: warn on invalid cursor blink intervals
> 
> On Fri, 20 May 2016, Scot Doyle wrote:
> > On Fri, 20 May 2016, Jeremy Kerr wrote:
> > > >Then looks there are two fix patches acked & tested:
> > > >
> > > > - the patch in this thread
> > > > - another one "[PATCH] tty: vt: Fix soft lockup in fbcon cursor
> > > >blink timer."
> > > > https://lkml.org/lkml/2016/5/17/455
> > > >
> > > >So which one will be pushed to linus?
> > >
> > > Not that it's my call, but we may want both; the first as a safety
> > > measure to prevent an invalid cur_blink_jiffies ever being set, and the
> > > second one to actually fix the initialisation of vc_cur_blink_ms (and
> > > address the warning introduced by the first).
> >
> > Tomi / Greg,
> >
> > I'd suggest
> > - applying "tty: vt: Fix soft lockup in fbcon cursor blink timer." to 4.7 and
> stable[4.2]
> > - applying "fbcon: warn on invalid cursor blink intervals" to 4.7
> > - ignoring "fbcon: use default if cursor blink interval is not valid"
> >
> > Note: the patches don't depend on each other
> 
> I applied both recommended patches on top of 4.4.11 for testing, and they
> made things a lot better here.
> 
> I suggest the second patch should be backported to stable too, might as well
> fix this thing for good *and keep the door closed*.

Is this patch available on some tree so that I can point to ?
And hope it will make it to linux-next soon ?

> 
> --
>   "One disk to rule them all, One disk to find them. One disk to bring
>   them all and in the darkness grind them. In the Land of Redmond
>   where the shadows lie." -- The Silicon Valley Tarot
>   Henrique Holschuh


More information about the dri-devel mailing list