xfs
[Top] [All Lists]
<prev [Date] next>
[Advanced]
 <prev [Thread] next>

Re: [PATCH] xfs: fix inode pincount check in fsync

from [Dave Chinner] [Permanent Link][Original]
To: Christoph Hellwig <hch@xxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>
Subject: Re: [PATCH] xfs: fix inode pincount check in fsync
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 18 Feb 2010 09:53:45 +1100
Cc: xfs@xxxxxxxxxxx
In-reply-to: <20100217193457.GA21840@xxxxxxxxxxxxx>
References: <20100217193457.GA21840@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17) 
On Wed, Feb 17, 2010 at 02:34:57PM -0500, Christoph Hellwig wrote:
> We need to hold the ilock to check the inode pincount safely.  While
> we're at it also remove the check for ip->i_itemp->ili_last_lsn, a
> pinned inode always has it set.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>

Looks good.

Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

-- 
Dave Chinner
david@xxxxxxxxxxxxx
[More with this subject...]
<Prev in Thread] Current Thread [Next in Thread>
Previous by Date:  Re: [PATCH 2/4] [PATCH 2/4] xfs: remove wrappers for read/write file operations, Dave Chinner
Next by Date:  Re: [PATCH] xfs: kill xfs_lrw.h, Dave Chinner
Previous by Thread:  [PATCH] xfs: fix inode pincount check in fsync, Christoph Hellwig
Next by Thread:  [PATCH] xfs: kill xfs_lrw.h, Christoph Hellwig
Indexes:  [Date] [Thread] [Top] [All Lists]