Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Andy Zhou
Thank you for pointing this out. Please disregard the patch. On Fri, Jun 21, 2013 at 10:12 AM, Jesse Gross wrote: > That sounds right to me. > > On Fri, Jun 21, 2013 at 10:06 AM, Andy Zhou wrote: > > You are right. > > > > The user space code needs to distinguish between two cases: > > 1) The

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Jesse Gross
That sounds right to me. On Fri, Jun 21, 2013 at 10:06 AM, Andy Zhou wrote: > You are right. > > The user space code needs to distinguish between two cases: > 1) The entire mask attribute is missing, then the entire key is exact match. > 2) The mask attribute is there, then the missing key attrib

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Andy Zhou
You are right. The user space code needs to distinguish between two cases: 1) The entire mask attribute is missing, then the entire key is exact match. 2) The mask attribute is there, then the missing key attributes should be treated as wildcarded fields. It seems we don't need this patch, but ma

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Jesse Gross
It doesn't really make sense that userspace would interpret a missing attribute as exact match because when the flow is installed a missing attribute means that it is wildcarded. Therefore, if you echoed back the exact same flow it would have a different meaning. On Fri, Jun 21, 2013 at 9:43 AM, A

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Andy Zhou
Correction. The ICMP mask will have to be fully masked in this example. But they still need to be present in the netlink. A missing ICMP mask attribute will be interpreted by the user space programs as exact match. On Fri, Jun 21, 2013 at 9:38 AM, Andy Zhou wrote: > Yes, you are right abou

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Andy Zhou
Yes, you are right about the root cause. A fully wildcarded ip proto in the key would prevent the mask of ICMP (may not be fully wildcarded) to be generated. This patch fixes it. Andy On Fri, Jun 21, 2013 at 9:27 AM, Jesse Gross wrote: > On Thu, Jun 20, 2013 at 11:38 PM, Andy Zhou wrote: >

Re: [ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-21 Thread Jesse Gross
On Thu, Jun 20, 2013 at 11:38 PM, Andy Zhou wrote: > Fix a bug where some mask attributes are missing in the netlink packets > from kernel to the user space. > > Reported-by: Justin Pettit > Signed-off-by: Andy Zhou Can you describe this some more? Presumably, this is because some of the prereq

[ovs-dev] [PATCH] datapath: Bug fix: Missing mask attributes

2013-06-20 Thread Andy Zhou
Fix a bug where some mask attributes are missing in the netlink packets from kernel to the user space. Reported-by: Justin Pettit Signed-off-by: Andy Zhou --- datapath/datapath.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath/datapath.c b/datapath/datapath.c ind