diff options
author | David S. Miller <davem@davemloft.net> | 2023-07-17 07:33:39 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2023-07-17 07:33:39 +0100 |
commit | 6e8778f8e0f58cac4955b17130549b156cf9d25f (patch) | |
tree | 1ea462ff6731f23f4fa69e65005e81913ced5e95 /drivers/net/dsa/microchip/ksz_common.c | |
parent | 0dd1805fe498e0cf64f68e451a8baff7e64494ec (diff) | |
parent | ac177a330077f264664f56259038e121bb214bec (diff) |
Merge branch 'sched-fixes'
Victor Nogueira says:
====================
net: sched: Fixes for classifiers
Four different classifiers (bpf, u32, matchall, and flower) are
calling tcf_bind_filter in their callbacks, but arent't undoing it by
calling tcf_unbind_filter if their was an error after binding.
This patch set fixes all this by calling tcf_unbind_filter in such
cases.
This set also undoes a refcount decrement in cls_u32 when an update
fails under specific conditions which are described in patch #3.
v1 -> v2:
* Remove blank line after fixes tag
* Fix reverse xmas tree issues pointed out by Simon
v2 -> v3:
* Inline functions cls_bpf_set_parms and fl_set_parms to avoid adding
yet another parameter (and a return value at it) to them.
* Remove similar fixes for u32 and matchall, which will be sent soon,
once we find a way to do the fixes without adding a return parameter
to their set_parms functions.
v3 -> v4:
* Inline mall_set_parms to avoid adding yet another parameter.
* Remove set_flags parameter from u32_set_parms and create a separate
function for calling tcf_bind_filter and tcf_unbind_filter in case of
failure.
* Change cover letter title to also encompass refcnt fix for u32
v4 -> v5:
* Change back tag to net
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/microchip/ksz_common.c')
0 files changed, 0 insertions, 0 deletions