From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id p6JN3vWS197833 for ; Tue, 19 Jul 2011 18:03:57 -0500 Subject: Re: [PATCH 3/4] xfs: convert AIL cursors to use struct list_head From: Alex Elder In-Reply-To: <1310960419-9875-4-git-send-email-david@fromorbit.com> References: <1310960419-9875-1-git-send-email-david@fromorbit.com> <1310960419-9875-4-git-send-email-david@fromorbit.com> Date: Tue, 19 Jul 2011 18:03:56 -0500 Message-ID: <1311116636.1964.40.camel@doink> MIME-Version: 1.0 Reply-To: aelder@sgi.com List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: xfs@oss.sgi.com On Mon, 2011-07-18 at 13:40 +1000, Dave Chinner wrote: > From: Dave Chinner > > The list of active AIL cursors uses a roll-your-own linked list with > special casing for the AIL push cursor. Simplify this code by > replacing the list with standard struct list_head lists, and use a > separate list_head to track the active cursors. This allows us to > treat the AIL push cursor as a generic cursor rather than as a > special case, further simplifying the code. > > Further, fix the duplicate push cursor initialisation that the > special case handling was hiding, and clean up all the comments > around the active cursor list handling. > > Signed-off-by: Dave Chinner > Reviewed-by: Christoph Hellwig You didn't fix one thing I mentioned before in a comment. I can fix that for you if you like. Other than that, this looks good. Reviewed-by: Alex Elder > --- > fs/xfs/xfs_trans_ail.c | 77 ++++++++++++++++------------------------------- > fs/xfs/xfs_trans_priv.h | 4 +- > 2 files changed, 28 insertions(+), 53 deletions(-) > > diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c > index 7908c79..8cd329b 100644 > --- a/fs/xfs/xfs_trans_ail.c > +++ b/fs/xfs/xfs_trans_ail.c . . . > @@ -181,17 +177,12 @@ xfs_trans_ail_cursor_init( . . . > - * Get the next item in the traversal and advance the cursor. > - * If the cursor was invalidated (inidicated by a lip of 1), > - * restart the traversal. > + * Get the next item in the traversal and advance the cursor. If the cursor > + * was invalidated (indicated by a lip of 1), restart the traversal. * was invalidated (indicated by its item pointer's low bit being set), * restart the traversal. > */ > struct xfs_log_item * > xfs_trans_ail_cursor_next( . . . _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs