[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20140825233443.GB13130@type.youpi.perso.aquilenet.fr>
Date: Tue, 26 Aug 2014 01:34:43 +0200
From: Samuel Thibault <samuel.thibault@...-lyon.org>
To: Hugh Dickins <hughd@...gle.com>
Cc: Sabrina Dubroca <sd@...asysnail.net>, Valdis.Kletnieks@...edu,
Vincent Donnefort <vdonnefort@...il.com>,
Bryan Wu <cooloney@...il.com>, linux-leds@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org
Subject: Re: 3.17-rc1: leds blink workqueue causes sleeping BUGs
Hugh Dickins, le Mon 25 Aug 2014 15:00:44 -0700, a écrit :
> On Mon, 25 Aug 2014, Samuel Thibault wrote:
> > Samuel Thibault, le Mon 25 Aug 2014 23:23:24 +0200, a écrit :
> > > We could indeed have a loop if the user was making the VT::* leds use
> > > the vt-* trigger,
> >
> > Actually, while there can be a loop, it wouldn't be possible to inject
> > events in it: a VT::* led only makes the corresponding vt-* trigger if
> > it got an event from its trigger, etc. So it's really a false positive,
> > the lock detector just can not know that it can not happen.
>
> I'm not suffering from this lockdep warning myself; but, false positive
> or not, it does need to be fixed (or annotated). Because once lockdep
> reports one issue, it turns itself off. So any developer who hits this
> warning is then unable test their own changes with lockdep afterwards.
Ew. I'll have a look.
Samuel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists