Bug report: GS908E, port statitstics byte counters overflow

[jess_last]

Cannot find an easy way to report bugs in firmware, so posting here. Obviously, I'm not interested in buying premium support just to report bugs.

GS908E managed switch, firmware v1.7.0.3.  Monitoring -> Port Statistics. Both "Bytes Received" and "Bytes Sent" use int32 as counter, which overflows multiple times a day. Should be at least 64-bit integer in order to be useful.

[schumaku]

Valid feature suggestion at least, however not a bug. Don't know if the switch platform in place does deliver more than an int32 to the tiny management controller. @YeZ please.

[schumaku]

Vaguely remember there was something similar in the past - GS108E (older firmware) mentioned - so here we go:

https://kb.netgear.com/30281/GS108Ev3-Firmware-Version-2-00-08
...

Bug Fixes:

• Fixed the web GUI counter issue where Bytes Received and Bytes Sent (under System > Monitoring > Port Statistics) gets reset to zero after reaching 4294967295.

...

This is the max 32-bit positive or unsigned binary integer (so double of what the GS908E does currently to in the Web UI) if my math is about right. #FFFFFFFF before flowing over to #00000000.

Now I'm uncertain what the effective max is now, hope for a 64-bit unsigned integer. Not collecting or monitoring these Smart Managed Plus switches, but have counters like 14407024373 or 31549920167 as or writing (other GSxxxE[MX]). So certainly a bug on the GS808E.

FWIW on the GS810EMX (SX10) with the similar Web UI we see numbers like 35352902536 as of writing. But we've tested this during the field testing on the SX10, probably probably missed this on the S8000.

@jess_last have you reviewed some JS code ref. the int32 max, or was this just based on the observations?

TIA,
-Kurt

[jess_last]

Based on observations. First noticed that it never shows over ~4GB, then confirmed it overflows at max_int. In case 64-bit integer is hard to do, maybe report in KB or MB? Who cares about single bytes for traffic counter.

Bonus: it would be nice to show thousand separator in the UI as well, otherwise hard to read.

Thanks!