On Wed, Feb 07, 2018 at 03:50:35PM +0300, Vladimir Sementsov-Ogievskiy wrote: > v2: > > 01: add block_latency_histogram_clear() > 02: fix spelling (sorry =() > some rewordings > remove histogram if latency parameter unspecified > > Vladimir Sementsov-Ogievskiy (2): > block/accounting: introduce latency histogram > qapi: add block latency histogram interface > > qapi/block-core.json | 73 +++++++++++++++++++++++++++++++++- > include/block/accounting.h | 9 +++++ > block/accounting.c | 97 > ++++++++++++++++++++++++++++++++++++++++++++++ > block/qapi.c | 31 +++++++++++++++ > blockdev.c | 19 +++++++++ > 5 files changed, 228 insertions(+), 1 deletion(-)
The feature looks good. I posted documentation and code readability suggestions. Stefan
signature.asc
Description: PGP signature