[ovs-dev] [PATCH v2 08/10] datapath-windows/Flow.c: FLOW_SET command handler.

[Ankur Sharma]

Registered FLOW_SET command handler. The same command
handler as FLOW_ADD is good enough to handle FLOW_SET
case as well.

Signed-off-by: Ankur Sharma ankursha...@vmware.com
Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com
Acked-by: Eitan Eliahu elia...@vmware.com
---
 datapath-windows/ovsext/Datapath.c | 5 +
 1 file changed, 5 insertions(+)

diff --git a/datapath-windows/ovsext/Datapath.c 
b/datapath-windows/ovsext/Datapath.c
index 9934a9a..5008aab 100644
--- a/datapath-windows/ovsext/Datapath.c
+++ b/datapath-windows/ovsext/Datapath.c
@@ -197,6 +197,11 @@ NETLINK_CMD nlFlowFamilyCmdOps[] = {
   .handler  = OvsFlowNlNewCmdHandler,
   .supportedDevOp   = OVS_TRANSACTION_DEV_OP,
   .validateDpIndex  = FALSE
+},
+{ .cmd  = OVS_FLOW_CMD_SET,
+  .handler  = OvsFlowNlNewCmdHandler,
+  .supportedDevOp   = OVS_TRANSACTION_DEV_OP,
+  .validateDpIndex  = FALSE
 }
 };
 
-- 
1.9.1

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 08/10] datapath-windows/Flow.c: FLOW_SET command handler.

[Nithin Raju]

On Sep 24, 2014, at 11:58 PM, Ankur Sharma ankursha...@vmware.com wrote:

 Registered FLOW_SET command handler. The same command
 handler as FLOW_ADD is good enough to handle FLOW_SET
 case as well.
 
 Signed-off-by: Ankur Sharma ankursha...@vmware.com
 Acked-by: Alin Gabriel Serdean aserd...@cloudbasesolutions.com
 Acked-by: Eitan Eliahu elia...@vmware.com
 ---
 datapath-windows/ovsext/Datapath.c | 5 +
 1 file changed, 5 insertions(+)
 
 diff --git a/datapath-windows/ovsext/Datapath.c 
 b/datapath-windows/ovsext/Datapath.c
 index 9934a9a..5008aab 100644
 --- a/datapath-windows/ovsext/Datapath.c
 +++ b/datapath-windows/ovsext/Datapath.c
 @@ -197,6 +197,11 @@ NETLINK_CMD nlFlowFamilyCmdOps[] = {
   .handler  = OvsFlowNlNewCmdHandler,
   .supportedDevOp   = OVS_TRANSACTION_DEV_OP,
   .validateDpIndex  = FALSE
 +},
 +{ .cmd  = OVS_FLOW_CMD_SET,
 +  .handler  = OvsFlowNlNewCmdHandler,
 +  .supportedDevOp   = OVS_TRANSACTION_DEV_OP,
 +  .validateDpIndex  = FALSE
 }
 };

validateDpIndex should be TRUE for both the commands. But, LG otherwise.

Acked-by: Nithin Raju nit...@vmware.com

-- Nithin
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev