On Thu, Aug 21, 2025 at 12:48:15AM +0100, Al Viro wrote: > On Tue, Aug 19, 2025 at 01:39:09AM +0100, Al Viro wrote: > > I'm still trying to come up with something edible for lock_mount() - > > the best approximation I've got so far is > > > > CLASS(lock_mount, mp)(path); > > if (IS_ERR(mp.mp)) > > bugger off > > ... and that does not work, since DEFINE_CLASS() has constructor return > a value that gets copied into the local variable in question. > > Which is unusable for situations when a part of what constructor is > doing is insertion of that local variable into a list. > > __cleanup() per se is still usable, but... no DEFINE_CLASS for that kind > of data structures ;-/ Just add the custom infrastructure that we need for this to work out imho. If it's useful outside of our own realm then we can add it to cleanup.h and if not we can just add our own header...
