diff options
| author |   Cosmin Ratiu <cratiu@nvidia.com> | 2025-09-18 13:15:06 +0300 |
|---|---|---|
| committer |   Jakub Kicinski <kuba@kernel.org> | 2025-09-18 07:47:18 -0700 |
| commit | 3191df0a4882c827cac29925e80ecb1775b904bd (patch) | |
| tree | 9fba9afdf24537a1158fe8d64965dd0e47cf5e58 /drivers/net/dsa/dsa_loop.c | |
| parent | MAINTAINERS: update sundance entry (diff) | |
| download | wireguard-linux-3191df0a4882c827cac29925e80ecb1775b904bd.tar.xz wireguard-linux-3191df0a4882c827cac29925e80ecb1775b904bd.zip | |
devlink rate: Remove unnecessary 'static' from a couple places
devlink_rate_node_get_by_name() and devlink_rate_nodes_destroy() have a
couple of unnecessary static variables for iterating over devlink rates.
This could lead to races/corruption/unhappiness if two concurrent
operations execute the same function.
Remove 'static' from both. It's amazing this was missed for 4+ years.
While at it, I confirmed there are no more examples of this mistake in
net/ with 1, 2 or 3 levels of indentation.
Fixes: a8ecb93ef03d ("devlink: Introduce rate nodes")
Signed-off-by: Cosmin Ratiu <cratiu@nvidia.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/dsa/dsa_loop.c')
0 files changed, 0 insertions, 0 deletions