qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 12/13] qapi: support for keyworded variable-leng


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 12/13] qapi: support for keyworded variable-length argument list
Date: Fri, 30 Mar 2012 09:49:20 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120216 Thunderbird/10.0.1

Il 29/03/2012 22:01, Anthony Liguori ha scritto:
>> Then, we open-code the marshaller to process the QDict, rather than
>> embedding
>> it in the script or passing it through to qmp_device_add().
> 
> You could also just do gen=False...
> 
> But I don't think open coding the marshaller is the right thing here. 
> You have to convert to strings and reparse anyway.

device_add expects strings, you don't have to convert no?  There is a
single parse point.

(Instead, a qom_add would accept the right types, but then it would also
use a more QAPI-friendly syntax with lists insteads of varargs).

Regarding device_add ? and device_add foo,? I would implement it as
separate QMP commands hooking into QOM, such as qom_list_types (taking
the superclass as an optional argument) and qom_properties.  But the
latter first needs static properties to move up from devices to objects.
 I'll take a look at that.

Paolo



reply via email to

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