[v3,0/2] plugging cleanup v3

Message

Pankaj Raghav Sept. 29, 2022, 7:47 a.m. UTC

Hi Jens,

1st patch modifies blk_mq_plug() function to disable plugging for
 write and write zeroes in zoned block devices.

2nd patch uses the blk_mq_plug function in the block layer consistently.

The patches are based on next-20220923.

Changes since v2:
- Enhance the commit log for the 2nd patch (Christoph)

Changes since v1:
- Explicltly check only for write and write zeroes as they require zone
  locking in blk_mq_plug
- create a new helper to check for ops that require zone locking for
  zoned devices and also reuse it in blk_req_needs_zone_write_lock

Pankaj Raghav (2):
  block: adapt blk_mq_plug() to not plug for writes that require a zone
    lock
  block: use blk_mq_plug() wrapper consistently in the block layer

 block/blk-core.c       | 2 +-
 block/blk-mq.c         | 6 ++++--
 block/blk-mq.h         | 3 ++-
 block/blk-zoned.c      | 9 +++------
 include/linux/blkdev.h | 9 +++++++++
 5 files changed, 19 insertions(+), 10 deletions(-)

Comments

Jens Axboe Sept. 29, 2022, 1:46 p.m. UTC | #1

On Thu, 29 Sep 2022 09:47:43 +0200, Pankaj Raghav wrote:
> 1st patch modifies blk_mq_plug() function to disable plugging for
>  write and write zeroes in zoned block devices.
> 
> 2nd patch uses the blk_mq_plug function in the block layer consistently.
> 
> The patches are based on next-20220923.
> 
> [...]

Applied, thanks!

[1/2] block: adapt blk_mq_plug() to not plug for writes that require a zone lock
      commit: 8cafdb5ab94cda3ebb0975be16e2d564a05132ea

Best regards,