[dm-devel] [PATCH 1/8] block: Remove REQ_OP_ZONE_RESET plugging
Javier González
javier at javigon.com
Tue Oct 29 12:13:16 UTC 2019
On 27.10.2019 23:05, Damien Le Moal wrote:
>REQ_OP_ZONE_RESET operations cannot be merged as these bios and requests
>do not have a size and are never sequential due to the zone start sector
>position required for their execution. As a result, there is no point in
>using a plug around blkdev_reset_zones() bio issuing loop. This patch
>removes this unnecessary plugging.
>
>Signed-off-by: Damien Le Moal <damien.lemoal at wdc.com>
>---
> block/blk-zoned.c | 4 ----
> 1 file changed, 4 deletions(-)
>
>diff --git a/block/blk-zoned.c b/block/blk-zoned.c
>index 4bc5f260248a..7fe376eede86 100644
>--- a/block/blk-zoned.c
>+++ b/block/blk-zoned.c
>@@ -258,7 +258,6 @@ int blkdev_reset_zones(struct block_device *bdev,
> sector_t zone_sectors;
> sector_t end_sector = sector + nr_sectors;
> struct bio *bio = NULL;
>- struct blk_plug plug;
> int ret;
>
> if (!blk_queue_is_zoned(q))
>@@ -283,7 +282,6 @@ int blkdev_reset_zones(struct block_device *bdev,
> end_sector != bdev->bd_part->nr_sects)
> return -EINVAL;
>
>- blk_start_plug(&plug);
> while (sector < end_sector) {
>
> bio = blk_next_bio(bio, 0, gfp_mask);
>@@ -301,8 +299,6 @@ int blkdev_reset_zones(struct block_device *bdev,
> ret = submit_bio_wait(bio);
> bio_put(bio);
>
>- blk_finish_plug(&plug);
>-
> return ret;
> }
> EXPORT_SYMBOL_GPL(blkdev_reset_zones);
>--
>2.21.0
>
Looks good to me.
Reviewed-by: Javier González <javier at javigon.com>
More information about the dm-devel
mailing list