[Top][All Lists]

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

Re: [Qemu-ppc] [Qemu-devel] [PATCH v3 18/18] qapi: Add parameter to visi

From: Eric Blake
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH v3 18/18] qapi: Add parameter to visit_end_*
Date: Tue, 3 May 2016 06:41:53 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

On 05/03/2016 05:53 AM, Markus Armbruster wrote:
> Okay, I'm confused.
> Consider BlockdevRef, defined as
>     { 'alternate': 'BlockdevRef',
>       'data': { 'definition': 'BlockdevOptions',
>                 'reference': 'str' } }
> where BlockdevOptions is a (flat) union.  Let's clone a BlockdevRef
> holding a str.  Sequence of calls:
>     qapi_BlockdevRef_clone(src)
>         qapi_clone_visitor_new(src)
>         // qcv->depth is now 0
>         visit_type_BlockdevRef(v, NULL, &dst, &error_abort)
>             visit_start_alternate(v, NULL, &dst,
>                                   sizeof(BlockdevRef), true, &error_abort)
>                 qapi_clone_start_alternate(v, NULL, &dst,
>                                   sizeof(BlockdevRef), true, &error_abort)

so far, so good,

>                     qapi_clone_start_struct(v, NULL, &dst,
>                                   sizeof(BlockdevRef), &error_abort)

Not reached if it holds a string. qapi_clone_start_alternate() sets
dst->type to QTYPE_QSTRING, but qapi_clone_start_struct() is only used

>                         // does not increment qcv->depth
>         visit_type_str(v, NULL, &dst->u.references, &error_abort)
>             qapi_clone_type_str(v, NULL, &dst->u.references, &error_abort)
>                 assert(qcv->depth)  // why does this hold?

Either we call:


or we elide the inner struct visit because the alternate's branch is
scalar.  Either way, we only need +1 to the depth over the entire
alternate, because we only need 1 level of g_memdup() at the time we
first enter the alternate.

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

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

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