Re: [PATCH 12/15] xfs: use vfs inode nlink field everywhere

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 12/15] xfs: use vfs inode nlink field everywhere
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Tue, 16 Feb 2016 23:35:06 -0800
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<1455693652-3899-13-git-send-email-david@xxxxxxxxxxxxx>
References:	<1455693652-3899-1-git-send-email-david@xxxxxxxxxxxxx> <1455693652-3899-13-git-send-email-david@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.24 (2015-08-30)

> index 685c7a7..1408be3 100644
> --- a/libxfs/libxfs_api_defs.h
> +++ b/libxfs/libxfs_api_defs.h
> @@ -108,4 +108,8 @@
>  
>  #define xfs_verify_cksum             libxfs_verify_cksum
>  
> +/* inode link counts */
> +#define set_nlink(inode, nlink)      ({ (inode)->i_nlink = (nlink); })
> +#define inc_nlink(inode)     (inode)->i_nlink++

Can we defines these as inlines?

> +             ip->i_vnode.i_nlink++;          /* account for . */

Shouldn't we use VFS_I() like everywhere else?

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 02/15] xfs: handle errors from ->free_blocks in xfs_btree_kill_iroot, (continued) [PATCH 02/15] xfs: handle errors from ->free_blocks in xfs_btree_kill_iroot, Dave Chinner [PATCH 04/15] xfs: move buffer invalidation to xfs_btree_free_block, Dave Chinner [PATCH 06/15] xfs: RT bitmap and summary buffers are not typed, Dave Chinner [PATCH 01/15] xfs: wire up Q_XGETNEXTQUOTA / get_nextdqblk, Dave Chinner Re: [PATCH 01/15] xfs: wire up Q_XGETNEXTQUOTA / get_nextdqblk, Christoph Hellwig [PATCH 03/15] xfs: factor btree block freeing into a helper, Dave Chinner [PATCH 08/15] xfs: introduce inode log format object, Dave Chinner [PATCH 05/15] xfs: remove unused function definitions, Dave Chinner [PATCH 10/15] xfs: cull unnecessary icdinode fields, Dave Chinner [PATCH 12/15] xfs: use vfs inode nlink field everywhere, Dave Chinner Re: [PATCH 12/15] xfs: use vfs inode nlink field everywhere, Christoph Hellwig <= Re: [PATCH 12/15] xfs: use vfs inode nlink field everywhere, Dave Chinner [PATCH 15/15] xfs: mode di_mode to vfs inode, Dave Chinner [PATCH 09/15] xfs: remove timestamps from incore inode, Dave Chinner Re: [PATCH 09/15] xfs: remove timestamps from incore inode, Brian Foster Re: [PATCH 09/15] xfs: remove timestamps from incore inode, Darrick J. Wong [PATCH 07/15] xfs: RT bitmap and summary buffers need verifiers, Dave Chinner [PATCH 14/15] xfs: move di_changecount to VFS inode, Dave Chinner [PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk, Dave Chinner Re: [PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk, Christoph Hellwig Re: [PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk, Dave Chinner

Previous by Date:	Re: [PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk, Christoph Hellwig
Next by Date:	Re: [PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk, Dave Chinner
Previous by Thread:	[PATCH 12/15] xfs: use vfs inode nlink field everywhere, Dave Chinner
Next by Thread:	Re: [PATCH 12/15] xfs: use vfs inode nlink field everywhere, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]