[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 3/3] hmp: show the backing file ancestors cou
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH V2 3/3] hmp: show the backing file ancestors count |
Date: |
Wed, 25 Jul 2012 13:22:37 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Am 25.07.2012 13:17, schrieb Benoît Canet:
> Le Wednesday 25 Jul 2012 à 12:59:00 (+0200), Kevin Wolf a écrit :
>> Am 25.07.2012 10:11, schrieb address@hidden:
>>> From: Benoît Canet <address@hidden>
>>>
>>> Signed-off-by: Benoit Canet <address@hidden>
>>> ---
>>> hmp.c | 2 ++
>>> 1 file changed, 2 insertions(+)
>>>
>>> diff --git a/hmp.c b/hmp.c
>>> index 6b72a64..19dcb65 100644
>>> --- a/hmp.c
>>> +++ b/hmp.c
>>> @@ -227,6 +227,8 @@ void hmp_info_block(Monitor *mon)
>>> if (info->value->inserted->has_backing_file) {
>>> monitor_printf(mon, " backing_file=");
>>> monitor_print_filename(mon,
>>> info->value->inserted->backing_file);
>>> + monitor_printf(mon, " backing_file_ancestors_count=%"
>>> PRId64,
>>> + info->value->inserted->backing_file_ancestors_count);
>>
>> This is not what a user interface looks like. Maybe something like "%d
>> backing files", and omit the message if there aren't any backing files.
>
> The whole line of info block look like this:
> virtio0: removable=0 io-status=ok file=/home/benoit/images/gruik.qed
> backing_file=/home/benoit/images/snap.qed backing_file_ancestors_count=1 ro=0
> drv=qed encrypted=0 bps=0 bps_rd=0 bps_wr=0 iops=0 iops_rd=0 iops_wr=0
> Again I tried to make something coherent.
I see. I wasn't really aware that it's already that bad today. The whole
output of this command should be changed, but it's not in the scope of
your series.
> It's already ommited when there is no backing file (the field is optional).
Yeah, sorry, I missed the context.
Kevin
[Qemu-devel] [PATCH V2 1/3] block: create bdrv_get_backing_file_ancestors_count(), benoit . canet, 2012/07/25
[Qemu-devel] [PATCH V2 3/3] hmp: show the backing file ancestors count, benoit . canet, 2012/07/25