Re: divide error in select_task_rq_fair()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Myron Stowe
Date: Thursday, November 11, 2010 - 11:28 am

On Fri, 2010-11-05 at 07:17 +0100, Eric Dumazet wrote:

I opened a BZ on this issue as it seems to be a regression -
https://bugzilla.kernel.org/show_bug.cgi?id=22662

I also, as indicated in the BZ, bisected the kernel which gave the
following results and reverting 50f2d7f682f9c0ed58191d0982fe77888d59d162
did re-enable booting on the box in question (an HP dl980g7).  Let me
know what further info you need or patches to test for debugging this.

Thanks,

commit 50f2d7f682f9c0ed58191d0982fe77888d59d162
Author: Nikanth Karthikesan <knikanth@suse.de>
Date:   Thu Sep 30 17:34:10 2010 +0530

    x86, numa: Assign CPUs to nodes in round-robin manner on fake NUMA

    commit d9c2d5ac6af87b4491bff107113aaf16f6c2b2d9 "x86, numa: Use near(er)
    online node instead of roundrobin for NUMA" changed NUMA initialization on
    Intel to choose the nearest online node or first node.  Fake NUMA would be
    better of with round-robin initialization, instead of the all CPUS on
    first node.  Change the choice of first node, back to round-robin.

    For testing NUMA kernel behaviour without cpusets and NUMA aware
    applications, it would be better to have cpus in different nodes, rather
    than all in a single node.  With cpusets migration of tasks scenarios
    cannot not be tested.

    I guess having it round-robin shouldn't affect the use cases for all cpus
    on the first node.

    The code comments in arch/x86/mm/numa_64.c:759 indicate that this used to
    be the case, which was changed by commit d9c2d5ac6.  It changed from
    roundrobin to nearer or first node.  And I couldn't find any reason for
    this change in its changelog.

    Signed-off-by: Nikanth Karthikesan <knikanth@suse.de>
    Cc: David Rientjes <rientjes@google.com>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>


-- 
Myron Stowe                             Linux Kernel Developer
Fort Collins, CO                        Office of Corporate Strategy and Technology

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
divide error in select_task_rq_fair(), Bjorn Helgaas, (Wed Nov 3, 9:12 pm)
Re: divide error in select_task_rq_fair(), Eric Dumazet, (Wed Nov 3, 10:19 pm)
Re: divide error in select_task_rq_fair(), Bjorn Helgaas, (Thu Nov 4, 7:28 am)
Re: divide error in select_task_rq_fair(), Eric Dumazet, (Thu Nov 4, 7:37 am)
Re: divide error in select_task_rq_fair(), Bjorn Helgaas, (Thu Nov 4, 7:00 pm)
Re: divide error in select_task_rq_fair(), Eric Dumazet, (Thu Nov 4, 11:17 pm)
Re: divide error in select_task_rq_fair(), Myron Stowe, (Thu Nov 11, 11:28 am)
Re: divide error in select_task_rq_fair(), Nikanth Karthikesan, (Thu Nov 11, 11:22 pm)
Re: divide error in select_task_rq_fair(), Myron Stowe, (Fri Nov 12, 7:06 am)
Re: divide error in select_task_rq_fair(), Yinghai Lu, (Sat Nov 13, 6:15 pm)
Re: divide error in select_task_rq_fair(), Myron Stowe, (Sun Nov 14, 10:36 am)
Re: divide error in select_task_rq_fair(), Yinghai Lu, (Sun Nov 14, 12:11 pm)
Re: divide error in select_task_rq_fair(), Myron Stowe, (Thu Nov 18, 4:32 pm)
Re: divide error in select_task_rq_fair(), Venkatesh Pallipadi, (Wed Dec 15, 3:09 pm)