diff options
| author |   Michael Chan <michael.chan@broadcom.com> | 2024-01-05 15:54:38 -0800 |
|---|---|---|
| committer |   Jakub Kicinski <kuba@kernel.org> | 2024-01-08 19:15:02 -0800 |
| commit | fd7769798de8a3748c286da65d7e32437f9854bf (patch) | |
| tree | 1a2eb839a940b8aace58639d2829a00fe705037c /tools/perf/scripts/python/gecko.py | |
| parent | bnxt_en: Remove unneeded variable in bnxt_hwrm_clear_vnic_filter() (diff) | |
| download | wireguard-linux-fd7769798de8a3748c286da65d7e32437f9854bf.tar.xz wireguard-linux-fd7769798de8a3748c286da65d7e32437f9854bf.zip | |
bnxt_en: Fix RCU locking for ntuple filters in bnxt_srxclsrldel()
After looking up an ntuple filter from a RCU hash list, the
rcu_read_unlock() call should be made after reading the structure,
or after determining that the filter cannot age out (by aRFS).
The existing code was calling rcu_read_unlock() too early in
bnxt_srxclsrldel().
As suggested by Simon Horman, change the code to handle the error
case of fltr_base not found in the if condition. The code looks
cleaner this way.
Fixes: 8d7ba028aa9a ("bnxt_en: Add support for ntuple filter deletion by ethtool.")
Suggested-by: Simon Horman <horms@kernel.org>
Reported-by: Jakub Kicinski <kuba@kernel.org>
Link: https://lore.kernel.org/netdev/20240104145955.5a6df702@kernel.org/
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20240105235439.28282-3-michael.chan@broadcom.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/gecko.py')
0 files changed, 0 insertions, 0 deletions