Messages in this thread | | | Subject | Re: [PATCH v7 0/9] bugfix and cleanup for blk-throttle | From | Yu Kuai <> | Date | Sat, 13 Aug 2022 13:59:44 +0800 |
| |
在 2022/08/02 22:04, Yu Kuai 写道: > From: Yu Kuai <yukuai3@huawei.com> > > Changes in v7: > - add patch 5 to improve handling of re-entered bio for bps limit > - as suggested by Tejun, add some comments > - sdd some Acked tag by Tejun > Changes in v6: > - rename parameter in patch 3 > - add comments and reviewed tag for patch 4 > Changes in v5: > - add comments in patch 4 > - clear bytes/io_skipped in throtl_start_new_slice_with_credit() in > patch 4 > - and cleanup patches 5-8 > Changes in v4: > - add reviewed-by tag for patch 1 > - add patch 2,3 > - use a different way to fix io hung in patch 4 > Changes in v3: > - fix a check in patch 1 > - fix link err in patch 2 on 32-bit platform > - handle overflow in patch 2 > Changes in v2: > - use a new solution suggested by Ming > - change the title of patch 1 > - add patch 2 > > Patch 1 fix that blk-throttle can't work if multiple bios are throttle. > Patch 2 fix overflow while calculating wait time. > Patch 3,4 fix io hung due to configuration updates. > Patch 5 improve handling of re-entered bio for bps limit. > Patch 6-9 are cleanup patches, there are no functional changes, just > some places that I think can be optimized during code review. > friendly ping ... > Previous version: > v1: https://lore.kernel.org/all/20220517134909.2910251-1-yukuai3@huawei.com/ > v2: https://lore.kernel.org/all/20220518072751.1188163-1-yukuai3@huawei.com/ > v3: https://lore.kernel.org/all/20220519085811.879097-1-yukuai3@huawei.com/ > v4: https://lore.kernel.org/all/20220523082633.2324980-1-yukuai3@huawei.com/ > v5: https://lore.kernel.org/all/20220528064330.3471000-1-yukuai3@huawei.com/ > v6: https://lore.kernel.org/all/20220701093441.885741-1-yukuai1@huaweicloud.com/ > > Yu Kuai (9): > blk-throttle: fix that io throttle can only work for single bio > blk-throttle: prevent overflow while calculating wait time > blk-throttle: factor out code to calculate ios/bytes_allowed > blk-throttle: fix io hung due to configuration updates > blk-throttle: improve handling of re-entered bio for bps limit > blk-throttle: use 'READ/WRITE' instead of '0/1' > blk-throttle: calling throtl_dequeue/enqueue_tg in pairs > blk-throttle: cleanup tg_update_disptime() > blk-throttle: clean up flag 'THROTL_TG_PENDING' > > block/blk-throttle.c | 175 ++++++++++++++++++++++++++++++------------- > block/blk-throttle.h | 20 ++++- > 2 files changed, 139 insertions(+), 56 deletions(-) >
| |