summaryrefslogtreecommitdiffstats
path: root/net/mac802154
diff options
context:
space:
mode:
authorEric Dumazet <edumazet@google.com>2016-03-17 17:23:36 -0700
committerDavid S. Miller <davem@davemloft.net>2016-03-18 23:14:15 -0400
commitfe30937b65354c7fec244caebbdaae68e28ca797 (patch)
tree76597e789e2eac26153f116c8dfe991f934bc96a /net/mac802154
parenteee577232203842b4dcadb7ab477a298479633ed (diff)
downloadlinux-fe30937b65354c7fec244caebbdaae68e28ca797.tar.gz
linux-fe30937b65354c7fec244caebbdaae68e28ca797.tar.bz2
linux-fe30937b65354c7fec244caebbdaae68e28ca797.zip
bonding: fix bond_get_stats()
bond_get_stats() can be called from rtnetlink (with RTNL held) or from /proc/net/dev seq handler (with RCU held) The logic added in commit 5f0c5f73e5ef ("bonding: make global bonding stats more reliable") kind of assumed only one cpu could run there. If multiple threads are reading /proc/net/dev, stats can be really messed up after a while. A second problem is that some fields are 32bit, so we need to properly handle the wrap around problem. Given that RTNL is not always held, we need to use bond_for_each_slave_rcu(). Fixes: 5f0c5f73e5ef ("bonding: make global bonding stats more reliable") Signed-off-by: Eric Dumazet <edumazet@google.com> Cc: Andy Gospodarek <gospo@cumulusnetworks.com> Cc: Jay Vosburgh <j.vosburgh@gmail.com> Cc: Veaceslav Falico <vfalico@gmail.com> Reviewed-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/mac802154')
0 files changed, 0 insertions, 0 deletions