Fri, May 22, 2015 at 02:11:41AM CEST, t...@herbertland.com wrote:
>Add a new flow key for TIPC addresses.
>
>Signed-off-by: Tom Herbert <t...@herbertland.com>
>---
> include/net/flow_dissector.h | 10 ++++++++++
> net/core/flow_dissector.c    | 20 ++++++++++++++------
> 2 files changed, 24 insertions(+), 6 deletions(-)
>
>diff --git a/include/net/flow_dissector.h b/include/net/flow_dissector.h
>index 306d461..3ee606a 100644
>--- a/include/net/flow_dissector.h
>+++ b/include/net/flow_dissector.h
>@@ -50,6 +50,14 @@ struct flow_dissector_key_ipv6_addrs {
> };
> 
> /**
>+ * struct flow_dissector_key_tipc_addrs:
>+ * @srcnode: source node address
>+ */
>+struct flow_dissector_key_tipc_addrs {
>+      __be32 srcnode;
>+};
>+
>+/**
>  * struct flow_dissector_key_addrs:
>  * @v4addrs: IPv4 addresses
>  * @v6addrs: IPv6 addresses
>@@ -58,6 +66,7 @@ struct flow_dissector_key_addrs {
>       union {
>               struct flow_dissector_key_ipv4_addrs v4addrs;
>               struct flow_dissector_key_ipv6_addrs v6addrs;
>+              struct flow_dissector_key_tipc_addrs tipcaddrs;
>       };
> };
> 
>@@ -97,6 +106,7 @@ enum flow_dissector_key_id {
>       FLOW_DISSECTOR_KEY_IPV6_HASH_ADDRS, /* struct flow_dissector_key_addrs 
> */
>       FLOW_DISSECTOR_KEY_PORTS, /* struct flow_dissector_key_ports */
>       FLOW_DISSECTOR_KEY_ETH_ADDRS, /* struct flow_dissector_key_eth_addrs */
>+      FLOW_DISSECTOR_KEY_TIPC_ADDRS, /* struct flow_dissector_key_tipc_addrs 
>*/
> 
>       FLOW_DISSECTOR_KEY_MAX,
> };
>diff --git a/net/core/flow_dissector.c b/net/core/flow_dissector.c
>index d77619f..8d6f089 100644
>--- a/net/core/flow_dissector.c
>+++ b/net/core/flow_dissector.c
>@@ -294,13 +294,12 @@ flow_label:
>               key_control->thoff = (u16)nhoff;
> 
>               if (skb_flow_dissector_uses_key(flow_dissector,
>-                                              
>FLOW_DISSECTOR_KEY_IPV6_HASH_ADDRS)) {
>+                              FLOW_DISSECTOR_KEY_TIPC_ADDRS)) {
>                       key_addrs = skb_flow_dissector_target(flow_dissector,
>-                                                            
>FLOW_DISSECTOR_KEY_IPV6_HASH_ADDRS,
>-                                                            target_container);
>-                      key_addrs->v4addrs.src = hdr->srcnode;
>-                      key_addrs->v4addrs.dst = 0;
>-                      key_control->addr_type = FLOW_DISSECTOR_KEY_IPV4_ADDRS;
>+                              FLOW_DISSECTOR_KEY_TIPC_ADDRS,
>+                              target_container);
>+                      key_addrs->tipcaddrs.srcnode = hdr->srcnode;
>+                      key_control->addr_type = FLOW_DISSECTOR_KEY_TIPC_ADDRS;


This patch looks good to me, leaving the addr_type part I mentioned in my
previous reply aside.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to