[XFS updates] XFS development tree branch, master, updated. v2.6.38-28508-g9e978d8
xfs at oss.sgi.com
xfs at oss.sgi.com
Thu Aug 11 09:08:09 CDT 2011
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
9e978d8 "xfs: fix error handling for synchronous writes" revisited
e44f411 xfs: set cursor in xfs_ail_splice() even when AIL was empty
from 2ddb4e94065470828e131351566102274ea9e83f (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 9e978d8f7db1c5de7cdc6450a8ca208db3b95f84
Author: Ajeet Yadav <ajeet.yadav.77 at gmail.com>
Date: Fri Jul 29 07:42:59 2011 +0000
"xfs: fix error handling for synchronous writes" revisited
xfs: fix for hang during synchronous buffer write error
If removed storage while synchronous buffer write underway,
"xfslogd" hangs.
Detailed log http://oss.sgi.com/archives/xfs/2011-07/msg00740.html
Related work bfc60177f8ab509bc225becbb58f7e53a0e33e81
"xfs: fix error handling for synchronous writes"
Given that xfs_bwrite actually does the shutdown already after
waiting for the b_iodone completion and given that we actually
found that calling xfs_force_shutdown from inside
xfs_buf_iodone_callbacks was a major contributor the problem
it better to drop this call.
Signed-off-by: Ajeet Yadav <ajeet.yadav.77 at gmail.com>
Reviewed-by: Christoph Hellwig <hch at lst.de>
Signed-off-by: Alex Elder <aelder at sgi.com>
commit e44f4112a46ca817fe2758aac7bf7893a64a8c0e
Author: Alex Elder <aelder at sgi.com>
Date: Fri Jul 22 16:04:41 2011 +0000
xfs: set cursor in xfs_ail_splice() even when AIL was empty
In xfs_ail_splice(), if a cursor is provided it is updated to
point to the last item on the list being spliced into the AIL.
But if the AIL was found to be empty, the cursor (if provided)
is just initialized instead.
There is no reason the empty AIL case needs to be treated any
differently. And treating it the same way allows this code
to be rearranged a bit, with a somewhat tidier result.
Signed-off-by: Alex Elder <aelder at sgi.com>
Reviewed-by: Dave Chinner <dchinner at redhat.com>
-----------------------------------------------------------------------
Summary of changes:
fs/xfs/xfs_buf_item.c | 1 -
fs/xfs/xfs_trans_ail.c | 67 ++++++++++++++++++++---------------------------
2 files changed, 29 insertions(+), 39 deletions(-)
hooks/post-receive
--
XFS development tree
More information about the xfs
mailing list