Re: [PATCH 4/5] conf: add options for disabling segment offloading

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

 



On 09/16/2014 12:30 AM, Eric Blake wrote:
> On 09/11/2014 05:43 AM, Ján Tomko wrote:
>> Add the following attributes:
>> csum, gso, guest_tso4, guest_tso6, guest_ecn
>> to the <driver> element of network interface
>> which control the virtio-net device properties
>> of the same names.
>> ---
>>  docs/formatdomain.html.in                          | 27 ++++++++
>>  docs/schemas/domaincommon.rng                      | 25 +++++++
>>  src/conf/domain_conf.c                             | 81 ++++++++++++++++++++++
>>  src/conf/domain_conf.h                             |  5 ++
>>  .../qemuxml2argv-net-virtio-disable-offloads.xml   | 32 +++++++++
>>  tests/qemuxml2xmltest.c                            |  1 +
>>  6 files changed, 171 insertions(+)
>>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-net-virtio-disable-offloads.xml
>>
>> diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
>> index a2ea758..5b2758a 100644
>> --- a/docs/formatdomain.html.in
>> +++ b/docs/formatdomain.html.in
>> @@ -3847,6 +3847,12 @@ qemu-kvm -net nic,model=? /dev/null
>>        &lt;model type='virtio'/&gt;
>>        <b>&lt;driver name='vhost' txmode='iothread' ioeventfd='on' event_idx='off' queues='5'/&gt;</b>
>>      &lt;/interface&gt;
>> +    &lt;interface type='network'&gt;
>> +      &lt;source network='default'/&gt;
>> +      &lt;target dev='vnet2'/&gt;
>> +      &lt;model type='virtio'/&gt;
>> +      <b>&lt;driver csum='off' gso='off' guest_tso4='off' guest_tso6='off' guest_ecn='off'/&gt;</b>
>> +    &lt;/interface&gt;
> 
> Are we stuck with names with underscores in our XML?  I'm still not sure
> if we've come up with the best naming for exposing all these knobs.

I'd rather not mix underscores (event_idx) and other word separators in the
same element.

Alternatively, we could do something like:
<driver csum='off' gso='off'>
  <guest tso4='off' tso6='off' ecn='off'/>
</driver>
to get rid of the multi-word attributes, but I like the underscores better
because they match QEMU arguments.

Jan

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]