Re: [patch 1/8] Page host virtual assist: unused / free pages.

[Martin Schwidefsky <schwidefsky@de.ibm.com>]

On Mon, 2006-04-24 at 16:49 +0200, Andi Kleen wrote:
> On Monday 24 April 2006 16:41, Martin Schwidefsky wrote:
> > On Mon, 2006-04-24 at 16:07 +0200, Andi Kleen wrote:
> > > On Monday 24 April 2006 14:34, Martin Schwidefsky wrote:
> > > 
> > > > +#define page_hva_set_unused(_page)		do { } while (0)
> > > > +#define page_hva_set_stable(_page)		do { } while (0)
> > > 
> > > The whole thing seems quite under commented in the code and illnamed
> > > (if you didn't know what page_hva_set_unused() is supposed to do
> > > already would you figure it out from the name?) 
> > 
> > Well, we can always add comments if something is unclear. The name
> > should give you a good hint though: page-(hva)-set-unused. You set the
> > page to the unused state. Is the name really that confusing ?
> 
> How about page_set_free / page_set_used ?

Ok, sounds reasonable. Do we need to drop the _hva name component? If we
do that then something like page_hva_unmap_all will be named
page_unmap_all which might be a bit confusing as well.

I've noted the lack of comments in the page_hva.h header on my to-do
list.

-- 
blue skies,
  Martin.

Martin Schwidefsky
Linux for zSeries Development & Services
IBM Deutschland Entwicklung GmbH

"Reality continues to ruin my life." - Calvin.


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>