On Wed, Nov 20, 2013 at 05:19:25PM +0900, Simon Horman wrote: > On Mon, Nov 18, 2013 at 03:48:29PM +0900, Simon Horman wrote: > > On Fri, Nov 15, 2013 at 02:33:14PM -0800, Ben Pfaff wrote: > > > Until now ofpacts_check() has been told either to enforce consistency or > > > not, but that means that the caller has to know exactly what protocol is > > > going to be in use (because some protocols require consistency to be > > > enforced and others don't). This commit changes ofpacts_check() to just > > > rule out protocols that require enforcement when it detects > > > inconsistencies. > > > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > > > Hi Ben, > > > > this approach seems reasonable to me. > > > > The problem I see with verification of VLAN actions that follow push_mpls > > when OpenFlow1.3 is not that different protocols require consistency to > > enforced while others don't. Rather the problem I as is that what is > > consistent is different for pre-OpenFlow1.3 and OpenFlow1.3+ tag ordering. > > > > I believe that logic to handle that difference can be built on top of your > > patch and what follows is a prototype of such logic. I'm not sure that this > > is the direction you want us to take so I would appreciate feedback on this > > approach. > > Hi Ben, > > I know this is rather complex and becoming rather drawn out. > But if you could take a few (maybe quite a few) moments > to look over this I would be most grateful.
Yeah, sorry, as usual I'm backlogged and I tend to be drawn toward an "review easiest patches first" strategy. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
