Re: [ovs-dev] [learning v2 15/19] Implement new "learn" action.

2011-09-13 Thread Ethan Jackson
Sounds good. Ethan On Tue, Sep 13, 2011 at 11:41, Ben Pfaff wrote: > On Tue, Sep 13, 2011 at 11:39:12AM -0700, Ethan Jackson wrote: >> > + * 3. Here's a recipe for a very simple-minded MAC learning switch. ?It >> > uses a >> > + * ? ?10-second MAC expiration time to make it easier to see what's

Re: [ovs-dev] [learning v2 15/19] Implement new "learn" action.

2011-09-13 Thread Ben Pfaff
On Tue, Sep 13, 2011 at 11:39:12AM -0700, Ethan Jackson wrote: > > + * 3. Here's a recipe for a very simple-minded MAC learning switch. ?It > > uses a > > + * ? ?10-second MAC expiration time to make it easier to see what's going > > on > > + * > > + * ? ? ?ovs-vsctl del-controller br0 > > + * ?

Re: [ovs-dev] [learning v2 15/19] Implement new "learn" action.

2011-09-13 Thread Ethan Jackson
> + * 3. Here's a recipe for a very simple-minded MAC learning switch.  It uses > a > + *    10-second MAC expiration time to make it easier to see what's going on > + * > + *      ovs-vsctl del-controller br0 > + *      ovs-ofctl del-flows br0 > + *      ovs-ofctl add-flow br0 "table=0 actions=le

Re: [ovs-dev] [learning v2 15/19] Implement new "learn" action.

2011-09-09 Thread Ethan Jackson
I get the following compiler warning: lib/learn.c: In function 'learn_format': lib/learn.c:463:32: error: variable 'src_field' set but not used [-Werror=unused-but-set-variable] > ovs-vsctl del-controller br0 > ovs-ofctl del-flows br0 > ovs-ofctl add-flow br0 "table=0 actions=learn(ta

[ovs-dev] [learning v2 15/19] Implement new "learn" action.

2011-08-19 Thread Ben Pfaff
There are a few loose ends here. First, learning actions cause too much flow revalidation. Upcoming commits will fix that problem. The following additional issues have not yet been addressed: * Resource limits: nothing yet limits the maximum number of flows that can be learned. It is