mm/huge_memory: mark racy access onhuge_anon_orders_always
BugLink: https://bugs.launchpad.net/bugs/2083488 [ Upstream commit 7f83bf14603ef41a44dc907594d749a283e22c37 ] huge_anon_orders_always is 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. Link: https://lkml.kernel.org/r/20240515104754889HqrahFPePOIE1UlANHVAh@zte.com.cn Signed-off-by: Ran Xiaokai <ran.xiaokai@zte.com.cn> Acked-by: David Hildenbrand <david@redhat.com> Reviewed-by: Lu Zhongjun <lu.zhongjun@zte.com.cn> Reviewed-by: xu xin <xu.xin16@zte.com.cn> Cc: Yang Yang <yang.yang29@zte.com.cn> Cc: Matthew Wilcox (Oracle) <willy@infradead.org> Cc: Yang Shi <shy828301@gmail.com> Cc: Zi Yan <ziy@nvidia.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Stable-dep-of: 00f58104202c ("mm: fix khugepaged activation policy") Signed-off-by: Sasha Levin <sashal@kernel.org> Signed-off-by: Koichiro Den <koichiro.den@canonical.com> Signed-off-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>
This commit is contained in:
committed by
Mehmet Basaran
parent
78bbfd5efe
commit
efe3fa97ff
@@ -128,8 +128,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)
|
||||
|
||||
Reference in New Issue
Block a user