Messages in this thread | | | Subject | Re: [PATCH] filelock: add stubs for new functions when CONFIG_FILE_LOCKING=n | From | Jeff Layton <> | Date | Mon, 05 Feb 2024 07:20:18 -0500 |
| |
On Mon, 2024-02-05 at 13:10 +0100, Christian Brauner wrote: > > Another thought too: "locks_" as a prefix is awfully generic. Might it be > > better to rename these new functions with a "filelock_" prefix instead? > > That would better distinguish to the casual reader that this is dealing > > with a file_lock object. I'm happy to respin the set if that's the > > consensus. > > If it's just a rename then just point me to a branch I can pull. I don't > think it's worth resending just because you effectively did some variant > of s/lock_*/filelock_*/g > > In any case, folded this one.
Thanks!
I haven't done a rename (yet). I was just trying to feel out whether it was worthwhile. At this point, I'm thinking I'll just leave them as-is., but let me know if anyone has opinions to the contrary. -- Jeff Layton <jlayton@kernel.org>
| |