Re: Lockdep false positive in sysfs

From: Tejun Heo
Date: Fri Apr 27 2012 - 12:09:08 EST


Hello,

On Fri, Apr 27, 2012 at 11:57:02AM -0400, Alan Stern wrote:
> By the way, do you know why attribute structures allow for dynamic keys
> as well as static keys? I see dynamic keys are used by attribute
> containers, but I don't understand why.

I have no idea. Eric?

> > > Another idea is to have A's method temporarily drop the sysfs readlock.
> > > Of course that would put the onus on the USB core of guaranteeing that
> > > A cannot be removed while this happens, but we can handle that.
> >
> > Yeah, that's an easier way out. Please make it a proper sysfs API
> > call tho so that people working on sysfs later can know of the special
> > case.
>
> I will.
>
> Would it be better to release just the lockdep annotation while
> continuing to hold the actual lock, or to really drop the lock?

Just the lockdep annotation, I think.

Thanks.

--
tejun
--
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/