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


Groups > linux.kernel > #1694045 > unrolled thread

[PATCH 1/2] acpi: add missing include in acpi_numa.h

Started byRoss Zwisler <ross.zwisler@linux.intel.com>
First post2017-07-22 01:00 +0200
Last post2017-07-24 22:40 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/2] acpi: add missing include in acpi_numa.h Ross Zwisler <ross.zwisler@linux.intel.com> - 2017-07-22 01:00 +0200
    [PATCH 2/2] acpi: Fix typo in SRAT table name Ross Zwisler <ross.zwisler@linux.intel.com> - 2017-07-22 01:00 +0200
      Re: [PATCH 2/2] acpi: Fix typo in SRAT table name "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-07-24 22:40 +0200

#1694045 — [PATCH 1/2] acpi: add missing include in acpi_numa.h

FromRoss Zwisler <ross.zwisler@linux.intel.com>
Date2017-07-22 01:00 +0200
Subject[PATCH 1/2] acpi: add missing include in acpi_numa.h
Message-ID<u5P3Q-2gm-7@gated-at.bofh.it>
Right now if a file includes acpi_numa.h and they don't happen to include
linux/numa.h before it, they get the following warning:

./include/acpi/acpi_numa.h:9:5: warning: "MAX_NUMNODES" is not defined [-Wundef]
 #if MAX_NUMNODES > 256
     ^~~~~~~~~~~~

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

These two small fixes were previously part of my HMAT enabling series, but
I thought I'd submit them seperately so they don't get held up while I'm
iterating that series.

---
 include/acpi/acpi_numa.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/acpi/acpi_numa.h b/include/acpi/acpi_numa.h
index d4b7294..1e3a74f 100644
--- a/include/acpi/acpi_numa.h
+++ b/include/acpi/acpi_numa.h
@@ -3,6 +3,7 @@
 
 #ifdef CONFIG_ACPI_NUMA
 #include <linux/kernel.h>
+#include <linux/numa.h>
 
 /* Proximity bitmap length */
 #if MAX_NUMNODES > 256
-- 
2.9.4

[toc] | [next] | [standalone]


#1694046 — [PATCH 2/2] acpi: Fix typo in SRAT table name

FromRoss Zwisler <ross.zwisler@linux.intel.com>
Date2017-07-22 01:00 +0200
Subject[PATCH 2/2] acpi: Fix typo in SRAT table name
Message-ID<u5P3Q-2gm-15@gated-at.bofh.it>
In reply to#1694045
To save someone the time of searching the ACPI spec for "Static Resource
Affinity Table".

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
---
 drivers/acpi/numa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c
index edb0c79..917f1cc 100644
--- a/drivers/acpi/numa.c
+++ b/drivers/acpi/numa.c
@@ -443,7 +443,7 @@ int __init acpi_numa_init(void)
 	 * So go over all cpu entries in SRAT to get apicid to node mapping.
 	 */
 
-	/* SRAT: Static Resource Affinity Table */
+	/* SRAT: System Resource Affinity Table */
 	if (!acpi_table_parse(ACPI_SIG_SRAT, acpi_parse_srat)) {
 		struct acpi_subtable_proc srat_proc[3];
 
-- 
2.9.4

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


#1695074 — Re: [PATCH 2/2] acpi: Fix typo in SRAT table name

From"Rafael J. Wysocki" <rjw@rjwysocki.net>
Date2017-07-24 22:40 +0200
SubjectRe: [PATCH 2/2] acpi: Fix typo in SRAT table name
Message-ID<u6Sj0-1xi-5@gated-at.bofh.it>
In reply to#1694046
On Friday, July 21, 2017 04:51:24 PM Ross Zwisler wrote:
> To save someone the time of searching the ACPI spec for "Static Resource
> Affinity Table".
> 
> Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> ---
>  drivers/acpi/numa.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c
> index edb0c79..917f1cc 100644
> --- a/drivers/acpi/numa.c
> +++ b/drivers/acpi/numa.c
> @@ -443,7 +443,7 @@ int __init acpi_numa_init(void)
>  	 * So go over all cpu entries in SRAT to get apicid to node mapping.
>  	 */
>  
> -	/* SRAT: Static Resource Affinity Table */
> +	/* SRAT: System Resource Affinity Table */
>  	if (!acpi_table_parse(ACPI_SIG_SRAT, acpi_parse_srat)) {
>  		struct acpi_subtable_proc srat_proc[3];
>  
> 

Both this and the [1/2] applied, thanks!

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web