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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ADB31E7849A for ; Mon, 2 Oct 2023 13:11:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=HMvWrsiv599+2gmnojeBk/cNkxEJP95BYrOgtD5jOVI=; b=qheU3dtsnLUv/WQGwShY7s3N2j Q3PoHzKJHFsuJBVkRruriIMBC+6OFkuPfjqk6we0cLCwZMfhmBUx0VXo+vm/2MtFx+RBql3dKqPNA qfArZt3FoEv9gDafZOfUAFGjeZcrPcnfwB5vPN5sKAxee2TxxEG7wgjJPYNSlITgXcmFlJ9oWQlXG 6Bt4Cjw/GNMmIKZurVq8SZ80vXLHZVKVNvzrpSqjbpMAYmC70+QC0X1QcWe3dccJocBicvIDFBjGL Q73OsuGqV4zOJ9ezIjZ1oAkEdaxtUIoNty7ZGEq3E9UKBdZSJ3CuWVmvtNkuw0TCxZiisci4PcYxH oJOu8d+w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qnIhq-00ClSu-2j; Mon, 02 Oct 2023 13:11:26 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qnIhn-00ClS4-1D for linux-riscv@lists.infradead.org; Mon, 02 Oct 2023 13:11:25 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id A31BA60E75; Mon, 2 Oct 2023 13:11:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 44544C433C7; Mon, 2 Oct 2023 13:11:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1696252282; bh=NJY7DZhSmTcXGEORtrXsTz9AsMnIx8Ld5dQpPWVHHTs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=sUTDN8IBbyw1yGoDkuRrJf8V7yv33vpjVPwhVtN96T0/3voKKRl92Efejo5nx9Obq Mhzh8cxEm3qkUmuIAXET4FUe9yL4If0fr3oH0AC2zUrSofinreoEMz+aXwfLy3WfR/ dkFQx0tdhfhT/0kP0f3btwDIHQX95EBkVd3qxpZgMyWV8O9dDHwW4i1mGSIXYsR0m6 1X/q+0vaNKU5p5rjX4g8ztaPwusQk0PY/mNL6+bF0zFzfLrqTrhJA/dWnOtnKqr1BS A0fXosGkGhnfSSCh9XcsEeKNt3RttXgb8q3Rbla+paFW2V74XBmJ3T8CpyaULQhn0r RJbSkBobKD8GQ== Date: Mon, 2 Oct 2023 14:11:17 +0100 From: Conor Dooley To: Alexandre Ghiti Cc: Alexandre Ghiti , Paul Walmsley , Palmer Dabbelt , Albert Ou , Andrew Jones , Qinglin Pan , Ryan Roberts , linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH -fixes 2/2] riscv: Fix set_huge_pte_at() for NAPOT mappings when a swap entry is set Message-ID: <20231002-unending-ether-1b9f73eb642c@spud> References: <20230928151846.8229-1-alexghiti@rivosinc.com> <20230928151846.8229-3-alexghiti@rivosinc.com> <20230930-unnoticed-slacked-0bf5696cc265@spud> <96984fd2-2e3b-d861-5555-5b5a4e5c55d6@ghiti.fr> MIME-Version: 1.0 In-Reply-To: <96984fd2-2e3b-d861-5555-5b5a4e5c55d6@ghiti.fr> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231002_061123_498433_A73F9D66 X-CRM114-Status: GOOD ( 31.08 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============6589535421213891356==" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org --===============6589535421213891356== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="wAGaI66Xrr9VnCv2" Content-Disposition: inline --wAGaI66Xrr9VnCv2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 02, 2023 at 09:18:52AM +0200, Alexandre Ghiti wrote: > Hi Conor, >=20 > On 30/09/2023 11:14, Conor Dooley wrote: > > On Thu, Sep 28, 2023 at 05:18:46PM +0200, Alexandre Ghiti wrote: > > > We used to determine the number of page table entries to set for a NA= POT > > > hugepage by using the pte value which actually fails when the pte to = set is > > > a swap entry. > > >=20 > > > So take advantage of a recent fix for arm64 reported in [1] which > > > introduces the size of the mapping as an argument of set_huge_pte_at(= ): we > > > can then use this size to compute the number of page table entries to= set > > > for a NAPOT region. > > >=20 > > > Fixes: 82a1a1f3bfb6 ("riscv: mm: support Svnapot in hugetlb page") > > > Reported-by: Ryan Roberts > > > Closes: https://lore.kernel.org/linux-arm-kernel/20230922115804.20437= 71-1-ryan.roberts@arm.com/ [1] > > > Signed-off-by: Alexandre Ghiti > > Breaks the build. Your $subject marks this for -fixes, but this will not > > build there, as it relies on content that's not yet in that branch. > > AFAICT, you're going to have to resend this with akpm on CC, as the > > dependency is in his tree... >=20 >=20 > I see, but I still don't understand why -fixes does not point to the late= st > rcX instead of staying on rc1? It's up to Palmer what he does with his fixes branch, but two thoughts. Doing what you suggest would require rebasing things not yet sent to Linus every week and fast-forwarding when PRs are actually merged. IIRC, Palmer used to do something like the latter, but IIRC he got some complaints about that and switched to the current method. At the very least, you should point out dependencies like this, as I figure an individual patch could be applied on top of -rc4 and merged in. Both Palmer and I have submitted things for b4 to improve support for doing things exactly like this ;) > The patch which this series depends on just made it to rc4. However, if you do not mention what the deps for your patches are explicitly, how are people supposed to know? The reference to the dependency makes it look like a report for a similar problem that also applies to riscv, not a pre-requisite for the patch. Thanks, Conor. --wAGaI66Xrr9VnCv2 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZRrBdQAKCRB4tDGHoIJi 0ogrAQDVtpLukDHQPeeMY8fXo2JQPSwQj/fZN5+cBLUGdPsfNQEAjVH9QNHTm+QI 5G/qgyYuZHGA027PSue0EO8EO8ZEWws= =JpKR -----END PGP SIGNATURE----- --wAGaI66Xrr9VnCv2-- --===============6589535421213891356== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv --===============6589535421213891356==--