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@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org