Re: [PATCH v11 08/10] dt, numa: Add NUMA dt binding implementation.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 03/01/2016 09:43 AM, Rob Herring wrote:
On Tue, Mar 1, 2016 at 10:57 AM, David Daney <ddaney@xxxxxxxxxxxxxxxxxx> wrote:
On 03/01/2016 08:47 AM, Rob Herring wrote:

On Thu, Feb 25, 2016 at 7:26 PM, David Daney <ddaney@xxxxxxxxxxxxxxxxxx>
wrote:

On 02/23/2016 11:36 AM, Rob Herring wrote:


On Fri, Feb 19, 2016 at 05:13:17PM -0800, David Daney wrote:


From: Ganapatrao Kulkarni <gkulkarni@xxxxxxxxxxxxxxxxxx>

ADD device tree node parsing for NUMA topology using device
"numa-node-id" property distance-map.



I still want an adequate explanation why NUMA setup cannot be done with
an unflattened tree. PowerPC manages to do that and should have a
similar init flow being memblock based, so I would expect arm64 can too.



Many things could be done.  Really, we want to know what *should* be
done.

In the context of the current arm64 memory initialization we (more or
less)
do:

   1) early_init_fdt_scan_reserved_mem();
   2) memory_present()
   3) sparse_init()
   4) other things
   5) unflatten_device_tree()

We are already reading information out of the FDT at #1.

This patch set adds a step between 1 and 2 where we read NUMA information
out of the FDT.


The dependency on unflattening is that memblock is up and we can
allocate a chunk from it. Isn't that dependency met by step 1


No.

Really, because it seems that numa_alloc_distance is essentially doing
a memblock alloc and that happens before memory_present.


or is
there a dependency on sparsemem (or something else)?


Will Deacon talked about this over here:

https://lkml.org/lkml/2016/2/26/782

I'm not saying to move memblock setup earlier nor before the MMU is
on, so I don't see how Will's reply is relevant other than PPC doesn't
serve as an example. Maybe PPC should be ignored because I think maybe
NUMA is only used on non-FDT systems.

In any case, no one has clearly explained what the dependencies are or
what happens if you moved the unflattening up sooner. You told me what
the current order is which doesn't equate to dependencies. For
example, step 4 may or may not be a dependency of step 5. These are
the dependencies I'm aware of:

memblock dependent on DT memory and reserved-memory parsing
unflattening dependent on memblock_alloc()
sparsemem dependent on NUMA parsing and memblock


I understand what you are saying.

Let me go back over the code looking to separate the issues of the inertia of the initial implementation, the need to cleanly support both EFI and non-EFI firmware, and your desire to unflatten the device tree much earlier than we currently do.

At this point, arm64 is the only user of the of_numa.c file. So, if you think it is not general purpose enough to live in drivers/of we could discuss the possibility of moving it under arch/arm64


David.


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux