[ovs-dev] [PATCH v3 1/4] datapath-windows: Making Flow parsing policies global.

Nithin Raju nithin at vmware.com
Wed Oct 15 18:34:05 UTC 2014


Acked-by: Nithin Raju <nithin at vmware.com>

On Oct 15, 2014, at 10:32 AM, Ankur Sharma <ankursharma at vmware.com>
 wrote:

> In this patch we have made the Flow parsing policies global.
> Pakcet execute handling requires these policies.
> 
> Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
> Acked-by: Eitan Eliahu <eliahue at vmware.com>
> ---
> datapath-windows/ovsext/Flow.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c
> index f6e7bdb..220d98a 100644
> --- a/datapath-windows/ovsext/Flow.c
> +++ b/datapath-windows/ovsext/Flow.c
> @@ -98,7 +98,7 @@ static NTSTATUS OvsDoDumpFlows(OvsFlowDumpInput *dumpInput,
> /* Flow family related netlink policies */
> 
> /* For Parsing attributes in FLOW_* commands */
> -static const NL_POLICY nlFlowPolicy[] = {
> +const NL_POLICY nlFlowPolicy[] = {
>     [OVS_FLOW_ATTR_KEY] = {.type = NL_A_NESTED, .optional = FALSE},
>     [OVS_FLOW_ATTR_MASK] = {.type = NL_A_NESTED, .optional = TRUE},
>     [OVS_FLOW_ATTR_ACTIONS] = {.type = NL_A_NESTED, .optional = TRUE},
> @@ -114,7 +114,7 @@ static const NL_POLICY nlFlowPolicy[] = {
>  * Some of the attributes like OVS_KEY_ATTR_RECIRC_ID
>  * & OVS_KEY_ATTR_MPLS are not supported yet. */
> 
> -static const NL_POLICY nlFlowKeyPolicy[] = {
> +const NL_POLICY nlFlowKeyPolicy[] = {
>     [OVS_KEY_ATTR_ENCAP] = {.type = NL_A_VAR_LEN, .optional = TRUE},
>     [OVS_KEY_ATTR_PRIORITY] = {.type = NL_A_UNSPEC, .minLen = 4,
>                                .maxLen = 4, .optional = TRUE},
> @@ -178,7 +178,7 @@ static const NL_POLICY nlFlowKeyPolicy[] = {
> };
> 
> /* For Parsing nested OVS_KEY_ATTR_TUNNEL attributes */
> -static const NL_POLICY nlFlowTunnelKeyPolicy[] = {
> +const NL_POLICY nlFlowTunnelKeyPolicy[] = {
>     [OVS_TUNNEL_KEY_ATTR_ID] = {.type = NL_A_UNSPEC, .minLen = 8,
>                                 .maxLen = 8, .optional = TRUE},
>     [OVS_TUNNEL_KEY_ATTR_IPV4_SRC] = {.type = NL_A_UNSPEC, .minLen = 4,
> @@ -200,7 +200,7 @@ static const NL_POLICY nlFlowTunnelKeyPolicy[] = {
> };
> 
> /* For Parsing nested OVS_FLOW_ATTR_ACTIONS attributes */
> -static const NL_POLICY nlFlowActionPolicy[] = {
> +const NL_POLICY nlFlowActionPolicy[] = {
>     [OVS_ACTION_ATTR_OUTPUT] = {.type = NL_A_UNSPEC, .minLen = sizeof(UINT32),
>                                 .maxLen = sizeof(UINT32), .optional = TRUE},
>     [OVS_ACTION_ATTR_USERSPACE] = {.type = NL_A_VAR_LEN, .optional = TRUE},
> -- 
> 1.9.1
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=ubrOpIWavCMqX4l4j1LEVpTfDj%2FD5Qyn8KCoJIBGvzo%3D%0A&m=3%2F%2BiHLw22CZOdfJD9JBSxdY7it21DHXWqJDVffarb1c%3D%0A&s=e71c168872bcb910020533518f96867232dbe145187eef56d3ff7a4a0e001e33

Thanks,
-- Nithin




More information about the dev mailing list