On Sat, Jul 22, 2017 at 8:53 AM, Hanjun Guo [off-list ref] wrote:

Hi Ganapat,

On 2017/6/8 12:44, Ganapatrao Kulkarni wrote:

quoted

Add code to parse proximity domain in SMMUv3 IORT table to
set numa node mapping for smmuv3 devices.

Signed-off-by: Ganapatrao Kulkarni <redacted>
---
 drivers/acpi/arm64/iort.c | 28 ++++++++++++++++++++++++++--
 1 file changed, 26 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/arm64/iort.c b/drivers/acpi/arm64/iort.c
index bba2b59..e804386 100644
--- a/drivers/acpi/arm64/iort.c
+++ b/drivers/acpi/arm64/iort.c

@@ -882,6 +882,23 @@ static bool __init arm_smmu_v3_is_coherent(struct acpi_iort_node *node)
      return smmu->flags & ACPI_IORT_SMMU_V3_COHACC_OVERRIDE;
 }

+/*
+ * 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));

                                                          ^^
Will have compile error in !CONFIG_NUMA, I think we need to introduce
a stub function in acpi_numa.h.

thanks Hanjun, i thought it is already there in header file, however,
there is stub for acpi_map_pxm_to_online_node and not for this
function.
it is better to have ifdef. will do it in next version.

Thanks
Hanjun

thanks
Ganapat