Hi Vlad, Can you try one simple test below with these patches?
#create an action sudo $TC actions add action skbedit mark 1 pipe # sudo $TC qdisc del dev lo parent ffff: sudo $TC qdisc add dev lo ingress # bind action to filter.... sudo $TC filter add dev lo parent ffff: protocol ip prio 1 \ u32 match ip dst 127.0.0.1/32 flowid 1:1 action skbedit index 1 #now delete that action multiple times while it is still bound sudo $TC actions del action skbedit index 1 sudo $TC actions del action skbedit index 1 sudo $TC actions del action skbedit index 1 #check the refcount and bindcount sudo $TC -s actions ls action skbedit #delete the filter (which should remove the bindcnt) sudo $TC filter del dev lo parent ffff: protocol ip prio 1 \ u32 match ip dst 127.0.0.1/32 flowid 1:1 #check the refcount and bindcount sudo $TC -s actions ls action skbedit Current behavior: i believe the action is gone in this last step. Your patches may change behavior so that the action action is still around. I dont think this is a big deal, but just wanted to be sure it is not something more unexpected. cheers, jamal
