[PATCH 2/4] xfs: remove confusing ail cursor wrapper
Alex Elder
aelder at sgi.com
Tue Jul 19 18:03:50 CDT 2011
On Mon, 2011-07-18 at 13:40 +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner at redhat.com>
>
> xfs_trans_ail_cursor_set() doesn't set the cursor to the current log
> item, it sets it to the next item. There is already a function for
> doing this - xfs_trans_ail_cursor_next() - and the _set function is
> simply a two line wrapper. Remove it and open code the setting of
> the cursor in the two locations that call it to remove the
> confusion.
>
> Signed-off-by: Dave Chinner <dchinner at redhat.com>
> Reviewed-by: Christoph Hellwig <hch at lst.de>
I already reviewed this but I do see you did a tiny change
or two. In any case, this (still) looks good.
Reviewed-by: Alex Elder <aelder at sgi.com>
More information about the xfs
mailing list