Re: [ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-27 Thread Joe Stringer
On 28 August 2014 10:34, Pravin Shelar wrote: > > > This is weird. I could not reproduce this on newer kernel. But I found > related bug. Can you try attached patch? It may be related. > I will submit formal patch later. > > --Pravin. > This patch doesn't fix the issue. Another slight variatio

Re: [ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-27 Thread Pravin Shelar
On Tue, Aug 26, 2014 at 11:43 PM, Joe Stringer wrote: > I've been having a bit of trouble with ovs after this patch was applied, > ovs-vswitchd locks up on Debian 7.6 (VBox VM). Kernel is 3.2.0-4-amd64 #1 > SMP Debian 3.2.60-1+deb7u3 x86_64 GNU/Linux. > > I have a fairly typical phy bridge + int b

Re: [ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-26 Thread Joe Stringer
I've been having a bit of trouble with ovs after this patch was applied, ovs-vswitchd locks up on Debian 7.6 (VBox VM). Kernel is 3.2.0-4-amd64 #1 SMP Debian 3.2.60-1+deb7u3 x86_64 GNU/Linux. I have a fairly typical phy bridge + int bridge setup, with dhclient running on br-int: # ovs-vsctl show

Re: [ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-25 Thread Pravin Shelar
On Thu, Aug 21, 2014 at 10:25 AM, Lorand Jakab wrote: > struct ovs_skb_cb is full on kernels < 3.11 due to compatibility code. > This patch removes the 'flow' member in order to make room for data > needed by layer 3 flow/port support that will be added in an upcoming > patch. The 'flow' memeber

Re: [ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-24 Thread Jesse Gross
On Thu, Aug 21, 2014 at 12:25 PM, Lorand Jakab wrote: > struct ovs_skb_cb is full on kernels < 3.11 due to compatibility code. > This patch removes the 'flow' member in order to make room for data > needed by layer 3 flow/port support that will be added in an upcoming > patch. The 'flow' memeber

[ovs-dev] [PATCH] datapath: remove flow member from struct ovs_skb_cb

2014-08-21 Thread Lorand Jakab
struct ovs_skb_cb is full on kernels < 3.11 due to compatibility code. This patch removes the 'flow' member in order to make room for data needed by layer 3 flow/port support that will be added in an upcoming patch. The 'flow' memeber was chosen for removal because it's only used in ovs_execute_ac