From: Vlad Buslov <vladbu@mellanox.com>
To: Jamal Hadi Salim <jhs@mojatatu.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net,
xiyou.wangcong@gmail.com, jiri@resnulli.us, pablo@netfilter.org,
kadlec@blackhole.kfki.hu, fw@strlen.de, ast@kernel.org,
daniel@iogearbox.net, edumazet@google.com, keescook@chromium.org,
marcelo.leitner@gmail.com, kliteyn@mellanox.com
Subject: Re: [PATCH net-next v4 00/11] Modify action API for implementing lockless actions
Date: Thu, 31 May 2018 15:38:53 +0300 [thread overview]
Message-ID: <vbfmuwgf19e.fsf@reg-r-vrt-018-180.mtr.labs.mlnx> (raw)
In-Reply-To: <262fbd11-401e-90cf-4226-39b1604eb16d@mojatatu.com>
On Thu 31 May 2018 at 10:01, Jamal Hadi Salim <jhs@mojatatu.com> wrote:
> 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
Hi Jamal,
On current net-next I still have action with single reference after last
step:
~$ sudo $TC -s actions ls action skbedit
total acts 1
action order 0: skbedit mark 1 pipe
index 1 ref 2 bind 1 installed 47 sec used 47 sec
Action statistics:
Sent 0 bytes 0 pkt (dropped 0, overlimits 0 requeues 0)
backlog 0b 0p requeues 0
~$ sudo $TC filter del dev lo parent ffff: protocol ip prio 1 \
> u32 match ip dst 127.0.0.1/32 flowid 1:1
~$ sudo $TC -s actions ls action skbedit
total acts 1
action order 0: skbedit mark 1 pipe
index 1 ref 1 bind 0 installed 80 sec used 80 sec
Action statistics:
Sent 0 bytes 0 pkt (dropped 0, overlimits 0 requeues 0)
backlog 0b 0p requeues 0
Which branch are you testing on?
Regards,
Vlad
next prev parent reply other threads:[~2018-05-31 12:39 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-31 7:58 [PATCH net-next v4 00/11] Modify action API for implementing lockless actions Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 01/11] net: sched: use rcu for action cookie update Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 02/11] net: sched: change type of reference and bind counters Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 03/11] net: sched: implement unlocked action init API Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 04/11] net: sched: always take reference to action Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 05/11] net: sched: implement action API that deletes action by index Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 06/11] net: sched: add 'delete' function to action ops Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 07/11] net: sched: implement reference counted action release Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 08/11] net: sched: don't release reference on action overwrite Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 09/11] net: sched: use reference counting action init Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 10/11] net: sched: atomically check-allocate action Vlad Buslov
2018-05-31 7:58 ` [PATCH net-next v4 11/11] net: sched: change action API to use array of pointers to actions Vlad Buslov
2018-05-31 10:01 ` [PATCH net-next v4 00/11] Modify action API for implementing lockless actions Jamal Hadi Salim
2018-05-31 12:38 ` Vlad Buslov [this message]
2018-06-01 12:24 ` Jamal Hadi Salim
2018-06-01 15:19 ` Vlad Buslov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=vbfmuwgf19e.fsf@reg-r-vrt-018-180.mtr.labs.mlnx \
--to=vladbu@mellanox.com \
--cc=ast@kernel.org \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=fw@strlen.de \
--cc=jhs@mojatatu.com \
--cc=jiri@resnulli.us \
--cc=kadlec@blackhole.kfki.hu \
--cc=keescook@chromium.org \
--cc=kliteyn@mellanox.com \
--cc=marcelo.leitner@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=pablo@netfilter.org \
--cc=xiyou.wangcong@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).