Re: [PATCH 03/11] xfs: rename xfs_ilock_map

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Sun, 8 Dec 2013 23:03:16 -0800
Cc:	Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20131208223359.GD31386@dastard>
References:	<20131206203006.914776999@xxxxxxxxxxxxxxxxxxxxxx> <20131206203128.511935311@xxxxxxxxxxxxxxxxxxxxxx> <20131208223359.GD31386@dastard>
User-agent:	Mutt/1.5.21 (2010-09-15)

On Mon, Dec 09, 2013 at 09:33:59AM +1100, Dave Chinner wrote:
> While we are changing this, I think it makes sense to move it to
> being a static inline function given how simple it is....

I actually tried that first, but with XFS_DINODE_FMT_BTREE in
fs/xfs/xfs_dinode.h, struct xfsicdinode in xfs_log_format.h
and the resulting dependencies it didn't seem workable.  My inode
reshuffle might provide relief eventually.

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 00/11] extent list locking fixes V3, Christoph Hellwig [PATCH 06/11] xfs: take the ilock around xfs_bmapi_read in xfs_zero_remaining_bytes, Christoph Hellwig Re: [PATCH 06/11] xfs: take the ilock around xfs_bmapi_read in xfs_zero_remaining_bytes, Dave Chinner [PATCH 09/11] xfs: use xfs_ilock_attr_map_shared in xfs_attr_get, Christoph Hellwig Re: [PATCH 09/11] xfs: use xfs_ilock_attr_map_shared in xfs_attr_get, Dave Chinner [PATCH 05/11] xfs: reinstate the ilock in xfs_readdir, Christoph Hellwig Re: [PATCH 05/11] xfs: reinstate the ilock in xfs_readdir, Dave Chinner [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Christoph Hellwig Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Dave Chinner Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Christoph Hellwig <= Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Dave Chinner [PATCH 10/11] xfs: use xfs_ilock_attr_map_shared in xfs_attr_list_int, Christoph Hellwig Re: [PATCH 10/11] xfs: use xfs_ilock_attr_map_shared in xfs_attr_list_int, Dave Chinner [PATCH 08/11] xfs: use xfs_ilock_data_map_shared in xfs_qm_dqiterate, Christoph Hellwig Re: [PATCH 08/11] xfs: use xfs_ilock_data_map_shared in xfs_qm_dqiterate, Dave Chinner [PATCH 07/11] xfs: use xfs_ilock_data_map_shared in xfs_qm_dqtobp, Christoph Hellwig Re: [PATCH 07/11] xfs: use xfs_ilock_data_map_shared in xfs_qm_dqtobp, Dave Chinner [PATCH 11/11] xfs: assert that we hold the ilock for extent map access, Christoph Hellwig Re: [PATCH 11/11] xfs: assert that we hold the ilock for extent map access, Dave Chinner [PATCH 01/11] xfs: no need to lock the inode in xfs_find_handle, Christoph Hellwig

Previous by Date:	Re: [PATCH v2 2/3] xfs: fix infinite loop by detaching the group/project hints from user dquot, Dave Chinner
Next by Date:	Re: [PATCH v2 2/3] xfs: fix infinite loop by detaching the group/project hints from user dquot, Christoph Hellwig
Previous by Thread:	Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Dave Chinner
Next by Thread:	Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]

Re: [PATCH 03/11] xfs: rename xfs_ilock_map_shared