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

Reply via email to