Re: [PATCH v4] xfs: fix the symbolic link assert in xfs

xfs

Re: [PATCH v4] xfs: fix the symbolic link assert in xfs_ifree

To:	Mark Tinguely <tinguely@xxxxxxx>
Subject:	Re: [PATCH v4] xfs: fix the symbolic link assert in xfs_ifree
From:	Ben Myers <bpm@xxxxxxx>
Date:	Tue, 18 Jun 2013 13:59:02 -0500
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<20130617203503.472644581@xxxxxxx>
References:	<20130617203503.472644581@xxxxxxx>
User-agent:	Mutt/1.5.20 (2009-06-14)

On Mon, Jun 17, 2013 at 03:35:57PM -0500, Mark Tinguely wrote:
> Adding an extended attribute to a symbolic link can force that
> link to an remote extent. xfs_inactive() incorrectly assumes
> that any symbolic link small enough to be in the inode core
> is incore, resulting in the remote extent to not be removed.
> xfs_ifree() will assert on presence of this leaked remote extent.
> 
> Signed-off-by: Mark Tinguely <tinguely@xxxxxxx>

Looks good.

Reviewed-by: Ben Myers <bpm@xxxxxxx>

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH v4] xfs: fix the symbolic link assert in xfs_ifree, Mark Tinguely Re: [PATCH v4] xfs: fix the symbolic link assert in xfs_ifree, Ben Myers <= Re: [PATCH v4] xfs: fix the symbolic link assert in xfs_ifree, Dave Chinner

Previous by Date:	Re:Urea,Potassium sulfate, Roland
Next by Date:	Re: [PATCH 19/27] xfs: consolidate xfs_vnodeops.c into xfs_inode_ops.c, Dave Chinner
Previous by Thread:	[PATCH v4] xfs: fix the symbolic link assert in xfs_ifree, Mark Tinguely
Next by Thread:	Re: [PATCH v4] xfs: fix the symbolic link assert in xfs_ifree, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]