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

[Ján Tomko]

On 09/17/2014 04:57 PM, Daniel P. Berrange wrote:
> On Mon, Sep 15, 2014 at 04:30:46PM -0600, 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
>>>        <model type='virtio'/>
>>>        <b><driver name='vhost' txmode='iothread' ioeventfd='on' event_idx='off' queues='5'/></b>
>>>      </interface>
>>> +    <interface type='network'>
>>> +      <source network='default'/>
>>> +      <target dev='vnet2'/>
>>> +      <model type='virtio'/>
>>> +      <b><driver csum='off' gso='off' guest_tso4='off' guest_tso6='off' guest_ecn='off'/></b>
>>> +    </interface>
>>
>> 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'm not really convinced having a 'guest_' prefix really buys
> us anything here, since there's no naming clash to avoid. Why
> don't we just kill the 'guest_' prefixes.

The clash is in the options I didn't expose:
http://git.qemu.org/?p=qemu.git;a=blob;f=include/hw/virtio/virtio-net.h;h=6ceb5aa92

because they weren't requested by the (private :() bug 1139364

Jan

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