From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 89F3AC48BE5 for ; Mon, 21 Jun 2021 09:06:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 65CC161040 for ; Mon, 21 Jun 2021 09:06:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230269AbhFUJI0 (ORCPT ); Mon, 21 Jun 2021 05:08:26 -0400 Received: from foss.arm.com ([217.140.110.172]:58958 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229618AbhFUJIY (ORCPT ); Mon, 21 Jun 2021 05:08:24 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 879C71FB; Mon, 21 Jun 2021 02:06:10 -0700 (PDT) Received: from lakrids.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C07933F718; Mon, 21 Jun 2021 02:06:07 -0700 (PDT) From: Mark Rutland To: linux-kernel@vger.kernel.org Cc: boqun.feng@gmail.com, bp@alien8.de, catalin.marinas@arm.com, dvyukov@google.com, elver@google.com, ink@jurassic.park.msu.ru, jonas@southpole.se, juri.lelli@redhat.com, linux@armlinux.org.uk, luto@kernel.org, mark.rutland@arm.com, mattst88@gmail.com, mingo@redhat.com, monstr@monstr.eu, paulmck@kernel.org, peterz@infradead.org, rth@twiddle.net, shorne@gmail.com, stefan.kristiansson@saunalahti.fi, tglx@linutronix.de, vincent.guittot@linaro.org, will@kernel.org Subject: [PATCH v2 0/9] thread_info: use helpers to snapshot thread flags Date: Mon, 21 Jun 2021 10:05:52 +0100 Message-Id: <20210621090602.16883-1-mark.rutland@arm.com> X-Mailer: git-send-email 2.11.0 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As thread_info::flags scan be manipulated by remote threads, it is necessary to use atomics or READ_ONCE() to ensure that code manipulates a consistent snapshot, but we open-code plain accesses to thread_info::flags across the kernel tree. Generally we get away with this, but tools like KCSAN legitimately warn that there is a data-race, and this is potentially fragile with compiler optimizations, LTO, etc. These patches introduce new helpers to snahpshot the thread flags, with the intent being that these should replace all plain accesses. Since v1 [1]: * Drop RFC * Make read_ti_thread_flags() __always_inline * Clarify commit messages * Fix typo in arm64 patch * Accumulate Reviewed-by / Acked-by tags * Drop powerpc patch to avoid potential conflicts (per [2]) [1] https://lore.kernel.org/r/20210609122001.18277-1-mark.rutland@arm.com [2] https://lore.kernel.org/r/87k0mvtgeb.fsf@mpe.ellerman.id.au Thanks, Mark. Mark Rutland (9): thread_info: add helpers to snapshot thread flags entry: snapshot thread flags sched: snapshot thread flags alpha: snapshot thread flags arm: snapshot thread flags arm64: snapshot thread flags microblaze: snapshot thread flags openrisc: snapshot thread flags x86: snapshot thread flags arch/alpha/kernel/signal.c | 2 +- arch/arm/kernel/signal.c | 2 +- arch/arm/mm/alignment.c | 2 +- arch/arm64/kernel/ptrace.c | 4 ++-- arch/arm64/kernel/signal.c | 2 +- arch/arm64/kernel/syscall.c | 4 ++-- arch/microblaze/kernel/signal.c | 2 +- arch/openrisc/kernel/signal.c | 2 +- arch/x86/kernel/process.c | 8 ++++---- arch/x86/kernel/process.h | 6 +++--- arch/x86/mm/tlb.c | 2 +- include/linux/entry-kvm.h | 2 +- include/linux/thread_info.h | 14 ++++++++++++++ kernel/entry/common.c | 4 ++-- kernel/entry/kvm.c | 4 ++-- kernel/sched/core.c | 2 +- 16 files changed, 38 insertions(+), 24 deletions(-) -- 2.11.0