Commit 0a80f0c2 authored by David S. Miller's avatar David S. Miller

Merge branch 'flow_dissector-Provide-basic-batman-adv-unicast-handling'

Sven Eckelmann says:

====================
flow_dissector: Provide basic batman-adv unicast handling

we are currently starting to use batman-adv as mesh protocol on multicore
embedded devices. These usually don't have a lot of CPU power per core but
are reasonable fast when using multiple cores.

It was noticed that sending was working very well but receiving was
basically only using on CPU core per neighbor. The reason for that is
format of the (normal) incoming packet:

  +--------------------+
  | ip(v6)hdr          |
  +--------------------+
  | inner ethhdr       |
  +--------------------+
  | batadv unicast hdr |
  +--------------------+
  | outer ethhdr       |
  +--------------------+

The flow dissector will therefore stop after parsing the outer ethernet
header and will not parse the actual ipv(4|6)/... header of the packet. Our
assumption was now that it would help us to add minimal support to the flow
dissector to jump over the batman-adv unicast and inner ethernet header
(like in gre ETH_P_TEB). The patch was implemented in a slightly hacky
way [1] and the results looked quite promising.

I didn't get any feedback how the files should actually be named. So I am
now just using the names from RFC v3

The discussion of the RFC v3 can be found in the related patches of
https://patchwork.ozlabs.org/cover/849345/

The discussion of the RFC v2 can be found in the related patches of
https://patchwork.ozlabs.org/cover/844783/

Changes in v4:
==============

* added  patch to change the u8/u16 to __u8/__u16 in
  include/uapi/linux/batadv_packet.h
  - requested by Willem de Bruijn <willemdebruijn.kernel@gmail.com>

Changes in v3:
==============

* removed change of uapi/linux/batman_adv.h to uapi/linux/batadv_genl.h
  - requested by Willem de Bruijn <willemdebruijn.kernel@gmail.com>
* removed naming fixes for enums/defines in uapi/linux/batadv_genl.h
  - requested by Willem de Bruijn <willemdebruijn.kernel@gmail.com>
* renamed uapi/linux/batadv.h to uapi/linux/batadv_packet.h
* moved batadv dissector functionality in own function
  - requested by Tom Herbert <tom@herbertland.com>
* added support for flags FLOW_DISSECTOR_F_STOP_AT_ENCAP and
  FLOW_DIS_ENCAPSULATION
  - requested by Willem de Bruijn <willemdebruijn.kernel@gmail.com>

Changes in v2:
==============

* removed the batman-adv unicast packet header definition from flow_dissector.c
* moved the batman-adv packet.h/uapi headers around to provide the correct
  definitions to flow_dissector.c
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents a0e2e75a 5b0890a9
...@@ -2564,6 +2564,7 @@ S: Maintained ...@@ -2564,6 +2564,7 @@ S: Maintained
F: Documentation/ABI/testing/sysfs-class-net-batman-adv F: Documentation/ABI/testing/sysfs-class-net-batman-adv
F: Documentation/ABI/testing/sysfs-class-net-mesh F: Documentation/ABI/testing/sysfs-class-net-mesh
F: Documentation/networking/batman-adv.rst F: Documentation/networking/batman-adv.rst
F: include/uapi/linux/batadv_packet.h
F: include/uapi/linux/batman_adv.h F: include/uapi/linux/batman_adv.h
F: net/batman-adv/ F: net/batman-adv/
......
...@@ -52,6 +52,7 @@ ...@@ -52,6 +52,7 @@
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <net/genetlink.h> #include <net/genetlink.h>
#include <net/netlink.h> #include <net/netlink.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "bat_algo.h" #include "bat_algo.h"
...@@ -63,7 +64,6 @@ ...@@ -63,7 +64,6 @@
#include "netlink.h" #include "netlink.h"
#include "network-coding.h" #include "network-coding.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
#include "translation-table.h" #include "translation-table.h"
......
...@@ -37,6 +37,7 @@ ...@@ -37,6 +37,7 @@
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <net/genetlink.h> #include <net/genetlink.h>
#include <net/netlink.h> #include <net/netlink.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "bat_algo.h" #include "bat_algo.h"
...@@ -49,7 +50,6 @@ ...@@ -49,7 +50,6 @@
#include "log.h" #include "log.h"
#include "netlink.h" #include "netlink.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
struct sk_buff; struct sk_buff;
......
...@@ -42,13 +42,13 @@ ...@@ -42,13 +42,13 @@
#include <linux/types.h> #include <linux/types.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <net/cfg80211.h> #include <net/cfg80211.h>
#include <uapi/linux/batadv_packet.h>
#include "bat_algo.h" #include "bat_algo.h"
#include "bat_v_ogm.h" #include "bat_v_ogm.h"
#include "hard-interface.h" #include "hard-interface.h"
#include "log.h" #include "log.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
......
...@@ -39,13 +39,13 @@ ...@@ -39,13 +39,13 @@
#include <linux/string.h> #include <linux/string.h>
#include <linux/types.h> #include <linux/types.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <uapi/linux/batadv_packet.h>
#include "bat_algo.h" #include "bat_algo.h"
#include "hard-interface.h" #include "hard-interface.h"
#include "hash.h" #include "hash.h"
#include "log.h" #include "log.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
#include "translation-table.h" #include "translation-table.h"
......
...@@ -50,6 +50,7 @@ ...@@ -50,6 +50,7 @@
#include <net/genetlink.h> #include <net/genetlink.h>
#include <net/netlink.h> #include <net/netlink.h>
#include <net/sock.h> #include <net/sock.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "hard-interface.h" #include "hard-interface.h"
...@@ -57,7 +58,6 @@ ...@@ -57,7 +58,6 @@
#include "log.h" #include "log.h"
#include "netlink.h" #include "netlink.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "sysfs.h" #include "sysfs.h"
#include "translation-table.h" #include "translation-table.h"
......
...@@ -24,9 +24,9 @@ ...@@ -24,9 +24,9 @@
#include <linux/compiler.h> #include <linux/compiler.h>
#include <linux/netdevice.h> #include <linux/netdevice.h>
#include <linux/types.h> #include <linux/types.h>
#include <uapi/linux/batadv_packet.h>
#include "originator.h" #include "originator.h"
#include "packet.h"
struct seq_file; struct seq_file;
struct sk_buff; struct sk_buff;
......
...@@ -33,10 +33,10 @@ ...@@ -33,10 +33,10 @@
#include <linux/slab.h> #include <linux/slab.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/string.h> #include <linux/string.h>
#include <uapi/linux/batadv_packet.h>
#include "hard-interface.h" #include "hard-interface.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
#include "soft-interface.h" #include "soft-interface.h"
......
...@@ -43,6 +43,7 @@ ...@@ -43,6 +43,7 @@
#include <linux/stddef.h> #include <linux/stddef.h>
#include <linux/udp.h> #include <linux/udp.h>
#include <net/sock.h> #include <net/sock.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "gateway_common.h" #include "gateway_common.h"
...@@ -50,7 +51,6 @@ ...@@ -50,7 +51,6 @@
#include "log.h" #include "log.h"
#include "netlink.h" #include "netlink.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "sysfs.h" #include "sysfs.h"
......
...@@ -27,10 +27,10 @@ ...@@ -27,10 +27,10 @@
#include <linux/netdevice.h> #include <linux/netdevice.h>
#include <linux/stddef.h> #include <linux/stddef.h>
#include <linux/string.h> #include <linux/string.h>
#include <uapi/linux/batadv_packet.h>
#include "gateway_client.h" #include "gateway_client.h"
#include "log.h" #include "log.h"
#include "packet.h"
#include "tvlv.h" #include "tvlv.h"
/** /**
......
...@@ -38,6 +38,7 @@ ...@@ -38,6 +38,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <net/net_namespace.h> #include <net/net_namespace.h>
#include <net/rtnetlink.h> #include <net/rtnetlink.h>
#include <uapi/linux/batadv_packet.h>
#include "bat_v.h" #include "bat_v.h"
#include "bridge_loop_avoidance.h" #include "bridge_loop_avoidance.h"
...@@ -46,7 +47,6 @@ ...@@ -46,7 +47,6 @@
#include "gateway_client.h" #include "gateway_client.h"
#include "log.h" #include "log.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "sysfs.h" #include "sysfs.h"
......
...@@ -44,11 +44,11 @@ ...@@ -44,11 +44,11 @@
#include <linux/string.h> #include <linux/string.h>
#include <linux/uaccess.h> #include <linux/uaccess.h>
#include <linux/wait.h> #include <linux/wait.h>
#include <uapi/linux/batadv_packet.h>
#include "hard-interface.h" #include "hard-interface.h"
#include "log.h" #include "log.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
static struct batadv_socket_client *batadv_socket_client_hash[256]; static struct batadv_socket_client *batadv_socket_client_hash[256];
......
...@@ -46,6 +46,7 @@ ...@@ -46,6 +46,7 @@
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <net/dsfield.h> #include <net/dsfield.h>
#include <net/rtnetlink.h> #include <net/rtnetlink.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "bat_algo.h" #include "bat_algo.h"
...@@ -63,7 +64,6 @@ ...@@ -63,7 +64,6 @@
#include "netlink.h" #include "netlink.h"
#include "network-coding.h" #include "network-coding.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
#include "soft-interface.h" #include "soft-interface.h"
......
...@@ -217,16 +217,14 @@ enum batadv_uev_type { ...@@ -217,16 +217,14 @@ enum batadv_uev_type {
/* Kernel headers */ /* Kernel headers */
#include <linux/bitops.h> /* for packet.h */
#include <linux/compiler.h> #include <linux/compiler.h>
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/if_ether.h> /* for packet.h */
#include <linux/if_vlan.h> #include <linux/if_vlan.h>
#include <linux/jiffies.h> #include <linux/jiffies.h>
#include <linux/percpu.h> #include <linux/percpu.h>
#include <linux/types.h> #include <linux/types.h>
#include <uapi/linux/batadv_packet.h>
#include "packet.h"
#include "types.h" #include "types.h"
struct net_device; struct net_device;
......
...@@ -55,11 +55,11 @@ ...@@ -55,11 +55,11 @@
#include <net/if_inet6.h> #include <net/if_inet6.h>
#include <net/ip.h> #include <net/ip.h>
#include <net/ipv6.h> #include <net/ipv6.h>
#include <uapi/linux/batadv_packet.h>
#include "hard-interface.h" #include "hard-interface.h"
#include "hash.h" #include "hash.h"
#include "log.h" #include "log.h"
#include "packet.h"
#include "translation-table.h" #include "translation-table.h"
#include "tvlv.h" #include "tvlv.h"
......
...@@ -40,6 +40,7 @@ ...@@ -40,6 +40,7 @@
#include <net/genetlink.h> #include <net/genetlink.h>
#include <net/netlink.h> #include <net/netlink.h>
#include <net/sock.h> #include <net/sock.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "bat_algo.h" #include "bat_algo.h"
...@@ -47,7 +48,6 @@ ...@@ -47,7 +48,6 @@
#include "gateway_client.h" #include "gateway_client.h"
#include "hard-interface.h" #include "hard-interface.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "tp_meter.h" #include "tp_meter.h"
#include "translation-table.h" #include "translation-table.h"
......
...@@ -49,12 +49,12 @@ ...@@ -49,12 +49,12 @@
#include <linux/stddef.h> #include <linux/stddef.h>
#include <linux/string.h> #include <linux/string.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <uapi/linux/batadv_packet.h>
#include "hard-interface.h" #include "hard-interface.h"
#include "hash.h" #include "hash.h"
#include "log.h" #include "log.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "routing.h" #include "routing.h"
#include "send.h" #include "send.h"
#include "tvlv.h" #include "tvlv.h"
......
...@@ -34,6 +34,7 @@ ...@@ -34,6 +34,7 @@
#include <linux/skbuff.h> #include <linux/skbuff.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/stddef.h> #include <linux/stddef.h>
#include <uapi/linux/batadv_packet.h>
#include "bitarray.h" #include "bitarray.h"
#include "bridge_loop_avoidance.h" #include "bridge_loop_avoidance.h"
...@@ -44,7 +45,6 @@ ...@@ -44,7 +45,6 @@
#include "log.h" #include "log.h"
#include "network-coding.h" #include "network-coding.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "tp_meter.h" #include "tp_meter.h"
......
...@@ -24,8 +24,7 @@ ...@@ -24,8 +24,7 @@
#include <linux/compiler.h> #include <linux/compiler.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/types.h> #include <linux/types.h>
#include <uapi/linux/batadv_packet.h>
#include "packet.h"
struct sk_buff; struct sk_buff;
......
...@@ -49,6 +49,7 @@ ...@@ -49,6 +49,7 @@
#include <linux/stddef.h> #include <linux/stddef.h>
#include <linux/string.h> #include <linux/string.h>
#include <linux/types.h> #include <linux/types.h>
#include <uapi/linux/batadv_packet.h>
#include "bat_algo.h" #include "bat_algo.h"
#include "bridge_loop_avoidance.h" #include "bridge_loop_avoidance.h"
...@@ -60,7 +61,6 @@ ...@@ -60,7 +61,6 @@
#include "multicast.h" #include "multicast.h"
#include "network-coding.h" #include "network-coding.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
#include "sysfs.h" #include "sysfs.h"
#include "translation-table.h" #include "translation-table.h"
......
...@@ -39,6 +39,7 @@ ...@@ -39,6 +39,7 @@
#include <linux/string.h> #include <linux/string.h>
#include <linux/stringify.h> #include <linux/stringify.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <uapi/linux/batadv_packet.h>
#include "bridge_loop_avoidance.h" #include "bridge_loop_avoidance.h"
#include "distributed-arp-table.h" #include "distributed-arp-table.h"
...@@ -47,7 +48,6 @@ ...@@ -47,7 +48,6 @@
#include "hard-interface.h" #include "hard-interface.h"
#include "log.h" #include "log.h"
#include "network-coding.h" #include "network-coding.h"
#include "packet.h"
#include "soft-interface.h" #include "soft-interface.h"
static struct net_device *batadv_kobj_to_netdev(struct kobject *obj) static struct net_device *batadv_kobj_to_netdev(struct kobject *obj)
......
...@@ -49,13 +49,13 @@ ...@@ -49,13 +49,13 @@
#include <linux/timer.h> #include <linux/timer.h>
#include <linux/wait.h> #include <linux/wait.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "hard-interface.h" #include "hard-interface.h"
#include "log.h" #include "log.h"
#include "netlink.h" #include "netlink.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
/** /**
......
...@@ -52,6 +52,7 @@ ...@@ -52,6 +52,7 @@
#include <net/genetlink.h> #include <net/genetlink.h>
#include <net/netlink.h> #include <net/netlink.h>
#include <net/sock.h> #include <net/sock.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "bridge_loop_avoidance.h" #include "bridge_loop_avoidance.h"
...@@ -60,7 +61,6 @@ ...@@ -60,7 +61,6 @@
#include "log.h" #include "log.h"
#include "netlink.h" #include "netlink.h"
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "soft-interface.h" #include "soft-interface.h"
#include "tvlv.h" #include "tvlv.h"
......
...@@ -36,9 +36,9 @@ ...@@ -36,9 +36,9 @@
#include <linux/stddef.h> #include <linux/stddef.h>
#include <linux/string.h> #include <linux/string.h>
#include <linux/types.h> #include <linux/types.h>
#include <uapi/linux/batadv_packet.h>
#include "originator.h" #include "originator.h"
#include "packet.h"
#include "send.h" #include "send.h"
#include "tvlv.h" #include "tvlv.h"
......
...@@ -35,10 +35,9 @@ ...@@ -35,10 +35,9 @@
#include <linux/types.h> #include <linux/types.h>
#include <linux/wait.h> #include <linux/wait.h>
#include <linux/workqueue.h> #include <linux/workqueue.h>
#include <uapi/linux/batadv_packet.h>
#include <uapi/linux/batman_adv.h> #include <uapi/linux/batman_adv.h>
#include "packet.h"
struct seq_file; struct seq_file;
#ifdef CONFIG_BATMAN_ADV_DAT #ifdef CONFIG_BATMAN_ADV_DAT
......
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <linux/tcp.h> #include <linux/tcp.h>
#include <net/flow_dissector.h> #include <net/flow_dissector.h>
#include <scsi/fc/fc_fcoe.h> #include <scsi/fc/fc_fcoe.h>
#include <uapi/linux/batadv_packet.h>
static void dissector_set_key(struct flow_dissector *flow_dissector, static void dissector_set_key(struct flow_dissector *flow_dissector,
enum flow_dissector_key_id key_id) enum flow_dissector_key_id key_id)
...@@ -437,6 +438,57 @@ __skb_flow_dissect_gre(const struct sk_buff *skb, ...@@ -437,6 +438,57 @@ __skb_flow_dissect_gre(const struct sk_buff *skb,
return FLOW_DISSECT_RET_PROTO_AGAIN; return FLOW_DISSECT_RET_PROTO_AGAIN;
} }
/**
* __skb_flow_dissect_batadv() - dissect batman-adv header
* @skb: sk_buff to with the batman-adv header
* @key_control: flow dissectors control key
* @data: raw buffer pointer to the packet, if NULL use skb->data
* @p_proto: pointer used to update the protocol to process next
* @p_nhoff: pointer used to update inner network header offset
* @hlen: packet header length
* @flags: any combination of FLOW_DISSECTOR_F_*
*
* ETH_P_BATMAN packets are tried to be dissected. Only
* &struct batadv_unicast packets are actually processed because they contain an
* inner ethernet header and are usually followed by actual network header. This
* allows the flow dissector to continue processing the packet.
*
* Return: FLOW_DISSECT_RET_PROTO_AGAIN when &struct batadv_unicast was found,
* FLOW_DISSECT_RET_OUT_GOOD when dissector should stop after encapsulation,
* otherwise FLOW_DISSECT_RET_OUT_BAD
*/
static enum flow_dissect_ret
__skb_flow_dissect_batadv(const struct sk_buff *skb,
struct flow_dissector_key_control *key_control,
void *data, __be16 *p_proto, int *p_nhoff, int hlen,
unsigned int flags)
{
struct {
struct batadv_unicast_packet batadv_unicast;
struct ethhdr eth;
} *hdr, _hdr;
hdr = __skb_header_pointer(skb, *p_nhoff, sizeof(_hdr), data, hlen,
&_hdr);
if (!hdr)
return FLOW_DISSECT_RET_OUT_BAD;
if (hdr->batadv_unicast.version != BATADV_COMPAT_VERSION)
return FLOW_DISSECT_RET_OUT_BAD;
if (hdr->batadv_unicast.packet_type != BATADV_UNICAST)
return FLOW_DISSECT_RET_OUT_BAD;
*p_proto = hdr->eth.h_proto;
*p_nhoff += sizeof(*hdr);
key_control->flags |= FLOW_DIS_ENCAPSULATION;
if (flags & FLOW_DISSECTOR_F_STOP_AT_ENCAP)
return FLOW_DISSECT_RET_OUT_GOOD;
return FLOW_DISSECT_RET_PROTO_AGAIN;
}
static void static void
__skb_flow_dissect_tcp(const struct sk_buff *skb, __skb_flow_dissect_tcp(const struct sk_buff *skb,
struct flow_dissector *flow_dissector, struct flow_dissector *flow_dissector,
...@@ -815,6 +867,11 @@ bool __skb_flow_dissect(const struct sk_buff *skb, ...@@ -815,6 +867,11 @@ bool __skb_flow_dissect(const struct sk_buff *skb,
nhoff, hlen); nhoff, hlen);
break; break;
case htons(ETH_P_BATMAN):
fdret = __skb_flow_dissect_batadv(skb, key_control, data,
&proto, &nhoff, hlen, flags);
break;
default: default:
fdret = FLOW_DISSECT_RET_OUT_BAD; fdret = FLOW_DISSECT_RET_OUT_BAD;
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment