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