]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
Revert "cgroup: Fix memory leak caused by missing cgroup_bpf_offline"
authorChen Ridong <chenridong@huawei.com>
Fri, 18 Oct 2024 08:15:19 +0000 (08:15 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 14 Dec 2024 18:50:47 +0000 (19:50 +0100)
[ Upstream commit feb301c60970bd2a1310a53ce2d6e4375397a51b ]

This reverts commit 04f8ef5643bcd8bcde25dfdebef998aea480b2ba.

Only cgroup v2 can be attached by cgroup by BPF programs. Revert this
commit and cgroup_bpf_inherit and cgroup_bpf_offline won't be called in
cgroup v1. The memory leak issue will be fixed with next patch.

Fixes: 04f8ef5643bc ("cgroup: Fix memory leak caused by missing cgroup_bpf_offline")
Link: https://lore.kernel.org/cgroups/aka2hk5jsel5zomucpwlxsej6iwnfw4qu5jkrmjhyfhesjlfdw@46zxhg5bdnr7/
Signed-off-by: Chen Ridong <chenridong@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
kernel/cgroup/cgroup.c

index 9ba87c5de1a879fec8f80e3e94db8c0986b08dfd..7346197f464cbed9cb2b4f9ebb2f214fa7e16cc8 100644 (file)
@@ -2228,10 +2228,8 @@ static void cgroup_kill_sb(struct super_block *sb)
         * And don't kill the default root.
         */
        if (list_empty(&root->cgrp.self.children) && root != &cgrp_dfl_root &&
-           !percpu_ref_is_dying(&root->cgrp.self.refcnt)) {
-               cgroup_bpf_offline(&root->cgrp);
+           !percpu_ref_is_dying(&root->cgrp.self.refcnt))
                percpu_ref_kill(&root->cgrp.self.refcnt);
-       }
        cgroup_put(&root->cgrp);
        kernfs_kill_sb(sb);
 }