Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change

xfs

Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok()

To:	Jan Kara <jack@xxxxxxx>
Subject:	Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok()
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Tue, 9 Aug 2016 02:35:31 -0700
Cc:	Christoph Hellwig <hch@xxxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Miklos Szeredi <miklos@xxxxxxxxxx>, xfs@xxxxxxxxxxx, "Yan, Zheng" <zyan@xxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, Ilya Dryomov <idryomov@xxxxxxxxx>, ceph-devel@xxxxxxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20160809093246.GB5107@xxxxxxxxxxxxxx>
References:	<1470223689-17783-1-git-send-email-jack@xxxxxxx> <1470223689-17783-2-git-send-email-jack@xxxxxxx> <20160809082756.GA11657@xxxxxxxxxxxxx> <20160809093246.GB5107@xxxxxxxxxxxxxx>
User-agent:	Mutt/1.6.1 (2016-04-27)

On Tue, Aug 09, 2016 at 11:32:46AM +0200, Jan Kara wrote:
> So do you suggest xfs_iop_change_ok and xfs_iop_setattr_nonsize? I don't
> really care, I just remember Dave had some ideas how names should be
> consistent and this naming was his idea...

Nah, just skip the prefix entirely, and maybe also already adopt to
the new setattr_prepare naming.  But if Dave already had an opinions
let's just keep it as-is and sort the naming out later.

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 0/5 v2] fs: Avoid premature clearing of file capabilities, Jan Kara [PATCH 5/5] fs: Avoid premature clearing of capabilities, Jan Kara Re: [PATCH 5/5] fs: Avoid premature clearing of capabilities, Christoph Hellwig [PATCH 4/5] fs: Give dentry to inode_change_ok() instead of inode, Jan Kara Re: [PATCH 4/5] fs: Give dentry to inode_change_ok() instead of inode, Christoph Hellwig [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Jan Kara Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Christoph Hellwig Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Jan Kara Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Christoph Hellwig <= [PATCH 2/5] ceph: Propagate dentry down to inode_change_ok(), Jan Kara [PATCH 3/5] fuse: Propagate dentry down to inode_change_ok(), Jan Kara

Previous by Date:	Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Jan Kara
Next by Date:	Re: Advice needed with file system corruption, Carlos E. R.
Previous by Thread:	Re: [PATCH 1/5] xfs: Propagate dentry down to inode_change_ok(), Jan Kara
Next by Thread:	[PATCH 2/5] ceph: Propagate dentry down to inode_change_ok(), Jan Kara
Indexes:	[Date] [Thread] [Top] [All Lists]