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 B9D0DC6379F for ; Fri, 13 Jan 2023 22:19:06 +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=NnSYjIExYqw0QlVvHjw7ZrTNc0NW7eSq5AR0bpk6xsM=; b=rBtPyLi9blsyS8AnENfWGealq0 ch49TUNiFxn9Z/wACOmEuK9DQGZsV9hhMflKIAhBf/c/aO6y/9O+9Np3F+SPghr8RDc45lU4A7u9F RA+VdypOglBM1tXzqPL/4A9lIXJh3uGp6Xj5dm2vYj/1TFbC0kP0Wowgw94hEQZ1HZs8HOGoVtVXb QwVsC0dXtuKv/C/C8VyzdmeVklfuXG3CRKUmiAhbglcZUMqBNJBRF34sX5GeN5vv39oj2Eselqwn6 tETgsqW2dECtgrIWPSuAoEKV73eGylxGU3gTJLYZHsb5zQGMmLi401a/SWi9zsn7A4lWs5H5WHjXz 4+cIG/Ag==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pGSNA-004iMP-Qv; Fri, 13 Jan 2023 22:18:04 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pGSN7-004iLk-3J for linux-arm-kernel@lists.infradead.org; Fri, 13 Jan 2023 22:18:02 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 69BFF62373; Fri, 13 Jan 2023 22:18:00 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 72136C433D2; Fri, 13 Jan 2023 22:17:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1673648279; bh=HUKCBg28UCRcJ10s3S5AA+c32vWb4jb4VQKxfxFg5pA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=NU9Ugpcv8hxTU5wYnSXtG7lUtd5S4D9JXKOT4Z/MaEOTTUjyCVzhZzkSPBiaQpRJY KwiqGshxlID4cA/qLY64d2pEQbwJLTpzHnsZ2g6WtFEfYtUL/FmJVFnmkkHDIxXtmq 0xpEgGfq32R+NtyXMRrsvbvPL73HyHedfk79+/KMKP+ehXqUZh/l87f3e1boM5mRgO F52dzEaQof3SQMbpHJD8svtD4qInTOHBwSU8l3LtG80jKAwD+rml9gQ3CoF8dsUKUG UIjrRAVgzCkDzDMotOx85xa5v2SNbg3aGjvj4kk+uowJW3UFy/0E45RsjWWAv+tZip 55kYchFvQUOoQ== Date: Fri, 13 Jan 2023 22:17:56 +0000 From: Conor Dooley To: Robin Murphy Cc: joro@8bytes.org, will@kernel.org, iommu@lists.linux.dev, linux-arm-kernel@lists.infradead.org, palmer@dabbelt.com Subject: Re: [PATCH] iommu: Tidy up io-pgtable dependencies Message-ID: References: <51d8c78e2ecc6696ac5907526580209ea6da167f.1673553587.git.robin.murphy@arm.com> MIME-Version: 1.0 In-Reply-To: <51d8c78e2ecc6696ac5907526580209ea6da167f.1673553587.git.robin.murphy@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230113_141801_197572_777FB21D X-CRM114-Status: GOOD ( 20.62 ) X-BeenThere: linux-arm-kernel@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="===============8364991790245965240==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============8364991790245965240== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="/ncmK7LH7mvy4WLh" Content-Disposition: inline --/ncmK7LH7mvy4WLh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hey Robin, On Thu, Jan 12, 2023 at 07:59:47PM +0000, Robin Murphy wrote: > Some io-pgtable implementations, and thus their users too, carry a > slightly odd dependency to get around the GENERIC_ATOMIC64 version of > cmpxchg64() often failing to compile. Since this is a functional > dependency, it's a bit misleading and untidy to tie it explicitly to > COMPILE_TEST while assuming that it's also implied by the other > platform/architecture options. Make things clearer by separating these > functional dependencies into distinct statements from those controlling > visibility, and since they do look a bit non-obvious to the uninitiated, > also commenting them for good measure. >=20 > Signed-off-by: Robin Murphy > --- > drivers/iommu/Kconfig | 18 ++++++++++++------ > 1 file changed, 12 insertions(+), 6 deletions(-) > config IPMMU_VMSA > bool "Renesas VMSA-compatible IPMMU" > - depends on ARCH_RENESAS || (COMPILE_TEST && !GENERIC_ATOMIC64) > + depends on ARCH_RENESAS || COMPILE_TEST > + depends on !GENERIC_ATOMIC64 # for IOMMU_IO_PGTABLE_LPAE > select IOMMU_API > select IOMMU_IO_PGTABLE_LPAE > select ARM_DMA_USE_IOMMU This is a fix for the broken riscv32 allmodconfig stuff that Palmer reported, right? https://lore.kernel.org/all/20221214180409.7354-1-palmer@rivosinc.com/ I did a dfn:drivers/iommu/Kconfig search on lore & saw this, but AFAICT the patch was applied to next rather than fixes. Apologies if I'm off here, I have had that report in my follow-up-on queue for a while & since Christmas happened in between I've lost track of when the build failure was introduced. Last I remember, Guo Ren sent a patch that was not to your liking, but I didn't see anything after that. Thanks, Conor. --/ncmK7LH7mvy4WLh Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCY8HYkwAKCRB4tDGHoIJi 0uHBAQDCqnz8+NwoYXjQnu2dFi70IzS4ZZKyh11h2wX7/xzlhwEA5Q/7WsWubNPt n2PIPk+ZrxOn+AZuOEMu/+EkByxUaA4= =T15g -----END PGP SIGNATURE----- --/ncmK7LH7mvy4WLh-- --===============8364991790245965240== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============8364991790245965240==--