[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