diff options
Diffstat (limited to 'fs/xfs/xfs_trans_buf.c')
-rw-r--r-- | fs/xfs/xfs_trans_buf.c | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/fs/xfs/xfs_trans_buf.c b/fs/xfs/xfs_trans_buf.c index 49130628d5ef..5ffd544434eb 100644 --- a/fs/xfs/xfs_trans_buf.c +++ b/fs/xfs/xfs_trans_buf.c | |||
@@ -75,13 +75,14 @@ xfs_trans_get_buf(xfs_trans_t *tp, | |||
75 | xfs_buf_log_item_t *bip; | 75 | xfs_buf_log_item_t *bip; |
76 | 76 | ||
77 | if (flags == 0) | 77 | if (flags == 0) |
78 | flags = XFS_BUF_LOCK | XFS_BUF_MAPPED; | 78 | flags = XBF_LOCK | XBF_MAPPED; |
79 | 79 | ||
80 | /* | 80 | /* |
81 | * Default to a normal get_buf() call if the tp is NULL. | 81 | * Default to a normal get_buf() call if the tp is NULL. |
82 | */ | 82 | */ |
83 | if (tp == NULL) | 83 | if (tp == NULL) |
84 | return xfs_buf_get(target_dev, blkno, len, flags | BUF_BUSY); | 84 | return xfs_buf_get(target_dev, blkno, len, |
85 | flags | XBF_DONT_BLOCK); | ||
85 | 86 | ||
86 | /* | 87 | /* |
87 | * If we find the buffer in the cache with this transaction | 88 | * If we find the buffer in the cache with this transaction |
@@ -117,14 +118,14 @@ xfs_trans_get_buf(xfs_trans_t *tp, | |||
117 | } | 118 | } |
118 | 119 | ||
119 | /* | 120 | /* |
120 | * We always specify the BUF_BUSY flag within a transaction so | 121 | * We always specify the XBF_DONT_BLOCK flag within a transaction |
121 | * that get_buf does not try to push out a delayed write buffer | 122 | * so that get_buf does not try to push out a delayed write buffer |
122 | * which might cause another transaction to take place (if the | 123 | * which might cause another transaction to take place (if the |
123 | * buffer was delayed alloc). Such recursive transactions can | 124 | * buffer was delayed alloc). Such recursive transactions can |
124 | * easily deadlock with our current transaction as well as cause | 125 | * easily deadlock with our current transaction as well as cause |
125 | * us to run out of stack space. | 126 | * us to run out of stack space. |
126 | */ | 127 | */ |
127 | bp = xfs_buf_get(target_dev, blkno, len, flags | BUF_BUSY); | 128 | bp = xfs_buf_get(target_dev, blkno, len, flags | XBF_DONT_BLOCK); |
128 | if (bp == NULL) { | 129 | if (bp == NULL) { |
129 | return NULL; | 130 | return NULL; |
130 | } | 131 | } |
@@ -290,15 +291,15 @@ xfs_trans_read_buf( | |||
290 | int error; | 291 | int error; |
291 | 292 | ||
292 | if (flags == 0) | 293 | if (flags == 0) |
293 | flags = XFS_BUF_LOCK | XFS_BUF_MAPPED; | 294 | flags = XBF_LOCK | XBF_MAPPED; |
294 | 295 | ||
295 | /* | 296 | /* |
296 | * Default to a normal get_buf() call if the tp is NULL. | 297 | * Default to a normal get_buf() call if the tp is NULL. |
297 | */ | 298 | */ |
298 | if (tp == NULL) { | 299 | if (tp == NULL) { |
299 | bp = xfs_buf_read(target, blkno, len, flags | BUF_BUSY); | 300 | bp = xfs_buf_read(target, blkno, len, flags | XBF_DONT_BLOCK); |
300 | if (!bp) | 301 | if (!bp) |
301 | return (flags & XFS_BUF_TRYLOCK) ? | 302 | return (flags & XBF_TRYLOCK) ? |
302 | EAGAIN : XFS_ERROR(ENOMEM); | 303 | EAGAIN : XFS_ERROR(ENOMEM); |
303 | 304 | ||
304 | if (XFS_BUF_GETERROR(bp) != 0) { | 305 | if (XFS_BUF_GETERROR(bp) != 0) { |
@@ -385,14 +386,14 @@ xfs_trans_read_buf( | |||
385 | } | 386 | } |
386 | 387 | ||
387 | /* | 388 | /* |
388 | * We always specify the BUF_BUSY flag within a transaction so | 389 | * We always specify the XBF_DONT_BLOCK flag within a transaction |
389 | * that get_buf does not try to push out a delayed write buffer | 390 | * so that get_buf does not try to push out a delayed write buffer |
390 | * which might cause another transaction to take place (if the | 391 | * which might cause another transaction to take place (if the |
391 | * buffer was delayed alloc). Such recursive transactions can | 392 | * buffer was delayed alloc). Such recursive transactions can |
392 | * easily deadlock with our current transaction as well as cause | 393 | * easily deadlock with our current transaction as well as cause |
393 | * us to run out of stack space. | 394 | * us to run out of stack space. |
394 | */ | 395 | */ |
395 | bp = xfs_buf_read(target, blkno, len, flags | BUF_BUSY); | 396 | bp = xfs_buf_read(target, blkno, len, flags | XBF_DONT_BLOCK); |
396 | if (bp == NULL) { | 397 | if (bp == NULL) { |
397 | *bpp = NULL; | 398 | *bpp = NULL; |
398 | return 0; | 399 | return 0; |
@@ -472,8 +473,8 @@ shutdown_abort: | |||
472 | if (XFS_BUF_ISSTALE(bp) && XFS_BUF_ISDELAYWRITE(bp)) | 473 | if (XFS_BUF_ISSTALE(bp) && XFS_BUF_ISDELAYWRITE(bp)) |
473 | cmn_err(CE_NOTE, "about to pop assert, bp == 0x%p", bp); | 474 | cmn_err(CE_NOTE, "about to pop assert, bp == 0x%p", bp); |
474 | #endif | 475 | #endif |
475 | ASSERT((XFS_BUF_BFLAGS(bp) & (XFS_B_STALE|XFS_B_DELWRI)) != | 476 | ASSERT((XFS_BUF_BFLAGS(bp) & (XBF_STALE|XBF_DELWRI)) != |
476 | (XFS_B_STALE|XFS_B_DELWRI)); | 477 | (XBF_STALE|XBF_DELWRI)); |
477 | 478 | ||
478 | trace_xfs_trans_read_buf_shut(bp, _RET_IP_); | 479 | trace_xfs_trans_read_buf_shut(bp, _RET_IP_); |
479 | xfs_buf_relse(bp); | 480 | xfs_buf_relse(bp); |