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 70DB9C25B67 for ; Thu, 26 Oct 2023 12:36:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E50DF6B036D; Thu, 26 Oct 2023 08:36:47 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DD9BC6B036E; Thu, 26 Oct 2023 08:36:47 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C52FD6B036F; Thu, 26 Oct 2023 08:36:47 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id AE17D6B036D for ; Thu, 26 Oct 2023 08:36:47 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 72C221A092B for ; Thu, 26 Oct 2023 12:36:47 +0000 (UTC) X-FDA: 81387561654.11.860F608 Received: from wout4-smtp.messagingengine.com (wout4-smtp.messagingengine.com [64.147.123.20]) by imf03.hostedemail.com (Postfix) with ESMTP id 12BEC2000B for ; Thu, 26 Oct 2023 12:36:43 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=flygoat.com header.s=fm1 header.b="ReK/1CPM"; dkim=pass header.d=messagingengine.com header.s=fm3 header.b=loijJ83r; spf=pass (imf03.hostedemail.com: domain of jiaxun.yang@flygoat.com designates 64.147.123.20 as permitted sender) smtp.mailfrom=jiaxun.yang@flygoat.com; dmarc=pass (policy=none) header.from=flygoat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1698323804; 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=7Ou35+gtN5wPlKenTK4K7QdEWtUFHuuBjgjvnmJt2so=; b=axAHY3cqX+WcFwE6gmxQrcUHPiioVawpZ1TZmZgAPiLBl4Z0gCJ9ItnGFlf1mMs12FONiR qMGQptbhBt1lAKHZCTu0iSws4LqiMwLEvL4ajYO7Y30vujynem8mCnMVWJtHtdwGNU3RzW UWNrexo3/m7y1/73DLhIo7wTaPdG7Vs= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1698323804; a=rsa-sha256; cv=none; b=LL2DQV6tEnesmF/UFK/zZs3QZv38gQIlkU6LnWXTH0JVwOyduzSEJcbvQP939p9+ecjKsG Gt46Royw6Nc0UDOjkBWE1CYkmEN9jXEqjqs2barSb5kRgOEDza5K5R8JFnODMZjsKB1GZY I+YGxyPt2QFUdNi7EW3jZ3i46qcaH/s= ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=flygoat.com header.s=fm1 header.b="ReK/1CPM"; dkim=pass header.d=messagingengine.com header.s=fm3 header.b=loijJ83r; spf=pass (imf03.hostedemail.com: domain of jiaxun.yang@flygoat.com designates 64.147.123.20 as permitted sender) smtp.mailfrom=jiaxun.yang@flygoat.com; dmarc=pass (policy=none) header.from=flygoat.com Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailout.west.internal (Postfix) with ESMTP id 6632E3200943; Thu, 26 Oct 2023 08:36:41 -0400 (EDT) Received: from imap44 ([10.202.2.94]) by compute3.internal (MEProxy); Thu, 26 Oct 2023 08:36:42 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=flygoat.com; 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= 1698323800; x=1698410200; bh=7Ou35+gtN5wPlKenTK4K7QdEWtUFHuuBjgj vnmJt2so=; b=ReK/1CPMHcj4oFv60BrYhPVG1zqUf01nfY1GKPWcNf69ULCkO8o cnP8yuy5WQmqBeNLBdnx5BJVYm6Z2qRPNSb+q6QXW4R2/Wrq1GKORxLZsnMNIxxY 6PG1jsYaHi/9fj96JAPLCKrr9OoudlC6RYE6s3RFVPOJ7vN/X7LpY01xWgNevNa7 +7RYD67P1hdjp4srkFlYBgiob57FfonvLTXqde6ri0zooVr+EH/pEoQRfKos+1Ce Og+toa/ApC7M0jWyY7X6ONmOShWHFE+3zpvt2vINo9n4c3Tke38wISEe4yipxtDc aGrfNOqerCiyDzq34glZqQRkLneKPgek2Cg== 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=fm3; t= 1698323800; x=1698410200; bh=7Ou35+gtN5wPlKenTK4K7QdEWtUFHuuBjgj vnmJt2so=; b=loijJ83r4Let7U+9tN186djBs/D0jVuaFqRfByNiqGvgbW7yuIH IQ8/m6s50xNKY0IdAKlVGkIB/4qRjrxAEKKypCIPTL0w0PoqoXhUhj7PFx4dGFny bxj04yVY9TpU2p41660X7hqq6Fu4r2k3qIBdbfmlzEBMbE3PDa4QU8qzsanHZ4E5 JA0NuwKApPdV5RXu+3uAayL+yZPFPb81lpcu/yEwS6jLq61QXKobVB4YcMeWelWO kwCXkgkxMiawKTxtP+8jpKLB9o0IMpMULEX7pxPQFVDIbSXoAoBG1nwDzapRYOut Eafmz0T75gCZLMLT+WgcQEvCTpR+efc8Erw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrledvgdehfecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvvefutgfgsehtqhertderreejnecuhfhrohhmpedflfhi rgiguhhnucgjrghnghdfuceojhhirgiguhhnrdihrghnghesfhhlhihgohgrthdrtghomh eqnecuggftrfgrthhtvghrnhepudefgeeftedugeehffdtheefgfevffelfefghefhjeeu geevtefhudduvdeihefgnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrg hilhhfrhhomhepjhhirgiguhhnrdihrghnghesfhhlhihgohgrthdrtghomh X-ME-Proxy: Feedback-ID: ifd894703:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id 06B5A36A0075; Thu, 26 Oct 2023 08:36:39 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-1048-g9229b632c5-fm-20231019.001-g9229b632 MIME-Version: 1.0 Message-Id: In-Reply-To: <20230921110424.215592-1-bhe@redhat.com> References: <20230921110424.215592-1-bhe@redhat.com> Date: Thu, 26 Oct 2023 13:36:13 +0100 From: "Jiaxun Yang" To: "Baoquan He" , linux-kernel@vger.kernel.org Cc: linux-arch@vger.kernel.org, linux-mm@kvack.org, "Andrew Morton" , "Arnd Bergmann" , mpe@ellerman.id.au, "Geert Uytterhoeven" , "Luis Chamberlain" , hch@infradead.org, "Thomas Bogendoerfer" , "Florian Fainelli" , deller@gmx.de Subject: Re: [PATCH v5 0/4] 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: 12BEC2000B X-Rspam-User: X-Stat-Signature: du5jz1i7fcnbg6grs9gzj7doep7z3n87 X-Rspamd-Server: rspam03 X-HE-Tag: 1698323803-90898 X-HE-Meta: U2FsdGVkX1/126ZhewwITguXBXNaZkEg+R0vH26g/2hjq0yFmKjrxzbuLBwkuKpCavIAXTuUQE8VSvyNjsP3FIGtggIz+NEBATjVHWcihBRVVnVmrzlA2fKqjSP0WgRCfEaRz0NEsBxuivSMH5pKNBtoWepHbAHFcUOin/HpTay6MNw5TUomfRHJp53iGtQ7DqbmoHnQ/qN2Hrjqb+Bj4n62Jjk0O85nr90qNE5k+QbV/foeR2q/jtlWrw6f4reD1iPqWjVo9V1EVu2nQ1q9xANZGrXAuOcqJrpxx/tzP3TpHG3r3GhhsFPMtwvlhb69whb4Bc8uXrQ7NVcAfNu1FlxgA8WF09y2W5q2ES/ezGqn9phUZzKIB1xZxPgiA3vITaHurBh4qLiwBnljxEqia4+eSFWGzdDXl2bPi0DseZdcYKNJB4qlSsVxX1oKPNPbB8oR/KP61kZxpq/aJcPupReUwnrIDe981XovNBan0UmVC9PiCxrNIXBgXSRoltNIFE/ObU21LcztNdTHDNGdnd6dfNlo8r8/I1TyFMDAJiyQ6U/9f5wRchSZ12frOO132mWRX6zlLEkVbGETlF5oLpTczsRItYhnSrnUJBeK6/ej8Fk+z5X1OOGYcBPY+iV21XkAWO4vFltDtvV+gUVRKMvlrLBm4/4RGDOxV4iMcx3cPPhmk+823NAUOt2h1x2GgpTkyGkmxmrDV47HFe6LKTBkiJOhNJsxbR9NMmplC1aqjV9o9fi4hdQeBE+nvVHLN+zm4ZjvhYCB7/KSSZsaOG48/26aGCltMtGYki6/UBC7rcgM4trfak6P9kN5mVURYQjJu4zpz36t42j6Hut8n1Bdg4OuGCelJYu7+z6XXXhpvhZLKxr+SGCo9kcuv3nzImIDZ/m4fRa6qD68fjB9HOn4RjB2IJ7OSwQ2NbcqM9pr8zo4IGofhm6dCexogD7u+b//B4D9gy+wQLhbQmr v8hBG9hl JDKkt/73TXi9n0fLo+t3Fwvl7qUDngVZHMHPp0i6SSP4QX+QN4XrJE4iI9xFJxCyAMsyo/ySpi8ZGTXiARnK74WUHq7NB2u6cJtordXSzLmWlYWjJKjGv2GCGuLBvk4kmb+LuJaroiw7kfz5hSEFYBFqtH4bx3UAucr5AntvxOIC/QobI0U/z7PcqpRh9f8OHZ0lShyv7dNyzY0WCLiWiTCI5BxuYFJoEQoB81Vd1uWldvqlLUxOUsDOJBfM7Uskn7x9e1nT1O3+BiStM+rH4kJ0DdogRE3PUZbNYhS43HV1jpVaUwotkiGOoP0ucIMjPnDxKouj1jqlAiKFAeCyxYhv69ovqv5gX9TCl3Z6DOSefMbjUsm6vImPYTyg14jWgHup8MB0IfjyawUMHrA54kis2IA== 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: =E5=9C=A82023=E5=B9=B49=E6=9C=8821=E6=97=A5=E4=B9=9D=E6=9C=88 =E4=B8=8B=E5= =8D=8812:04=EF=BC=8CBaoquan He=E5=86=99=E9=81=93=EF=BC=9A > This patchset tries to remove ioremap_uc() in the current architectures > except of x86 and ia64. They will use the default ioremap_uc version > in which returns NULL. Anyone who wants to add new > invocation of ioremap_uc(), please consider using ioremap() instead or > adding a new ARCH specific ioremap_uc(), or refer to the callsite > in drivers/video/fbdev/aty/atyfb_base.c. > > This change won't cuase breakage to the current kernel because in the > only ioremap_uc callsite, an adjustment is made to eliminate impact in > patch 1 of this series. > > To get rid of all of them other than x86 and ia64, add asm-generic/io.h > to asm/io.h of mips ARCH. With this adding, we can get rid of the > ioremap_uc() in mips too. This is done in patch 2. And a followup patch > 4 is added to remove duplicated code according to Arnd's suggestion. > > Test: > =3D=3D=3D=3D=3D > Except of Jiaxun's efficient testing on patch 2/4, I also did cross co= mpiling > of this series on mips64, building passed. > For whole series: Tested-by: Jiaxun Yang Hi Arnd and Thomas, I've got some work pending based on this series, however I'm unclear abo= ut which tree this series will go since both of you give acked-by. Given that there are some tree-wide modifications, I guess it should go = into Arnd's asm-generic tree? Thanks --=20 - Jiaxun