diff options
| author | Christoph Hellwig <hch@infradead.org> | 2012-04-23 15:58:38 +1000 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2012-05-14 16:20:30 -0500 |
| commit | 960c60af8b9481595e68875e79b2602e73169c29 (patch) | |
| tree | 26756622c5a0c63ead727621602cc1670604c0a3 /fs/xfs/xfs_trans_buf.c | |
| parent | fe7257fd4b8ae9a3e354d9edb61890973e373ef0 (diff) | |
| download | linux-stable-960c60af8b9481595e68875e79b2602e73169c29.tar.gz linux-stable-960c60af8b9481595e68875e79b2602e73169c29.tar.bz2 linux-stable-960c60af8b9481595e68875e79b2602e73169c29.zip | |
xfs: do not add buffers to the delwri queue until pushed
Instead of adding buffers to the delwri list as soon as they are logged,
even if they can't be written until commited because they are pinned
defer adding them to the delwri list until xfsaild pushes them. This
makes the code more similar to other log items and prepares for writing
buffers directly from xfsaild.
The complication here is that we need to fail buffers that were added
but not logged yet in xfs_buf_item_unpin, borrowing code from
xfs_bioerror.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_trans_buf.c')
| -rw-r--r-- | fs/xfs/xfs_trans_buf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/xfs/xfs_trans_buf.c b/fs/xfs/xfs_trans_buf.c index 1302d1d95a58..296a7995a007 100644 --- a/fs/xfs/xfs_trans_buf.c +++ b/fs/xfs/xfs_trans_buf.c @@ -626,8 +626,6 @@ xfs_trans_log_buf(xfs_trans_t *tp, bp->b_iodone = xfs_buf_iodone_callbacks; bip->bli_item.li_cb = xfs_buf_iodone; - xfs_buf_delwri_queue(bp); - trace_xfs_trans_log_buf(bip); /* |