diff options
| author | NeilBrown <[email protected]> | 2012-11-20 14:11:15 +1100 |
|---|---|---|
| committer | NeilBrown <[email protected]> | 2012-12-13 19:52:32 +1100 |
| commit | 749586b7d34df910118bff2c248d08877d772e81 (patch) | |
| tree | 2785d1a7601ff4d717545b43ab60c32b45d683ba | |
| parent | 0a19caabf01ac138bf3668786939e50ea4d9c8ac (diff) | |
md/raid5: use async_tx_quiesce() instead of open-coding it.
handle_stripe_expansion contains:
if (tx) {
async_tx_ack(tx);
dma_wait_for_async_tx(tx);
}
which is very similar to the body of async_tx_quiesce(),
except that the later handles an error from dma_wait_for_async_tx()
(admittedly by panicing, but that decision belongs in the dma
code, not the md code).
So just us async_tx_quiesce().
Acked-by: Dan Williams <[email protected]>
Reported-by: Bartlomiej Zolnierkiewicz <[email protected]>
Signed-off-by: NeilBrown <[email protected]>
| -rw-r--r-- | drivers/md/raid5.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index a4502686e7a8..2cf23f23ddca 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c @@ -3224,10 +3224,7 @@ static void handle_stripe_expansion(struct r5conf *conf, struct stripe_head *sh) } /* done submitting copies, wait for them to complete */ - if (tx) { - async_tx_ack(tx); - dma_wait_for_async_tx(tx); - } + async_tx_quiesce(&tx); } /* |