Re: 2.6.16-rc1-mm1

[Arjan van de Ven]

On Wed, 2006-01-18 at 23:13 +0000, Alan Cox wrote:
> On Mer, 2006-01-18 at 14:09 -0500, Dave Jones wrote:
> > On Wed, Jan 18, 2006 at 03:27:16AM -0800, Andrew Morton wrote:
> > 
> >  > Well yes, that code is kfree()ing a locked mutex.  It's somewhat weird to
> >  > take a lock on a still-private object but whatever.  The code's legal
> >  > enough.
> >  > 
> 
> If someone else can be waiting on it then it doesn't look legal ?

This is why Ingo made the decision to just declare it illegal period for
mutexes (and check for it in debug mode). The obvious exceptions are
error paths, but those aren't perf criticial in any way.


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/