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 /rust/helpers/signal.c | |
| parent | 7736aff4704188d4483b7b36ff06d201c8be4844 (diff) | |
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 'rust/helpers/signal.c')
0 files changed, 0 insertions, 0 deletions