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

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

[Max Reitz]

Several events were missing from the QAPI enum, add them.

Reported-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
---
 qapi/block-core.json | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

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
-- 
1.9.3

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

[Kevin Wolf]

Am 07.11.2014 um 16:51 hat Max Reitz geschrieben:
> Several events were missing from the QAPI enum, add them.
> 
> Reported-by: Kevin Wolf <kwolf@redhat.com>
> Signed-off-by: Max Reitz <mreitz@redhat.com>

Thanks, applied to the block branch.

Kevin

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

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 1269 bytes --]

On 11/07/2014 04:51 PM, Max Reitz wrote:
> Several events were missing from the QAPI enum, add them.
> 
> Reported-by: Kevin Wolf <kwolf@redhat.com>
> Signed-off-by: Max Reitz <mreitz@redhat.com>
> ---
>  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?

> 
> 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
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 539 bytes --]

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

[Max Reitz]

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 <kwolf@redhat.com>
>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>   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
>>