diff --git a/batman-adv/Makefile b/batman-adv/Makefile index 9df29a7..9572a28 100644 --- a/batman-adv/Makefile +++ b/batman-adv/Makefile @@ -10,7 +10,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=batman-adv PKG_VERSION:=2018.1 -PKG_RELEASE:=13 +PKG_RELEASE:=14 PKG_HASH:=b866b28dbbe5c9238abbdf5abbc30fc526dea56898ce4c1bd76d5c017843048b PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz diff --git a/batman-adv/patches/0055-batman-adv-set-.owner-to-THIS_MODULE.patch b/batman-adv/patches/0055-batman-adv-set-.owner-to-THIS_MODULE.patch new file mode 100644 index 0000000..d3804c1 --- /dev/null +++ b/batman-adv/patches/0055-batman-adv-set-.owner-to-THIS_MODULE.patch @@ -0,0 +1,25 @@ +From: Taehee Yoo +Date: Sun, 15 Nov 2020 10:30:04 +0000 +Subject: batman-adv: set .owner to THIS_MODULE + +If THIS_MODULE is not set, the module would be removed while debugfs is +being used. +It eventually makes kernel panic. + +Fixes: 24571be14371 ("batman-adv: add routing debug log accessible via debugfs") +Signed-off-by: Taehee Yoo +Signed-off-by: Sven Eckelmann +Origin: upstream, https://git.open-mesh.org/batman-adv.git/commit/6e2937539a2c9f8a8536515258bc1f08bed19a06 + +diff --git a/net/batman-adv/log.c b/net/batman-adv/log.c +index 853773e45f7921bccc142ed4208ff2d377134b49..837f67c9fad3505448d12bec631d9bec71e66e5f 100644 +--- a/net/batman-adv/log.c ++++ b/net/batman-adv/log.c +@@ -205,6 +205,7 @@ static const struct file_operations batadv_log_fops = { + .read = batadv_log_read, + .poll = batadv_log_poll, + .llseek = no_llseek, ++ .owner = THIS_MODULE, + }; + + /** diff --git a/batman-adv/patches/0056-batman-adv-Consider-fragmentation-for-needed_headroo.patch b/batman-adv/patches/0056-batman-adv-Consider-fragmentation-for-needed_headroo.patch new file mode 100644 index 0000000..2362096 --- /dev/null +++ b/batman-adv/patches/0056-batman-adv-Consider-fragmentation-for-needed_headroo.patch @@ -0,0 +1,30 @@ +From: Sven Eckelmann +Date: Thu, 26 Nov 2020 18:15:06 +0100 +Subject: batman-adv: Consider fragmentation for needed_headroom + +If a batman-adv packets has to be fragmented, then the original batman-adv +packet header is not stripped away. Instead, only a new header is added in +front of the packet after it was split. + +This size must be considered to avoid cost intensive reallocations during +the transmission through the various device layers. + +Fixes: e2b4301f4e2d ("batman-adv: Add lower layer needed_(head|tail)room to own ones") +Reported-by: Linus Lüssing +Signed-off-by: Sven Eckelmann +Origin: upstream, https://git.open-mesh.org/batman-adv.git/commit/92064deda9b063ca2d5a53b307c6127a9453357c + +diff --git a/net/batman-adv/hard-interface.c b/net/batman-adv/hard-interface.c +index c4e0435c952db87c89727633c184320820812cda..fc732b78daf77f7ce2f9d5ba81bf69b4b458d301 100644 +--- a/net/batman-adv/hard-interface.c ++++ b/net/batman-adv/hard-interface.c +@@ -565,6 +565,9 @@ static void batadv_hardif_recalc_extra_skbroom(struct net_device *soft_iface) + needed_headroom = lower_headroom + (lower_header_len - ETH_HLEN); + needed_headroom += batadv_max_header_len(); + ++ /* fragmentation headers don't strip the unicast/... header */ ++ needed_headroom += sizeof(struct batadv_frag_packet); ++ + soft_iface->needed_headroom = needed_headroom; + soft_iface->needed_tailroom = lower_tailroom; + } diff --git a/batman-adv/patches/0057-batman-adv-Reserve-needed_-room-for-fragments.patch b/batman-adv/patches/0057-batman-adv-Reserve-needed_-room-for-fragments.patch new file mode 100644 index 0000000..8b6e098 --- /dev/null +++ b/batman-adv/patches/0057-batman-adv-Reserve-needed_-room-for-fragments.patch @@ -0,0 +1,81 @@ +From: Sven Eckelmann +Date: Wed, 25 Nov 2020 13:16:43 +0100 +Subject: batman-adv: Reserve needed_*room for fragments + +The batadv net_device is trying to propagate the needed_headroom and +needed_tailroom from the lower devices. This is needed to avoid cost +intensive reallocations using pskb_expand_head during the transmission. + +But the fragmentation code split the skb's without adding extra room at the +end/beginning of the various fragments. This reduced the performance of +transmissions over complex scenarios (batadv on vxlan on wireguard) because +the lower devices had to perform the reallocations at least once. + +Fixes: db56e4ecf5c2 ("batman-adv: Fragment and send skbs larger than mtu") +Signed-off-by: Sven Eckelmann +Origin: upstream, https://git.open-mesh.org/batman-adv.git/commit/0966d5424bb87e863037301488519ccdd69e4d26 + +diff --git a/net/batman-adv/fragmentation.c b/net/batman-adv/fragmentation.c +index 5b71a289d04fc80de6c20e7a24d621727c77825a..2dbd870221e4063d0368a078f2fa82c4d3562592 100644 +--- a/net/batman-adv/fragmentation.c ++++ b/net/batman-adv/fragmentation.c +@@ -403,6 +403,7 @@ bool batadv_frag_skb_fwd(struct sk_buff *skb, + + /** + * batadv_frag_create() - create a fragment from skb ++ * @net_dev: outgoing device for fragment + * @skb: skb to create fragment from + * @frag_head: header to use in new fragment + * @fragment_size: size of new fragment +@@ -413,22 +414,25 @@ bool batadv_frag_skb_fwd(struct sk_buff *skb, + * + * Return: the new fragment, NULL on error. + */ +-static struct sk_buff *batadv_frag_create(struct sk_buff *skb, ++static struct sk_buff *batadv_frag_create(struct net_device *net_dev, ++ struct sk_buff *skb, + struct batadv_frag_packet *frag_head, + unsigned int fragment_size) + { ++ unsigned int ll_reserved = LL_RESERVED_SPACE(net_dev); ++ unsigned int tailroom = net_dev->needed_tailroom; + struct sk_buff *skb_fragment; + unsigned int header_size = sizeof(*frag_head); + unsigned int mtu = fragment_size + header_size; + +- skb_fragment = netdev_alloc_skb(NULL, mtu + ETH_HLEN); ++ skb_fragment = dev_alloc_skb(ll_reserved + mtu + tailroom); + if (!skb_fragment) + goto err; + + skb_fragment->priority = skb->priority; + + /* Eat the last mtu-bytes of the skb */ +- skb_reserve(skb_fragment, header_size + ETH_HLEN); ++ skb_reserve(skb_fragment, ll_reserved + header_size); + skb_split(skb, skb_fragment, skb->len - fragment_size); + + /* Add the header */ +@@ -451,11 +455,12 @@ int batadv_frag_send_packet(struct sk_buff *skb, + struct batadv_orig_node *orig_node, + struct batadv_neigh_node *neigh_node) + { ++ struct net_device *net_dev = neigh_node->if_incoming->net_dev; + struct batadv_priv *bat_priv; + struct batadv_hard_iface *primary_if = NULL; + struct batadv_frag_packet frag_header; + struct sk_buff *skb_fragment; +- unsigned int mtu = neigh_node->if_incoming->net_dev->mtu; ++ unsigned int mtu = net_dev->mtu; + unsigned int header_size = sizeof(frag_header); + unsigned int max_fragment_size, num_fragments; + int ret; +@@ -515,7 +520,7 @@ int batadv_frag_send_packet(struct sk_buff *skb, + goto put_primary_if; + } + +- skb_fragment = batadv_frag_create(skb, &frag_header, ++ skb_fragment = batadv_frag_create(net_dev, skb, &frag_header, + max_fragment_size); + if (!skb_fragment) { + ret = -ENOMEM; diff --git a/batman-adv/patches/0058-batman-adv-Don-t-always-reallocate-the-fragmentation.patch b/batman-adv/patches/0058-batman-adv-Don-t-always-reallocate-the-fragmentation.patch new file mode 100644 index 0000000..1b1a355 --- /dev/null +++ b/batman-adv/patches/0058-batman-adv-Don-t-always-reallocate-the-fragmentation.patch @@ -0,0 +1,41 @@ +From: Sven Eckelmann +Date: Thu, 26 Nov 2020 18:24:49 +0100 +Subject: batman-adv: Don't always reallocate the fragmentation skb head + +When a packet is fragmented by batman-adv, the original batman-adv header +is not modified. Only a new fragmentation is inserted between the original +one and the ethernet header. The code must therefore make sure that it has +a writable region of this size in the skbuff head. + +But it is not useful to always reallocate the skbuff by this size even when +there would be more than enough headroom still in the skb. The reallocation +is just to costly during in this codepath. + +Fixes: db56e4ecf5c2 ("batman-adv: Fragment and send skbs larger than mtu") +Signed-off-by: Sven Eckelmann +Origin: upstream, https://git.open-mesh.org/batman-adv.git/commit/3e3ff987876d3be70d928561acbefe5a48ab1654 + +diff --git a/net/batman-adv/fragmentation.c b/net/batman-adv/fragmentation.c +index 2dbd870221e4063d0368a078f2fa82c4d3562592..cc062b69fc8d89b5e7e7c6820ffc751487a9093b 100644 +--- a/net/batman-adv/fragmentation.c ++++ b/net/batman-adv/fragmentation.c +@@ -539,13 +539,14 @@ int batadv_frag_send_packet(struct sk_buff *skb, + frag_header.no++; + } + +- /* Make room for the fragment header. */ +- if (batadv_skb_head_push(skb, header_size) < 0 || +- pskb_expand_head(skb, header_size + ETH_HLEN, 0, GFP_ATOMIC) < 0) { +- ret = -ENOMEM; ++ /* make sure that there is at least enough head for the fragmentation ++ * and ethernet headers ++ */ ++ ret = skb_cow_head(skb, ETH_HLEN + header_size); ++ if (ret < 0) + goto put_primary_if; +- } + ++ skb_push(skb, header_size); + memcpy(skb->data, &frag_header, header_size); + + /* Send the last fragment */