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

Nithin Raju nithin at vmware.com
Thu Sep 25 15:50:56 UTC 2014


On Sep 24, 2014, at 11:58 PM, Ankur Sharma <ankursharma at 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 <ankursharma at vmware.com>
> Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
> Acked-by: Eitan Eliahu <eliahue at 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 <nithin at vmware.com>

-- Nithin


More information about the dev mailing list