[libvirt] [PATCH] Use camelCase for XML attribute numQueues
Eric Blake
eblake at redhat.com
Tue Apr 23 23:44:06 UTC 2013
On 04/23/2013 05:32 AM, Martin Kletzander wrote:
> On 04/22/2013 04:43 PM, Laine Stump wrote:
>> On 04/22/2013 10:13 AM, Laine Stump wrote:
>>> Of course in this case, we *could* avoid the camelCase vs underscore
>>> entirely by just naming the attribute "queues" instead of numQueues or
>>> num_queues (the fact that the attribute is a number makes it pretty
>>> obvious that it is the "number of queues").
>>
>> And I notice that the patches for supporting multiple queues in
>> interfaces uses <driver queues='n'/>, so consistency would vote in favor
>> of using the same thing for <controller>.
>>
>
> I'm not the '+1' kind of guy, but this makes way more sense in the
> driver element and I think we would definitely make a use of that in the
> future, even though it increases the complexity of XML. Since there's
> still time, should I try to change it to <driver queues... or is
> somebody else against (or already did) that? (Last time I wrongly
> assumed that sending the patch would speed up solving the question, so
> I'm rather asking now).
It still hasn't been done. Martin, can you jump in and do it? Laine,
speak up now if you were doing it as part of your vfio <driver> work,
although it didn't look like that to me.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130423/a24a3b85/attachment-0001.sig>
More information about the libvir-list
mailing list