This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "XFS development tree".
The branch, master has been updated
55fb25d5 xfs: add size update tracepoint to IO completion
af3e402 xfs: convert AIL cursors to use struct list_head
16b5902 xfs: remove confusing ail cursor wrapper
1d8c95a xfs: use a cursor for bulk AIL insertion
ad1a2c8 xfs: failure mapping nfs fh to inode should return ESTALE
adab0f6 xfs: Remove the second parameter to xfs_sb_count()
from d0f9e8fb4cc6dd5d07c72eeecc2f332b6e85e221 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 55fb25d5b39320242b41af894921f5cef0c7e293
Author: Dave Chinner <dchinner@xxxxxxxxxx>
Date: Mon Jul 18 03:40:19 2011 +0000
xfs: add size update tracepoint to IO completion
For improving insight into IO completion behaviour.
Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
commit af3e40228fb2dbc18d94fbd5103f07344a720ae7
Author: Dave Chinner <dchinner@xxxxxxxxxx>
Date: Mon Jul 18 03:40:18 2011 +0000
xfs: convert AIL cursors to use struct list_head
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 <dchinner@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
commit 16b5902943c793e632cf8c9526619f59893bdc29
Author: Dave Chinner <dchinner@xxxxxxxxxx>
Date: Mon Jul 18 03:40:17 2011 +0000
xfs: remove confusing ail cursor wrapper
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@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
commit 1d8c95a363bf8cd4d4182dd19c01693b635311c2
Author: Dave Chinner <dchinner@xxxxxxxxxx>
Date: Mon Jul 18 03:40:16 2011 +0000
xfs: use a cursor for bulk AIL insertion
Delayed logging can insert tens of thousands of log items into the
AIL at the same LSN. When the committing of log commit records
occur, we can get insertions occurring at an LSN that is not at the
end of the AIL. If there are thousands of items in the AIL on the
tail LSN, each insertion has to walk the AIL to find the correct
place to insert the new item into the AIL. This can consume large
amounts of CPU time and block other operations from occurring while
the traversals are in progress.
To avoid this repeated walk, use a AIL cursor to record
where we should be inserting the new items into the AIL without
having to repeat the walk. The cursor infrastructure already
provides this functionality for push walks, so is a simple extension
of existing code. While this will not avoid the initial walk, it
will avoid repeating it tens of thousands of times during a single
checkpoint commit.
This version includes logic improvements from Christoph Hellwig.
Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
commit ad1a2c878ca70829874b4fcc83223cccb4e26dab
Author: J. Bruce Fields <bfields@xxxxxxxxxx>
Date: Thu Jul 14 20:50:36 2011 +0000
xfs: failure mapping nfs fh to inode should return ESTALE
On xfs exports, nfsd is incorrectly returning ENOENT instead of
ESTALE on attempts to use a filehandle of a deleted file (spotted
with pynfs test PUTFH3). The ENOENT was coming from xfs_iget.
(It's tempting to wonder whether we should just map all xfs_iget
errors to ESTALE, but I don't believe so--xfs_iget can also return
ENOMEM at least, which we wouldn't want mapped to ESTALE.)
While we're at it, the other return of ENOENT in xfs_nfs_get_inode()
also looks wrong.
Signed-off-by: J. Bruce Fields <bfields@xxxxxxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
commit adab0f67d1cdaf468bbc311bce4d61f17626a536
Author: Chandra Seetharaman <sekharan@xxxxxxxxxx>
Date: Wed Jun 29 22:10:14 2011 +0000
xfs: Remove the second parameter to xfs_sb_count()
Remove the second parameter to xfs_sb_count() since all callers of
the function set them.
Also, fix the header comment regarding it being called periodically.
Signed-off-by: Chandra Seetharaman <sekharan@xxxxxxxxxx>
Signed-off-by: Alex Elder <aelder@xxxxxxx>
-----------------------------------------------------------------------
Summary of changes:
fs/xfs/linux-2.6/xfs_aops.c | 1 +
fs/xfs/linux-2.6/xfs_export.c | 4 +-
fs/xfs/linux-2.6/xfs_sync.c | 2 +-
fs/xfs/linux-2.6/xfs_trace.h | 12 ++-
fs/xfs/xfs_mount.c | 15 +--
fs/xfs/xfs_mount.h | 2 +-
fs/xfs/xfs_trans.c | 27 +++++-
fs/xfs/xfs_trans_ail.c | 214 +++++++++++++++++++++++------------------
fs/xfs/xfs_trans_priv.h | 14 ++-
9 files changed, 171 insertions(+), 120 deletions(-)
hooks/post-receive
--
XFS development tree
|