From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qt1-f170.google.com (mail-qt1-f170.google.com [209.85.160.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1CBC8338F45 for ; Wed, 21 Jan 2026 02:24:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768962301; cv=none; b=mYda775fWACJG19pvEOnMcZeSRAuaPnzbGBzCjc9X2LiJgwklKsCVyOqY9qKeYFnudJR/Z9DeYdiKjSxgLlSpNB8yTWB2p4Qq2uwpegJ5QjTIQP/fyh4atADmFNtpaHyc1NBVyCQLmobEOT9VVNQOBMhYjN5+YN6SEnQ6FplxhI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768962301; c=relaxed/simple; bh=YycTiJrPuZ50XWbU+aKf4d35PZCgfYUN6bofn1fuMIY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=OYH68lIU44s0ojSLahJWHClfj2BuvCYZ/5KDs+ibIhHlumAcIJj4Qj/7EhuwFMYQE3oW8M8qTBZaAYrIN2GMjg7J3WNEt9fd3ylZIlmmFpYB2e5BxrWzQYXgNCiY1mKyJ4zea6uhRj3lZ6tuwKB9ndqvs83d7OeZN5xEWkavlq4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=MJ9J2Xfi; arc=none smtp.client-ip=209.85.160.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="MJ9J2Xfi" Received: by mail-qt1-f170.google.com with SMTP id d75a77b69052e-501c6665144so61732481cf.0 for ; Tue, 20 Jan 2026 18:24:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768962299; x=1769567099; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:feedback-id:from:to:cc:subject:date :message-id:reply-to; bh=/SYWzQFciEmh1FoU8deqlPVu+NRiIm4WLmRuTDTk8Qc=; b=MJ9J2XfiyVW4oHxXetYWBGpo/NCaP5oJEAK9JG/UgaHKeANy/DnoPHPI8OPaJbiLpQ zdpnso+l2XEYTIWB9Po8vOfokdJNR0UbTk31YbXUUXwJ3DEHJuA7TJna4/1NjgjSBVpf EdmXC2gkkjGWVujoY6uKmlK2ry0k3kdReGGFOsswYqgSYrhco3GxTU6iZT7/GHm9DqwY LKO1fqM1DvK5u0Adt0Qbj2/PFuRJmLs5LabGG35Q7n2LTevqB535QabTkN6SBuJpyVqD MmGTjFkWzyvue3dg9ngyaQciADe1ed+vCjCr51nIM4CgG681g4eL0Fv51n3sLULJMDPI QoFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768962299; x=1769567099; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:feedback-id:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=/SYWzQFciEmh1FoU8deqlPVu+NRiIm4WLmRuTDTk8Qc=; b=Ye9/tn1Vmtm1dinBI1w46u+NptuAs5sKaEQT089gDy5wk5djXK/PwwECiQcf1vIBi0 f+bsTcJb2qcL8mqmdrQ9PbjJUNbRl10tNFZgQVhjMKF1HSHdLgywqQTSoOjXU5OflaWh JGbtjihFjGfenZI+Ry3jhz5w4STx38uKzANMhBGqIB1usCOKDS1rjEMOJcQy//G90u0B jIoxfWQDXZxPK18dBeVU+YcMto4oUyqP4LkxNHtLroh2YWkdt2REE4i/brZA1blVCpiF 2Z/vg3NEzkoQrvE67iwAyRRSbLLCZnOcIr9IxrM7o8lwzmcZwgfF00mvojiZ6o7yGXOP Bw0Q== X-Forwarded-Encrypted: i=1; AJvYcCUqv9uKQ8JBu1Ewo2t/5aBsm540Nk8VitroP8lC+48MlBcqT2JH2WtxPUX5Do+qjUie2sjb/hwkJp418iM=@vger.kernel.org X-Gm-Message-State: AOJu0YxwI5asxZrkSbq10eXVc3KVamjUyKkumz3xD5LIlcz7iCj8PhCj RYo+AX2mPbafexXKW0ChDGNIoDriCnKA+1rkkWvfh53y091H+EJOr1gHtp4cdAV7 X-Gm-Gg: AZuq6aLIcQH0rBsUOkSkRjtRD3FdOKh0h42jeEX+aANIMunwK8zINVNTnuhwxYlEZFM zSNyz2iSTD0LjrVAaaCmmOyeUjfOvEH41DHFuTBXA1LI+JjTSBvdRZGkc2jhf/RGiHdbXWi3mVm Bvyoxiv+xFTOAz/EwvmY/9KLE2ELb7dZCp112z4De7+X5G0FXWgApqXb4Wa9riHbXiy5AtNbI8v 6I0IJALgXLmZUnxjtOXV4+wp+bDHQMh2vdgjmwv7yihuBhvTKQ8nxzpC90dImrvja31tJhHkkAP oo0MZQ6WdLsqR946wq95WLEPHQqoN8xE/8qFQaIiGLykwvfYB376qzBklpVoxKMMAQ1h5M5oncd usC/8vdjKL5pvvekCLRSKHHtQDZibfxBSaAM0MSDa1uKHULpT5K+rifqslgMRCMBHPYHl7x4wyj isfcoFfxLv4tCo2Y6a3QRNmqstFHShYyFu3+Zjdd5c+bUSB1gHPexD26ZeBvB9NXuEQYdRcrMhH ISSoCrzG6eFDTg= X-Received: by 2002:ac8:5e11:0:b0:4f4:d298:b47c with SMTP id d75a77b69052e-502a1e07becmr238961631cf.28.1768956457813; Tue, 20 Jan 2026 16:47:37 -0800 (PST) Received: from fauth-a1-smtp.messagingengine.com (fauth-a1-smtp.messagingengine.com. [103.168.172.200]) by smtp.gmail.com with ESMTPSA id af79cd13be357-8c6a7269277sm1143950785a.43.2026.01.20.16.47.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 20 Jan 2026 16:47:37 -0800 (PST) Received: from phl-compute-02.internal (phl-compute-02.internal [10.202.2.42]) by mailfauth.phl.internal (Postfix) with ESMTP id 9A38FF40070; Tue, 20 Jan 2026 19:47:35 -0500 (EST) Received: from phl-frontend-03 ([10.202.2.162]) by phl-compute-02.internal (MEProxy); Tue, 20 Jan 2026 19:47:35 -0500 X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefgedrtddtgddugedukeelucetufdoteggodetrf dotffvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfurfetoffkrfgpnffqhgenuceu rghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujf gurhepfffhvfevuffkfhggtggujgesthdtredttddtvdenucfhrhhomhepuehoqhhunhcu hfgvnhhguceosghoqhhunhdrfhgvnhhgsehgmhgrihhlrdgtohhmqeenucggtffrrghtth gvrhhnpeehudfgudffffetuedtvdehueevledvhfelleeivedtgeeuhfegueevieduffei vdenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegsoh hquhhnodhmvghsmhhtphgruhhthhhpvghrshhonhgrlhhithihqdeiledvgeehtdeigedq udejjeekheehhedvqdgsohhquhhnrdhfvghngheppehgmhgrihhlrdgtohhmsehfihigmh gvrdhnrghmvgdpnhgspghrtghpthhtohepfedupdhmohguvgepshhmthhpohhuthdprhgt phhtthhopegrlhhitggvrhihhhhlsehgohhoghhlvgdrtghomhdprhgtphhtthhopeifih hllheskhgvrhhnvghlrdhorhhgpdhrtghpthhtohepphgvthgvrhiisehinhhfrhgruggv rggurdhorhhgpdhrtghpthhtoheprhhitghhrghrugdrhhgvnhguvghrshhonheslhhinh grrhhordhorhhgpdhrtghpthhtohepmhgrthhtshhtkeeksehgmhgrihhlrdgtohhmpdhr tghpthhtoheplhhinhhmrghgjeesghhmrghilhdrtghomhdprhgtphhtthhopegtrghtrg hlihhnrdhmrghrihhnrghssegrrhhmrdgtohhmpdhrtghpthhtohepohhjvggurgeskhgv rhhnvghlrdhorhhgpdhrtghpthhtohepghgrrhihsehgrghrhihguhhordhnvght X-ME-Proxy: Feedback-ID: iad51458e:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Tue, 20 Jan 2026 19:47:34 -0500 (EST) Date: Wed, 21 Jan 2026 08:47:32 +0800 From: Boqun Feng To: Alice Ryhl Cc: Will Deacon , Peter Zijlstra , Richard Henderson , Matt Turner , Magnus Lindholm , Catalin Marinas , Miguel Ojeda , Gary Guo , =?iso-8859-1?Q?Bj=F6rn?= Roy Baron , Benno Lossin , Andreas Hindborg , Trevor Gross , Danilo Krummrich , Mark Rutland , FUJITA Tomonori , Frederic Weisbecker , Lyude Paul , Thomas Gleixner , Anna-Maria Behnsen , John Stultz , Stephen Boyd , Alexander Viro , Christian Brauner , Jan Kara , linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, rust-for-linux@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 5/5] rust: fs: use READ_ONCE instead of read_volatile Message-ID: References: <20251231-rwonce-v1-0-702a10b85278@google.com> <20251231-rwonce-v1-5-702a10b85278@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251231-rwonce-v1-5-702a10b85278@google.com> On Wed, Dec 31, 2025 at 12:22:29PM +0000, Alice Ryhl wrote: > Using `READ_ONCE` is the correct way to read the `f_flags` field. > > Signed-off-by: Alice Ryhl > --- > rust/kernel/fs/file.rs | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/rust/kernel/fs/file.rs b/rust/kernel/fs/file.rs > index 23ee689bd2400565223181645157d832a836589f..6b07f08e7012f512e53743266096ce0076d29e1c 100644 > --- a/rust/kernel/fs/file.rs > +++ b/rust/kernel/fs/file.rs > @@ -335,12 +335,8 @@ pub fn cred(&self) -> &Credential { > /// The flags are a combination of the constants in [`flags`]. > #[inline] > pub fn flags(&self) -> u32 { > - // This `read_volatile` is intended to correspond to a READ_ONCE call. > - // > - // SAFETY: The file is valid because the shared reference guarantees a nonzero refcount. > - // > - // FIXME(read_once): Replace with `read_once` when available on the Rust side. > - unsafe { core::ptr::addr_of!((*self.as_ptr()).f_flags).read_volatile() } > + // SAFETY: The `f_flags` field of `struct file` is readable with `READ_ONCE`. > + unsafe { kernel::sync::READ_ONCE(&raw const (*self.as_ptr()).f_flags) } Not a question directly to this patch, but for FS folks: I see we read and write `f_flags` normally (i.e. without *_ONCE() or any atomic), and I don't see any synchronization between these read and write (maybe I'm missing something?), if read and write can happen at the same time, it's data race. So I assume we must have some assumption on the atomicity of these accesses to `f_flags`, could you may share or confirm this? Thanks Regards, Boqun > } > } > > > -- > 2.52.0.351.gbe84eed79e-goog >