[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v2 3/3] i386/cpu: net: Flatten simple union Gues

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v2 3/3] i386/cpu: net: Flatten simple union GuestPanicInformationType
Date: Wed, 22 Feb 2017 19:46:20 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Eric Blake <address@hidden> writes:

> On 02/21/2017 02:46 PM, Markus Armbruster wrote:
>> Simple unions are simpler than flat unions in the schema, but more
>> complicated in C and on the QMP wire: there's extra indirection in C
>> and extra nesting on the wire, both pointless.  They're best avoided
>> in new code.  Fix up recent commit d187e08 accordingly.
>> Cc: Anton Nefedov <address@hidden>
>> Cc: Denis V. Lunev <address@hidden>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>>  qapi-schema.json  | 14 ++++++++++++--
>>  target/i386/cpu.c | 17 ++++++-----------
>>  vl.c              | 12 ++++++------
>>  3 files changed, 24 insertions(+), 19 deletions(-)
> Very similar to Anton's patch which is on Paolo's queue:
> https://lists.gnu.org/archive/html/qemu-devel/2017-02/msg04448.html

Wasn't cc'ed, so I missed it.  No biggie.

> I don't care which version goes in, but will leave it to Markus and
> Paolo to decide which queue it should go through.

My commit message is more verbose, and I fix space style in the QAPI
schema.  On the other hand, Anton has an extra QAPI schema comment line.

Picking Anton's is probably simpler, because it's 2/3, and mine looks
like it conflicts with his 1/3.

> Reviewed-by: Eric Blake <address@hidden>


reply via email to

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