On Fri, Apr 03, 2020 at 09:38:17AM -0400, Tejun Heo wrote: > On Fri, Apr 03, 2020 at 12:56:27PM +0300, Dmitry Yakunin wrote: > > This patch adds cgroup v2 ID to common inet diag message attributes. > > Cgroup v2 ID is kernfs ID (ino or ino+gen). This attribute allows filter > > inet diag output by cgroup ID obtained by name_to_handle_at() syscall. > > When net_cls or net_prio cgroup is activated this ID is equal to 1 (root > > cgroup ID) for newly created sockets. > > > > Some notes about this ID: > > > > 1) gets initialized in socket() syscall > > 2) incoming socket gets ID from listening socket > > (not during accept() syscall) > > How would this work with things like inetd? Would it make sense to associate the > socket on the first actual send/recv? Oh, it's not a problem with your patch as you're just following the associated ptr, so we can have that discussion separately. Looks good to me from cgroup side. Please feel free to add my acked-by. Thanks. -- tejun
