RE: [PATCH] fbcon: warn on invalid cursor blink intervals

use.com>; David > Airlie <airl...@linux.ie>; David Daney <ddaney.c...@gmail.com>; dri- > de...@lists.freedesktop.org; linux-fb...@vger.kernel.org; Linux Kernel > Mailing List <linux-kernel@vger.kernel.org>; stable > <sta...@vger.kernel.org> > Subject: Re: [PATCH] f

RE: [PATCH] fbcon: warn on invalid cursor blink intervals

t ; stable > > 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: > > > > > > &g

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." > > >

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." > > >

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Thu, 19 May 2016, Scot Doyle wrote: > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. > > [1] https://bugs.launchpad.net/bugs/1574814 > > Suggested-by: David Daney

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Thu, 19 May 2016, Scot Doyle wrote: > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. > > [1] https://bugs.launchpad.net/bugs/1574814 > > Suggested-by: David Daney >

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: > > Hi Ming, > > > > >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." > > >

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: > > Hi Ming, > > > > >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." > > >

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, 20 May 2016, Jeremy Kerr wrote: > Hi Ming, > > >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

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, 20 May 2016, Jeremy Kerr wrote: > Hi Ming, > > >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

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Ming, >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

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Ming, >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

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, May 20, 2016 at 10:26 AM, Jeremy Kerr wrote: > Hi Ming, > >> Not sure this one is needed for stable because it justs dumps >> a warning, and not set a valid period to ops->cur_blink_jiffies. >> >> So I guess other fix patch is still required for the soft lockup >> issue,

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, May 20, 2016 at 10:26 AM, Jeremy Kerr wrote: > Hi Ming, > >> Not sure this one is needed for stable because it justs dumps >> a warning, and not set a valid period to ops->cur_blink_jiffies. >> >> So I guess other fix patch is still required for the soft lockup >> issue, right? > > The

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Ming, > Not sure this one is needed for stable because it justs dumps > a warning, and not set a valid period to ops->cur_blink_jiffies. > > So I guess other fix patch is still required for the soft lockup > issue, right? The main thing is that we don't set cur_blink_jiffies to the < 50ms

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Ming, > Not sure this one is needed for stable because it justs dumps > a warning, and not set a valid period to ops->cur_blink_jiffies. > > So I guess other fix patch is still required for the soft lockup > issue, right? The main thing is that we don't set cur_blink_jiffies to the < 50ms

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, May 20, 2016 at 6:31 AM, Scot Doyle wrote: > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. > > [1] https://bugs.launchpad.net/bugs/1574814 > >

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On Fri, May 20, 2016 at 6:31 AM, Scot Doyle wrote: > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. > > [1] https://bugs.launchpad.net/bugs/1574814 > > Suggested-by: David Daney >

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Scot, > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. This prevents a lockup on AST BMC machines, but (as expected) generates a warning against the fbcon driver, which is a

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

Hi Scot, > Two systems are locking on boot [1] because ops->cur_blink_jiffies > is set to zero from vc->vc_cur_blink_ms. > > Ignore such invalid intervals and log a warning. This prevents a lockup on AST BMC machines, but (as expected) generates a warning against the fbcon driver, which is a

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On 05/19/2016 03:31 PM, Scot Doyle wrote: Two systems are locking on boot [1] because ops->cur_blink_jiffies is set to zero from vc->vc_cur_blink_ms. Ignore such invalid intervals and log a warning. [1] https://bugs.launchpad.net/bugs/1574814 Suggested-by: David Daney

Re: [PATCH] fbcon: warn on invalid cursor blink intervals

On 05/19/2016 03:31 PM, Scot Doyle wrote: Two systems are locking on boot [1] because ops->cur_blink_jiffies is set to zero from vc->vc_cur_blink_ms. Ignore such invalid intervals and log a warning. [1] https://bugs.launchpad.net/bugs/1574814 Suggested-by: David Daney Signed-off-by: Scot

[PATCH] fbcon: warn on invalid cursor blink intervals

Two systems are locking on boot [1] because ops->cur_blink_jiffies is set to zero from vc->vc_cur_blink_ms. Ignore such invalid intervals and log a warning. [1] https://bugs.launchpad.net/bugs/1574814 Suggested-by: David Daney Signed-off-by: Scot Doyle

[PATCH] fbcon: warn on invalid cursor blink intervals

Two systems are locking on boot [1] because ops->cur_blink_jiffies is set to zero from vc->vc_cur_blink_ms. Ignore such invalid intervals and log a warning. [1] https://bugs.launchpad.net/bugs/1574814 Suggested-by: David Daney Signed-off-by: Scot Doyle Cc: [v4.2] ---