Re: [patch 2] Xen core patch : arch_free_page return value

[Andrew Morton <akpm@osdl.org>]

Jeff Dike <jdike@addtoit.com> wrote:
>
> akpm@osdl.org said:
> > But heck - why bother?  The current patch adds just one line of code
> > in one place, and the compiler will toss it away anyway for all but
> > xen and um. 
> 
> Agree, but on a conceptual level, this is allowing the arch to swipe pages 
> arbitrarily out from under the nose of the generic page allocator.  Plus, the 
> code that's being bypassed has to be implemented in some form in the arch.  
> That's not my concern, or yours, necessarily, but it does suggest that the 
> interface is being abused.
> 

True.  But we can take a look at that once we've seen the rest of the xen
patches.  I wouldn't be merging up any of these patches until we've seen
the whole submission.