Re: [PATCH] xfs: fix stale inode flush avoidance

xfs

Re: [PATCH] xfs: fix stale inode flush avoidance

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH] xfs: fix stale inode flush avoidance
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Mon, 11 Jan 2010 16:47:33 -0500
Cc:	xfs@xxxxxxxxxxx
In-reply-to:	<1263210321-21068-1-git-send-email-david@xxxxxxxxxxxxx>
References:	<1263210321-21068-1-git-send-email-david@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.19 (2009-01-05)

On Mon, Jan 11, 2010 at 10:45:21PM +1100, Dave Chinner wrote:
> When reclaiming stale inodes, we need to guarantee that inodes are
> unpinned before returning with a "clean" status. If we don't we can
> reclaim inodes that are pinned, leading to use after free in the
> transaction subsystem as transactions complete.

Looks good.  Per discussion this morning the clean check might not be
good enough for delwri buffers either, but as let's leave that for the
more throughout redo of the inode buffer flushing.

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH] xfs: fix stale inode flush avoidance, Dave Chinner Re: [PATCH] xfs: fix stale inode flush avoidance, Christoph Hellwig <=

Previous by Date:	Re: [PATCH V3] mkfs: handle 4k sector devices more cleanly, Christoph Hellwig
Next by Date:	Re: [PATCH 1/3] XFS: Use list_heads for log recovery item lists, Christoph Hellwig
Previous by Thread:	[PATCH] xfs: fix stale inode flush avoidance, Dave Chinner
Next by Thread:	[PATCH 0/10] Perag lookup rework, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]