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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2002BC678D5 for ; Tue, 7 Mar 2023 07:17:57 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6A8AF6B0072; Tue, 7 Mar 2023 02:17:56 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 630D26B0075; Tue, 7 Mar 2023 02:17:56 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4AA49280001; Tue, 7 Mar 2023 02:17:56 -0500 (EST) 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 384A06B0072 for ; Tue, 7 Mar 2023 02:17:56 -0500 (EST) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id E5F82A0D14 for ; Tue, 7 Mar 2023 07:17:55 +0000 (UTC) X-FDA: 80541247710.20.A115496 Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by imf04.hostedemail.com (Postfix) with ESMTP id BB1C04000E for ; Tue, 7 Mar 2023 07:17:53 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=EVRS9l0S; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=drzmil33; dmarc=none; spf=pass (imf04.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.25 as permitted sender) smtp.mailfrom=arnd@arndb.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1678173474; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=qEXKdcaZZ2RtX2iYZ7qIlOWk0bUKh5e0zsPy9GctdU4=; b=jwCXPkvORGAh/XjPxnp6LjaIQfeEty+dmm2NzMvgqR35SES/gpTZ5M1M/vRVaAPVfWQiK/ W2bYmwL8XfSmf3dXLeTtOWqUWSM8T5bDpJdvFlQxzoJPis1gwDdWqYyMzhZtktis+7CEBG Zcn6oNPDJ7vq0rzLjE2AX4yqL9XUnzc= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=arndb.de header.s=fm1 header.b=EVRS9l0S; dkim=pass header.d=messagingengine.com header.s=fm1 header.b=drzmil33; dmarc=none; spf=pass (imf04.hostedemail.com: domain of arnd@arndb.de designates 66.111.4.25 as permitted sender) smtp.mailfrom=arnd@arndb.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1678173474; a=rsa-sha256; cv=none; b=lQb30uvAZcZNL4qzugoRLZfdUatPdT3s4QsZ5GJA7hAuQRCP6hgl8gQvFerzZ0zx/9CRX+ QvNndG26gDmpMCRRn4uh+4cLaj7s8wFkG42Kw+V3uAsa8BDzYSl+T7k/rO5dNIkbX55SAm fWZRBF9TJJnTpa88nra/bILkZARpN0s= Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 0C2795C0267; Tue, 7 Mar 2023 02:17:53 -0500 (EST) Received: from imap51 ([10.202.2.101]) by compute6.internal (MEProxy); Tue, 07 Mar 2023 02:17:53 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1678173473; x=1678259873; bh=qEXKdcaZZ2RtX2iYZ7qIlOWk0bUKh5e0zsP y9GctdU4=; b=EVRS9l0SgvuuYtQJV4VfCOBDd5P19/ufqL/xok5l9oWD530qvRl 32OGHmJT2bayLahPVS0gWRhnLxK6Z+Jlk++srAgloB6tZ4t6RnEQYl9dtmHEbUEF 41yZP0BmrDRa116+CrN+4XevwsaS770TCHXUy+xB6oK0X09IiOjiuVuH0C/ARmo6 fO6SZlgLkFxz5cjkZRClchxi0pyAZ+FHLG3nl/CBKqBfT5jRISfkTQNn0YYywD4T Wj92noz9HFGKpzdsy/cwomQIzdk3+YO7/KK9SAc605foLfc32u4qJDvXpP1n5P7c peGtwY/JqW+0XBFgu8QLiruwVmEZvdvu2Mw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t= 1678173473; x=1678259873; bh=qEXKdcaZZ2RtX2iYZ7qIlOWk0bUKh5e0zsP y9GctdU4=; b=drzmil3385Lov2w7beShJrFQ3GATqdBgomeK+nZBz6RibzrgaH9 Ph4oAZFDXe1gA4dz02jni1x8rYLPU8csmNt0y/kcxAHHuEqFKFb3SExrpsrl2+ox U3hGgfD2s/cIMt8ObvuYqHtHz6rCzw4uyLsanVK3GWvMbQj1h7ydE9biXdh6IlAo vbBrXEwKvNYO63OEEXlqSJsO+MvOnXKAazVOsOpMwDKqVBcWpraKL3cF2FUOLPH8 UeMggh88CpSEmj+eZVgGHUKWgsHEc9fW6HXemWUPFDglt5cDYOzE/+dXaTvRkKAT +fxjvbLqJBJC5gz7jKtljcIml1zGKCfQHUA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrvddtledguddtgecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefofgggkfgjfhffhffvvefutgfgsehtqhertderreejnecuhfhrohhmpedf tehrnhguuceuvghrghhmrghnnhdfuceorghrnhgusegrrhhnuggsrdguvgeqnecuggftrf grthhtvghrnhepgfekueelgeeigefhudduledtkeefffejueelheelfedutedttdfgveeu feefieegnecuffhomhgrihhnpehkvghrnhgvlhdrohhrghenucevlhhushhtvghrufhiii gvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegrrhhnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 13F47B60086; Tue, 7 Mar 2023 02:17:52 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-206-g57c8fdedf8-fm-20230227.001-g57c8fded Mime-Version: 1.0 Message-Id: <9d8292ad-c865-4b82-b6e3-d7db75820d0f@app.fastmail.com> In-Reply-To: References: <20230303102817.212148-1-bhe@redhat.com> <20230303102817.212148-3-bhe@redhat.com> <87sfej1rie.fsf@mpe.ellerman.id.au> <5dec69d0-0bc9-4f6c-8d0d-ee5422783100@app.fastmail.com> <87jzzt1ioc.fsf@mpe.ellerman.id.au> Date: Tue, 07 Mar 2023 08:17:30 +0100 From: "Arnd Bergmann" To: "Baoquan He" , "Michael Ellerman" Cc: "Geert Uytterhoeven" , Linux-Arch , linux-sh@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-hexagon@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org, "Christoph Hellwig" , linux-mm@kvack.org, "Luis Chamberlain" , linux-parisc@vger.kernel.org, linux-alpha@vger.kernel.org, sparclinux@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH v3 2/2] arch/*/io.h: remove ioremap_uc in some architectures Content-Type: text/plain;charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: BB1C04000E X-Rspamd-Server: rspam09 X-Rspam-User: X-Stat-Signature: 9kxk79kohwb54tu9aeupdbdsyu3pp9ee X-HE-Tag: 1678173473-256054 X-HE-Meta: U2FsdGVkX19n+Noxy0k39ALtXDSNqqVBOm1WCyJGXtRbSu0xpwpk2VXi5er0LX7rMYtNAc8WVwWK662C4MSd1eAkJ+12gKBF2F1uFuHnFuS3VqMUmFJCIj1NbsIAb+fl+tzX+BqCmRJYVmcmMeqEGEuEWj8E9r0fZLTOQ+TZyH9013yJQifMtOD5IcOrdCb/VU5KTsBnvlUd78xQYPpYQfp7Z+BVC9ifAiZ1k4iZmpgXS1hTorn62PVczqEt8kGa4OR5371iLxQWaaf6OHtY+SVG9bg+g2pLW93KeIyjwDgXBQBQQTEBAru6WihD8bBHldvGwTLD25/sKbj4BibC4j5ivyn0wYn2xErLAuWol8BLAYrxSTCiir0iI+oUPaNGlycYY2iJDNiD7ChRRNIv+RKhqjQCUVsZsIQjVRD2BqsRQ5/LLdhrQVeTDjxKDs5Ep5bL62dmVDIrNq3FxGoRWULg7UOAEAe/GTl8iotl2zdI3MQOc85NkATdwyDIuJBlFvW89rtgdHQhl3RabPrK28sNDNh+6xeHMGVsc+iX21rJM0dxsP7fo0WRuRrzZbtnY+qjma+Adi9KOFGzChp6KnqUddXU0pKCW0p/CeCHvsLUF9pe5r+zltumFt6p7W08bA18IHPgN6Ps8c2IUjD1FTVRtYnw3sYjOs+rdCiq0QkLZv+ViU7Qs0P7vSZeaq5TQcSboWKvmevSbc9gL3HOXruOLindOPNsCVVqPM3DFhBilrRKhkfn+OkWnpsyHeLtBOETqnMZQbRq7HCmRaC+b3L9rf1OioGSwxcZECcZDI7SdsoYUUdxR4rU7bXAdX0B/RYSNEYsdk6NZ4UgUuKscQk0z5EBbzCBEPSA6DpUc4mGluaSDsr1t5bweCtwSPylp550301Vnl/stQ/X4DLHC4sq8mOZCLkfj3Th/oZ+i+2xT1F+tfJljea2zVMPLhBIJCDxO4FTVJj9HMxYpHO h0HybNJi jzazU0nEufwAHhrbjoZ8BzS5GchIQKKVEpznKvPI8Ls0+Ey/WjhpHOMGoGhN75Ciw60RxR3s+l8h7K50fty2CPCMwQ3fYJ52n4OPplszxK6WTj4pakoWHQ0s40FZEAZjELF+g7hKu+3wgIsyD3kupgu6rx+2yhY7+69wjI/27ZW34kRl0O1J1kj2lHT2KisgEa1Icr4sAQRJsK0RXPbxZMKu2w+V0daK3sqDLjKHasmECgjNupY2+VBISTwQPq4WE0aeNJfrBEGiSYGY4p80GMosmelsrJiGV/94zuoyPgaeK+BY= 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 Tue, Mar 7, 2023, at 02:30, Baoquan He wrote: > On 03/07/23 at 11:58am, Michael Ellerman wrote: >> "Arnd Bergmann" writes: >> > On Sun, Mar 5, 2023, at 10:29, Geert Uytterhoeven wrote: >> >> On Sun, Mar 5, 2023 at 10:23=E2=80=AFAM Michael Ellerman wrote: >> >>> Maybe that exact code path is only reachable on x86/ia64? But if = so >> >>> please explain why. >> >>> >> >>> Otherwise it looks like this series could break that driver on po= werpc >> >>> at least. >> >> >> >> Indeed. >> > >> > When I last looked into this, I sent a patch to use ioremap() >> > on non-x86: >> > >> > https://lore.kernel.org/all/20191111192258.2234502-1-arnd@arndb.de/ >>=20 >> OK thanks. >>=20 >> Baoquan can you add that patch to the start of this series if/when you >> post the next version? > > Sure, will do. Wondering if we need make change to cover powerpc other > than x86 and ia64 in Arnd's patch as you and Geert pointed out. The patch fixes the aty driver for all architectures, including the ones that were already broken before your series with the 'return NULL' version. The only other callers of ioremap_uc() and devm_ioremap_uc() are in architecture specific code and in drivers/mfd/intel-lpss.c, which is x86 specific. Arnd