From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758904Ab3APJfT (ORCPT ); Wed, 16 Jan 2013 04:35:19 -0500 Received: from cantor2.suse.de ([195.135.220.15]:39485 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758217Ab3APJfQ (ORCPT ); Wed, 16 Jan 2013 04:35:16 -0500 Date: Wed, 16 Jan 2013 10:35:12 +0100 Message-ID: From: Takashi Iwai To: sedat.dilek@gmail.com Cc: Jiri Kosina , linux-fbdev@vger.kernel.org, LKML , alan@lxorguk.ukuu.org.uk, Andrew Morton , "Rafael J. Wysocki" , Greg Kroah-Hartman , Jiri Slaby Subject: Re: 3.8-rc2 lockdep complains about console_lock vs. fb_notifier_list.rwsem In-Reply-To: References: User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.2 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Wed, 16 Jan 2013 10:21:46 +0100, Sedat Dilek wrote: > > On Tue, Jan 15, 2013 at 3:25 PM, Takashi Iwai wrote: > > At Sat, 5 Jan 2013 13:13:27 +0100, > > Sedat Dilek wrote: > >> > >> Hi Jiri, > >> > >> ...known issue (see thread in [1]), please feel free to test patches > >> from Alan and Andrew (see [1], [2] and [3]) and report. > >> > >> Regards, > >> - Sedat - > >> > >> [1] http://marc.info/?t=135309396400003&r=1&w=2 > >> [2] http://ozlabs.org/~akpm/mmots/broken-out/fb-rework-locking-to-fix-lock-ordering-on-takeover.patch > >> [3] http://ozlabs.org/~akpm/mmots/broken-out/fb-rework-locking-to-fix-lock-ordering-on-takeover-fix.patch > >> [4] http://ozlabs.org/~akpm/mmots/broken-out/fb-rework-locking-to-fix-lock-ordering-on-takeover-fix-2.patch > > > > I've hit this bug and tried the patch [2] ([3] and [4] are gone). > > Unfortunately the deadlock is still reported, as seen below. > > > > A similar fix for fbcon_unbind(), splitting an unlocked version of > > unbind_con_driver() and call it? > > > > (BTW, the patch [2] contains strange characters in the comments, and > > has a few coding issues easily detected by checkpatch.pl.) > > > > [ CCing Rafael as he asked in another thread if I had sent a patch ] > > I noticed also some these "strange" chars in the patch from Andrew and > a patch of mine was sent as > "fb-Rework-locking-to-fix-lock-ordering-on-takeover-fix-comments.patch" > to the lists. > > It is strange to me that Andrew or other maintainers himself did not > check with "checkpatch.pl". > This should be a common testcase! > Hey, noone is perfect :-). > ( /me has also not checked with that script - I just saw it. ) > > Just as a note to the issue in general: > Andrew took over the patch from Alan which is very honest - he is not > the active TTY maintainer! > Didn't Greg takeover maintenance from Alan (after a dispute and blaming Alan)? > If this is true, why the hell is Greg not CCed? Let's do it :) Greg, Jiri, this bug hits already quite a few people. I can reproduce the bug easily on a machine with a radeon graphics. It appears always at boot when lockdep is enabled. > Thanks for the real patch in the followup of this thread! > Who will take care of it :-)? Either tty maintainer, or fb maintainer, or Andrew? FWIW, Andrew took my patch in mm: http://ozlabs.org/~akpm/mmots/broken-out/fb-rework-locking-to-fix-lock-ordering-on-takeover.patch http://ozlabs.org/~akpm/mmots/broken-out/fb-yet-another-band-aid-for-fixing-lockdep-mess.patch thanks, Takashi