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=-6.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 1AE0AC48BE5 for ; Wed, 16 Jun 2021 23:49:24 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 8ECED611EE for ; Wed, 16 Jun 2021 23:49:23 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8ECED611EE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 20F866B0070; Wed, 16 Jun 2021 19:49:23 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1991A6B0071; Wed, 16 Jun 2021 19:49:23 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F3C516B0072; Wed, 16 Jun 2021 19:49:22 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id B47A96B0070 for ; Wed, 16 Jun 2021 19:49:22 -0400 (EDT) Received: from smtpin08.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 425F0181AEF07 for ; Wed, 16 Jun 2021 23:49:22 +0000 (UTC) X-FDA: 78261230964.08.2BE78A4 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf14.hostedemail.com (Postfix) with ESMTP id EC05EC0091AE for ; Wed, 16 Jun 2021 23:49:10 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 0FCFF613AE; Wed, 16 Jun 2021 23:49:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1623887359; bh=qIUOjJaXnnZDIyqjA+8Rc1PjQR2ZFeNFFdkcFco2deg=; h=In-Reply-To:References:Date:From:To:Cc:Subject:From; b=tLLlTT5QYvEBvhHyFgtlIUon6gCPPxmjtFckUGTgYydeAEqnZDsIDRjwwA4edaT4i SxjOwyOiiypMhRc5geGG5OgstPcC5Zydag4FyTShAXeUicNHyag/uwd+Bf8addiz00 r8/1njn1ijQGDlnV6k+rfHTFwz46KsHf/SjGCMx9fE58OFAK//3XD6VMtAMLWQzk6n Ev9xVT4vvdAIgmQ8AxLgsQmZwVDU9W6/It1AubbzeAKkkNmTvi/EmCaGXdyXb5HH2s XJk7RGwQv/SyH4TP+iKmDKoEdgMvZtMBHTC+hzuUNyU6fbMwMEk2cicq8FRa861MWQ vOcPKj4sc0mRw== Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailauth.nyi.internal (Postfix) with ESMTP id 1C2F927C0054; Wed, 16 Jun 2021 19:49:17 -0400 (EDT) Received: from imap21 ([10.202.2.71]) by compute2.internal (MEProxy); Wed, 16 Jun 2021 19:49:17 -0400 X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrfeeftddgvddvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvffutgesthdtredtreerjeenucfhrhhomhepfdetnhgu hicunfhuthhomhhirhhskhhifdcuoehluhhtoheskhgvrhhnvghlrdhorhhgqeenucggtf frrghtthgvrhhnpeegjefghfdtledvfeegfeelvedtgfevkeeugfekffdvveeffeetieeh ueetveekfeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhroh hmpegrnhguhidomhgvshhmthhprghuthhhphgvrhhsohhnrghlihhthidqudduiedukeeh ieefvddqvdeifeduieeitdekqdhluhhtoheppehkvghrnhgvlhdrohhrgheslhhinhhugi drlhhuthhordhush X-ME-Proxy: Received: by mailuser.nyi.internal (Postfix, from userid 501) id 8540951C0060; Wed, 16 Jun 2021 19:49:15 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.5.0-alpha0-526-gf020ecf851-fm-20210616.001-gf020ecf8 Mime-Version: 1.0 Message-Id: <57dae79c-fff8-411f-a375-1aa8a1cd10ac@www.fastmail.com> In-Reply-To: <1e248763-9372-6e4e-5dea-cda999000aeb@kernel.org> References: <07a8b963002cb955b7516e61bad19514a3acaa82.1623813516.git.luto@kernel.org> <1623818343.eko1v01gvr.astroid@bobo.none> <1e248763-9372-6e4e-5dea-cda999000aeb@kernel.org> Date: Wed, 16 Jun 2021 16:48:54 -0700 From: "Andy Lutomirski" To: "Nicholas Piggin" , "the arch/x86 maintainers" Cc: "Andrew Morton" , "Benjamin Herrenschmidt" , "Catalin Marinas" , "Dave Hansen" , linux-arm-kernel@lists.infradead.org, "Linux Kernel Mailing List" , linux-mm@kvack.org, linuxppc-dev@lists.ozlabs.org, "Mathieu Desnoyers" , "Michael Ellerman" , "Paul Mackerras" , "Peter Zijlstra (Intel)" , stable@vger.kernel.org, "Will Deacon" Subject: =?UTF-8?Q?Re:_[PATCH_8/8]_membarrier:_Rewrite_sync=5Fcore=5Fbefore=5Fuse?= =?UTF-8?Q?rmode()_and_improve_documentation?= Content-Type: text/plain Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=tLLlTT5Q; spf=pass (imf14.hostedemail.com: domain of luto@kernel.org designates 198.145.29.99 as permitted sender) smtp.mailfrom=luto@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Stat-Signature: so3zzhekz8w8nf5rqk6a7t5x4h5zhcc4 X-Rspamd-Queue-Id: EC05EC0091AE X-Rspamd-Server: rspam06 X-HE-Tag: 1623887350-925711 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: On Wed, Jun 16, 2021, at 11:52 AM, Andy Lutomirski wrote: > On 6/15/21 9:45 PM, Nicholas Piggin wrote: > > Excerpts from Andy Lutomirski's message of June 16, 2021 1:21 pm: > >> The old sync_core_before_usermode() comments suggested that a non-icache-syncing > >> return-to-usermode instruction is x86-specific and that all other > >> architectures automatically notice cross-modified code on return to > >> userspace. > > >> +/* > >> + * XXX: can a powerpc person put an appropriate comment here? > >> + */ > >> +static inline void membarrier_sync_core_before_usermode(void) > >> +{ > >> +} > >> + > >> +#endif /* _ASM_POWERPC_SYNC_CORE_H */ > > > > powerpc's can just go in asm/membarrier.h > > $ ls arch/powerpc/include/asm/membarrier.h > ls: cannot access 'arch/powerpc/include/asm/membarrier.h': No such file > or directory Which is because I deleted it. Duh. I'll clean this up. > > > > > > /* > > * The RFI family of instructions are context synchronising, and > > * that is how we return to userspace, so nothing is required here. > > */ > > Thanks! >