On 22/1/2018 11:47 PM, Eric Blake wrote:
On 01/19/2018 06:50 AM, Anton Nefedov wrote:
Signed-off-by: Anton Nefedov <anton.nefe...@virtuozzo.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
Reviewed-by: Alberto Garcia <be...@igalia.com>
---
  qapi/block-core.json       | 29 +++++++++++++++++++++++------
  include/block/accounting.h |  1 +
  block/qapi.c               |  6 ++++++
  3 files changed, 30 insertions(+), 6 deletions(-)


@@ -688,6 +693,9 @@
  #
  # @rd_total_time_ns: Total_time_spend on reads in nano-seconds (since 0.15.0).

While we are here, we could change s/Total_time_spend/Total time spent/

  #
+# @unmap_total_time_ns: Total time spent on unmap operations in nano-seconds
+#                       (Since 2.12)

Also, s/nano-seconds/nanoseconds/ (for both lines, if we are touching both).

The QAPI maintainer can touch that up (that may be me, depending on
Markus' schedule in the next few weeks); but I'm not seeing any UI
problems with the addition, so

Reviewed-by: Eric Blake <ebl...@redhat.com>


Thanks, fixed

Reply via email to