From patchwork Wed Aug 2 16:21:19 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lorenzo Pieralisi X-Patchwork-Id: 9877017 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id C433A6035F for ; Wed, 2 Aug 2017 16:19:58 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id AE008287EF for ; Wed, 2 Aug 2017 16:19:58 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id A2484287F5; Wed, 2 Aug 2017 16:19:58 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 0BB60287EF for ; Wed, 2 Aug 2017 16:19:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.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=D7DHYPhhW1vzJWwAKlp8BgLT6ENcO4Mr2jRsbeaGQaA=; b=FbMH0i8uAcYbve TqYEe/Df8wBu6Gm7IUYHlfTEsgOO/jRm3/ND5b7oRtC5mbWX8T9MA1VraeK4a2LTc/wcabH2UZCAA vvgoDcpT63gcq1ZtgVY3hS7pWg2rX0hBpHwcT1750E0yWuP+0KuITI1rlkjgvMjYOQL7imWc96a67 J0/b4uwvz6g6+8N22V0a3cbwZRBnL3kSh7DMAVjdCGP0TcYuwGdvK/jZAkxID5y10U0QXkFsrQQnD 8A++qCD8zBrXUIT12y+IadG6IVqsIw5amM0RWDrEtI8zarMSvWeDCo6lNTS5apYev0f87ksDRamX6 QkRj4st4gUCGMBqMqSEQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dcwND-0007sB-TW; Wed, 02 Aug 2017 16:19:51 +0000 Received: from foss.arm.com ([217.140.101.70]) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dcwNA-0007r9-8h for linux-arm-kernel@lists.infradead.org; Wed, 02 Aug 2017 16:19:50 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 8430D1596; Wed, 2 Aug 2017 09:19:27 -0700 (PDT) Received: from red-moon (red-moon.cambridge.arm.com [10.1.206.55]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A44173F577; Wed, 2 Aug 2017 09:19:24 -0700 (PDT) Date: Wed, 2 Aug 2017 17:21:19 +0100 From: Lorenzo Pieralisi To: Robert Richter Subject: Re: [PATCH v4] acpi/iort: numa: Add numa node mapping for smmuv3 devices Message-ID: <20170802162119.GA3984@red-moon> References: <20170725050237.11033-1-ganapatrao.kulkarni@cavium.com> <20170725151305.GU17961@rric.localdomain> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170725151305.GU17961@rric.localdomain> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170802_091948_361785_B0D15BFC X-CRM114-Status: GOOD ( 28.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: rjw@rjwysocki.net, joro@8bytes.org, sudeep.holla@arm.com, Will.Deacon@arm.com, linux-kernel@vger.kernel.org, gklkml16@gmail.com, jnair@caviumnetworks.com, linux-acpi@vger.kernel.org, iommu@lists.linux-foundation.org, hanjun.guo@linaro.org, Ganapatrao Kulkarni , lenb@kernel.org, robin.murphy@arm.com, linux-arm-kernel@lists.infradead.org, lv.zheng@intel.com Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP On Tue, Jul 25, 2017 at 05:13:05PM +0200, Robert Richter wrote: > On 25.07.17 10:32:37, Ganapatrao Kulkarni wrote: > > ARM IORT specification(rev. C) has added provision to define proximity > > domain in SMMUv3 IORT table. Adding required code to parse Proximity > > domain and set numa_node of smmv3 platform devices. > > > > Add code to parse proximity domain in SMMUv3 IORT table to > > set numa node mapping for smmuv3 devices. > > > > Signed-off-by: Ganapatrao Kulkarni > > --- > > > > This patch has dependency on header file patch [1], which is > > already merged to linux-pm. > > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/?h=linux-next&id=c944230064eb65e4fa018d86779b4fd200b1d7e7 > > > > v4: > > - Fix compilation issue in !CONFIG_NUMA > > > > v3: > > - Addressed Lorenzo Pieralisi comment. > > > > v2: > > - Changed as per Lorenzo Pieralisi and Hanjun Guo suggestions. > > > > v1: > > - Initial patch > > > > drivers/acpi/arm64/iort.c | 33 +++++++++++++++++++++++++++++++-- > > 1 file changed, 31 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/acpi/arm64/iort.c b/drivers/acpi/arm64/iort.c > > index a3215ee..c5c82c3 100644 > > --- a/drivers/acpi/arm64/iort.c > > +++ b/drivers/acpi/arm64/iort.c > > @@ -908,6 +908,28 @@ static bool __init arm_smmu_v3_is_coherent(struct acpi_iort_node *node) > > return smmu->flags & ACPI_IORT_SMMU_V3_COHACC_OVERRIDE; > > } > > > > +#if defined(CONFIG_ACPI_NUMA) && (ACPI_CA_VERSION >= 0x20170629) > > This acpica version check was very unhandy for backports when we did > this last time. If we backport just c944230064 and this single patch > it wont work as the version check prevents the code from being > enabled. Isn't there another way to handle cross tree dependencies of > pull requests? E.g. we could base this patch on acpica (ab539eaa50) > and remove the check. > > Alternatively we could do in this case: > > #if defined(CONFIG_ACPI_NUMA) && defined(ACPI_IORT_SMMU_V3_PXM_VALID) > > > +/* > > + * set numa proximity domain for smmuv3 device > > + */ > > +static void __init arm_smmu_v3_set_proximity(struct acpi_iort_node *node, > > + struct device *dev) > > +{ > > + struct acpi_iort_smmu_v3 *smmu; > > + > > + smmu = (struct acpi_iort_smmu_v3 *)node->node_data; > > + if (smmu->flags & ACPI_IORT_SMMU_V3_PXM_VALID) { > > + set_dev_node(dev, acpi_map_pxm_to_node(smmu->pxm)); > > + pr_info("SMMUV3[%llx] Mapped to Proximity domain %d\n", > > Better use dev_info() here. > > > + smmu->base_address, > > + smmu->pxm); > > + } > > +} > > +#else > > +static void __init arm_smmu_v3_set_proximity(struct acpi_iort_node *node, > > + struct device *dev) { } > > Just do: > > #define arm_smmu_v3_set_proximity NULL > > > +#endif > > + > > static int __init arm_smmu_count_resources(struct acpi_iort_node *node) > > { > > struct acpi_iort_smmu *smmu; > > @@ -977,20 +999,24 @@ struct iort_iommu_config { > > int (*iommu_count_resources)(struct acpi_iort_node *node); > > void (*iommu_init_resources)(struct resource *res, > > struct acpi_iort_node *node); > > + void (*iommu_set_proximity)(struct acpi_iort_node *node, > > + struct device *dev); > > Wrt pseudo-OO programming this is more a method for struct dev than > node, so swap args: iommu_set_proximity(dev, node) > > > }; > > > > static const struct iort_iommu_config iort_arm_smmu_v3_cfg __initconst = { > > .name = "arm-smmu-v3", > > .iommu_is_coherent = arm_smmu_v3_is_coherent, > > .iommu_count_resources = arm_smmu_v3_count_resources, > > - .iommu_init_resources = arm_smmu_v3_init_resources > > + .iommu_init_resources = arm_smmu_v3_init_resources, > > + .iommu_set_proximity = arm_smmu_v3_set_proximity > > Add comma at the end here. > > > }; > > > > static const struct iort_iommu_config iort_arm_smmu_cfg __initconst = { > > .name = "arm-smmu", > > .iommu_is_coherent = arm_smmu_is_coherent, > > .iommu_count_resources = arm_smmu_count_resources, > > - .iommu_init_resources = arm_smmu_init_resources > > + .iommu_init_resources = arm_smmu_init_resources, > > + .iommu_set_proximity = NULL > > No need for null assignment. Patch that I will send upstream below, please check, thanks. -- >8 -- Subject: [PATCH] ACPI/IORT: numa: Add numa node mapping for smmuv3 devices ARM IORT specification(rev. C) has added provision to define proximity domain in SMMUv3 IORT table. Adding required code to parse Proximity domain and set numa_node of smmv3 platform devices. Add code to parse proximity domain in SMMUv3 IORT table to set numa node mapping for smmuv3 devices. Link: http://lkml.kernel.org/r/20170706112017.GA16981@rric.localdomain Signed-off-by: Ganapatrao Kulkarni [lorenzo.pieralisi@arm.com: updated review comments] Signed-off-by: Lorenzo Pieralisi --- drivers/acpi/arm64/iort.c | 29 ++++++++++++++++++++++++++++- 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/drivers/acpi/arm64/iort.c b/drivers/acpi/arm64/iort.c index a3215ee..35dd803 100644 --- a/drivers/acpi/arm64/iort.c +++ b/drivers/acpi/arm64/iort.c @@ -908,6 +908,27 @@ static bool __init arm_smmu_v3_is_coherent(struct acpi_iort_node *node) return smmu->flags & ACPI_IORT_SMMU_V3_COHACC_OVERRIDE; } +#if defined(CONFIG_ACPI_NUMA) && defined(ACPI_IORT_SMMU_v3_PXM_VALID) +/* + * set numa proximity domain for smmuv3 device + */ +static void __init arm_smmu_v3_set_proximity(struct device *dev, + struct acpi_iort_node *node) +{ + struct acpi_iort_smmu_v3 *smmu; + + smmu = (struct acpi_iort_smmu_v3 *)node->node_data; + if (smmu->flags & ACPI_IORT_SMMU_V3_PXM_VALID) { + set_dev_node(dev, acpi_map_pxm_to_node(smmu->pxm)); + dev_info(dev, "SMMUV3[%llx] Mapped to Proximity domain %d\n", + smmu->base_address, + smmu->pxm); + } +} +#else +#define arm_smmu_v3_set_proximity NULL +#endif + static int __init arm_smmu_count_resources(struct acpi_iort_node *node) { struct acpi_iort_smmu *smmu; @@ -977,13 +998,16 @@ struct iort_iommu_config { int (*iommu_count_resources)(struct acpi_iort_node *node); void (*iommu_init_resources)(struct resource *res, struct acpi_iort_node *node); + void (*iommu_set_proximity)(struct device *dev, + struct acpi_iort_node *node); }; static const struct iort_iommu_config iort_arm_smmu_v3_cfg __initconst = { .name = "arm-smmu-v3", .iommu_is_coherent = arm_smmu_v3_is_coherent, .iommu_count_resources = arm_smmu_v3_count_resources, - .iommu_init_resources = arm_smmu_v3_init_resources + .iommu_init_resources = arm_smmu_v3_init_resources, + .iommu_set_proximity = arm_smmu_v3_set_proximity, }; static const struct iort_iommu_config iort_arm_smmu_cfg __initconst = { @@ -1028,6 +1052,9 @@ static int __init iort_add_smmu_platform_device(struct acpi_iort_node *node) if (!pdev) return -ENOMEM; + if (ops->iommu_set_proximity) + ops->iommu_set_proximity(&pdev->dev, node); + count = ops->iommu_count_resources(node); r = kcalloc(count, sizeof(*r), GFP_KERNEL);