Re: [PATCH 7/9] sock, cgroup: add sock->sk_cgroup

From
Daniel Wagner <hidden>
Date
2015-11-23 15:53:33
Also in
cgroups, lkml, netfilter-devel
Reply to
Tejun Heo, 2015-11-23
On 11/23/2015 04:48 PM, Tejun Heo wrote:
On Mon, Nov 23, 2015 at 02:02:03PM +0100, Daniel Wagner wrote:
quoted
On 11/21/2015 05:13 PM, Tejun Heo wrote:
quoted
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Daniel Wagner <redacted>
I did a quick test and for new connection the cgroup2 match worked as
expected. For an existing connection I wasn't able to trigger the match.

It is quite likely I do something wrong:

	ssh into the box
	# mkdir /sys/fs/cgroup/test
	# echo $$ > /sys/fs/cgroup/test/cgroup.procs
	# echo $PPID > /sys/fs/cgroup/test/cgroup.procs
	# iptables -A OUTPUT -m cgroup --path test

Should I see matches with the existing ssh session?
Socket is associated with the creating cgroup and stays associated
with that cgroup until it's released.  Migrating the process doesn't
change the ownership of the sockets it has created.  This is in line
with how other stateful resources such as memory are handled in
cgroup2 hierarchy.
Thanks for the explanation. Looks good to me:

Tested-by: Daniel Wagner <redacted>
Acked-by: Daniel Wagner <redacted>

Thanks,
Daniel
Thread (37 messages) 37 messages, 8 authors, last reply 2015-11-23
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help