Message ID | 1496063932-1619-1-git-send-email-freifunk@adrianschmutzler.de |
---|---|
State | Superseded |
Headers | show |
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.patch 1970-01-01 01:00:00.000000000 +0100 ++++ target/linux/ar71xx/patches-4.4/fix-l2tp-stats-couter-on-32-Bit-Systems.patch 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@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 ++
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@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 > ++
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@meskal.net] Sent: Dienstag, 30. Mai 2017 00:48 To: franken-dev@freifunk.net; Adrian Schmutzler <freifunk@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@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 > ++
Hallo, Am Dienstag, 30. Mai 2017, 10:31:04 CEST schrieb mail@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@meskal.net] > Sent: Dienstag, 30. Mai 2017 00:48 > To: franken-dev@freifunk.net; Adrian Schmutzler > <freifunk@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@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 > > ++
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@meskal.net] Sent: Dienstag, 30. Mai 2017 13:15 To: franken-dev@freifunk.net; mail@adrianschmutzler.de Cc: 'Adrian Schmutzler' <freifunk@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@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@meskal.net] > Sent: Dienstag, 30. Mai 2017 00:48 > To: franken-dev@freifunk.net; Adrian Schmutzler > <freifunk@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@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 > > ++