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

Groups > linux.kernel > #1415841

[PATCH v4 02/14] of/numa: fix a memory@ node can only contains one memory block

Show all headers | View raw

For a normal memory@ devicetree node, its reg property can contains more
memory blocks.

Because we don't known how many memory blocks maybe contained, so we try
from index=0, increase 1 until error returned(the end).

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
---
 drivers/of/of_numa.c | 29 ++++++++++-------------------
 1 file changed, 10 insertions(+), 19 deletions(-)

diff --git a/drivers/of/of_numa.c b/drivers/of/of_numa.c
index fb71b4e..7b3fbdc 100644
--- a/drivers/of/of_numa.c
+++ b/drivers/of/of_numa.c
@@ -63,13 +63,9 @@ static int __init of_numa_parse_memory_nodes(void)
 	struct device_node *np = NULL;
 	struct resource rsrc;
 	u32 nid;
-	int r = 0;
-
-	for (;;) {
-		np = of_find_node_by_type(np, "memory");
-		if (!np)
-			break;
+	int i, r;

+	for_each_node_by_type(np, "memory") {
 		r = of_property_read_u32(np, "numa-node-id", &nid);
 		if (r == -EINVAL)
 			/*
@@ -78,23 +74,18 @@ static int __init of_numa_parse_memory_nodes(void)
 			 * "numa-node-id" property
 			 */
 			continue;
-		else if (r)
-			/* some other error */
-			break;

-		r = of_address_to_resource(np, 0, &rsrc);
-		if (r) {
-			pr_err("NUMA: bad reg property in memory node\n");
-			break;
-		}
+		for (i = 0; !r && !of_address_to_resource(np, i, &rsrc); i++)
+			r = numa_add_memblk(nid, rsrc.start, rsrc.end + 1);

-		r = numa_add_memblk(nid, rsrc.start, rsrc.end + 1);
-		if (r)
-			break;
+		if (!i || r) {
+			of_node_put(np);
+			pr_err("NUMA: bad property in memory node\n");
+			return r ? : -EINVAL;
+		}
 	}
-	of_node_put(np);

-	return r;
+	return 0;
 }

 static int __init of_numa_parse_distance_map_v1(struct device_node *map)
--
2.5.0

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v4 00/14] fix some type infos and bugs for arm64/of numa Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 06/14] of_numa: Use of_get_next_parent to simplify code Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
    Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES Ganapatrao Kulkarni <gpkulkarni@gmail.com> - 2016-06-07 10:40 +0200
      Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES "Leizhen (ThunderTown)" <thunder.leizhen@huawei.com> - 2016-06-07 15:00 +0200
        Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES Ganapatrao Kulkarni <gpkulkarni@gmail.com> - 2016-06-07 16:10 +0200
          Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES "Leizhen (ThunderTown)" <thunder.leizhen@huawei.com> - 2016-06-08 04:30 +0200
            Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES Ganapatrao Kulkarni <gpkulkarni@gmail.com> - 2016-06-08 06:50 +0200
              Re: [PATCH v4 11/14] arm64/numa: support HAVE_MEMORYLESS_NODES "Leizhen (ThunderTown)" <thunder.leizhen@huawei.com> - 2016-06-08 10:00 +0200
  [PATCH v4 08/14] arm64: numa: Use pr_fmt() Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 02/14] of/numa: fix a memory@ node can only contains one memory block Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 01/14] of/numa: remove a duplicated pr_debug information Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 05/14] arm64/numa: avoid inconsistent information to be printed Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 09/14] arm64/numa: support HAVE_SETUP_PER_CPU_AREA Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
  [PATCH v4 12/14] arm64/numa: remove some useless code Zhen Lei <thunder.leizhen@huawei.com> - 2016-06-07 10:20 +0200
    Re: [PATCH v4 12/14] arm64/numa: remove some useless code Ganapatrao Kulkarni <gpkulkarni@gmail.com> - 2016-06-07 10:30 +0200
      Re: [PATCH v4 12/14] arm64/numa: remove some useless code "Leizhen (ThunderTown)" <thunder.leizhen@huawei.com> - 2016-06-07 14:50 +0200
  Re: [PATCH v4 00/14] fix some type infos and bugs for arm64/of numa Will Deacon <will.deacon@arm.com> - 2016-06-07 16:00 +0200
    Re: [PATCH v4 00/14] fix some type infos and bugs for arm64/of numa "Leizhen (ThunderTown)" <thunder.leizhen@huawei.com> - 2016-06-08 11:10 +0200
  Re: [PATCH v4 00/14] fix some type infos and bugs for arm64/of numa Rob Herring <robh+dt@kernel.org> - 2016-06-07 22:40 +0200

csiph-web