Re: [ovs-dev] [PATCH] ofp-util: Recognise wildcarded VLAN as NXM only.

2012-04-04 Thread Ben Pfaff
On Sat, Mar 31, 2012 at 01:30:59AM -0700, Ethan Jackson wrote: > Is there any reason we can't simply ignore the CFI bit? Something like the > following? I'm having trouble working out exactly how the CFI bit in OpenFlow > matches interacts with the various versions and the classifier. OpenFlow 1

Re: [ovs-dev] [PATCH] ofp-util: Recognise wildcarded VLAN as NXM only.

2012-04-02 Thread Ethan Jackson
Is there any reason we can't simply ignore the CFI bit? Something like the following? I'm having trouble working out exactly how the CFI bit in OpenFlow matches interacts with the various versions and the classifier. Ethan --- lib/ofp-util.c |9 + 1 file changed, 9 insertions(+) d

Re: [ovs-dev] [PATCH] ofp-util: Recognise wildcarded VLAN as NXM only.

2012-03-29 Thread Ben Pfaff
On Thu, Mar 29, 2012 at 03:00:06PM -0700, Ethan Jackson wrote: > Without this patch, using ovs-ofctl to add a rule which masks the > vlan_tci field would not work as expected unless the protocol was > explicitly set to NXM. > > Signed-off-by: Ethan Jackson This is the right idea but I think that

[ovs-dev] [PATCH] ofp-util: Recognise wildcarded VLAN as NXM only.

2012-03-29 Thread Ethan Jackson
Without this patch, using ovs-ofctl to add a rule which masks the vlan_tci field would not work as expected unless the protocol was explicitly set to NXM. Signed-off-by: Ethan Jackson --- lib/ofp-util.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/ofp-util.c b/lib/ofp-ut