Thread (26 messages) 26 messages, 3 authors, 2025-11-11

Re: [PATCH v5 09/17] vfs: clean up argument list for vfs_create()

From: NeilBrown <hidden>
Date: 2025-11-05 21:27:46
Also in: linux-cifs, linux-fsdevel, linux-nfs, linux-unionfs, linux-xfs, lkml, netfs

On Thu, 06 Nov 2025, Jeff Layton wrote:
As Neil points out:

"I would be in favour of dropping the "dir" arg because it is always
d_inode(dentry->d_parent) which is stable."

...and...

"Also *every* caller of vfs_create() passes ".excl = true".  So maybe we
don't need that arg at all."

Drop both arguments from vfs_create() and fix up the callers.

Suggested-by: NeilBrown <redacted>
Signed-off-by: Jeff Layton <jlayton@kernel.org>
This I like.

Reviewed-by: NeilBrown <neil@brown.name>

It would be consistent to also remove the 'dir' arg from vfs_mkdir(),
vfs_mknod(), etc.  I wouldn't do that until we find out what other
people think of the change.

Thanks,
NeilBrown

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help