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

[Adrian Schmutzler]

Typo ist bereits geändert und das mit Tested-by hab ich auch bei allen relevanten Commits eingefügt.

Fixed #39 hab ich weggelassen, da es ja strenggenommen das Problem nicht fixed, sondern nur die Message unterdrückt. Wäre aber wohl am sinnvollsten, die Nr. 39 zu schließen (die betrifft ja primär den Patch) und eine neue für das eigentliche Problem aufzumachen, was dann aber wohl niemand fixen wird, da es wahrscheinlich zu haarig ist.

Grüße

Adrian

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

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-System
> s.pat
> 
> > ch	1970-01-01 01:00:00.000000000 +0100 ++++
> 
> target/linux/ar71xx/patches-4.4/fix-l2tp-stats-couter-on-32-Bit-System
> s.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
> > ++