On Fri, 2006-09-01 at 17:43 +0200, Martin Schwidefsky wrote: > This question did come up already. arch_free_page() is done before the > PageReserved() check so it isn't suitable for stable/unused state > transitions. You can argue that arch_free_page() should be moved but who > knows what the architecture defined function is supposed to do? > page_set_stable/page_set_unused on the other hand have a clearly defined > meaning. With a patch set this large, I think it would at least be a nice thing to go through and review the other architectures' uses. -- Dave
