[libvirt] [PATCH] Add support for 'passthru' mode for direct network interfaces

Eric Blake eblake at redhat.com
Wed May 18 14:22:23 UTC 2011


On 05/18/2011 05:32 AM, D. Herrendoerfer wrote:
>>>> destination
>>>>      device is in <code>private</code> mode.</dd>
>>>> +      <dt><code>passthru</code></dt>
>>>> +      <dd>This feature allows to attach a virtual function of a SRIOV
>>>> capable
>>>
>>  That patch looks fine to me, ACK,
>> The only unresolved uncertainty to me is what happen if you try to use
>> it on a kernel without the support ?
>>
> 
> This looks fine, except in formatdomain.html.in the feature is still called
> passthru.
> 
> Used on a system without passthru support this raises an error indicating a
> failed macvtap setup.

I fixed that one last place, and found one more issue:

> 
> --- a/src/util/macvtap.h
> +++ b/src/util/macvtap.h
> @@ -92,6 +92,7 @@ void delMacvtap(const char *ifname,
> #  define MACVTAP_MODE_PRIVATE_STR  "private"
> #  define MACVTAP_MODE_VEPA_STR     "vepa"
> #  define MACVTAP_MODE_BRIDGE_STR   "bridge"
> +#  define MACVTAP_MODE_PASSTHRU_STR "passthru" 

No one was using these defines (they were leftovers from before we
switched to virEnum), so I deleted them.

Now pushed.  Thanks for the submission!

-- 
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/20110518/83ad8831/attachment-0001.sig>


More information about the libvir-list mailing list