lkml.org 
[lkml]   [2005]   [Nov]   [16]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Lhms-devel] Re: 2.6.14-mm2
FromDave Hansen <>
DateWed, 16 Nov 2005 17:31:44 +0100
On Thu, 2005-11-17 at 01:02 +0900, Kamezawa Hiroyuki wrote: 
> > Can you explain in a little bit more detail why this matters, and
> > exactly how it fixes your problem.  I'm not sure it's correct.
> > > Ah, okay.> > It's just because free_area[] is not initaialized at all if this is not called.
> It is list.next and list.prev has bad value.> Then, the first free_page(page) will cause panic.

Hmmm.  I _think_ you're just trying to do some things at runtime that I
didn't intend.  In the patch I pointed to in the last mail, look at what
I did in hot_add_zone_init().  It does some of what
free_area_init_core() does, but only the most minimal bits.  Basically:

       zone_wait_table_init(zone, size_pages);
       init_currently_empty_zone(zone, phys_start_pfn, size_pages);
       zone_pcp_init(zone);

Your way may also be valid, but I broke out init_currently_empty_zone()
for a reason, and I think this was it.  I don't think we want to be
calling free_area_init_core() itself at runtime.

-- Dave

-
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-11-16 17:34    [from the cache]
©2003-2008