![]() | |||||||||||||
Messages in this thread |
On Wed, 4 Feb 2004, Keith Mannthey wrote:
>
> Martin sent me a patch that fixed the X panics (NUMA and DISCONTIG
> enabled). (Thanks Martin!) I don't have the same X panics and issues I
> had before. I don't know if this will work for the generic case. It
> compiles with a simple memory situation just fine but I didn't boot it.
Looks ok, but the thing should be made a function (possibly inline,
depending on how big the code generated ends up being). As it is, it now
uses its arguments several times, and while I don't see anything where
that could screw up, it's just a tad scary.
Also, related to this whole mess, what the _heck_ is this in mm/rmap.c:
if (!pfn_valid(page_to_pfn(page)) || PageReserved(page))
return pte_chain;
that "pfn_valid(page_to_pfn(page))" just looks totally nonsensical. Can
somebody really pass in random page pointers to this thing, and if so, are
they guaranteed to be "not-random enough" to not cause bogus behaviour
when the "page_to_pfn()" happens to be valid..
If VM_IO gets rid of this, then we should immediately apply the patch.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
| ||||||||||||
| Last update: 2005-03-22 13:00 [from the cache] ©2003-2008 | |||||||||||||