Thread (31 messages) 31 messages, 6 authors, 2015-01-28

Re: mmotm 2015-01-22-15-04: qemu failure due to 'mm: memcontrol: remove unnecessary soft limit tree node test'

From: Christoph Lameter <hidden>
Date: 2015-01-23 20:20:03
Also in: linux-fsdevel, linux-mm, lkml

On Fri, 23 Jan 2015, Johannes Weiner wrote:
quoted hunk ↗ jump to hunk
        struct mem_cgroup_tree_per_node *rtpn;
        struct mem_cgroup_tree_per_zone *rtpz;
-       int tmp, node, zone;
+       int node, zone;

        for_each_node(node) {
Do for_each_online_node(node) {

instead?
-               tmp = node;
-               if (!node_state(node, N_NORMAL_MEMORY))
-                       tmp = -1;
-               rtpn = kzalloc_node(sizeof(*rtpn), GFP_KERNEL, tmp);
+               rtpn = kzalloc_node(sizeof(*rtpn), GFP_KERNEL, node);
                BUG_ON(!rtpn);

                soft_limit_tree.rb_tree_per_node[node] = rtpn;

--

Is the assumption of this patch wrong?  Does the specified node have
to be online for the fallback to work?

Thanks
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help