Messages in this thread | | | Date | Tue, 21 Dec 2021 20:57:36 +0100 | From | Andrew Lunn <> | Subject | Re: [RFC PATCH v12 00/17] dlb: introduce DLB device driver |
| |
> Hey Greg, > > This is my fault. > > To date Mike has been patiently and diligently following my review > feedback to continue to make the driver smaller and more Linux > idiomatic. Primarily this has been ripping and replacing a pile of > object configuration ioctls with configfs. While my confidence in that > review feedback was high, my confidence in the current round of deeper > architecture reworks is lower and they seemed to raise questions that > are likely FAQs with using configfs. Specifically the observation that > configfs, like sysfs, lacks an "atomically update multiple attributes" > capability. To my knowledge that's just the expected tradeoff with > pseudo-fs based configuration and it is up to userspace to coordinate > multiple configuration writers.
Hi Dan
If this is considered a network accelerator, it probably should use the same configuration mechanisms all of networking uses, netlink. I'm not aware of anything network related using configfs, but it could exist. netlink messages should also solve your atomisity problem.
But it does not really help with cleanup when the userspace user goes away. Is there anything from GPU drivers which can be reused? They must have some sort of cleanup when the user space DRM driver exits.
Andrew
| |