RE: [PATCH] fix memory less node allocation

From: Luck, Tony <tony.luck_at_intel.com>
Date: 2005-11-04 09:33:53
-	if (!ptr)
-		panic("NO memory for memory less node\n");

Why did you drop this panic?  Although your patch now
makes sure to never try to alloc from a memory less
node ... surely it is still possible (though unlikely)
that the call to __alloc_bootmem_node() might fail. A
panic here might be easier to debug than letting it run
until fill_pernode() does a memset(__va(0), 0, ...).

-Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Fri Nov 04 09:34:28 2005

This archive was generated by hypermail 2.1.8 : 2005-11-04 09:34:35 EST