[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v3 1/8] qapi: group BlockDeviceStats fields
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [PATCH v3 1/8] qapi: group BlockDeviceStats fields |
Date: |
Fri, 15 Jun 2018 14:58:35 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 13 Jun 2018 07:44:19 PM CEST, Anton Nefedov <address@hidden> wrote:
> Make the stat fields definition slightly more readable.
> Also reorder total_time_ns stats read-write-flush as done elsewhere.
> Cosmetic change only.
>
> Signed-off-by: Anton Nefedov <address@hidden>
> ---
> qapi/block-core.json | 34 +++++++++++++++++++++++-----------
> 1 file changed, 23 insertions(+), 11 deletions(-)
>
> diff --git a/qapi/block-core.json b/qapi/block-core.json
> index b3e94d8..ee75aef 100644
> --- a/qapi/block-core.json
> +++ b/qapi/block-core.json
> @@ -784,12 +784,12 @@
> # @flush_operations: The number of cache flush operations performed by the
> # device (since 0.15.0)
> #
> -# @flush_total_time_ns: Total time spend on cache flushes in nano-seconds
> -# (since 0.15.0).
> +# @rd_total_time_ns: Total time spent on reads in nanoseconds (since 0.15.0).
> #
> -# @wr_total_time_ns: Total time spend on writes in nano-seconds (since
> 0.15.0).
> +# @wr_total_time_ns: Total time spent on writes in nanoseconds (since
> 0.15.0).
> #
> -# @rd_total_time_ns: Total_time_spend on reads in nano-seconds (since
> 0.15.0).
> +# @flush_total_time_ns: Total time spent on cache flushes in nanoseconds
> +# (since 0.15.0).
> #
> # @wr_highest_offset: The offset after the greatest byte written to the
> # device. The intended use of this information is for
> @@ -842,14 +842,26 @@
> # Since: 0.14.0
> ##
> { 'struct': 'BlockDeviceStats',
> - 'data': {'rd_bytes': 'int', 'wr_bytes': 'int', 'rd_operations': 'int',
> - 'wr_operations': 'int', 'flush_operations': 'int',
> - 'flush_total_time_ns': 'int', 'wr_total_time_ns': 'int',
> - 'rd_total_time_ns': 'int', 'wr_highest_offset': 'int',
> - 'rd_merged': 'int', 'wr_merged': 'int', '*idle_time_ns': 'int',
> + 'data': {'rd_bytes': 'int', 'wr_bytes': 'int',
> +
> + 'rd_operations': 'int', 'wr_operations': 'int',
> + 'flush_operations': 'int',
> +
> + 'rd_total_time_ns': 'int', 'wr_total_time_ns': 'int',
> + 'flush_total_time_ns': 'int',
This patch looks good, but I don't think the empty lines here are
necessary (is there any other data structure using them?).
Berto
- [Qemu-block] [PATCH v3 0/8] discard blockstats, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 3/8] ide: account UNMAP (TRIM) operations, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 4/8] scsi: store unmap offset and nb_sectors in request struct, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 2/8] qapi: add unmap to BlockDeviceStats, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 5/8] scsi: move unmap error checking to the complete callback, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 1/8] qapi: group BlockDeviceStats fields, Anton Nefedov, 2018/06/13
- Re: [Qemu-block] [PATCH v3 1/8] qapi: group BlockDeviceStats fields,
Alberto Garcia <=
- [Qemu-block] [PATCH v3 7/8] file-posix: account discard operations, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 6/8] scsi: account unmap operations, Anton Nefedov, 2018/06/13
- [Qemu-block] [PATCH v3 8/8] qapi: query-blockstat: add driver specific file-posix stats, Anton Nefedov, 2018/06/13
- Re: [Qemu-block] [Qemu-devel] [PATCH v3 0/8] discard blockstats, no-reply, 2018/06/13
- Re: [Qemu-block] [Qemu-devel] [PATCH v3 0/8] discard blockstats, no-reply, 2018/06/13