|From:||James Bottomley <James.Bottomley-AT-HansenPartnership.com>|
|To:||Christoph Lameter <cl-AT-linux.com>|
|Subject:||Re: slub: fix panic with DISCONTIGMEM|
|Date:||Tue, 19 Apr 2011 16:52:50 -0500|
|Cc:||linux-arch-AT-vger.kernel.org, linux-mm-AT-kvack.org, "hugh.dickins-AT-tiscali.co.uk" <hugh.dickins-AT-tiscali.co.uk>, Parisc List <linux-parisc-AT-vger.kernel.org>|
On Tue, 2011-04-19 at 16:38 -0500, Christoph Lameter wrote: > On Tue, 19 Apr 2011, James Bottomley wrote: > > > Slub makes assumptions about page_to_nid() which are violated by > > DISCONTIGMEM and !NUMA. This violation results in a panic because > > Fix this by stating correctly by saying "The kernel makes assumptions in > various subsystems ..." Slub is a subset of the kernel, so the original wording is a bit more precise. > > page_to_nid() can be non-zero for pages in the discontiguous ranges and > > this leads to a null return by get_node(). The assertion by the > > maintainer is that DISCONTIGMEM should only be allowed when NUMA is also > > defined. However, at least six architectures: alpha, ia64, m32r, m68k, > > That is not what I said. DISCONTIG support needs to be fixed so that the > core subsystems using page_to_nid() will operate correctly with a !NUMA > discontig configuration. Core will expect page_to_nid() to only return 0 > on !NUMA. Well, we can discuss how to proceed going forwards. The current fact is that any prior kernel that enables SLUB with DISCONTIGMEM and !NUMA will eventually go boom when the page allocator returns a page not in the first pfn array. That has to be fixed in -stable. I don't really think a DISCONTIGMEM re-engineering effort would be the best thing for the -stable series. James -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to firstname.lastname@example.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"email@example.com"> firstname.lastname@example.org </a>
Copyright © 2011, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds