From patchwork Fri Oct 4 11:43:26 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan Cameron X-Patchwork-Id: 175214 Delivered-To: patch@linaro.org Received: by 2002:a92:7e96:0:0:0:0:0 with SMTP id q22csp580806ill; Fri, 4 Oct 2019 04:44:11 -0700 (PDT) X-Google-Smtp-Source: APXvYqyVWUVmou1P4PyZrHEVUJzRgD2fAbopxeHilTU4uSUUv4T6pnWnYJxuY5wGLESVlTANAbpn X-Received: by 2002:a05:6402:7c1:: with SMTP id u1mr14757501edy.198.1570189451646; Fri, 04 Oct 2019 04:44:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570189451; cv=none; d=google.com; s=arc-20160816; b=PvQTlsI2ikdLhYnlZUEB3Ko5yjokZXNkdjs3UYySftYe+NwehTTsKLb0eukO86yoMe SxSZkcaEbuQ1/jdKwnsQTI4DAahPA5rf6WQdsmTVzh+eumnNQ3e+fxbe2BIwAz/NVTmk +9RGfg7Vlm6JxCy4g0RTuEV2p7T4W6/ZMc06JHo68ejrV71wVd3eB4rUSmChoAhUmYa4 f7p5josTD+ywiQJykt2eUSEGP+psw5BfFVADurb9v+8+R3T+eeQ0Zj8aApmEkGWR4Gt6 6xUJW0ugDplnboQXGSEjyJWfRrivjwN9swpSJqS+KP3UH3SNc2JH9d3bjISN6G4hi67+ bCLQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=PK5Vugi9HwRie5v+3TNAVpnI+FDy0NvKE+6VhmlzUrU=; b=w9bWzCCMsWzUjqhR54OLhUCWqKlWu+NneY+Q6TOKKd75+lJB60lqwSBUFS7i41gwR4 64rLVw87BIrwp5UwCV2CbZe58qCzZan/RnnDm0Y295m59DjFrDgkMcFvrnPVoN1aKi/a +JMXymWZncBsg8HugzxN9wnD8SN5IjaA2+Kn2w3rF3Q+g+ErmiXuEdT0KX9iHnv10dXY lVpG8mM0/94wRkOjV+DGZlPQKmDpKfPs8R8Lq/A3KPkg4GAHgg/0vlUE4zqf5N09jPka K6/0Qie1lRhafDORpuhUCP4Xf9U5q9AKSMkUJz7Vk/6cYhlxMkoZOQA+NCQVDDrEOIfA 0oJg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-acpi-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-acpi-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id si6si2822248ejb.195.2019.10.04.04.44.11; Fri, 04 Oct 2019 04:44:11 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-acpi-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-acpi-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-acpi-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729548AbfJDLoH (ORCPT + 7 others); Fri, 4 Oct 2019 07:44:07 -0400 Received: from szxga05-in.huawei.com ([45.249.212.191]:3246 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728962AbfJDLoH (ORCPT ); Fri, 4 Oct 2019 07:44:07 -0400 Received: from DGGEMS401-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 05A2E588E3FAE8D78D9E; Fri, 4 Oct 2019 19:44:05 +0800 (CST) Received: from lhrphicprd00229.huawei.com (10.123.41.22) by DGGEMS401-HUB.china.huawei.com (10.3.19.201) with Microsoft SMTP Server id 14.3.439.0; Fri, 4 Oct 2019 19:43:57 +0800 From: Jonathan Cameron To: , , , , CC: Keith Busch , , "Rafael J . Wysocki" , , Andrew Morton , Dan Williams , Jonathan Cameron Subject: [PATCH V5 0/4] ACPI: Support Generic Initiator proximity domains Date: Fri, 4 Oct 2019 19:43:26 +0800 Message-ID: <20191004114330.104746-1-Jonathan.Cameron@huawei.com> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 X-Originating-IP: [10.123.41.22] X-CFilter-Loop: Reflected Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org Introduces a new type of NUMA node for cases where we want to represent the access characteristics of a non CPU initiator of memory requests, as these differ from all those for existing nodes containing CPUs and/or memory. This patch set has been sitting around for a long time (rebases only since v3 in April) without significant review. I would appreciate it very much if anyone has time to take a look. One outstanding question to highlight in this series is whether we should assume all ACPI supporting architectures support Generic Initiator domains, or whether to introduce an ARCH_HAS_GENERIC_INITIATOR_DOMAINS entry in Kconfig. Changes since V4: At Rafael's suggestion: Rebase on top of Dan William's Specific Purpose Memory series as that moves srat.c Original patches cherry-picked fine onto mmotm with Dan's patches applied. Applies to mmotm-2019-09-25 + https://lore.kernel.org/linux-acpi/156140036490.2951909.1837804994781523185.stgit@dwillia2-desk3.amr.corp.intel.com/ [PATCH v4 00/10] EFI Specific Purpose Memory Support (note there are some trivial conflicts to deal with when applying the SPM series). Change since V3. * Rebase. Changes since RFC V2. * RFC dropped as now we have x86 support, so the lack of guards in in the ACPI code etc should now be fine. * Added x86 support. Note this has only been tested on QEMU as I don't have a convenient x86 NUMA machine to play with. Note that this fitted together rather differently from arm64 so I'm particularly interested in feedback on the two solutions. Since RFC V1. * Fix incorrect interpretation of the ACPI entry noted by Keith Busch * Use the acpica headers definitions that are now in mmotm. It's worth noting that, to safely put a given device in a GI node, may require changes to the existing drivers as it's not unusual to assume you have local memory or processor core. There may be further constraints not yet covered by this patch. Original cover letter... ACPI 6.3 introduced a new entity that can be part of a NUMA proximity domain. It may share such a domain with the existing options (memory, CPU etc) but it may also exist on it's own. The intent is to allow the description of the NUMA properties (particularly via HMAT) of accelerators and other initiators of memory activity that are not the host processor running the operating system. This patch set introduces 'just enough' to make them work for arm64 and x86. It should be trivial to support other architectures, I just don't suitable NUMA systems readily available to test. There are a few quirks that need to be considered. 1. Fall back nodes ****************** As pre ACPI 6.3 supporting operating systems do not have Generic Initiator Proximity Domains it is possible to specify, via _PXM in DSDT that another device is part of such a GI only node. This currently blows up spectacularly. Whilst we can obviously 'now' protect against such a situation (see the related thread on PCI _PXM support and the threadripper board identified there as also falling into the problem of using non existent nodes https://patchwork.kernel.org/patch/10723311/ ), there is no way to be sure we will never have legacy OSes that are not protected against this. It would also be 'non ideal' to fallback to a default node as there may be a better (non GI) node to pick if GI nodes aren't available. The work around is that we also have a new system wide OSC bit that allows an operating system to 'announce' that it supports Generic Initiators. This allows, the firmware to us DSDT magic to 'move' devices between the nodes dependent on whether our new nodes are there or not. 2. New ways of assigning a proximity domain for devices ******************************************************* Until now, the only way firmware could indicate that a particular device (outside the 'special' set of cpus etc) was to be found in a particular Proximity Domain by the use of _PXM in DSDT. That is equally valid with GI domains, but we have new options. The SRAT affinity structure includes a handle (ACPI or PCI) to identify devices with the system and specify their proximity domain that way. If both _PXM and this are provided, they should give the same answer. For now this patch set completely ignores that feature as we don't need it to start the discussion. It will form a follow up set at some point (if no one else fancies doing it). Jonathan Cameron (4): ACPI: Support Generic Initiator only domains arm64: Support Generic Initiator only domains x86: Support Generic Initiator only proximity domains ACPI: Let ACPI know we support Generic Initiator Affinity Structures arch/arm64/kernel/smp.c | 8 +++++ arch/x86/include/asm/numa.h | 2 ++ arch/x86/kernel/setup.c | 1 + arch/x86/mm/numa.c | 14 ++++++++ drivers/acpi/bus.c | 1 + drivers/acpi/numa/srat.c | 62 +++++++++++++++++++++++++++++++++- drivers/base/node.c | 3 ++ include/asm-generic/topology.h | 3 ++ include/linux/acpi.h | 1 + include/linux/nodemask.h | 1 + include/linux/topology.h | 7 ++++ 11 files changed, 102 insertions(+), 1 deletion(-) -- 2.20.1