| Message ID | 20210203151019.27036-1-guoqing.jiang@cloud.ionos.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | block: add two statistic tables | expand |
Hi Jens, Any chance can this be considered for 5.12? Thanks, Guoqing On 2/3/21 16:10, Guoqing Jiang wrote: > Hi Jens, > > This version adds Reviewed-by tag from Johannes. > > Thanks, > Guoqing > > PATCH V3: https://lore.kernel.org/linux-block/7f78132a-affc-eb03-735a-4da43e143b6e@cloud.ionos.com/T/#t > * reorgnize the patchset per Johannes's suggestion. > > PATCH V2: https://lore.kernel.org/linux-block/20210201012727.28305-1-guoqing.jiang@cloud.ionos.com/T/#t > *. remove BLK_ADDITIONAL_DISKSTAT option per Christoph's comment. > *. move blk_queue_io_extra_stat into blk_additional_{latency,sector} > per Christoph's comment. > *. simplify blk_additional_latency by pass duration time directly. > > PATCH V1: https://marc.info/?l=linux-block&m=161176000024443&w=2 > * add Jack's reviewed-by. > > RFC V4: https://marc.info/?l=linux-block&m=161027198729158&w=2 > * rebase with latest code. > > RFC V3: https://marc.info/?l=linux-block&m=159730633416534&w=2 > * Move the #ifdef CONFIG_BLK_ADDITIONAL_DISKSTAT into the function body > per Johannes's comment. > * Tweak the output of two tables to make they are more intuitive > > RFC V2: https://marc.info/?l=linux-block&m=159467483514062&w=2 > * don't call ktime_get_ns and drop unnecessary patches. > * add io_extra_stats to avoid potential overhead. > > RFC V1: https://marc.info/?l=linux-block&m=159419516730386&w=2 > > Guoqing Jiang (3): > block: add io_extra_stats node > block: add a statistic table for io latency > block: add a statistic table for io sector > > Documentation/ABI/testing/sysfs-block | 26 ++++++++++ > Documentation/block/queue-sysfs.rst | 5 ++ > block/blk-core.c | 43 ++++++++++++++++ > block/blk-sysfs.c | 3 ++ > block/genhd.c | 74 +++++++++++++++++++++++++++ > include/linux/blkdev.h | 2 + > include/linux/part_stat.h | 6 +++ > 7 files changed, 159 insertions(+) >
Hallo Jens, any updates on this patchset? Thank you, Danil On Tue, Feb 9, 2021 at 1:42 AM Guoqing Jiang <guoqing.jiang@cloud.ionos.com> wrote: > > Hi Jens, > > Any chance can this be considered for 5.12? > > Thanks, > Guoqing > > On 2/3/21 16:10, Guoqing Jiang wrote: > > Hi Jens, > > > > This version adds Reviewed-by tag from Johannes. > > > > Thanks, > > Guoqing > > > > PATCH V3: https://lore.kernel.org/linux-block/7f78132a-affc-eb03-735a-4da43e143b6e@cloud.ionos.com/T/#t > > * reorgnize the patchset per Johannes's suggestion. > > > > PATCH V2: https://lore.kernel.org/linux-block/20210201012727.28305-1-guoqing.jiang@cloud.ionos.com/T/#t > > *. remove BLK_ADDITIONAL_DISKSTAT option per Christoph's comment. > > *. move blk_queue_io_extra_stat into blk_additional_{latency,sector} > > per Christoph's comment. > > *. simplify blk_additional_latency by pass duration time directly. > > > > PATCH V1: https://marc.info/?l=linux-block&m=161176000024443&w=2 > > * add Jack's reviewed-by. > > > > RFC V4: https://marc.info/?l=linux-block&m=161027198729158&w=2 > > * rebase with latest code. > > > > RFC V3: https://marc.info/?l=linux-block&m=159730633416534&w=2 > > * Move the #ifdef CONFIG_BLK_ADDITIONAL_DISKSTAT into the function body > > per Johannes's comment. > > * Tweak the output of two tables to make they are more intuitive > > > > RFC V2: https://marc.info/?l=linux-block&m=159467483514062&w=2 > > * don't call ktime_get_ns and drop unnecessary patches. > > * add io_extra_stats to avoid potential overhead. > > > > RFC V1: https://marc.info/?l=linux-block&m=159419516730386&w=2 > > > > Guoqing Jiang (3): > > block: add io_extra_stats node > > block: add a statistic table for io latency > > block: add a statistic table for io sector > > > > Documentation/ABI/testing/sysfs-block | 26 ++++++++++ > > Documentation/block/queue-sysfs.rst | 5 ++ > > block/blk-core.c | 43 ++++++++++++++++ > > block/blk-sysfs.c | 3 ++ > > block/genhd.c | 74 +++++++++++++++++++++++++++ > > include/linux/blkdev.h | 2 + > > include/linux/part_stat.h | 6 +++ > > 7 files changed, 159 insertions(+) > >
Hi Jens, This version adds Reviewed-by tag from Johannes. Thanks, Guoqing PATCH V3: https://lore.kernel.org/linux-block/7f78132a-affc-eb03-735a-4da43e143b6e@cloud.ionos.com/T/#t * reorgnize the patchset per Johannes's suggestion. PATCH V2: https://lore.kernel.org/linux-block/20210201012727.28305-1-guoqing.jiang@cloud.ionos.com/T/#t *. remove BLK_ADDITIONAL_DISKSTAT option per Christoph's comment. *. move blk_queue_io_extra_stat into blk_additional_{latency,sector} per Christoph's comment. *. simplify blk_additional_latency by pass duration time directly. PATCH V1: https://marc.info/?l=linux-block&m=161176000024443&w=2 * add Jack's reviewed-by. RFC V4: https://marc.info/?l=linux-block&m=161027198729158&w=2 * rebase with latest code. RFC V3: https://marc.info/?l=linux-block&m=159730633416534&w=2 * Move the #ifdef CONFIG_BLK_ADDITIONAL_DISKSTAT into the function body per Johannes's comment. * Tweak the output of two tables to make they are more intuitive RFC V2: https://marc.info/?l=linux-block&m=159467483514062&w=2 * don't call ktime_get_ns and drop unnecessary patches. * add io_extra_stats to avoid potential overhead. RFC V1: https://marc.info/?l=linux-block&m=159419516730386&w=2 Guoqing Jiang (3): block: add io_extra_stats node block: add a statistic table for io latency block: add a statistic table for io sector Documentation/ABI/testing/sysfs-block | 26 ++++++++++ Documentation/block/queue-sysfs.rst | 5 ++ block/blk-core.c | 43 ++++++++++++++++ block/blk-sysfs.c | 3 ++ block/genhd.c | 74 +++++++++++++++++++++++++++ include/linux/blkdev.h | 2 + include/linux/part_stat.h | 6 +++ 7 files changed, 159 insertions(+)