[libvirt] [PATCH 0/7] Add support for setting QoS

Eric Blake eblake at redhat.com
Fri Jun 24 14:38:40 UTC 2011


On 06/24/2011 01:34 AM, Michal Privoznik wrote:
>> 5) I see you've put the data structures and parsing/formatting functions
>> that are used by both network_conf.c and domain_conf.c in the util
>> directory. I've been wondering myself what to do with similar
>> structs/functions that will be used in both network and domain XML (eg,
>> <virtualport> will soon be used by both) - does everyone agree this is
>> the right way to handle it? Or should there be a common file in the conf
>> directory instead? (I don't have an opinion, but want to "do the right
>> thing" when I write my own code)
> Yeah, I was thinking the same way. When I wrote the code, I've decided
> for src/util. But I would not hesitate if somebody wants it in src/conf.

We already have src/util/sysinfo.[ch]	 for common parsing/formatting of
sysinfo XML, so I'm okay with src/util.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110624/46e4d4c9/attachment-0001.sig>


More information about the libvir-list mailing list