lkml.org 
[lkml]   [2013]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [Part1 PATCH v5 10/22] x86, mm, numa: Move two functions calling on successful path later
Hello,

Does the subject match the patch content? What two functions? The
patch is separating out the actual registration part so that the
discovery part can happen earlier, right?

> Currently, parsing numa info needs to allocate some buffer and need to be
> called after init_mem_mapping. So try to split parsing numa info procedure
> into two steps:
> - The first step will be called before init_mem_mapping, and it
> should not need allocate buffers.

Document the requirement somewhere in the source code?

> - The second step will cantain all the buffer related code and be
> executed later.
>
> At last we will have early_initmem_init() and initmem_init().

Do you mean "eventually" or "in the end" by "at last"?

> This patch implements only the first step.
>
> setup_node_data() and numa_init_array() are only called for successful
> path, so we can move these two callings to x86_numa_init(). That will also
> make numa_init() smaller and more readable.

I find the description somewhat difficult to follow. :(

> -v2: remove online_node_map clear in numa_init(), as it is only
> set in setup_node_data() at last in successful path.

I don't get this. What prevents specific numa init functions (numaq,
x86_acpi, amd...) from updating node_online_map?

Thanks.

--
tejun


\
 
 \ /
  Last update: 2013-06-18 03:21    [W:0.567 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site