[PATCH v2 0/9] md: fix is_mddev_idle()

Yu Kuai posted 9 patches 7 months, 3 weeks ago
There is a newer version of this series
block/blk-core.c          |   2 +-
block/blk-mq.c            |  22 ++---
block/blk-mq.h            |   5 +-
block/blk.h               |   1 -
block/genhd.c             |  69 ++++++++------
drivers/md/dm-raid.c      |   3 +
drivers/md/md.c           | 193 +++++++++++++++++++++++++++-----------
drivers/md/md.h           |  18 +---
drivers/md/raid1.c        |   3 -
drivers/md/raid10.c       |   9 --
drivers/md/raid5.c        |   8 --
include/linux/blkdev.h    |   1 -
include/linux/part_stat.h |   2 +
13 files changed, 194 insertions(+), 142 deletions(-)
[PATCH v2 0/9] md: fix is_mddev_idle()
Posted by Yu Kuai 7 months, 3 weeks ago
From: Yu Kuai <yukuai3@huawei.com>

Changes in v2:
 - add patch 1-5
 - add reviewed-by in patch 6,7,9
 - rename mddev->last_events to mddev->normal_IO_events in patch 8

Yu Kuai (9):
  blk-mq: remove blk_mq_in_flight()
  block: reuse part_in_flight_rw for part_in_flight
  block: WARN if bdev inflight counter is negative
  block: cleanup blk_mq_in_flight_rw()
  block: export API to get the number of bdev inflight IO
  md: record dm-raid gendisk in mddev
  md: add a new api sync_io_depth
  md: fix is_mddev_idle()
  md: cleanup accounting for issued sync IO

 block/blk-core.c          |   2 +-
 block/blk-mq.c            |  22 ++---
 block/blk-mq.h            |   5 +-
 block/blk.h               |   1 -
 block/genhd.c             |  69 ++++++++------
 drivers/md/dm-raid.c      |   3 +
 drivers/md/md.c           | 193 +++++++++++++++++++++++++++-----------
 drivers/md/md.h           |  18 +---
 drivers/md/raid1.c        |   3 -
 drivers/md/raid10.c       |   9 --
 drivers/md/raid5.c        |   8 --
 include/linux/blkdev.h    |   1 -
 include/linux/part_stat.h |   2 +
 13 files changed, 194 insertions(+), 142 deletions(-)

-- 
2.39.2
Re: [PATCH v2 0/9] md: fix is_mddev_idle()
Posted by Paul Menzel 7 months, 3 weeks ago
Dear Kuai,


Thank you for your patch series. Some minor comments below.


Am 27.04.25 um 10:29 schrieb Yu Kuai:
> From: Yu Kuai <yukuai3@huawei.com>

If a full cover letter is not warranted, maybe state, what patch to look 
at? (In this case 8/9.)

> Changes in v2:
>   - add patch 1-5
>   - add reviewed-by in patch 6,7,9
>   - rename mddev->last_events to mddev->normal_IO_events in patch 8
> 
> Yu Kuai (9):
>    blk-mq: remove blk_mq_in_flight()
>    block: reuse part_in_flight_rw for part_in_flight
>    block: WARN if bdev inflight counter is negative
>    block: cleanup blk_mq_in_flight_rw()

cleanup → clean up

>    block: export API to get the number of bdev inflight IO
>    md: record dm-raid gendisk in mddev
>    md: add a new api sync_io_depth

add a new → add new

>    md: fix is_mddev_idle()
>    md: cleanup accounting for issued sync IO

cleanup → clean up

>   block/blk-core.c          |   2 +-
>   block/blk-mq.c            |  22 ++---
>   block/blk-mq.h            |   5 +-
>   block/blk.h               |   1 -
>   block/genhd.c             |  69 ++++++++------
>   drivers/md/dm-raid.c      |   3 +
>   drivers/md/md.c           | 193 +++++++++++++++++++++++++++-----------
>   drivers/md/md.h           |  18 +---
>   drivers/md/raid1.c        |   3 -
>   drivers/md/raid10.c       |   9 --
>   drivers/md/raid5.c        |   8 --
>   include/linux/blkdev.h    |   1 -
>   include/linux/part_stat.h |   2 +
>   13 files changed, 194 insertions(+), 142 deletions(-)



Kind regards,

Paul