Received-SPF: pass (google.com: domain of linux-kernel+bounces-49063-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Date: Fri, 2 Feb 2024 09:02:54 +0800
From: Yan Zhao <yan.y.zhao@intel.com>
To: Arnd Bergmann <arnd@arndb.de>
CC: Linus Walleij <linus.walleij@linaro.org>, guoren <guoren@kernel.org>,
	Brian Cain <bcain@quicinc.com>, Jonas Bonn <jonas@southpole.se>, "Stefan
 Kristiansson" <stefan.kristiansson@saunalahti.fi>, Stafford Horne
	<shorne@gmail.com>, Linux-Arch <linux-arch@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, "linux-csky@vger.kernel.org"
	<linux-csky@vger.kernel.org>, <linux-hexagon@vger.kernel.org>,
	"linux-openrisc@vger.kernel.org" <linux-openrisc@vger.kernel.org>
Subject: Re: [PATCH 0/4] apply page shift to PFN instead of VA in pfn_to_virt
Message-ID: <Zbw/PpNkCQCbXPdP@yzhao56-desk.sh.intel.com>
Reply-To: Yan Zhao <yan.y.zhao@intel.com>
References: <20240131055159.2506-1-yan.y.zhao@intel.com>
 <5e55b5c0-6c8d-45b4-ac04-cf694bcb08d3@app.fastmail.com>
 <ZbrfcTaiuu2gaa2A@yzhao56-desk.sh.intel.com>
 <9f27c23b-ea8b-443c-b09c-03ecaa210cd5@app.fastmail.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <9f27c23b-ea8b-443c-b09c-03ecaa210cd5@app.fastmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?3Oi18poG+Y3jLaKIHy6TEFDs0ekb220EztpBIy7FRmbPa1OIAY3OKsmMFP9f?=
 =?us-ascii?Q?HM4V9794irivqQTyj5XirSdJ3y+XjrY49yW2oU84pDzEm73xdtGZ8DJz3BBk?=
 =?us-ascii?Q?rwTcxGIkyopWiWpkMddtTAWhRk6eb44pyxU/ibla9Wa0bDyEaAEC1FEJhJb/?=
 =?us-ascii?Q?nU0lM/hn2mnJGyieoxT3ACFjopUur1YODF1Owxbb3cSJ6XQGdPGzvfIgWdIh?=
 =?us-ascii?Q?2wmHJeMxOcKMsXycJxlXVPqWDPQ/f8KorKGV0FGn1nPjHLMlahjnXRS/gErw?=
 =?us-ascii?Q?qXjKqSuw3IJ4y3NLP5RDfOoDV0MqfWwj9TU1flG63aY5jKxZGgsI78G3ufFf?=
 =?us-ascii?Q?urJJnnB/FKEJH31/kiZdWZ4qYSWhtZ19QGSHtEZ8T4JcnFIagr2DBFtx6LAs?=
 =?us-ascii?Q?yKFyIpdHIoHt7zIxMwN9e5uU4jTMJ7nQn0MT1yn9Omp4QNkdnqtnDMag9r9F?=
 =?us-ascii?Q?57hitURELsmJaq7affLFgDA9x1i5arMkc9xmYvWRKfmFcczyZqEGqSwUMYyP?=
 =?us-ascii?Q?3thziOCiQghcxMyxfYN+25KfPF0Y5ciVli/d86yo815tIj6/sAvxafQNVlZx?=
 =?us-ascii?Q?qhPrH6bsJ/hIAAVJt9p0jOF+3e4VYwcrke9WHzzXKezCvdmESMRH/QMh9JPF?=
 =?us-ascii?Q?BIga4OV9VUcwto3JvoK0Jgiib9pfieusftKWaGFIufqoa4p7mdEDMDKOb18E?=
 =?us-ascii?Q?8X7vyRQTxG5xKSTt2+d9m86MUdoVy9rQfF5S96oVtOJfaGNyiNglAwW0bC/p?=
 =?us-ascii?Q?jszOsbZHWCH4cbIX6TwHEzqKSbFVg02bRCb53q6ZrhFBKHkFdaeu7dLA+DfS?=
 =?us-ascii?Q?eIL+7HEh7zui6e5Pw+iunpeDzp3IhmwG42xqGMNapknlgPBnmdjyagS3FL/h?=
 =?us-ascii?Q?B8paiy9O0Nyz3pkB0gUcUyx7SAPbhbpKplvwim4HHk3qner8zShI82CLZRyb?=
 =?us-ascii?Q?5pX28XrdayEtt5HsrHP8wNpWVSarGpYLKSiI+bOyMlFXDJH51g/lrH35ETRP?=
 =?us-ascii?Q?ylrR1qXij7nEsN2r9ZUivHnefdN5ot61MSRZUHYxX750/1obsEweI2akFxzC?=
 =?us-ascii?Q?uZESY9m7iytx8Kg2D1PRAvyDQB7NJDyv1r0jHL3caKHMjIgZCYOcvOB6vRLB?=
 =?us-ascii?Q?wDtVO/bQVEIR5d/Qa3ZmgKCaS56Ivub7EYd9hzixG6BxQU11Tpv7VaJ9iq8P?=
 =?us-ascii?Q?IZKR3KbTvRj9oJYXASMtMd/Y+KHu898s5CXYpeA9poHchgkSiRFnKogqlm3c?=
 =?us-ascii?Q?8TDWBRZhD8vZBKDEe99Y+LCJ/SUCCrzFAFAHUfbpupfcDu7e1l/69OLp0mYb?=
 =?us-ascii?Q?H7IRTuksf8skbfCRRHpLEzvLVXLYbUR75aUc8RsEf1LyWYnnF3lka+2AiUh+?=
 =?us-ascii?Q?C5BavqKvr5ju65OKDPEZeOELH3qrjgd0DFl2eHR0qlpJmKTQIOLilq3C4EKN?=
 =?us-ascii?Q?Z2Fa5utPHDd0JCD5V06nuvuy6zHk+//XtlBYkNVlvSLASSBmRS+1IOv5Cj6G?=
 =?us-ascii?Q?IUNB5xarbjjw1F/xYAPfsNe/Q2mB7w8fBgfbTsXLS+ZFKKg8ALbL3kJUA74r?=
 =?us-ascii?Q?jr0nEi3THmlwgG9gjlJ89xK+4EwLs5noNhAOBAmf?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 77e3cd15-2eb7-43d6-4f9e-08dc238ecfe8
X-MS-Exchange-CrossTenant-AuthSource: DS7PR11MB5966.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Feb 2024 01:32:26.1261
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 05cStZuAIsWuYWeC1UawUBTwi+dhB/pDXWnb/5+kEjwA4We5tKZq7NEdghcNIMFMbMldtqQmMUbwbAKDlygUBg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5257
X-OriginatorOrg: intel.com

On Thu, Feb 01, 2024 at 06:46:46AM +0100, Arnd Bergmann wrote:
> On Thu, Feb 1, 2024, at 01:01, Yan Zhao wrote:
> > On Wed, Jan 31, 2024 at 12:48:38PM +0100, Arnd Bergmann wrote:
> >> On Wed, Jan 31, 2024, at 06:51, Yan Zhao wrote:
> >> 
> >> How exactly did you notice the function being wrong,
> >> did you try to add a user somewhere, or just read through
> >> the code?
> > I came across them when I was debugging an unexpected kernel page fault
> > on x86, and I was not sure whether page_to_virt() was compiled in
> > asm-generic/page.h or linux/mm.h.
> > Though finally, it turned out that the one in linux/mm.h was used, which
> > yielded the right result and the unexpected kernel page fault in my case
> > was not related to page_to_virt(), it did lead me to noticing that the
> > pfn_to_virt() in asm-generic/page.h and other 3 archs did not look right.
> >
> > Yes, unlike virt_to_pfn() which still has a caller in openrisc (among
> > csky, Hexagon, openrisc), pfn_to_virt() now does not have a caller in
> > the 3 archs. Though both virt_to_pfn() and pfn_to_virt() are referenced
> > in asm-generic/page.h, I also not sure if we need to remove the
> > asm-generic/page.h which may serve as a template to future archs ?
> >
> > So, either way looks good to me :)
> 
> I think it's fair to assume we won't need asm-generic/page.h any
> more, as we likely won't be adding new NOMMU architectures.
> I can have a look myself at removing any such unused headers in
> include/asm-generic/, it's probably not the only one.
> 
> Can you just send a patch to remove the unused pfn_to_virt()
> functions?
Ok. I'll do it!
BTW: do you think it's also good to keep this fixing series though we'll
remove the unused function later?
So if people want to revert the removal some day, they can get right one.

Or maybe I'm just paranoid, and explanation about the fix in the commit
message of patch for function removal is enough.

What's your preference? :)