Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1216952 > unrolled thread

[PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT

Started byLinda Knippers <linda.knippers@hp.com>
First post2015-09-01 18:30 +0200
Last post2015-09-01 21:30 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT Linda Knippers <linda.knippers@hp.com> - 2015-09-01 18:30 +0200
    Re: [PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT Thomas Gleixner <tglx@linutronix.de> - 2015-09-01 21:20 +0200
      Re: [PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT Thomas Gleixner <tglx@linutronix.de> - 2015-09-01 21:30 +0200
      Re: [PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT Linda Knippers <linda.knippers@hp.com> - 2015-09-01 21:30 +0200

#1216952 — [PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT

FromLinda Knippers <linda.knippers@hp.com>
Date2015-09-01 18:30 +0200
Subject[PATCH] arch/x86/mm/srat: Print non-volatile flag in SRAT
Message-ID<q3WBA-3os-13@gated-at.bofh.it>
Nobody checks this flag but it would be interesting to know if it's being
set on any platforms.

Signed-off-by: Linda Knippers <linda.knippers@hp.com>
---
 arch/x86/mm/srat.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/x86/mm/srat.c b/arch/x86/mm/srat.c
index 66338a6..c2aea63 100644
--- a/arch/x86/mm/srat.c
+++ b/arch/x86/mm/srat.c
@@ -192,10 +192,11 @@ acpi_numa_memory_affinity_init(struct acpi_srat_mem_affinity *ma)
 
 	node_set(node, numa_nodes_parsed);
 
-	pr_info("SRAT: Node %u PXM %u [mem %#010Lx-%#010Lx]%s\n",
+	pr_info("SRAT: Node %u PXM %u [mem %#010Lx-%#010Lx]%s%s\n",
 		node, pxm,
 		(unsigned long long) start, (unsigned long long) end - 1,
-		hotpluggable ? " hotplug" : "");
+		hotpluggable ? " hotplug" : "",
+		ma->flags & ACPI_SRAT_MEM_NON_VOLATILE ? " non-volatile" : "");
 
 	/* Mark hotplug range in memblock. */
 	if (hotpluggable && memblock_mark_hotplug(start, ma->length))
-- 
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1217031

FromThomas Gleixner <tglx@linutronix.de>
Date2015-09-01 21:20 +0200
Message-ID<q3Zg5-7e7-1@gated-at.bofh.it>
In reply to#1216952
On Tue, 1 Sep 2015, Linda Knippers wrote:
> Nobody checks this flag but it would be interesting to know if it's being
> set on any platforms.

What you're omitting to explain, is WHY it would be interesting.

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1217036

FromThomas Gleixner <tglx@linutronix.de>
Date2015-09-01 21:30 +0200
Message-ID<q3ZpM-7px-9@gated-at.bofh.it>
In reply to#1217031
On Tue, 1 Sep 2015, Linda Knippers wrote:
> On 9/1/2015 3:17 PM, Thomas Gleixner wrote:
> > On Tue, 1 Sep 2015, Linda Knippers wrote:
> >> Nobody checks this flag but it would be interesting to know if it's being
> >> set on any platforms.
> > 
> > What you're omitting to explain, is WHY it would be interesting.
> 
> With the addition of NVDIMM support, a question came up as to whether
> NVDIMM ranges should be in the SRAT with this bit set.  I think the
> consensus was no because the ranges are in the NFIT with proximity
> domain information there.
> 
> ACPI is not clear on the meaning of this bit in the SRAT.
> If someone is setting it, we might want to ask them what they expect
> to happen with it.
> 
> Right now this bit is only printed if all the ACPI debug information is
> turned on.
> 
> Sorry, I should have explained more.

Indeed. And that explanation wants to be in the changelog. Care to resend?

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1217040

FromLinda Knippers <linda.knippers@hp.com>
Date2015-09-01 21:30 +0200
Message-ID<q3ZpM-7px-11@gated-at.bofh.it>
In reply to#1217031
On 9/1/2015 3:17 PM, Thomas Gleixner wrote:
> On Tue, 1 Sep 2015, Linda Knippers wrote:
>> Nobody checks this flag but it would be interesting to know if it's being
>> set on any platforms.
> 
> What you're omitting to explain, is WHY it would be interesting.

With the addition of NVDIMM support, a question came up as to whether
NVDIMM ranges should be in the SRAT with this bit set.  I think the
consensus was no because the ranges are in the NFIT with proximity
domain information there.

ACPI is not clear on the meaning of this bit in the SRAT.
If someone is setting it, we might want to ask them what they expect
to happen with it.

Right now this bit is only printed if all the ACPI debug information is
turned on.

Sorry, I should have explained more.

-- ljk

> 
> Thanks,
> 
> 	tglx
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web