Re: [PATCH 3/5] xfs: fold xfs_attr_remove_int into xfs_attr_remove

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Mon, May 05, 2014 at 04:21:28PM -0400, Brian Foster wrote:
> On Sat, May 03, 2014 at 05:20:13PM +0200, Christoph Hellwig wrote:
> > Also remove a useless ilock roundtrip for the first attr fork check, it's
> > racy anyway and we redo it later under the ilock before we start the 
> > removal.
> > 
> > Plus various minor style fixes to the new xfs_attr_remove.
> > 
> > Signed-off-by: Christoph Hellwig <hch@xxxxxx>
.....
> > @@ -487,35 +497,26 @@ xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name 
> > *name, int flags)
> >      */
> >     xfs_trans_ijoin(args.trans, dp, 0);
> >  
> > -   /*
> > -    * Decide on what work routines to call based on the inode size.
> > -    */
> >     if (!xfs_inode_hasattr(dp)) {
> >             error = XFS_ERROR(ENOATTR);
> 
> I suppose we probably want to nuke the XFS_ERROR() while we're here..?
> Otherwise, it looks good.

No need, I'll do that at the end of the dev cycle for everything.

Cheers,

Dave.

-- 
Dave Chinner
david@xxxxxxxxxxxxx