Registered FLOW_DEL command handler. The same command handler as FLOW_ADD is good enough to handle FLOW_DEL case as well with minor changes for checking to action attribute.
Signed-off-by: Ankur Sharma <[email protected]> Acked-by: Alin Gabriel Serdean <[email protected]> Acked-by: Eitan Eliahu <[email protected]> Acked-by: Nithin Raju <[email protected]> Acked-by: Samuel Ghinet <[email protected]> Tested-by: Ankur Sharma <[email protected]> --- datapath-windows/ovsext/Datapath.c | 7 ++++++- datapath-windows/ovsext/Flow.c | 7 +++++-- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windows/ovsext/Datapath.c index 865c14c..660c958 100644 --- a/datapath-windows/ovsext/Datapath.c +++ b/datapath-windows/ovsext/Datapath.c @@ -201,7 +201,12 @@ NETLINK_CMD nlFlowFamilyCmdOps[] = { { .cmd = OVS_FLOW_CMD_SET, .handler = OvsFlowNlNewCmdHandler, .supportedDevOp = OVS_TRANSACTION_DEV_OP, - .validateDpIndex = FALSE + .validateDpIndex = TRUE + }, + { .cmd = OVS_FLOW_CMD_DEL, + .handler = OvsFlowNlNewCmdHandler, + .supportedDevOp = OVS_TRANSACTION_DEV_OP, + .validateDpIndex = TRUE } }; diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c index f68db19..2eb6174 100644 --- a/datapath-windows/ovsext/Flow.c +++ b/datapath-windows/ovsext/Flow.c @@ -352,8 +352,11 @@ _MapNlToFlowPut(POVS_MESSAGE msgIn, PNL_ATTR keyAttr, mappedFlow); /* Map the action */ - mappedFlow->actionsLen = NlAttrGetSize(actionAttr); - mappedFlow->actions = NlAttrGet(actionAttr); + if (actionAttr) { + mappedFlow->actionsLen = NlAttrGetSize(actionAttr); + mappedFlow->actions = NlAttrGet(actionAttr); + } + mappedFlow->dpNo = ovsHdr->dp_ifindex; _MapNlToFlowPutFlags(genlMsgHdr, flowAttrClear, -- 1.9.1 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
