Re: [ovs-dev] [PATCH] ofproto: Check for overlapping flows only in the target table.

2011-10-24 Thread Ben Pfaff
Thanks, I pushed this one. On Fri, Oct 21, 2011 at 07:18:13PM -0700, Ethan Jackson wrote: > Looks good. > > Ethan > > > On Wed, Oct 19, 2011 at 16:03, Ben Pfaff wrote: > > There's no reason to check for overlapping flows in table A if the flow > > is going to be inserted into table B. > > > >

Re: [ovs-dev] [PATCH] ofproto: Check for overlapping flows only in the target table.

2011-10-21 Thread Ethan Jackson
Looks good. Ethan On Wed, Oct 19, 2011 at 16:03, Ben Pfaff wrote: > There's no reason to check for overlapping flows in table A if the flow > is going to be inserted into table B. > > (I doubt anyone actually uses OFPFF_CHECK_OVERLAP though.) > --- >  ofproto/ofproto.c |   17 ++---

[ovs-dev] [PATCH] ofproto: Check for overlapping flows only in the target table.

2011-10-19 Thread Ben Pfaff
There's no reason to check for overlapping flows in table A if the flow is going to be inserted into table B. (I doubt anyone actually uses OFPFF_CHECK_OVERLAP though.) --- ofproto/ofproto.c | 17 ++--- 1 files changed, 6 insertions(+), 11 deletions(-) diff --git a/ofproto/ofproto.