Re: [ovs-dev] [subfacet 3/4] ofproto-dpif: Set flow-eviction-threshold globally.

2013-06-07 Thread Ethan Jackson
> Whoa. Declaring a static variable in a header file? Far out. But it > isn't going to work. Doh you're right. I'll just make it an extern. See the incremental below. Looks alright otherwise? --- ofproto/ofproto-provider.h |3 +-- ofproto/ofproto.c |2 ++ 2 files changed, 3

Re: [ovs-dev] [subfacet 3/4] ofproto-dpif: Set flow-eviction-threshold globally.

2013-06-07 Thread Ben Pfaff
On Fri, Jun 07, 2013 at 01:32:12PM -0700, Ethan Jackson wrote: > > Whoa. Declaring a static variable in a header file? Far out. But it > > isn't going to work. > > Doh you're right. I'll just make it an extern. See the incremental below. > Looks alright otherwise? Yes, thanks. __

Re: [ovs-dev] [subfacet 3/4] ofproto-dpif: Set flow-eviction-threshold globally.

2013-06-06 Thread Ben Pfaff
On Wed, Jun 05, 2013 at 01:46:20PM -0700, Ethan Jackson wrote: > With the signal datapath, it no longer makes sense to have a per "single" datapath > ofproto flow eviction threshold. This patch moves the flow > eviction threshold to the Open_vSwitch table making the setting > global, though stil