gen_stats: Add instead Set the value in __gnet_stats_copy_basic().
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Sat, 16 Oct 2021 08:49:02 +0000 (10:49 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 18 Oct 2021 11:54:40 +0000 (12:54 +0100)
commitfbf307c89eb08c51da4dd039f68c19afbcf5949d
tree8959dbffc0540940976953e50a2b3421b7f8fe5a
parent254ec036db1123b10e23e1412c191a3cf70dce71
gen_stats: Add instead Set the value in __gnet_stats_copy_basic().

__gnet_stats_copy_basic() always assigns the value to the bstats
argument overwriting the previous value. The later added per-CPU version
always accumulated the values in the returning gnet_stats_basic_packed
argument.

Based on review there are five users of that function as of today:
- est_fetch_counters(), ___gnet_stats_copy_basic()
  memsets() bstats to zero, single invocation.

- mq_dump(), mqprio_dump(), mqprio_dump_class_stats()
  memsets() bstats to zero, multiple invocation but does not use the
  function due to !qdisc_is_percpu_stats().

Add the values in __gnet_stats_copy_basic() instead overwriting. Rename
the function to gnet_stats_add_basic() to make it more obvious.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/gen_stats.h
net/core/gen_estimator.c
net/core/gen_stats.c
net/sched/sch_mq.c
net/sched/sch_mqprio.c