* Re: slub: fix panic with DISCONTIGMEM [not found] ` <alpine.DEB.2.00.1104191633250.23077@router.home> @ 2011-04-19 21:52 ` James Bottomley 2011-04-19 22:04 ` Christoph Lameter 0 siblings, 1 reply; 2+ messages in thread From: James Bottomley @ 2011-04-19 21:52 UTC (permalink / raw) To: Christoph Lameter; +Cc: linux-arch, linux-mm, hugh.dickins, Parisc List 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 majordomo@kvack.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:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: slub: fix panic with DISCONTIGMEM 2011-04-19 21:52 ` slub: fix panic with DISCONTIGMEM James Bottomley @ 2011-04-19 22:04 ` Christoph Lameter 0 siblings, 0 replies; 2+ messages in thread From: Christoph Lameter @ 2011-04-19 22:04 UTC (permalink / raw) To: James Bottomley; +Cc: linux-arch, linux-mm, hugh.dickins, Parisc List On Tue, 19 Apr 2011, James Bottomley wrote: > 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. F.e. hugepage support does the same thing. So it not slub specific. > 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. As far as I can tell: It will go boom even with other subsystems. I am surprised that we have never seen this before. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.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:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-04-19 22:04 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1303248576.11237.23.camel@mulgrave.site>
[not found] ` <alpine.DEB.2.00.1104191633250.23077@router.home>
2011-04-19 21:52 ` slub: fix panic with DISCONTIGMEM James Bottomley
2011-04-19 22:04 ` Christoph Lameter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox