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=-5.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 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 ACCD6C433DB for ; Fri, 5 Feb 2021 18:56:59 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 48DC364E06 for ; Fri, 5 Feb 2021 18:56:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 48DC364E06 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=A6mCoxlPSkXUIKSvrUZCiUKRtU/GQTPlpBIoedWhht0=; b=L15h6doKxDwk7Xwo9mBkYcYB6 UXNyKTBtv4WQXkglMgEiOYhM9/yliOM7jouzcy72t7dCXmVGb76NuRlZHdIfWloPf1MFWEyVOitmz k9IeXE4az4I3dYFqDa28lwQxt0ajWFzexfcEk+xzGAj7/5fd/ZTflmunHU+xvNdYRatS2BZ9mDL6A AvCHtvVZJI+Ur89bwS9Uz58jMSIRDBRsfiOhMLtFO6FKSYWU8yDA6UaS+ajczLEhJxieSXsbkntXp K0y86CpsEQfriQS3xOQZiXbAhD+V8biTkfY2OqHyw8SXLSJ51HMJL9SKN8WJxnGCSBVP0fRJJ9z6e 24ai7ni+Q==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l86Gw-0007DG-VU; Fri, 05 Feb 2021 18:56:03 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1l86Gt-0007CC-Kj for linux-arm-kernel@lists.infradead.org; Fri, 05 Feb 2021 18:56:00 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 6FD4C64F19; Fri, 5 Feb 2021 18:55:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1612551358; bh=Dut+c1FVyHBMznczx7YPl/WFEcO5YCjuRBcQn/cce0o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=HIQ53G/rdyIEmK4O5Pwo+d7o20ZMKNsfvwD6vY3+oeW7Ok3RpP/ZH9udF/0EyxiAa SjnrxihAS62M2VaFx3DlhLvwLDhoMg6VD6xQnjWRZ/fb5JwYNvj7/t98zkcRLthvBe 3VuZzf4WM00pXfnbVr/AjfIZsbfmV+Z/GMZui1RwHZf0gV5pEyOasuftjk5tsjYufU fxMGNxa1BYmz5J4PgajSgCMtxPDF2HAqR6iDqG+BrzldnTxifqSgpUNwzXYtxYKOao 8KPhxvNGtvM2oHfebe5rRQohaqYoS+j/Fx8HvazL2Xf+gRTlNTZ794sw6Htljq1ant maJG6gpTixiaQ== Date: Fri, 5 Feb 2021 18:55:53 +0000 From: Will Deacon To: Anshuman Khandual Subject: Re: [PATCH V2 1/2] arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory Message-ID: <20210205185552.GA23216@willie-the-truck> References: <1612239114-28428-1-git-send-email-anshuman.khandual@arm.com> <1612239114-28428-2-git-send-email-anshuman.khandual@arm.com> <20210202123215.GA16868@willie-the-truck> <20210202123524.GB16868@willie-the-truck> <20210202125152.GC16868@willie-the-truck> <4d8f5156-8628-5531-1485-322ad92aa15c@redhat.com> <0e649f28-4d54-319d-f876-8a93870cda7f@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <0e649f28-4d54-319d-f876-8a93870cda7f@arm.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210205_135559_768635_2028E937 X-CRM114-Status: GOOD ( 21.06 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , David Hildenbrand , Catalin Marinas , linux-kernel@vger.kernel.org, Mike Rapoport , linux-mm@kvack.org, =?iso-8859-1?B?Suly9G1l?= Glisse , James Morse , Dan Williams , Robin Murphy , Ard Biesheuvel , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Feb 03, 2021 at 09:20:39AM +0530, Anshuman Khandual wrote: > On 2/2/21 6:26 PM, David Hildenbrand wrote: > > On 02.02.21 13:51, Will Deacon wrote: > >> On Tue, Feb 02, 2021 at 01:39:29PM +0100, David Hildenbrand wrote: > >>> As I expressed already, long term we should really get rid of the arm64 > >>> variant and rather special-case the generic one. Then we won't go out of > >>> sync - just as it happened with ZONE_DEVICE handling here. > >> > >> Why does this have to be long term? This ZONE_DEVICE stuff could be the > >> carrot on the stick :) > > > > Yes, I suggested to do it now, but Anshuman convinced me that doing a > > simple fix upfront might be cleaner --- for example when it comes to > > backporting :) > > Right. The current pfn_valid() breaks for ZONE_DEVICE memory and this fixes > the problem in the present context which can be easily backported if required. > > Changing or rather overhauling the generic code with new configs as proposed > earlier (which I am planning to work on subsequently) would definitely be an > improvement for the current pfn_valid() situation in terms of maintainability > but then it should not stop us from fixing the problem now. Alright, I've mulled this over a bit. I don't agree that this patch helps with maintainability (quite the opposite, in fact), but perfection is the enemy of the good so I'll queue the series for 5.12. However, I'll revert the changes at the first sign of a problem, so please do work towards a generic solution which can replace this in the medium term. Even just adding hooks with well-defined (documented) semantics to pfn_valid() instead of having architectures override it wholesale would be a massive step forward in my opinion. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel