Re: [PATCH v2 1/1] mm: pgtable: fix pte_swp_exclusive

2025-06-01 Thread Al Viro
On Sat, Apr 05, 2025 at 06:09:11PM +0100, Sam James wrote: > Sam James writes: > > > Lovely cleanup and a great suggestion from Al. > > > > Reviewed-by: Sam James > > > > I'd suggest adding a: > > Suggested-by: Al Viro > > Al, were yo

Re: [PATCH 1/6] Change inode_operations.mkdir to return struct dentry *

2025-02-25 Thread Al Viro
On Wed, Feb 26, 2025 at 02:18:01PM +1100, NeilBrown wrote: > Thanks. I'll submit a patch through the VFS tree as I have other VFS > patches in the works that will depend on that so having them together > would be good. Do it on top of mainline, please (say, -rc4) and let's put it into a separate

Re: [PATCH 4/6] fuse: return correct dentry for ->mkdir

2025-02-23 Thread Al Viro
On Mon, Feb 24, 2025 at 01:26:18PM +1100, NeilBrown wrote: > Probably now. It would require S_IFDIR to be passed in the mode to > vfs_mknod(). I don't think any current callers do that, but I don't see > any code in vfs_mknod() to prevent it. Not allowed (and that's caller's responsibility to e

Re: [PATCH 1/6] Change inode_operations.mkdir to return struct dentry *

2025-02-23 Thread Al Viro
On Mon, Feb 24, 2025 at 12:34:06PM +1100, NeilBrown wrote: > On Sat, 22 Feb 2025, Al Viro wrote: > > On Fri, Feb 21, 2025 at 10:36:30AM +1100, NeilBrown wrote: > > > > > +In general, filesystems which use d_instantiate_new() to install the new > > > +inode can

Re: [PATCH 1/6] Change inode_operations.mkdir to return struct dentry *

2025-02-21 Thread Al Viro
On Fri, Feb 21, 2025 at 10:36:30AM +1100, NeilBrown wrote: > Not all filesystems reliably result in a positive hashed dentry: > > - NFS, cifs, hostfs will sometimes need to perform a lookup of > the name to get inode information. Races could result in this > returning something different. No

Re: [PATCH 5/6] nfs: change mkdir inode_operation to return alternate dentry if needed.

2025-02-21 Thread Al Viro
On Fri, Feb 21, 2025 at 10:36:34AM +1100, NeilBrown wrote: > nfs3_proc_mkdir(struct inode *dir, struct dentry *dentry, struct iattr > *sattr) > { > struct posix_acl *default_acl, *acl; > @@ -612,15 +612,18 @@ nfs3_proc_mkdir(struct inode *dir, struct dentry > *dentry, struct iattr *sattr

Re: [PATCH 1/6] Change inode_operations.mkdir to return struct dentry *

2025-02-21 Thread Al Viro
On Fri, Feb 21, 2025 at 10:36:30AM +1100, NeilBrown wrote: > +In general, filesystems which use d_instantiate_new() to install the new > +inode can safely return NULL. Filesystems which may not have an I_NEW inode > +should use d_drop();d_splice_alias() and return the result of the latter. IMO t

Re: [PATCH 4/6] fuse: return correct dentry for ->mkdir

2025-02-21 Thread Al Viro
On Fri, Feb 21, 2025 at 10:36:33AM +1100, NeilBrown wrote: > @@ -871,7 +870,12 @@ static int fuse_mknod(struct mnt_idmap *idmap, struct > inode *dir, > args.in_args[0].value = &inarg; > args.in_args[1].size = entry->d_name.len + 1; > args.in_args[1].value = entry->d_name.name; >

[PATCH] hostfs: fix string handling in __dentry_name()

2025-01-17 Thread Al Viro
uot;hostfs: Remove open coded strcpy()" Signed-off-by: Al Viro --- diff --git a/fs/hostfs/hostfs_kern.c b/fs/hostfs/hostfs_kern.c index 7e51d2cec64b..bd6503b73142 100644 --- a/fs/hostfs/hostfs_kern.c +++ b/fs/hostfs/hostfs_kern.c @@ -95,32 +95,17 @@ __uml_setup("hostfs=", hostfs_args, s

Re: [PATCH v2 08/92] fs: new helper: simple_rename_timestamp

2023-09-09 Thread Al Viro
On Wed, Jul 05, 2023 at 02:58:11PM -0400, Jeff Layton wrote: > + * POSIX mandates that the old and new parent directories have their ctime > and > + * mtime updated, and that inodes of @old_dentry and @new_dentry (if any), > have > + * their ctime updated. APPLICATION USAGE Some implementations

Re: [PATCH 01/30] block: also call ->open for incremental partition opens

2023-08-24 Thread Al Viro
On Thu, Jun 08, 2023 at 01:02:29PM +0200, Christoph Hellwig wrote: > --- a/block/bdev.c > +++ b/block/bdev.c > @@ -683,9 +683,6 @@ static int blkdev_get_part(struct block_device *part, > fmode_t mode) > struct gendisk *disk = part->bd_disk; > int ret; > > - if (atomic_read(&part