On Fri, May 4, 2012 at 1:19 PM, Srinivasa R Kanduru <[email protected]>wrote:
> Thanks a lot guys ! I was using kvm directly to launch VM as libvirt > (0.9.11.3) was giving an internal error even with a simple config file > which replaces the default network with openvswitch. > > What error was that? Can you provide your Domain config XML file? > May I ask which version of libvirt that I should use ? > > Regards, > Srinivas. > > > On Fri, May 4, 2012 at 1:00 PM, Chris Wright <[email protected]> wrote: > >> * Srinivasa R Kanduru ([email protected]) wrote: >> > The physical interface is a wireless NIC. I assume that shouldn't have >> any >> > bearing on the behaviour. >> >> Yes, it does. Seems that OVS ought to do something like: >> >> Subject: [PATCH] openvswitch: honour IFF_DONT_BRIDGE >> From: Chris Wright <[email protected]> >> >> Especially in the case of wireless, the netdev may not work properly. >> So honor IFF_DONT_BRIDGE. >> >> Signed-off-by: Chris Wright <[email protected]> >> -- >> NOTE: Completely untested >> >> diff --git a/net/openvswitch/vport-netdev.c >> b/net/openvswitch/vport-netdev.c >> index c1068ae..ee04d44 100644 >> --- a/net/openvswitch/vport-netdev.c >> +++ b/net/openvswitch/vport-netdev.c >> @@ -96,6 +96,11 @@ static struct vport *netdev_create(const struct >> vport_parms *parms) >> goto error_put; >> } >> >> + if (netdev_vport->dev->priv_flags & IFF_DONT_BRIDGE) { >> + err = -EOPNOTSUPP; >> + goto error_put; >> + } >> + >> err = netdev_rx_handler_register(netdev_vport->dev, >> netdev_frame_hook, >> vport); >> if (err) >> > > > > > > _______________________________________________ > discuss mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/discuss > >
_______________________________________________ discuss mailing list [email protected] http://openvswitch.org/mailman/listinfo/discuss
