From patchwork Sat May 11 06:44:36 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: xu.xin16@zte.com.cn X-Patchwork-Id: 13662289 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id BA70EC04FFE for ; Sat, 11 May 2024 06:44:50 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2F8576B0162; Sat, 11 May 2024 02:44:50 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 281636B0163; Sat, 11 May 2024 02:44:50 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 1226B6B0164; Sat, 11 May 2024 02:44:50 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id E71326B0162 for ; Sat, 11 May 2024 02:44:49 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 3A78B1C1FE9 for ; Sat, 11 May 2024 06:44:49 +0000 (UTC) X-FDA: 82105177098.28.9061239 Received: from mxhk.zte.com.cn (mxhk.zte.com.cn [63.216.63.40]) by imf20.hostedemail.com (Postfix) with ESMTP id D9EB11C000D for ; Sat, 11 May 2024 06:44:46 +0000 (UTC) Authentication-Results: imf20.hostedemail.com; dkim=none; spf=pass (imf20.hostedemail.com: domain of xu.xin16@zte.com.cn designates 63.216.63.40 as permitted sender) smtp.mailfrom=xu.xin16@zte.com.cn; dmarc=pass (policy=none) header.from=zte.com.cn ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1715409887; a=rsa-sha256; cv=none; b=h5L6pMibdBkYYaR+/sxefcx388AdmtFUCaL3wUxaVBZKX5mn9RO+PCwCw7CA4BFkPUA+H2 V5V+nKmRV7Utqf+qqZsHVfAFOWl9eGOAPZFl6uUIY4/3CPBYLoB5OiD/BFshm/fmT/ralJ cbusvV00WKva6g3eFqIwLpHqyZ7tkcw= ARC-Authentication-Results: i=1; imf20.hostedemail.com; dkim=none; spf=pass (imf20.hostedemail.com: domain of xu.xin16@zte.com.cn designates 63.216.63.40 as permitted sender) smtp.mailfrom=xu.xin16@zte.com.cn; dmarc=pass (policy=none) header.from=zte.com.cn ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1715409887; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding:in-reply-to: references; bh=6bdr0N9TaM4YPIOdxhc5FdKzzdCLL+HFnHl4C6/zK0M=; b=IfF3mYjXBFEeO8rnBS2STUa+r+W63W9TrXzMRX1OF6utNvbtYCdbqZCp38l6h7zG2zPl1S AptP+V7EFX/Bj0glCrbZ/Y7mK4wlwFtdI3+grMQwz4BhuzmCzGwwB6YasXPLM+zPgqkchG g60yzTiPWejDubi8KX60KxWFM1ZWFDw= Received: from mse-fl2.zte.com.cn (unknown [10.5.228.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mxhk.zte.com.cn (FangMail) with ESMTPS id 4Vbx7X5tphz8XrXG; Sat, 11 May 2024 14:44:40 +0800 (CST) Received: from xaxapp01.zte.com.cn ([10.88.99.176]) by mse-fl2.zte.com.cn with SMTP id 44B6iY3k075463; Sat, 11 May 2024 14:44:34 +0800 (+08) (envelope-from xu.xin16@zte.com.cn) Received: from mapi (xaxapp03[null]) by mapi (Zmail) with MAPI id mid32; Sat, 11 May 2024 14:44:36 +0800 (CST) Date: Sat, 11 May 2024 14:44:36 +0800 (CST) X-Zmail-TransId: 2afb663f13d40bb-06f01 X-Mailer: Zmail v1.0 Message-ID: <20240511144436754EiKfJM4xjMSTyCbEExwcL@zte.com.cn> Mime-Version: 1.0 From: To: Cc: , , , , , Subject: =?utf-8?q?=C2=A0=5BPATCH_linux-next=5D_mm/huge=5Fmemory=3A_mark_rac?= =?utf-8?q?y_access_on=C2=A0huge=5Fanon=5Forders=5Falways?= X-MAIL: mse-fl2.zte.com.cn 44B6iY3k075463 X-Fangmail-Anti-Spam-Filtered: true X-Fangmail-MID-QID: 663F13D8.001/4Vbx7X5tphz8XrXG X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: D9EB11C000D X-Stat-Signature: 9ezfkabyuei3cn1r5jxhqnny4gwzuad5 X-HE-Tag: 1715409886-932982 X-HE-Meta: U2FsdGVkX18NyOrp5hwUcOKHE9BYlTdSjkcGvWfVpVQhl5zKBelowP0+N90MuhvaMklaXBDjncEEKsWHWyUeT7BUNt7KGLnG7LMPbpoNqV+BZjuaBA8wfE7+wNvkvdT6jCJ6xwdiAfkjjtjEUeNp5vH/WXIuWmeHhy7jlLc0g/aWMNfuXyx9heVXpisffh8BJQDSlDylf8CcJ2kurA4m4P1z8fg67f5BhuJO95cABbf82kUBomsX5jQvL/Fe3wWtsydtyQsJmcshep2gning3h2SzjLi9td58QRGQrWgLPL0Y7+UICnE0bGj9vgvQKd9CrWaDrcZ1Y8iv4i8KcSpVTi93CaHB7JnnUhzy2/RIHacEFoBu9yxECDXrPuiZ8r0Kp49Lx06myhhCZKUAxNC4KHlDQwnDHDuuaw/gt0QZiQRUB05ysf6HjsEk0GxtM75Ex1mLi4auowh9xYWc7S4cgFw3uUOUysVceWEevRiydlo8tYLyrdEyk1QcSP+jSUW1FlbwmKorbE7443AEyzXXTigplqsMxk9O0CFIyG/jggaKc1RIBezAwLg/ik4O/W00z1TQYDMCifWAUGWExPT2SGiQ6MDAFx2pU0YGDxXvNvDE4NWVdcprA9UprDlZx859cIUwasFexPW9v7CdRI8ErizfCpkG/5ZZT6ahElRphdjGt3kD0PhKN39IgYgzRB5JABk+nfhupgez6JkvHvvEnYedmrlokGB+JhqMvzWX9IW8IQVw44/Q5lLNGjMNlKi1vmO/9mg19ptCyHxuc5kxn0ZPGIDbn7/JXvweE0gxDbZcUSDU82N3uugJ7GYqwX/1AcAbT4ngut983tkbJQh5mXPaNxM29LAOAE1vxjvY9KoYuWXQ9YJ6CnhAteJ0/IddDdmIple/4F6xcOQWclxotq5BVsefki5/6dziP/WHTbwNfS3hHosvEu9bfovSYOwwpSGyePIDCf22EF+7AU vl36esiR udSuqhEIJGbMPXrUZs4tav0EHRtDy5+7UhXCl8KCjBvDptLLM9XorgRXPxwZXGd3tqdPOI8+ihQXCZHC6xy8K4wbW2c5ImZDQHHOIQD/CIXIiox18pdcwiWV0MzboSL8X6yVee6hvKMEO4mj+U27/zXf25AVO9JxtNBXIEGrKPp4D16zRPkLju9xKt4LFmRiGO71arWycPBRvgvj3Ds/94yttHJBlMNDZzJci X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: Ran Xiaokai huge_anon_orders_always and huge_anon_orders_always are accessed lockless, it is better to use the READ_ONCE() wrapper. This is not fixing any visible bug, hopefully this can cease some KCSAN complains in the future. Also do that for huge_anon_orders_madvise. Signed-off-by: Ran Xiaokai Acked-by: David Hildenbrand --- include/linux/huge_mm.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h index de0c89105076..6573430ea600 100644 --- a/include/linux/huge_mm.h +++ b/include/linux/huge_mm.h @@ -122,8 +122,8 @@ static inline bool hugepage_flags_enabled(void) * So we don't need to look at huge_anon_orders_inherit. */ return hugepage_global_enabled() || - huge_anon_orders_always || - huge_anon_orders_madvise; + READ_ONCE(huge_anon_orders_always) || + READ_ONCE(huge_anon_orders_madvise); } static inline int highest_order(unsigned long orders)