Re: [Xen-devel] [PATCH] x86/NUMA: fix setup_node()

2015-08-31 Thread Wei Liu
On Fri, Aug 28, 2015 at 07:58:46AM -0600, Jan Beulich wrote: > The function referenced an __initdata object (nodes_found). Since this > being a node mask was more complicated than needed, the variable gets > replaced by a simple counter. Check at once that the count of nodes > doesn't go beyond MAX

Re: [Xen-devel] [PATCH] x86/NUMA: fix setup_node()

2015-08-28 Thread Andrew Cooper
On 28/08/15 14:58, Jan Beulich wrote: > The function referenced an __initdata object (nodes_found). Since this > being a node mask was more complicated than needed, the variable gets > replaced by a simple counter. Check at once that the count of nodes > doesn't go beyond MAX_NUMNODES. > > Also con

[Xen-devel] [PATCH] x86/NUMA: fix setup_node()

2015-08-28 Thread Jan Beulich
The function referenced an __initdata object (nodes_found). Since this being a node mask was more complicated than needed, the variable gets replaced by a simple counter. Check at once that the count of nodes doesn't go beyond MAX_NUMNODES. Also consolidate four printk()s related to the function's