From: Sasha Levin <sashal@kernel.org>
To: linux-kernel@vger.kernel.org, stable@vger.kernel.org
Cc: Tejun Heo <tj@kernel.org>,
Abhishek Shah <abhishek.shah@columbia.edu>,
Gabriel Ryan <gabe@cs.columbia.edu>,
Christian Brauner <brauner@kernel.org>,
Sasha Levin <sashal@kernel.org>,
lizefan.x@bytedance.com, hannes@cmpxchg.org,
cgroups@vger.kernel.org
Subject: [PATCH AUTOSEL 5.4 03/13] cgroup: Remove data-race around cgrp_dfl_visible
Date: Mon, 17 Oct 2022 20:10:52 -0400 [thread overview]
Message-ID: <20221018001102.2731930-3-sashal@kernel.org> (raw)
In-Reply-To: <20221018001102.2731930-1-sashal@kernel.org>
From: Tejun Heo <tj@kernel.org>
[ Upstream commit dc79ec1b232ad2c165d381d3dd2626df4ef9b5a4 ]
There's a seemingly harmless data-race around cgrp_dfl_visible detected by
kernel concurrency sanitizer. Let's remove it by throwing WRITE/READ_ONCE at
it.
Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Abhishek Shah <abhishek.shah@columbia.edu>
Cc: Gabriel Ryan <gabe@cs.columbia.edu>
Reviewed-by: Christian Brauner (Microsoft) <brauner@kernel.org>
Link: https://lore.kernel.org/netdev/20220819072256.fn7ctciefy4fc4cu@wittgenstein/
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
kernel/cgroup/cgroup.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c
index d14575c0e464..194060a6492a 100644
--- a/kernel/cgroup/cgroup.c
+++ b/kernel/cgroup/cgroup.c
@@ -2178,7 +2178,7 @@ static int cgroup_get_tree(struct fs_context *fc)
struct cgroup_fs_context *ctx = cgroup_fc2context(fc);
int ret;
- cgrp_dfl_visible = true;
+ WRITE_ONCE(cgrp_dfl_visible, true);
cgroup_get_live(&cgrp_dfl_root.cgrp);
ctx->root = &cgrp_dfl_root;
@@ -6022,7 +6022,7 @@ int proc_cgroup_show(struct seq_file *m, struct pid_namespace *ns,
struct cgroup *cgrp;
int ssid, count = 0;
- if (root == &cgrp_dfl_root && !cgrp_dfl_visible)
+ if (root == &cgrp_dfl_root && !READ_ONCE(cgrp_dfl_visible))
continue;
seq_printf(m, "%d:", root->hierarchy_id);
--
2.35.1
next prev parent reply other threads:[~2022-10-18 0:16 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 0:10 [PATCH AUTOSEL 5.4 01/13] crypto: qcom-rng - Fix qcom_rng_of_match unused warning Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 02/13] gfs2: Switch from strlcpy to strscpy Sasha Levin
2022-10-18 0:10 ` Sasha Levin [this message]
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 04/13] of/fdt: Don't calculate initrd size from DT if start > end Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 05/13] objtool,x86: Teach decode about LOOP* instructions Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 06/13] gfs2: Check sb_bsize_shift after reading superblock Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 07/13] powerpc/64: don't refer nr_cpu_ids in asm code when it's undefined Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 08/13] m68knommu: fix non-specific 68328 choice interrupt build failure Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 09/13] m68knommu: fix non-mmu classic 68000 legacy timer tick selection Sasha Levin
2022-10-18 0:10 ` [PATCH AUTOSEL 5.4 10/13] kmsan: disable physical page merging in biovec Sasha Levin
2022-10-18 0:22 ` Eric Biggers
2022-10-18 9:23 ` Vlastimil Babka
2022-10-18 0:11 ` [PATCH AUTOSEL 5.4 11/13] 9p: trans_fd/p9_conn_cancel: drop client lock earlier Sasha Levin
2022-10-18 0:11 ` [PATCH AUTOSEL 5.4 12/13] 9p/trans_fd: always use O_NONBLOCK read/write Sasha Levin
2022-10-18 0:11 ` [PATCH AUTOSEL 5.4 13/13] virtio_pci: don't try to use intxif pin is zero Sasha Levin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221018001102.2731930-3-sashal@kernel.org \
--to=sashal@kernel.org \
--cc=abhishek.shah@columbia.edu \
--cc=brauner@kernel.org \
--cc=cgroups@vger.kernel.org \
--cc=gabe@cs.columbia.edu \
--cc=hannes@cmpxchg.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lizefan.x@bytedance.com \
--cc=stable@vger.kernel.org \
--cc=tj@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox