* [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set
@ 2022-10-20 23:54 Dan Williams
2022-10-21 3:18 ` Verma, Vishal L
2022-10-21 17:56 ` Rafael J. Wysocki
0 siblings, 2 replies; 3+ messages in thread
From: Dan Williams @ 2022-10-20 23:54 UTC (permalink / raw)
To: linux-cxl
Cc: stable, Alison Schofield, Rafael J. Wysocki, Vishal Verma,
Vishal Verma, linux-acpi
The ACPI CEDT.CFMWS indicates a range of possible address where new CXL
regions can appear. Each range is associated with a QTG id (QoS
Throttling Group id). For each range + QTG pair that is not covered by a proximity
domain in the SRAT, Linux creates a new NUMA node. However, the commit
that added the new ranges missed updating the node_possible mask which
causes memory_group_register() to fail. Add the new nodes to the
nodes_possible mask.
Cc: <stable@vger.kernel.org>
Fixes: fd49f99c1809 ("ACPI: NUMA: Add a node and memblk for each CFMWS not in SRAT")
Cc: Alison Schofield <alison.schofield@intel.com>
Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reported-by: Vishal Verma <vishal.l.verma@intel.com>
Tested-by: Vishal Verma <vishal.l.verma@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
---
Rafael, I can take this through the CXL tree with some other pending
fixes.
drivers/acpi/numa/srat.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c
index 3b818ab186be..1f4fc5f8a819 100644
--- a/drivers/acpi/numa/srat.c
+++ b/drivers/acpi/numa/srat.c
@@ -327,6 +327,7 @@ static int __init acpi_parse_cfmws(union acpi_subtable_headers *header,
pr_warn("ACPI NUMA: Failed to add memblk for CFMWS node %d [mem %#llx-%#llx]\n",
node, start, end);
}
+ node_set(node, numa_nodes_parsed);
/* Set the next available fake_pxm value */
(*fake_pxm)++;
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set
2022-10-20 23:54 [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set Dan Williams
@ 2022-10-21 3:18 ` Verma, Vishal L
2022-10-21 17:56 ` Rafael J. Wysocki
1 sibling, 0 replies; 3+ messages in thread
From: Verma, Vishal L @ 2022-10-21 3:18 UTC (permalink / raw)
To: Williams, Dan J, linux-cxl
Cc: Schofield, Alison, stable, Wysocki, Rafael J, linux-acpi
On Thu, 2022-10-20 at 16:54 -0700, Dan Williams wrote:
> The ACPI CEDT.CFMWS indicates a range of possible address where new CXL
> regions can appear. Each range is associated with a QTG id (QoS
> Throttling Group id). For each range + QTG pair that is not covered by a proximity
> domain in the SRAT, Linux creates a new NUMA node. However, the commit
> that added the new ranges missed updating the node_possible mask which
> causes memory_group_register() to fail. Add the new nodes to the
> nodes_possible mask.
>
> Cc: <stable@vger.kernel.org>
> Fixes: fd49f99c1809 ("ACPI: NUMA: Add a node and memblk for each CFMWS not in SRAT")
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> Reported-by: Vishal Verma <vishal.l.verma@intel.com>
> Tested-by: Vishal Verma <vishal.l.verma@intel.com>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
> ---
> Rafael, I can take this through the CXL tree with some other pending
> fixes.
>
> drivers/acpi/numa/srat.c | 1 +
> 1 file changed, 1 insertion(+)
Looks good,
Reviewed-by: Vishal Verma <vishal.l.verma@intel.com>
>
> diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c
> index 3b818ab186be..1f4fc5f8a819 100644
> --- a/drivers/acpi/numa/srat.c
> +++ b/drivers/acpi/numa/srat.c
> @@ -327,6 +327,7 @@ static int __init acpi_parse_cfmws(union acpi_subtable_headers *header,
> pr_warn("ACPI NUMA: Failed to add memblk for CFMWS node %d [mem %#llx-%#llx]\n",
> node, start, end);
> }
> + node_set(node, numa_nodes_parsed);
>
> /* Set the next available fake_pxm value */
> (*fake_pxm)++;
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set
2022-10-20 23:54 [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set Dan Williams
2022-10-21 3:18 ` Verma, Vishal L
@ 2022-10-21 17:56 ` Rafael J. Wysocki
1 sibling, 0 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2022-10-21 17:56 UTC (permalink / raw)
To: Dan Williams
Cc: linux-cxl, stable, Alison Schofield, Rafael J. Wysocki,
Vishal Verma, linux-acpi
On Fri, Oct 21, 2022 at 1:55 AM Dan Williams <dan.j.williams@intel.com> wrote:
>
> The ACPI CEDT.CFMWS indicates a range of possible address where new CXL
> regions can appear. Each range is associated with a QTG id (QoS
> Throttling Group id). For each range + QTG pair that is not covered by a proximity
> domain in the SRAT, Linux creates a new NUMA node. However, the commit
> that added the new ranges missed updating the node_possible mask which
> causes memory_group_register() to fail. Add the new nodes to the
> nodes_possible mask.
>
> Cc: <stable@vger.kernel.org>
> Fixes: fd49f99c1809 ("ACPI: NUMA: Add a node and memblk for each CFMWS not in SRAT")
> Cc: Alison Schofield <alison.schofield@intel.com>
> Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> Reported-by: Vishal Verma <vishal.l.verma@intel.com>
> Tested-by: Vishal Verma <vishal.l.verma@intel.com>
> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
> ---
> Rafael, I can take this through the CXL tree with some other pending
> fixes.
Sure.
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> drivers/acpi/numa/srat.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c
> index 3b818ab186be..1f4fc5f8a819 100644
> --- a/drivers/acpi/numa/srat.c
> +++ b/drivers/acpi/numa/srat.c
> @@ -327,6 +327,7 @@ static int __init acpi_parse_cfmws(union acpi_subtable_headers *header,
> pr_warn("ACPI NUMA: Failed to add memblk for CFMWS node %d [mem %#llx-%#llx]\n",
> node, start, end);
> }
> + node_set(node, numa_nodes_parsed);
>
> /* Set the next available fake_pxm value */
> (*fake_pxm)++;
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-10-21 17:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-20 23:54 [PATCH] ACPI: NUMA: Add CXL CFMWS 'nodes' to the possible nodes set Dan Williams
2022-10-21 3:18 ` Verma, Vishal L
2022-10-21 17:56 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).