[ovs-discuss] (no subject)

[Ansis Atteka]

On Fri, May 4, 2012 at 1:19 PM, Srinivasa R Kanduru <skanduru at gmail.com>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 <chrisw at sous-sol.org> wrote:
>
>> * Srinivasa R Kanduru (skanduru at gmail.com) 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 <chrisw at sous-sol.org>
>>
>> Especially in the case of wireless, the netdev may not work properly.
>> So honor IFF_DONT_BRIDGE.
>>
>> Signed-off-by: Chris Wright <chrisw at sous-sol.org>
>> --
>> 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
> discuss at openvswitch.org
> http://openvswitch.org/mailman/listinfo/discuss
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://openvswitch.org/pipermail/ovs-discuss/attachments/20120504/18e1e692/attachment.html>