* [PATCH 00/11] Remove the parent_node() for each arch
@ 2017-07-26 13:34 Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (10 more replies)
0 siblings, 11 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang
Michael reports the parent_node() will never be invoked since the
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of it.
So we start removing it from the topology.h headers for each arch.
Dou Liyang (11):
arm64: numa: Remove the unused parent_node() macro
ia64: topology: Remove the unused parent_node() macro
metag/numa: Remove the unused parent_node() macro
MIPS: numa: Remove the unused parent_node() macro
powerpc/numa: Remove the unused parent_node() macro
s390/topology: Remove the unused parent_node() macro
sh/numa: Remove the unused parent_node() macro
sparc64/topology: Remove the unused parent_node() macro
tile/topology: Remove the unused parent_node() macro
x86/topology: Remove the unused parent_node() macro
asm-generic: numa: Remove the unused parent_node() macro
arch/arm64/include/asm/numa.h | 3 ---
arch/ia64/include/asm/topology.h | 7 -------
arch/metag/include/asm/topology.h | 1 -
arch/mips/include/asm/mach-ip27/topology.h | 1 -
arch/mips/include/asm/mach-loongson64/topology.h | 1 -
arch/powerpc/include/asm/topology.h | 2 --
arch/s390/include/asm/topology.h | 6 ------
arch/sh/include/asm/topology.h | 1 -
arch/sparc/include/asm/topology_64.h | 2 --
arch/tile/include/asm/topology.h | 6 ------
arch/x86/include/asm/topology.h | 6 ------
include/asm-generic/topology.h | 3 ---
12 files changed, 39 deletions(-)
--
2.5.5
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Will Deacon, linux-arm-kernel
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in ARM64 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Will Deacon <will.deacon@arm.com>
Cc: linux-arm-kernel@lists.infradead.org
---
arch/arm64/include/asm/numa.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
index bf466d1..ef7b238 100644
--- a/arch/arm64/include/asm/numa.h
+++ b/arch/arm64/include/asm/numa.h
@@ -7,9 +7,6 @@
#define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
-/* currently, arm64 implements flat NUMA topology */
-#define parent_node(node) (node)
-
int __node_distance(int from, int to);
#define node_distance(a, b) __node_distance(a, b)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-arm-kernel
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in ARM64 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Will Deacon <will.deacon@arm.com>
Cc: linux-arm-kernel at lists.infradead.org
---
arch/arm64/include/asm/numa.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
index bf466d1..ef7b238 100644
--- a/arch/arm64/include/asm/numa.h
+++ b/arch/arm64/include/asm/numa.h
@@ -7,9 +7,6 @@
#define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
-/* currently, arm64 implements flat NUMA topology */
-#define parent_node(node) (node)
-
int __node_distance(int from, int to);
#define node_distance(a, b) __node_distance(a, b)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 02/11] ia64: topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Tony Luck, Fenghua Yu, linux-ia64
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in IA64(Itanium) platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: linux-ia64@vger.kernel.org
---
arch/ia64/include/asm/topology.h | 7 -------
1 file changed, 7 deletions(-)
diff --git a/arch/ia64/include/asm/topology.h b/arch/ia64/include/asm/topology.h
index 3ad8f69..82f9bf7 100644
--- a/arch/ia64/include/asm/topology.h
+++ b/arch/ia64/include/asm/topology.h
@@ -34,13 +34,6 @@
&node_to_cpu_mask[node])
/*
- * Returns the number of the node containing Node 'nid'.
- * Not implemented here. Multi-level hierarchies detected with
- * the help of node_distance().
- */
-#define parent_node(nid) (nid)
-
-/*
* Determines the node for a given pci bus
*/
#define pcibus_to_node(bus) PCI_CONTROLLER(bus)->node
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 02/11] ia64: topology: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Tony Luck, Fenghua Yu, linux-ia64
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in IA64(Itanium) platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: linux-ia64@vger.kernel.org
---
arch/ia64/include/asm/topology.h | 7 -------
1 file changed, 7 deletions(-)
diff --git a/arch/ia64/include/asm/topology.h b/arch/ia64/include/asm/topology.h
index 3ad8f69..82f9bf7 100644
--- a/arch/ia64/include/asm/topology.h
+++ b/arch/ia64/include/asm/topology.h
@@ -34,13 +34,6 @@
&node_to_cpu_mask[node])
/*
- * Returns the number of the node containing Node 'nid'.
- * Not implemented here. Multi-level hierarchies detected with
- * the help of node_distance().
- */
-#define parent_node(nid) (nid)
-
-/*
* Determines the node for a given pci bus
*/
#define pcibus_to_node(bus) PCI_CONTROLLER(bus)->node
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 03/11] metag/numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, James Hogan, akpm, linux-metag
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in METAG architecture is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: James Hogan <james.hogan@imgtec.com>
Cc: akpm@linux-foundation.org
Cc: linux-metag@vger.kernel.org
---
arch/metag/include/asm/topology.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/metag/include/asm/topology.h b/arch/metag/include/asm/topology.h
index e95f874..707c7f7 100644
--- a/arch/metag/include/asm/topology.h
+++ b/arch/metag/include/asm/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) ((void)(cpu), 0)
-#define parent_node(node) ((void)(node), 0)
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 03/11] metag/numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: mpe-Gsx/Oe8HsFggBc27wqDAHg, Dou Liyang, James Hogan,
akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b,
linux-metag-u79uwXL29TY76Z2rM5mHXA
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in METAG architecture is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe-Gsx/Oe8HsFggBc27wqDAHg@public.gmane.org>
Signed-off-by: Dou Liyang <douly.fnst-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org>
Cc: James Hogan <james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>
Cc: akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org
Cc: linux-metag-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
---
arch/metag/include/asm/topology.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/metag/include/asm/topology.h b/arch/metag/include/asm/topology.h
index e95f874..707c7f7 100644
--- a/arch/metag/include/asm/topology.h
+++ b/arch/metag/include/asm/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) ((void)(cpu), 0)
-#define parent_node(node) ((void)(node), 0)
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 04/11] MIPS: numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Ralf Baechle, James Hogan, linux-mips
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macros in both IP27 and Loongson64 are unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: James Hogan <james.hogan@imgtec.com>
Cc: linux-mips@linux-mips.org
---
arch/mips/include/asm/mach-ip27/topology.h | 1 -
arch/mips/include/asm/mach-loongson64/topology.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/arch/mips/include/asm/mach-ip27/topology.h b/arch/mips/include/asm/mach-ip27/topology.h
index defd135..3fb7a0e 100644
--- a/arch/mips/include/asm/mach-ip27/topology.h
+++ b/arch/mips/include/asm/mach-ip27/topology.h
@@ -23,7 +23,6 @@ struct cpuinfo_ip27 {
extern struct cpuinfo_ip27 sn_cpu_info[NR_CPUS];
#define cpu_to_node(cpu) (sn_cpu_info[(cpu)].p_nodeid)
-#define parent_node(node) (node)
#define cpumask_of_node(node) ((node) == -1 ? \
cpu_all_mask : \
&hub_data(node)->h_cpus)
diff --git a/arch/mips/include/asm/mach-loongson64/topology.h b/arch/mips/include/asm/mach-loongson64/topology.h
index 0d8f3b5..bcb8856 100644
--- a/arch/mips/include/asm/mach-loongson64/topology.h
+++ b/arch/mips/include/asm/mach-loongson64/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) (cpu_logical_map(cpu) >> 2)
-#define parent_node(node) (node)
#define cpumask_of_node(node) (&__node_data[(node)]->cpumask)
struct pci_bus;
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 04/11] MIPS: numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Ralf Baechle, James Hogan, linux-mips
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macros in both IP27 and Loongson64 are unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: James Hogan <james.hogan@imgtec.com>
Cc: linux-mips@linux-mips.org
---
arch/mips/include/asm/mach-ip27/topology.h | 1 -
arch/mips/include/asm/mach-loongson64/topology.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/arch/mips/include/asm/mach-ip27/topology.h b/arch/mips/include/asm/mach-ip27/topology.h
index defd135..3fb7a0e 100644
--- a/arch/mips/include/asm/mach-ip27/topology.h
+++ b/arch/mips/include/asm/mach-ip27/topology.h
@@ -23,7 +23,6 @@ struct cpuinfo_ip27 {
extern struct cpuinfo_ip27 sn_cpu_info[NR_CPUS];
#define cpu_to_node(cpu) (sn_cpu_info[(cpu)].p_nodeid)
-#define parent_node(node) (node)
#define cpumask_of_node(node) ((node) == -1 ? \
cpu_all_mask : \
&hub_data(node)->h_cpus)
diff --git a/arch/mips/include/asm/mach-loongson64/topology.h b/arch/mips/include/asm/mach-loongson64/topology.h
index 0d8f3b5..bcb8856 100644
--- a/arch/mips/include/asm/mach-loongson64/topology.h
+++ b/arch/mips/include/asm/mach-loongson64/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) (cpu_logical_map(cpu) >> 2)
-#define parent_node(node) (node)
#define cpumask_of_node(node) (&__node_data[(node)]->cpumask)
struct pci_bus;
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 05/11] powerpc/topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
` (3 preceding siblings ...)
2017-07-26 13:34 ` Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-27 2:21 ` Michael Ellerman
2017-08-18 12:50 ` [05/11] " Michael Ellerman
2017-07-26 13:34 ` Dou Liyang
` (5 subsequent siblings)
10 siblings, 2 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel
Cc: mpe, Dou Liyang, Benjamin Herrenschmidt, Paul Mackerras, linuxppc-dev
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in POWERPC platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: linuxppc-dev@lists.ozlabs.org
---
arch/powerpc/include/asm/topology.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/powerpc/include/asm/topology.h b/arch/powerpc/include/asm/topology.h
index dc4e159..2d84bca 100644
--- a/arch/powerpc/include/asm/topology.h
+++ b/arch/powerpc/include/asm/topology.h
@@ -16,8 +16,6 @@ struct device_node;
#include <asm/mmzone.h>
-#define parent_node(node) (node)
-
#define cpumask_of_node(node) ((node) == -1 ? \
cpu_all_mask : \
node_to_cpumask_map[node])
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 06/11] s390/topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel
Cc: mpe, Dou Liyang, Martin Schwidefsky, Heiko Carstens,
Michael Holzheu, linux-s390
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in S390 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com>
Cc: linux-s390@vger.kernel.org
---
arch/s390/include/asm/topology.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/s390/include/asm/topology.h b/arch/s390/include/asm/topology.h
index fa1bfce..5222da1 100644
--- a/arch/s390/include/asm/topology.h
+++ b/arch/s390/include/asm/topology.h
@@ -77,12 +77,6 @@ static inline const struct cpumask *cpumask_of_node(int node)
return &node_to_cpumask_map[node];
}
-/*
- * Returns the number of the node containing node 'node'. This
- * architecture is flat, so it is a pretty simple function!
- */
-#define parent_node(node) (node)
-
#define pcibus_to_node(bus) __pcibus_to_node(bus)
#define node_distance(a, b) __node_distance(a, b)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 06/11] s390/topology: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel
Cc: mpe, Dou Liyang, Martin Schwidefsky, Heiko Carstens,
Michael Holzheu, linux-s390
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in S390 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com>
Cc: linux-s390@vger.kernel.org
---
arch/s390/include/asm/topology.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/s390/include/asm/topology.h b/arch/s390/include/asm/topology.h
index fa1bfce..5222da1 100644
--- a/arch/s390/include/asm/topology.h
+++ b/arch/s390/include/asm/topology.h
@@ -77,12 +77,6 @@ static inline const struct cpumask *cpumask_of_node(int node)
return &node_to_cpumask_map[node];
}
-/*
- * Returns the number of the node containing node 'node'. This
- * architecture is flat, so it is a pretty simple function!
- */
-#define parent_node(node) (node)
-
#define pcibus_to_node(bus) __pcibus_to_node(bus)
#define node_distance(a, b) __node_distance(a, b)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 07/11] sh/numa: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Yoshinori Sato, Rich Felker, linux-sh
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in SUPERH platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Rich Felker <dalias@libc.org>
Cc: linux-sh@vger.kernel.org
---
arch/sh/include/asm/topology.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/sh/include/asm/topology.h b/arch/sh/include/asm/topology.h
index 358e3f5..6931f50 100644
--- a/arch/sh/include/asm/topology.h
+++ b/arch/sh/include/asm/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) ((void)(cpu),0)
-#define parent_node(node) ((void)(node),0)
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 07/11] sh/numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Yoshinori Sato, Rich Felker, linux-sh
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in SUPERH platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Cc: Rich Felker <dalias@libc.org>
Cc: linux-sh@vger.kernel.org
---
arch/sh/include/asm/topology.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/sh/include/asm/topology.h b/arch/sh/include/asm/topology.h
index 358e3f5..6931f50 100644
--- a/arch/sh/include/asm/topology.h
+++ b/arch/sh/include/asm/topology.h
@@ -4,7 +4,6 @@
#ifdef CONFIG_NUMA
#define cpu_to_node(cpu) ((void)(cpu),0)
-#define parent_node(node) ((void)(node),0)
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, David S. Miller, sparclinux
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in SPARC64 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: sparclinux@vger.kernel.org
---
arch/sparc/include/asm/topology_64.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/sparc/include/asm/topology_64.h b/arch/sparc/include/asm/topology_64.h
index ad5293f..0fcc9a0 100644
--- a/arch/sparc/include/asm/topology_64.h
+++ b/arch/sparc/include/asm/topology_64.h
@@ -10,8 +10,6 @@ static inline int cpu_to_node(int cpu)
return numa_cpu_lookup_table[cpu];
}
-#define parent_node(node) (node)
-
#define cpumask_of_node(node) ((node) == -1 ? \
cpu_all_mask : \
&numa_cpumask_lookup_table[node])
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, David S. Miller, sparclinux
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in SPARC64 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: sparclinux@vger.kernel.org
---
arch/sparc/include/asm/topology_64.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/sparc/include/asm/topology_64.h b/arch/sparc/include/asm/topology_64.h
index ad5293f..0fcc9a0 100644
--- a/arch/sparc/include/asm/topology_64.h
+++ b/arch/sparc/include/asm/topology_64.h
@@ -10,8 +10,6 @@ static inline int cpu_to_node(int cpu)
return numa_cpu_lookup_table[cpu];
}
-#define parent_node(node) (node)
-
#define cpumask_of_node(node) ((node) = -1 ? \
cpu_all_mask : \
&numa_cpumask_lookup_table[node])
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 09/11] tile/topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
` (7 preceding siblings ...)
2017-07-26 13:34 ` Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 18:00 ` Chris Metcalf
2017-07-26 13:34 ` [PATCH 10/11] x86/topology: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
10 siblings, 1 reply; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Chris Metcalf
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in tile platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Chris Metcalf <cmetcalf@mellanox.com>
---
arch/tile/include/asm/topology.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/tile/include/asm/topology.h b/arch/tile/include/asm/topology.h
index b11d5fc..635a0a4 100644
--- a/arch/tile/include/asm/topology.h
+++ b/arch/tile/include/asm/topology.h
@@ -29,12 +29,6 @@ static inline int cpu_to_node(int cpu)
return cpu_2_node[cpu];
}
-/*
- * Returns the number of the node containing Node 'node'.
- * This architecture is flat, so it is a pretty simple function!
- */
-#define parent_node(node) (node)
-
/* Returns a bitmask of CPUs on Node 'node'. */
static inline const struct cpumask *cpumask_of_node(int node)
{
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 10/11] x86/topology: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
` (8 preceding siblings ...)
2017-07-26 13:34 ` [PATCH 09/11] tile/topology: " Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-27 8:58 ` [tip:sched/core] " tip-bot for Dou Liyang
2017-07-26 13:34 ` Dou Liyang
10 siblings, 1 reply; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel
Cc: mpe, Dou Liyang, Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in X86 platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: x86@kernel.org
---
arch/x86/include/asm/topology.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/x86/include/asm/topology.h b/arch/x86/include/asm/topology.h
index 6358a85..c1d2a98 100644
--- a/arch/x86/include/asm/topology.h
+++ b/arch/x86/include/asm/topology.h
@@ -75,12 +75,6 @@ static inline const struct cpumask *cpumask_of_node(int node)
extern void setup_node_to_cpumask_map(void);
-/*
- * Returns the number of the node containing Node 'node'. This
- * architecture is flat, so it is a pretty simple function!
- */
-#define parent_node(node) (node)
-
#define pcibus_to_node(bus) __pcibus_to_node(bus)
extern int __node_distance(int, int);
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 11/11] asm-generic: numa: Remove the unused parent_node() macro
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
@ 2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` Dou Liyang
` (9 subsequent siblings)
10 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Arnd Bergmann, linux-arch
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in generic situation is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arch@vger.kernel.org
---
include/asm-generic/topology.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/asm-generic/topology.h b/include/asm-generic/topology.h
index fc824e2..a91d842 100644
--- a/include/asm-generic/topology.h
+++ b/include/asm-generic/topology.h
@@ -44,9 +44,6 @@
#define cpu_to_mem(cpu) ((void)(cpu),0)
#endif
-#ifndef parent_node
-#define parent_node(node) ((void)(node),0)
-#endif
#ifndef cpumask_of_node
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
#endif
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* [PATCH 11/11] asm-generic: numa: Remove the unused parent_node() macro
@ 2017-07-26 13:34 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-07-26 13:34 UTC (permalink / raw)
To: linux-kernel; +Cc: mpe, Dou Liyang, Arnd Bergmann, linux-arch
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in generic situation is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arch@vger.kernel.org
---
include/asm-generic/topology.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/include/asm-generic/topology.h b/include/asm-generic/topology.h
index fc824e2..a91d842 100644
--- a/include/asm-generic/topology.h
+++ b/include/asm-generic/topology.h
@@ -44,9 +44,6 @@
#define cpu_to_mem(cpu) ((void)(cpu),0)
#endif
-#ifndef parent_node
-#define parent_node(node) ((void)(node),0)
-#endif
#ifndef cpumask_of_node
#define cpumask_of_node(node) ((void)node, cpu_online_mask)
#endif
--
2.5.5
^ permalink raw reply related [flat|nested] 37+ messages in thread
* Re: [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` Dou Liyang
@ 2017-07-26 16:46 ` David Miller
-1 siblings, 0 replies; 37+ messages in thread
From: David Miller @ 2017-07-26 16:46 UTC (permalink / raw)
To: douly.fnst; +Cc: linux-kernel, mpe, sparclinux
From: Dou Liyang <douly.fnst@cn.fujitsu.com>
Date: Wed, 26 Jul 2017 21:34:33 +0800
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in SPARC64 platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
@ 2017-07-26 16:46 ` David Miller
0 siblings, 0 replies; 37+ messages in thread
From: David Miller @ 2017-07-26 16:46 UTC (permalink / raw)
To: douly.fnst; +Cc: linux-kernel, mpe, sparclinux
From: Dou Liyang <douly.fnst@cn.fujitsu.com>
Date: Wed, 26 Jul 2017 21:34:33 +0800
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in SPARC64 platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 09/11] tile/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` [PATCH 09/11] tile/topology: " Dou Liyang
@ 2017-07-26 18:00 ` Chris Metcalf
0 siblings, 0 replies; 37+ messages in thread
From: Chris Metcalf @ 2017-07-26 18:00 UTC (permalink / raw)
To: Dou Liyang, linux-kernel; +Cc: mpe
On 7/26/2017 9:34 AM, Dou Liyang wrote:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in tile platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman<mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang<douly.fnst@cn.fujitsu.com>
> Cc: Chris Metcalf<cmetcalf@mellanox.com>
> ---
> arch/tile/include/asm/topology.h | 6 ------
> 1 file changed, 6 deletions(-)
Acked-by: Chris Metcalf <cmetcalf@mellanox.com>
--
Chris Metcalf, Mellanox Technologies
http://www.mellanox.com
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 05/11] powerpc/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` [PATCH 05/11] powerpc/topology: " Dou Liyang
@ 2017-07-27 2:21 ` Michael Ellerman
2017-07-27 2:49 ` Dou Liyang
2017-08-18 12:50 ` [05/11] " Michael Ellerman
1 sibling, 1 reply; 37+ messages in thread
From: Michael Ellerman @ 2017-07-27 2:21 UTC (permalink / raw)
To: Dou Liyang, linux-kernel
Cc: Dou Liyang, Benjamin Herrenschmidt, Paul Mackerras, linuxppc-dev
Dou Liyang <douly.fnst@cn.fujitsu.com> writes:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in POWERPC platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: Paul Mackerras <paulus@samba.org>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: linuxppc-dev@lists.ozlabs.org
> ---
> arch/powerpc/include/asm/topology.h | 2 --
> 1 file changed, 2 deletions(-)
Thanks for doing this series.
Acked-by: Michael Ellerman <mpe@ellerman.id.au>
cheers
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 05/11] powerpc/topology: Remove the unused parent_node() macro
2017-07-27 2:21 ` Michael Ellerman
@ 2017-07-27 2:49 ` Dou Liyang
2017-08-16 11:13 ` Michael Ellerman
0 siblings, 1 reply; 37+ messages in thread
From: Dou Liyang @ 2017-07-27 2:49 UTC (permalink / raw)
To: Michael Ellerman, linux-kernel
Cc: Benjamin Herrenschmidt, Paul Mackerras, linuxppc-dev
Hi Michael,
At 07/27/2017 10:21 AM, Michael Ellerman wrote:
> Dou Liyang <douly.fnst@cn.fujitsu.com> writes:
>
>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>> __register_one_node()") removes the last user of parent_node().
>>
>> The parent_node() macro in POWERPC platform is unnecessary.
>>
>> Remove it for cleanup.
>>
>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>> Cc: Paul Mackerras <paulus@samba.org>
>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>> Cc: linuxppc-dev@lists.ozlabs.org
>> ---
>> arch/powerpc/include/asm/topology.h | 2 --
>> 1 file changed, 2 deletions(-)
>
> Thanks for doing this series.
>
It's my pleasure. :)
Sincerely,
dou
> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
>
> cheers
>
>
>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 06/11] s390/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` Dou Liyang
(?)
@ 2017-07-27 8:19 ` Heiko Carstens
2017-08-16 12:56 ` Dou Liyang
-1 siblings, 1 reply; 37+ messages in thread
From: Heiko Carstens @ 2017-07-27 8:19 UTC (permalink / raw)
To: Dou Liyang
Cc: linux-kernel, mpe, Martin Schwidefsky, Michael Holzheu, linux-s390
On Wed, Jul 26, 2017 at 09:34:31PM +0800, Dou Liyang wrote:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in S390 platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
> Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com>
> Cc: linux-s390@vger.kernel.org
> ---
Are arch maintainers supposed to pick these patches up?
In any case: Acked-by: Heiko Carstens <heiko.carstens@de.ibm.com>
^ permalink raw reply [flat|nested] 37+ messages in thread
* [tip:sched/core] x86/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` [PATCH 10/11] x86/topology: " Dou Liyang
@ 2017-07-27 8:58 ` tip-bot for Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: tip-bot for Dou Liyang @ 2017-07-27 8:58 UTC (permalink / raw)
To: linux-tip-commits
Cc: mingo, douly.fnst, tglx, mpe, torvalds, hpa, peterz, linux-kernel
Commit-ID: dbe04493eddfaa89756ec9af8dde56206290182a
Gitweb: http://git.kernel.org/tip/dbe04493eddfaa89756ec9af8dde56206290182a
Author: Dou Liyang <douly.fnst@cn.fujitsu.com>
AuthorDate: Wed, 26 Jul 2017 21:34:35 +0800
Committer: Ingo Molnar <mingo@kernel.org>
CommitDate: Thu, 27 Jul 2017 10:53:00 +0200
x86/topology: Remove the unused parent_node() macro
Commit:
a7be6e5a7f8d ("mm: drop useless local parameters of __register_one_node()")
... removed the last user of parent_node(), so remove the macro.
Reported-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1501076076-1974-11-git-send-email-douly.fnst@cn.fujitsu.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
arch/x86/include/asm/topology.h | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/x86/include/asm/topology.h b/arch/x86/include/asm/topology.h
index 6358a85..c1d2a98 100644
--- a/arch/x86/include/asm/topology.h
+++ b/arch/x86/include/asm/topology.h
@@ -75,12 +75,6 @@ static inline const struct cpumask *cpumask_of_node(int node)
extern void setup_node_to_cpumask_map(void);
-/*
- * Returns the number of the node containing Node 'node'. This
- * architecture is flat, so it is a pretty simple function!
- */
-#define parent_node(node) (node)
-
#define pcibus_to_node(bus) __pcibus_to_node(bus)
extern int __node_distance(int, int);
^ permalink raw reply related [flat|nested] 37+ messages in thread
* Re: [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
2017-07-26 13:34 ` Dou Liyang
@ 2017-08-04 13:30 ` Will Deacon
-1 siblings, 0 replies; 37+ messages in thread
From: Will Deacon @ 2017-08-04 13:30 UTC (permalink / raw)
To: Dou Liyang; +Cc: linux-kernel, mpe, linux-arm-kernel, catalin.marinas
On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in ARM64 platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: linux-arm-kernel@lists.infradead.org
> ---
> arch/arm64/include/asm/numa.h | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
> index bf466d1..ef7b238 100644
> --- a/arch/arm64/include/asm/numa.h
> +++ b/arch/arm64/include/asm/numa.h
> @@ -7,9 +7,6 @@
>
> #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
>
> -/* currently, arm64 implements flat NUMA topology */
> -#define parent_node(node) (node)
> -
Acked-by: Will Deacon <will.deacon@arm.com>
Where you intending for this to go via the arm64 tree?
Will
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
@ 2017-08-04 13:30 ` Will Deacon
0 siblings, 0 replies; 37+ messages in thread
From: Will Deacon @ 2017-08-04 13:30 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in ARM64 platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: linux-arm-kernel at lists.infradead.org
> ---
> arch/arm64/include/asm/numa.h | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
> index bf466d1..ef7b238 100644
> --- a/arch/arm64/include/asm/numa.h
> +++ b/arch/arm64/include/asm/numa.h
> @@ -7,9 +7,6 @@
>
> #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
>
> -/* currently, arm64 implements flat NUMA topology */
> -#define parent_node(node) (node)
> -
Acked-by: Will Deacon <will.deacon@arm.com>
Where you intending for this to go via the arm64 tree?
Will
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
2017-08-04 13:30 ` Will Deacon
@ 2017-08-04 15:28 ` Dou Liyang
-1 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-08-04 15:28 UTC (permalink / raw)
To: Will Deacon; +Cc: linux-kernel, mpe, linux-arm-kernel, catalin.marinas
Hi will,
At 08/04/2017 09:30 PM, Will Deacon wrote:
> On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>> __register_one_node()") removes the last user of parent_node().
>>
>> The parent_node() macro in ARM64 platform is unnecessary.
>>
>> Remove it for cleanup.
>>
>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>> Cc: Will Deacon <will.deacon@arm.com>
>> Cc: linux-arm-kernel@lists.infradead.org
>> ---
>> arch/arm64/include/asm/numa.h | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>> diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
>> index bf466d1..ef7b238 100644
>> --- a/arch/arm64/include/asm/numa.h
>> +++ b/arch/arm64/include/asm/numa.h
>> @@ -7,9 +7,6 @@
>>
>> #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
>>
>> -/* currently, arm64 implements flat NUMA topology */
>> -#define parent_node(node) (node)
>> -
>
> Acked-by: Will Deacon <will.deacon@arm.com>
>
> Where you intending for this to go via the arm64 tree?
>
I intend for this to go to upstream tree. shall we?
Thanks
dou.
> Will
>
>
>
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
@ 2017-08-04 15:28 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-08-04 15:28 UTC (permalink / raw)
To: linux-arm-kernel
Hi will,
At 08/04/2017 09:30 PM, Will Deacon wrote:
> On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>> __register_one_node()") removes the last user of parent_node().
>>
>> The parent_node() macro in ARM64 platform is unnecessary.
>>
>> Remove it for cleanup.
>>
>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>> Cc: Will Deacon <will.deacon@arm.com>
>> Cc: linux-arm-kernel at lists.infradead.org
>> ---
>> arch/arm64/include/asm/numa.h | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>> diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
>> index bf466d1..ef7b238 100644
>> --- a/arch/arm64/include/asm/numa.h
>> +++ b/arch/arm64/include/asm/numa.h
>> @@ -7,9 +7,6 @@
>>
>> #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
>>
>> -/* currently, arm64 implements flat NUMA topology */
>> -#define parent_node(node) (node)
>> -
>
> Acked-by: Will Deacon <will.deacon@arm.com>
>
> Where you intending for this to go via the arm64 tree?
>
I intend for this to go to upstream tree. shall we?
Thanks
dou.
> Will
>
>
>
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
2017-08-04 15:28 ` Dou Liyang
@ 2017-08-15 13:38 ` Catalin Marinas
-1 siblings, 0 replies; 37+ messages in thread
From: Catalin Marinas @ 2017-08-15 13:38 UTC (permalink / raw)
To: Dou Liyang; +Cc: Will Deacon, mpe, linux-kernel, linux-arm-kernel
On Fri, Aug 04, 2017 at 11:28:36PM +0800, Dou Liyang wrote:
> Hi will,
>
> At 08/04/2017 09:30 PM, Will Deacon wrote:
> > On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
> > > Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> > > __register_one_node()") removes the last user of parent_node().
> > >
> > > The parent_node() macro in ARM64 platform is unnecessary.
> > >
> > > Remove it for cleanup.
> > >
> > > Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> > > Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> > > Cc: Michael Ellerman <mpe@ellerman.id.au>
> > > Cc: Will Deacon <will.deacon@arm.com>
> > > Cc: linux-arm-kernel@lists.infradead.org
> > > ---
> > > arch/arm64/include/asm/numa.h | 3 ---
> > > 1 file changed, 3 deletions(-)
> > >
> > > diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
> > > index bf466d1..ef7b238 100644
> > > --- a/arch/arm64/include/asm/numa.h
> > > +++ b/arch/arm64/include/asm/numa.h
> > > @@ -7,9 +7,6 @@
> > >
> > > #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
> > >
> > > -/* currently, arm64 implements flat NUMA topology */
> > > -#define parent_node(node) (node)
> > > -
> >
> > Acked-by: Will Deacon <will.deacon@arm.com>
> >
> > Where you intending for this to go via the arm64 tree?
>
> I intend for this to go to upstream tree. shall we?
Queued for 4.14. Thanks.
--
Catalin
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
@ 2017-08-15 13:38 ` Catalin Marinas
0 siblings, 0 replies; 37+ messages in thread
From: Catalin Marinas @ 2017-08-15 13:38 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Aug 04, 2017 at 11:28:36PM +0800, Dou Liyang wrote:
> Hi will,
>
> At 08/04/2017 09:30 PM, Will Deacon wrote:
> > On Wed, Jul 26, 2017 at 09:34:26PM +0800, Dou Liyang wrote:
> > > Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> > > __register_one_node()") removes the last user of parent_node().
> > >
> > > The parent_node() macro in ARM64 platform is unnecessary.
> > >
> > > Remove it for cleanup.
> > >
> > > Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> > > Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> > > Cc: Michael Ellerman <mpe@ellerman.id.au>
> > > Cc: Will Deacon <will.deacon@arm.com>
> > > Cc: linux-arm-kernel at lists.infradead.org
> > > ---
> > > arch/arm64/include/asm/numa.h | 3 ---
> > > 1 file changed, 3 deletions(-)
> > >
> > > diff --git a/arch/arm64/include/asm/numa.h b/arch/arm64/include/asm/numa.h
> > > index bf466d1..ef7b238 100644
> > > --- a/arch/arm64/include/asm/numa.h
> > > +++ b/arch/arm64/include/asm/numa.h
> > > @@ -7,9 +7,6 @@
> > >
> > > #define NR_NODE_MEMBLKS (MAX_NUMNODES * 2)
> > >
> > > -/* currently, arm64 implements flat NUMA topology */
> > > -#define parent_node(node) (node)
> > > -
> >
> > Acked-by: Will Deacon <will.deacon@arm.com>
> >
> > Where you intending for this to go via the arm64 tree?
>
> I intend for this to go to upstream tree. shall we?
Queued for 4.14. Thanks.
--
Catalin
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 05/11] powerpc/topology: Remove the unused parent_node() macro
2017-07-27 2:49 ` Dou Liyang
@ 2017-08-16 11:13 ` Michael Ellerman
0 siblings, 0 replies; 37+ messages in thread
From: Michael Ellerman @ 2017-08-16 11:13 UTC (permalink / raw)
To: Dou Liyang, linux-kernel
Cc: Benjamin Herrenschmidt, Paul Mackerras, linuxppc-dev
Dou Liyang <douly.fnst@cn.fujitsu.com> writes:
> Hi Michael,
>
> At 07/27/2017 10:21 AM, Michael Ellerman wrote:
>> Dou Liyang <douly.fnst@cn.fujitsu.com> writes:
>>
>>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>>> __register_one_node()") removes the last user of parent_node().
>>>
>>> The parent_node() macro in POWERPC platform is unnecessary.
>>>
>>> Remove it for cleanup.
>>>
>>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>>> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>>> Cc: Paul Mackerras <paulus@samba.org>
>>> Cc: Michael Ellerman <mpe@ellerman.id.au>
>>> Cc: linuxppc-dev@lists.ozlabs.org
>>> ---
>>> arch/powerpc/include/asm/topology.h | 2 --
>>> 1 file changed, 2 deletions(-)
>>
>> Thanks for doing this series.
>
> It's my pleasure. :)
Seems other arch maintainers are merging these patches individually, so
I grabbed this one and put it in the powerpc tree.
cheers
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 06/11] s390/topology: Remove the unused parent_node() macro
2017-07-27 8:19 ` Heiko Carstens
@ 2017-08-16 12:56 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-08-16 12:56 UTC (permalink / raw)
To: Heiko Carstens
Cc: linux-kernel, mpe, Martin Schwidefsky, Michael Holzheu, linux-s390
Hi Heiko,
At 07/27/2017 04:19 PM, Heiko Carstens wrote:
> On Wed, Jul 26, 2017 at 09:34:31PM +0800, Dou Liyang wrote:
>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>> __register_one_node()") removes the last user of parent_node().
>>
>> The parent_node() macro in S390 platform is unnecessary.
>>
>> Remove it for cleanup.
>>
>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
>> Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
>> Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com>
>> Cc: linux-s390@vger.kernel.org
>> ---
>
> Are arch maintainers supposed to pick these patches up?
>
Yes, As Michael said, It seems these patches are merging individually.
can you merge this patch in s390 tree?
> In any case: Acked-by: Heiko Carstens <heiko.carstens@de.ibm.com>
>
>
Thanks,
dou.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 06/11] s390/topology: Remove the unused parent_node() macro
@ 2017-08-16 12:56 ` Dou Liyang
0 siblings, 0 replies; 37+ messages in thread
From: Dou Liyang @ 2017-08-16 12:56 UTC (permalink / raw)
To: Heiko Carstens
Cc: linux-kernel, mpe, Martin Schwidefsky, Michael Holzheu, linux-s390
Hi Heiko,
At 07/27/2017 04:19 PM, Heiko Carstens wrote:
> On Wed, Jul 26, 2017 at 09:34:31PM +0800, Dou Liyang wrote:
>> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
>> __register_one_node()") removes the last user of parent_node().
>>
>> The parent_node() macro in S390 platform is unnecessary.
>>
>> Remove it for cleanup.
>>
>> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
>> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
>> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
>> Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
>> Cc: Michael Holzheu <holzheu@linux.vnet.ibm.com>
>> Cc: linux-s390@vger.kernel.org
>> ---
>
> Are arch maintainers supposed to pick these patches up?
>
Yes, As Michael said, It seems these patches are merging individually.
can you merge this patch in s390 tree?
> In any case: Acked-by: Heiko Carstens <heiko.carstens@de.ibm.com>
>
>
Thanks,
dou.
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [05/11] powerpc/topology: Remove the unused parent_node() macro
2017-07-26 13:34 ` [PATCH 05/11] powerpc/topology: " Dou Liyang
2017-07-27 2:21 ` Michael Ellerman
@ 2017-08-18 12:50 ` Michael Ellerman
1 sibling, 0 replies; 37+ messages in thread
From: Michael Ellerman @ 2017-08-18 12:50 UTC (permalink / raw)
To: Dou Liyang, linux-kernel; +Cc: linuxppc-dev, Paul Mackerras, Dou Liyang
On Wed, 2017-07-26 at 13:34:30 UTC, Dou Liyang wrote:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in POWERPC platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman <mpe@ellerman.id.au>
> Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: Paul Mackerras <paulus@samba.org>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: linuxppc-dev@lists.ozlabs.org
> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/7baebe54a64af44dc34d0bd51462f0
cheers
^ permalink raw reply [flat|nested] 37+ messages in thread
end of thread, other threads:[~2017-08-18 12:50 UTC | newest]
Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-26 13:34 [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang
2017-07-26 13:34 ` [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-08-04 13:30 ` Will Deacon
2017-08-04 13:30 ` Will Deacon
2017-08-04 15:28 ` Dou Liyang
2017-08-04 15:28 ` Dou Liyang
2017-08-15 13:38 ` Catalin Marinas
2017-08-15 13:38 ` Catalin Marinas
2017-07-26 13:34 ` [PATCH 02/11] ia64: topology: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` [PATCH 03/11] metag/numa: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` [PATCH 04/11] MIPS: numa: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` [PATCH 05/11] powerpc/topology: " Dou Liyang
2017-07-27 2:21 ` Michael Ellerman
2017-07-27 2:49 ` Dou Liyang
2017-08-16 11:13 ` Michael Ellerman
2017-08-18 12:50 ` [05/11] " Michael Ellerman
2017-07-26 13:34 ` [PATCH 06/11] s390/topology: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-27 8:19 ` Heiko Carstens
2017-08-16 12:56 ` Dou Liyang
2017-08-16 12:56 ` Dou Liyang
2017-07-26 13:34 ` [PATCH 07/11] sh/numa: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-26 13:34 ` [PATCH 08/11] sparc64/topology: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
2017-07-26 16:46 ` David Miller
2017-07-26 16:46 ` David Miller
2017-07-26 13:34 ` [PATCH 09/11] tile/topology: " Dou Liyang
2017-07-26 18:00 ` Chris Metcalf
2017-07-26 13:34 ` [PATCH 10/11] x86/topology: " Dou Liyang
2017-07-27 8:58 ` [tip:sched/core] " tip-bot for Dou Liyang
2017-07-26 13:34 ` [PATCH 11/11] asm-generic: numa: " Dou Liyang
2017-07-26 13:34 ` Dou Liyang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.