Mail Archive Search: <+subject:/^(?:^\s*(re|sv|fwd|fw)[\[\]\d]*[:>-]+\s*)*\[PATCH\s+3\/5\]\s+xfs\:\s+remove\s+confusing\s+ail\s+cursor\s+wrapper\s*$/>

Results:

References: [ +subject:/^(?:^\s*(re|sv|fwd|fw)[\[\]\d]*[:>-]+\s*)*\[PATCH\s+3\/5\]\s+xfs\:\s+remove\s+confusing\s+ail\s+cursor\s+wrapper\s*$/: 3 ]

Total 3 documents matching your query.

1. [PATCH 3/5] xfs: remove confusing ail cursor wrapper (score: 1): Author: Dave Chinner <david@xxxxxxxxxxxxx>; Date: Mon, 4 Jul 2011 15:27:38 +1000; 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 functio; /archives/xfs/2011-07/msg00054.html (11,979 bytes)
2. Re: [PATCH 3/5] xfs: remove confusing ail cursor wrapper (score: 1): Author: Christoph Hellwig <hch@xxxxxxxxxxxxx>; Date: Mon, 4 Jul 2011 04:16:03 -0400; The lip = NULL above could nbe turned into a direct return NULL; Otherwise looks good, Reviewed-by: Christoph Hellwig <hch@xxxxxx>; /archives/xfs/2011-07/msg00062.html (10,435 bytes)
3. Re: [PATCH 3/5] xfs: remove confusing ail cursor wrapper (score: 1): Author: Alex Elder <aelder@xxxxxxx>; Date: Thu, 7 Jul 2011 15:33:56 -0500; Good idea. I also appreciate the comment improvement; the "returns NULL" is something I wanted to suggest while looking at the previous patch, but that section of the file wasn't in that patch. Revie; /archives/xfs/2011-07/msg00220.html (10,129 bytes)

Current List: 1 - 3
Page: [1]