From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Date: Fri, 09 Oct 2015 20:48:35 +0000 Subject: Re: [PATCH] fbcon: initialize blink interval before calling fb_set_par Message-Id: <1444423715.2845.29.camel@kernel.crashing.org> List-Id: References: <2505354.BllTl4uaOg@new-mexico> <3998353.WdEhcyQWSK@new-mexico> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Scot Doyle , Tomi Valkeinen , Jean-Christophe Plagniol-Villard Cc: Greg Kroah-Hartman , Alistair Popple , Pavel Machek , airlied@redhat.com, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org On Fri, 2015-10-09 at 15:08 +0000, Scot Doyle wrote: > Since commit 27a4c827c34ac4256a190cc9d24607f953c1c459 > fbcon: use the cursor blink interval provided by vt > > a PPC64LE kernel fails to boot when fbcon_add_cursor_timer uses an > uninitialized ops->cur_blink_jiffies. Prevent by initializing > in fbcon_init before the call to info->fbops->fb_set_par. > Reported-and-tested-by: Alistair Popple > Signed-off-by: Scot Doyle Please add CC: [v4.2] This bug will cause any machine using fbcon to occasionally fail to boot due to having a timer callback try to mod_timer with "jiffies + 0" without an exit condition. Cheers, Ben. > --- > drivers/video/console/fbcon.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/video/console/fbcon.c > b/drivers/video/console/fbcon.c > index 1aaf893..92f3949 100644 > --- a/drivers/video/console/fbcon.c > +++ b/drivers/video/console/fbcon.c > @@ -1093,6 +1093,7 @@ static void fbcon_init(struct vc_data *vc, int > init) > con_copy_unimap(vc, svc); > > ops = info->fbcon_par; > + ops->cur_blink_jiffies = msecs_to_jiffies(vc > ->vc_cur_blink_ms); > p->con_rotate = initial_rotation; > set_blitting_type(vc, info); > > -- > 2.1.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux > -fbdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html