[PATCH 4/4] build_patches: reintroduce updated l2tp stats patch

[mail at adrianschmutzler.de]

Der Typo ist aus dem Original-Patch :-)

Getestet habe ich ihn selbst, macht da ein "tested-by:" Sinn?

fixes #39 einfach in den Fließtext oder gibt es da eine Syntax?

-----Original Message-----
From: Tobias Klaus [mailto:tk+ff at meskal.net] 
Sent: Dienstag, 30. Mai 2017 00:48
To: franken-dev at freifunk.net; Adrian Schmutzler
<freifunk at adrianschmutzler.de>
Subject: Re: [PATCH 4/4] build_patches: reintroduce updated l2tp stats patch

Hallo Adrian,

auch hier fehlt leider das Signed-off-by. Bis auf den Typo (couter statt
counter) im Dateinamen des aus dem Patch resultierenden Kernelpatche, sieht
der Patch gut aus.

Leider kann ich ihn gerade nicht testen. Laut Mantis hast du es auch schon
getestet. Daher wäre es gut, wenn du noch ein Tested-by: dazuschicken
könntest.

Zu guter Letzt wäre es noch recht cool, wenn du ein "fixes #39" in der
Commit- Nachricht unterbringen könntest, dann würde sich der Mantis-Bug
nämlich
(hoffentlich) automatisch schließen :-).

Vielen Dank für deinen Einsatz! Ich schätze dein Engagement und hoffe dass
dich die noch fehlenden Kleinigkeiten nicht entmutigen.

Viele Grüße und jetzt auch gute Nacht
Tobias

Am Montag, 29. Mai 2017, 15:18:52 CEST schrieb Adrian Schmutzler:
> ---
>  .../openwrt/0004-ar71xx-4.4-l2tp-stats.patch       | 41
> ++++++++++++++++++++++ 1 file changed, 41 insertions(+)
>  create mode 100755 
> build_patches/openwrt/0004-ar71xx-4.4-l2tp-stats.patch
> 
> diff --git a/build_patches/openwrt/0004-ar71xx-4.4-l2tp-stats.patch
> b/build_patches/openwrt/0004-ar71xx-4.4-l2tp-stats.patch new file mode
> 100755
> index 0000000..3928002
> --- /dev/null
> +++ b/build_patches/openwrt/0004-ar71xx-4.4-l2tp-stats.patch
> @@ -0,0 +1,41 @@
> +---
>
target/linux/ar71xx/patches-4.4/fix-l2tp-stats-couter-on-32-Bit-Systems.pat
> ch	1970-01-01 01:00:00.000000000 +0100 ++++
>
target/linux/ar71xx/patches-4.4/fix-l2tp-stats-couter-on-32-Bit-Systems.pat
> ch	2016-03-12 18:41:32.818602442 +0100 +@@ -0,0 +1,38 @@
> ++From 19b1c8733b99f13005f2d8918bce588f0b2556f8 Mon Sep 17 00:00:00 
> ++2001
> ++From: Dominik Heidler <dominik at heidler.eu>
> ++Date: Sat, 12 Mar 2016 18:37:42 +0100
> ++Subject: [PATCH] Fix l2tp stats couter on 32 Bit Systems
> ++
> ++---
> ++ net/l2tp/l2tp_eth.c | 9 +++++++++
> ++ 1 file changed, 9 insertions(+)
> ++
> ++diff --git a/net/l2tp/l2tp_eth.c b/net/l2tp/l2tp_eth.c index 
> ++e253c26..a18c2ff 100644
> ++--- a/net/l2tp/l2tp_eth.c
> +++++ b/net/l2tp/l2tp_eth.c
> ++@@ -111,12 +111,21 @@ static struct rtnl_link_stats64
> *l2tp_eth_get_stats64(struct net_device *dev, ++ {
> ++ 	struct l2tp_eth *priv = netdev_priv(dev);
> ++
> +++	#if BITS_PER_LONG == 64
> ++ 	stats->tx_bytes   = atomic_long_read(&priv->tx_bytes);
> ++ 	stats->tx_packets = atomic_long_read(&priv->tx_packets);
> ++ 	stats->tx_dropped = atomic_long_read(&priv->tx_dropped);
> ++ 	stats->rx_bytes   = atomic_long_read(&priv->rx_bytes);
> ++ 	stats->rx_packets = atomic_long_read(&priv->rx_packets);
> ++ 	stats->rx_errors  = atomic_long_read(&priv->rx_errors);
> +++	#else
> +++	stats->tx_bytes   = (unsigned long)
atomic_long_read(&priv->tx_bytes);
> +++	stats->tx_packets = (unsigned long) atomic_long_read(&priv-
>tx_packets);
> +++	stats->tx_dropped = (unsigned long)
> atomic_long_read(&priv->tx_dropped); +++	stats->rx_bytes   =
(unsigned
> long) atomic_long_read(&priv->rx_bytes); +++	stats->rx_packets =
(unsigned
> long) atomic_long_read(&priv->rx_packets); +++	stats->rx_errors  =
> (unsigned long) atomic_long_read(&priv->rx_errors); +++	#endif
> ++ 	return stats;
> ++ }
> ++
> ++--
> ++2.7.2
> ++