Re: [PATCH 1/3] xfs: rcu free inodes

["Paul E. McKenney" <paulmck@linux.vnet.ibm.com>]

On Mon, Dec 13, 2010 at 12:32:35PM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
> 
> Introduce RCU freeing of XFS inodes so that we can convert lookup
> traversals to use rcu_read_lock() protection. This patch only
> introduces the RCU freeing to minimise the potential conflicts with
> mainline if this is merged into mainline via a VFS patchset. It
> abuses the i_dentry list for the RCU callback structure because the
> VFS patches make this a union so it is safe to use like this and
> simplifies and merge issues.
> 
> This patch uses basic RCU freeing rather than SLAB_DESTROY_BY_RCU.
> The later lookup patches need the same "found free inode" protection
> regardless of the RCU freeing method used, so once again the RCU
> freeing method can be dealt with apprpriately at merge time without
> affecting any other code.

There are only two call sites that free into xfs_inode_zone.  One of them
is initialization, before readers have access to the data strcuture,
and the other is covered by this patch.  So looks good to me!

Reviewed-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
>  fs/xfs/xfs_iget.c |   14 +++++++++++++-
>  1 files changed, 13 insertions(+), 1 deletions(-)
> 
> diff --git a/fs/xfs/xfs_iget.c b/fs/xfs/xfs_iget.c
> index cdb1c25..9fae475 100644
> --- a/fs/xfs/xfs_iget.c
> +++ b/fs/xfs/xfs_iget.c
> @@ -105,6 +105,18 @@ xfs_inode_alloc(
>  }
> 
>  void
> +__xfs_inode_free(
> +	struct rcu_head		*head)
> +{
> +	struct inode		*inode = container_of((void *)head,
> +							struct inode, i_dentry);
> +	struct xfs_inode	*ip = XFS_I(inode);
> +
> +	INIT_LIST_HEAD(&inode->i_dentry);
> +	kmem_zone_free(xfs_inode_zone, ip);
> +}
> +
> +void
>  xfs_inode_free(
>  	struct xfs_inode	*ip)
>  {
> @@ -147,7 +159,7 @@ xfs_inode_free(
>  	ASSERT(!spin_is_locked(&ip->i_flags_lock));
>  	ASSERT(completion_done(&ip->i_flush));
> 
> -	kmem_zone_free(xfs_inode_zone, ip);
> +	call_rcu((struct rcu_head *)&VFS_I(ip)->i_dentry, __xfs_inode_free);
>  }
> 
>  /*
> -- 
> 1.7.2.3
> 

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs