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

Groups > linux.kernel > #1697098 > unrolled thread

[PATCH 00/11] Remove the parent_node() for each arch

Started byDou Liyang <douly.fnst@cn.fujitsu.com>
First post2017-07-26 15:40 +0200
Last post2017-07-26 15:40 +0200
Articles 9 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 00/11] Remove the parent_node() for each arch Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
    [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
    [PATCH 10/11] x86/topology: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
      [tip:sched/core] x86/topology: Remove the unused parent_node()  macro tip-bot for Dou Liyang <tipbot@zytor.com> - 2017-07-27 11:10 +0200
    [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
      Re: [PATCH 08/11] sparc64/topology: Remove the unused  parent_node() macro David Miller <davem@davemloft.net> - 2017-07-26 18:50 +0200
    [PATCH 04/11] MIPS: numa: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
    [PATCH 03/11] metag/numa: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200
    [PATCH 07/11] sh/numa: Remove the unused parent_node() macro Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-07-26 15:40 +0200

#1697098 — [PATCH 00/11] Remove the parent_node() for each arch

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 00/11] Remove the parent_node() for each arch
Message-ID<u7uHD-SQ-7@gated-at.bofh.it>
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

[toc] | [next] | [standalone]

#1697099 — [PATCH 01/11] arm64: numa: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 01/11] arm64: numa: Remove the unused parent_node() macro
Message-ID<u7uHE-SQ-31@gated-at.bofh.it>
In reply to#1697098
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

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

#1697104 — [PATCH 10/11] x86/topology: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 10/11] x86/topology: Remove the unused parent_node() macro
Message-ID<u7uHF-SQ-49@gated-at.bofh.it>
In reply to#1697098
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

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

#1697826 — [tip:sched/core] x86/topology: Remove the unused parent_node() macro

Fromtip-bot for Dou Liyang <tipbot@zytor.com>
Date2017-07-27 11:10 +0200
Subject[tip:sched/core] x86/topology: Remove the unused parent_node() macro
Message-ID<u7MXV-4d2-23@gated-at.bofh.it>
In reply to#1697104
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);

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

#1697105 — [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
Message-ID<u7uHF-SQ-51@gated-at.bofh.it>
In reply to#1697098
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

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

#1697355 — Re: [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro

FromDavid Miller <davem@davemloft.net>
Date2017-07-26 18:50 +0200
SubjectRe: [PATCH 08/11] sparc64/topology: Remove the unused parent_node() macro
Message-ID<u7xFx-2JL-27@gated-at.bofh.it>
In reply to#1697105
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>

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

#1697106 — [PATCH 04/11] MIPS: numa: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 04/11] MIPS: numa: Remove the unused parent_node() macro
Message-ID<u7uHF-SQ-45@gated-at.bofh.it>
In reply to#1697098
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

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

#1697107 — [PATCH 03/11] metag/numa: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 03/11] metag/numa: Remove the unused parent_node() macro
Message-ID<u7uHF-SQ-53@gated-at.bofh.it>
In reply to#1697098
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

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

#1697108 — [PATCH 07/11] sh/numa: Remove the unused parent_node() macro

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-07-26 15:40 +0200
Subject[PATCH 07/11] sh/numa: Remove the unused parent_node() macro
Message-ID<u7uHF-SQ-47@gated-at.bofh.it>
In reply to#1697098
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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web