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

[Tobias Klaus]

Hallo,

Am Dienstag, 30. Mai 2017, 10:31:04 CEST schrieb mail at adrianschmutzler.de:
> Der Typo ist aus dem Original-Patch :-)
Ja das ziehen wir schon länger rum. :-) ist ja eigentlich auch egal.

> Getestet habe ich ihn selbst, macht da ein "tested-by:" Sinn?
Ja, finde ich schon. Ein Patch ist ja schnell geschickt und es zeigt nichts 
an, dass der Code auch wirklich mal gelaufen ist(Signed-off tut das auch 
nicht). Auch ein "Review-by:" sagt bei uns nicht aus, dass es getestet wurde, 
sondern nur, dass der Patch plausibel aussieht. Daher finde ich das schon 
immer ganz hilfreich, wenn es jemand mal getestet hat. Genau so sollte man 
eigentlich auch vorgehen wenn man Patches für neue Modell schickt.

Dank patchwork/maintainer reicht es ja auch aus ein "Tested-by" einfach als 
Antwort auf den Patch zu schicken.

> fixes #39 einfach in den Fließtext oder gibt es da eine Syntax?
So richtig viel über den Syntax steht nicht da:
https://www.mantisbt.org/wiki/doku.php/mantisbt:git_commit_messages

Aber ich denke eine neue Zeile mit "Fixes #39" sollte passen.

Viele Grüße
Tobias


> -----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
> > ++