Merge branch 'sched-fixes'
authorDavid S. Miller <davem@davemloft.net>
Mon, 17 Jul 2023 06:33:39 +0000 (07:33 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Jul 2023 06:33:39 +0000 (07:33 +0100)
commit6e8778f8e0f58cac4955b17130549b156cf9d25f
tree1ea462ff6731f23f4fa69e65005e81913ced5e95
parent0dd1805fe498e0cf64f68e451a8baff7e64494ec
parentac177a330077f264664f56259038e121bb214bec
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>