Re: [PATCH v2 0/2] cgroup: Sync cgroup id and inode number

From: Tejun Heo
Date: Mon Nov 04 2019 - 11:49:13 EST


On Mon, Nov 04, 2019 at 05:45:18PM +0900, Namhyung Kim wrote:
> Hello,
>
> This patchset changes cgroup inode number and id management to be in
> sync with kernfs. The cgroup inode number is managed by kernfs but
> cgroup id is allocated by a separate idr. The idea is to have a
> single id for internal usage, inode number and file handle which can
> be accessed from userspace. Actually this work is from Tejun who also
> provided the idea. I just took over the work and fixed some errors
> and finally was able to run perf for testing.
>
> The background of this work is that I want to add cgroup sampling
> feature in the perf event subsystem. As Tejun mentioned that using
> cgroup id is not enough and it'd better using file handle instead.
> But getting file handle in perf NMI handler is not possible so I want
> to get the info from a cgroup node.
>
> The first patch converted kernfs id into a single 64bit number and in
> the second patch cgroup uses the kernfs id as cgroup id.
>
> The patches are based on the for-next branch in Tejun's cgroup tree.
> Tested with tools/testing/selftests/cgroup/test_stress.sh.

Sorry about the delay. I'm still working on the patch series and the
draft patches currently posted aren't quite correct (e.g. netprio is
now trying to allocate arrays which use u64 values as index and that
u64 value can start with 33rd bit set from the get-go).

So, please hold on w/ this series for now.

Thanks.

--
tejun