[PATCH v4 7/7] configfs: Fix LOCKDEP nesting issues with fragment semaphores

Christoph Hellwig hch at lst.de
Thu Aug 11 08:09:39 PDT 2022


On Fri, Aug 05, 2022 at 03:15:53PM +0100, Mike Leach wrote:
> You are correct - in this patchset we are adding the use of a binary
> attribute to load and unload Coresight configurations and features -
> which action has a side effect of adding and removing entries in
> particular sub-directories in our configfs sub-system.

I don't think configfs was designed to be that dynamic.  Especially
the fact that a write to a binary attribute, which is intended to
be just for passthrough to hardware is a bit concerning.

>From your further description it sounds like this binary attribute
is loading structured data interpreted by the kernel, which really
isn't how binary attributes in configfs or sysfs are intended to
be used.



More information about the linux-arm-kernel mailing list