All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guoqing Jiang <guoqing.jiang@cloud.ionos.com>
To: axboe@kernel.dk
Cc: linux-block@vger.kernel.org,
	Guoqing Jiang <guoqing.jiang@cloud.ionos.com>
Subject: [PATCH RFC V2 0/4] block: add two statistic tables
Date: Mon, 13 Jul 2020 23:13:17 +0200	[thread overview]
Message-ID: <20200713211321.21123-1-guoqing.jiang@cloud.ionos.com> (raw)

Hi,

Now, a new io_extra_stats node is introduced which works with the config
option. with the double insurance, I think no one will suffer from the
additional overhead unless people really need the statistic tables.

And I will delete the kernel option if it doesn't make sense to have both
kernel option and sysfs node. Review and comment are welcomed.

Thanks,
Guoqing

RFC V1 -> RFC V2:
* don't call ktime_get_ns and drop unnecessary patches.
* add io_extra_stats to avoid potential overhead.

RFC V1 at https://marc.info/?l=linux-block&m=159419516730386&w=2

Guoqing Jiang (4):
  block: add a statistic table for io latency
  block: add a statistic table for io sector
  block: add io_extra_stats node
  block: call blk_additional_{latency,sector} only when io_extra_stats
    is true

 Documentation/block/queue-sysfs.rst |  6 ++++
 block/Kconfig                       |  9 +++++
 block/blk-core.c                    | 54 +++++++++++++++++++++++++++++
 block/blk-sysfs.c                   |  8 +++++
 block/genhd.c                       | 47 +++++++++++++++++++++++++
 include/linux/blkdev.h              |  2 ++
 include/linux/part_stat.h           |  8 +++++
 7 files changed, 134 insertions(+)

-- 
2.17.1


             reply	other threads:[~2020-07-13 21:13 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-13 21:13 Guoqing Jiang [this message]
2020-07-13 21:13 ` [PATCH RFC V2 1/4] block: add a statistic table for io latency Guoqing Jiang
2020-07-14  7:43   ` Johannes Thumshirn
2020-07-14  8:25     ` Guoqing Jiang
2020-08-11  7:22   ` Danil Kipnis
2020-07-13 21:13 ` [PATCH RFC V2 2/4] block: add a statistic table for io sector Guoqing Jiang
2020-08-11 15:04   ` Aleksei Marov
2020-08-11 15:48     ` Guoqing Jiang
2020-07-13 21:13 ` [PATCH RFC V2 3/4] block: add io_extra_stats node Guoqing Jiang
2020-07-13 21:13 ` [PATCH RFC V2 4/4] block: call blk_additional_{latency,sector} only when io_extra_stats is true Guoqing Jiang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200713211321.21123-1-guoqing.jiang@cloud.ionos.com \
    --to=guoqing.jiang@cloud.ionos.com \
    --cc=axboe@kernel.dk \
    --cc=linux-block@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.