diff options
| author | Guoqing Jiang <guoqing.jiang@cloud.ionos.com> | 2020-06-16 11:25:52 +0200 |
|---|---|---|
| committer | Song Liu <songliubraving@fb.com> | 2020-07-16 10:14:08 -0700 |
| commit | 1684e97538b9484c72bcaff2961569c3b0021473 (patch) | |
| tree | c3c4fd7f6aeddf02193d2367056b6524a46decd7 /drivers/md/raid5.c | |
| parent | cb9902db3827135bfd6ae70bcb65b653418bb775 (diff) | |
raid5: remove the meaningless check in raid5_make_request
We can't guarntee the batched stripe to be set with STRIPE_HANDLE since
there are lots of functions could set the flag, such as sync_request,
ops_complete_* and end_{read,write}_request etc.
Also clear_batch_ready called in handle_stripe ensures the batched list
can't continue to be handled by handle_stripe.
Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com>
Signed-off-by: Song Liu <songliubraving@fb.com>
Diffstat (limited to 'drivers/md/raid5.c')
| -rw-r--r-- | drivers/md/raid5.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index 3519560d35b0..73128f46924c 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c @@ -5738,8 +5738,7 @@ static bool raid5_make_request(struct mddev *mddev, struct bio * bi) do_flush = false; } - if (!sh->batch_head || sh == sh->batch_head) - set_bit(STRIPE_HANDLE, &sh->state); + set_bit(STRIPE_HANDLE, &sh->state); clear_bit(STRIPE_DELAYED, &sh->state); if ((!sh->batch_head || sh == sh->batch_head) && (bi->bi_opf & REQ_SYNC) && |