Thread (7 messages) 7 messages, 3 authors, 2021-04-08

Re: [PATCH V5 0/3] block: add two statistic tables

From: Khazhy Kumykov <hidden>
Date: 2021-04-08 22:23:06

On Thu, Apr 8, 2021 at 6:59 AM Md Haris Iqbal [off-list ref] wrote:
Hi Jens,

This version is rebased against the latest for-next.

Thanks,
Haris

PATCH V4: https://lore.kernel.org/linux-block/20210203151019.27036-1-guoqing.jiang@cloud.ionos.com/ (local)
* Adds Reviewed-by tag from Johannes.

PATCH V3: https://lore.kernel.org/linux-block/7f78132a-affc-eb03-735a-4da43e143b6e@cloud.ionos.com/T/#t (local)
* 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 (local)
*. 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
We've also found it useful to have the kernel export latency
statistics. In particular we've been using a 2d-histogram exporting
latency per iosize (there was a proposal a loong time ago that seemed
to fizzle out https://lwn.net/Articles/383515/)

Do you think having a format like that would be useful?

# cat /sys/block/sda/read_request_histo
rows = bytes columns = ms
        10      20      50      100     200     500     1000    2000
 5000    10000   20000
   2048 33      0       0       0       0       0       0       0
 0       0       0
   4096 51798   1114    1271    11      0       0       0       0
 0       0       0
   8192 8803    81      109     1       0       0       0       0
 0       0       0
  16384 10321   116     150     5       0       0       0       0
 0       0       0
  32768 8341    166     215     6       0       0       0       0
 0       0       0
  65536 7550    248     389     2       0       0       0       0
 0       0       0
 131072 40440   923     1317    10      0       0       0       0
 0       0       0
 262144 357     1       0       0       0       0       0       0
 0       0       0
 524288 13      1       0       0       0       0       0       0
 0       0       0
1048576 7       2       0       0       0       0       0       0
 0       0       0

Khazhy

Attachments

  • smime.p7s [application/pkcs7-signature] 3996 bytes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help