b.a.t.m.a.n.lists.open-mesh.org archive mirror
 help / color / mirror / Atom feed
* [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif
@ 2018-09-11 15:59 Sven Eckelmann
  2018-09-11 16:07 ` Sven Eckelmann
  2018-09-14 17:48 ` Sven Eckelmann
  0 siblings, 2 replies; 3+ messages in thread
From: Sven Eckelmann @ 2018-09-11 15:59 UTC (permalink / raw)
  To: b.a.t.m.a.n

The batadv interfaces are virtual interfaces which just tunnel the traffic
over other ethernet compatible interfaces. It doesn't need serialization
during the tx phase and is using RCU for most of its internal
datastructures. Since it doesn't have actual queues which could be locked
independently, the throughput gets significantly reduced by the extra lock
in the core net code.

8 parallel TCP connections forwarded by an IPQ4019 based hardware over
5GHz could reach:

* without LLX: 349 Mibit/s
* with LLX:    563 Mibit/s

Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
 net/batman-adv/soft-interface.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/batman-adv/soft-interface.c b/net/batman-adv/soft-interface.c
index 626ddca3..b1596c23 100644
--- a/net/batman-adv/soft-interface.c
+++ b/net/batman-adv/soft-interface.c
@@ -1062,6 +1062,7 @@ static void batadv_softif_init_early(struct net_device *dev)
 	dev->needs_free_netdev = true;
 	dev->priv_destructor = batadv_softif_free;
 	dev->features |= NETIF_F_HW_VLAN_CTAG_FILTER | NETIF_F_NETNS_LOCAL;
+	dev->features |= NETIF_F_LLTX;
 	dev->priv_flags |= IFF_NO_QUEUE;
 
 	/* can't call min_mtu, because the needed variables
-- 
2.11.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif
  2018-09-11 15:59 [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif Sven Eckelmann
@ 2018-09-11 16:07 ` Sven Eckelmann
  2018-09-14 17:48 ` Sven Eckelmann
  1 sibling, 0 replies; 3+ messages in thread
From: Sven Eckelmann @ 2018-09-11 16:07 UTC (permalink / raw)
  To: b.a.t.m.a.n

[-- Attachment #1: Type: text/plain, Size: 182 bytes --]

On Dienstag, 11. September 2018 17:59:42 CEST Sven Eckelmann wrote:
[...]
> * without LLX: 349 Mibit/s
> * with LLX:    563 Mibit/s

I know, it is called LLTX :(

Kind regards,
	Sven

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif
  2018-09-11 15:59 [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif Sven Eckelmann
  2018-09-11 16:07 ` Sven Eckelmann
@ 2018-09-14 17:48 ` Sven Eckelmann
  1 sibling, 0 replies; 3+ messages in thread
From: Sven Eckelmann @ 2018-09-14 17:48 UTC (permalink / raw)
  To: b.a.t.m.a.n

[-- Attachment #1: Type: text/plain, Size: 898 bytes --]

On Dienstag, 11. September 2018 17:59:42 CEST Sven Eckelmann wrote:
> The batadv interfaces are virtual interfaces which just tunnel the traffic
> over other ethernet compatible interfaces. It doesn't need serialization
> during the tx phase and is using RCU for most of its internal
> datastructures. Since it doesn't have actual queues which could be locked
> independently, the throughput gets significantly reduced by the extra lock
> in the core net code.
> 
> 8 parallel TCP connections forwarded by an IPQ4019 based hardware over
> 5GHz could reach:
> 
> * without LLX: 349 Mibit/s
> * with LLX:    563 Mibit/s
> 
> Signed-off-by: Sven Eckelmann <sven@narfation.org>
> ---
>  net/batman-adv/soft-interface.c | 1 +
>  1 file changed, 1 insertion(+)

Applied as e3ed8ad1a7b3 [1]

Kind regards,
	Sven

[1] https://git.open-mesh.org/batman-adv.git/commit/e3ed8ad1a7b36bc63b31df6fc9771dc63f8cac3e

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-09-14 17:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-11 15:59 [B.A.T.M.A.N.] [PATCH] batman-adv: Enable LockLess TX for softif Sven Eckelmann
2018-09-11 16:07 ` Sven Eckelmann
2018-09-14 17:48 ` Sven Eckelmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).