[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <c3221491-18e9-4de5-961c-38b9ac184601@gmail.com>
Date: Wed, 13 May 2020 21:28:28 -0600
From: David Ahern <dsahern@...il.com>
To: Дмитрий Якунин
<zeil@...dex-team.ru>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Cc: "cgroups@...r.kernel.org" <cgroups@...r.kernel.org>
Subject: Re: [PATCH iproute2-next v2 1/3] ss: introduce cgroup2 cache and
helper functions
On 5/13/20 1:27 AM, Дмитрий Якунин wrote:
>
>
> 13.05.2020, 05:03, "David Ahern" <dsahern@...il.com>:
>> On 5/9/20 10:52 AM, Dmitry Yakunin wrote:
>>> This patch prepares infrastructure for matching sockets by cgroups.
>>> Two helper functions are added for transformation between cgroup v2 ID
>>> and pathname. Cgroup v2 cache is implemented as hash table indexed by ID.
>>> This cache is needed for faster lookups of socket cgroup.
>>>
>>> v2:
>>> - style fixes (David Ahern)
>>
>> you missed my other comment about this set. Running this new command on
>> a kernel without support should give the user a better error message
>> than a string of Invalid arguments:
>>
>> $ uname -r
>> 5.3.0-51-generic
>>
>> $ ss -a cgroup /sys/fs/cgroup/unified
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>> RTNETLINK answers: Invalid argument
>
> No, i didn't miss your comment. This patchset was extended with the third patch which includes bytecode filter checking.
>
ok. missed that. applied to iproute2-next.
in the future, please provide a cover letter
Powered by blists - more mailing lists