On Thu, 03 Apr 2008 17:51:12 -0700
Dave Hansen <dave@xxxxxxxxxxxxxxxxxx> wrote:
On Thu, 2008-04-03 at 17:05 -0700, Jeremy Fitzhardinge wrote:
Add a per-section "section_ops" structure, allowing each section to haveThis is kinda a lot of code and mucking around for what we actually get
specific functions for onlining and offlining pages within the section.
This is used by the Xen balloon driver to make sure that pages are not
onlined without some physical memory backing them.
out of it, especially since you just condensed down all the actual
online_page() instances.
I think it might just be nicer to have a global list of these handlers
somewhere. The Xen driver can just say "put me on the list of
callbacks" and we'll call them at online_page(). I really don't think
we need to be passing an ops structure around.
KAME, did you have some other ideas about this?
At first. please don't call online_page() handler in add_memory() phase.
online_page() handler should be called in online_pages().
Passing handler to online_pages() is much easier and it's ok to me.