qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] qapi: Complete BlkdebugEvent


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCH] qapi: Complete BlkdebugEvent
Date: Fri, 07 Nov 2014 22:01:55 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0

On 07.11.2014 21:57, Eric Blake wrote:
On 11/07/2014 04:51 PM, Max Reitz wrote:
Several events were missing from the QAPI enum, add them.

Reported-by: Kevin Wolf <address@hidden>
Signed-off-by: Max Reitz <address@hidden>
---
  qapi/block-core.json | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)
Do we need any documentation on these? Or is this one of those types
where the only people using it are already expected to be familiar with
the C code?

Well, you have to be familiar with the block driver as well. The documentation wouldn't be very useful, it'd be something like "refblock_alloc.write_blocks: Emitted when the new refblocks are written during qcow2 refblock allocation" or "empty_image_prepare: Emitted when qcow2 prepares to empty an image". So I'd leave the documentation out. Most users don't want to use this anyway, blkdebug is for debugging purposes only, so you need to know what the event does in the context of the block driver you're debugging anyway.

Max

diff --git a/qapi/block-core.json b/qapi/block-core.json
index 77a0cfb..8c3e45d 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -1422,7 +1422,9 @@
              'refblock_alloc.write_blocks', 'refblock_alloc.write_table',
              'refblock_alloc.switch_table', 'cluster_alloc',
              'cluster_alloc_bytes', 'cluster_free', 'flush_to_os',
-            'flush_to_disk' ] }
+            'flush_to_disk', 'pwritev_rmw.head', 'pwritev_rmw.after_head',
+            'pwritev_rmw.tail', 'pwritev_rmw.after_tail', 'pwritev',
+            'pwritev_zero', 'pwritev_done', 'empty_image_prepare' ] }
##
  # @BlkdebugInjectErrorOptions





reply via email to

[Prev in Thread] Current Thread [Next in Thread]