Re: [PATCH 01/17] mm: sl[au]b: Add knowledge of PFMEMALLOC reservepages

From: Mel Gorman
Date: Wed Jun 20 2012 - 07:44:50 EST


On Wed, Jun 20, 2012 at 01:05:13PM +0200, Sebastian Andrzej Siewior wrote:
> On Wed, Jun 20, 2012 at 10:35:04AM +0100, Mel Gorman wrote:
> > [a.p.zijlstra@xxxxxxxxx: Original implementation]
> > Signed-off-by: Mel Gorman <mgorman@xxxxxxx>
> > ---
> > diff --git a/mm/slab.c b/mm/slab.c
> > index e901a36..b190cac 100644
> > --- a/mm/slab.c
> > +++ b/mm/slab.c
> > @@ -1851,6 +1984,7 @@ static void kmem_freepages(struct kmem_cache *cachep, void *addr)
> > while (i--) {
> > BUG_ON(!PageSlab(page));
> > __ClearPageSlab(page);
> > + __ClearPageSlabPfmemalloc(page);
> > page++;
> > }
> > if (current->reclaim_state)
> > @@ -3120,16 +3254,19 @@ bad:
> > diff --git a/mm/slub.c b/mm/slub.c
> > index 8c691fa..43738c9 100644
> > --- a/mm/slub.c
> > +++ b/mm/slub.c
> > @@ -1414,6 +1418,7 @@ static void __free_slab(struct kmem_cache *s, struct page *page)
> > -pages);
> >
> > __ClearPageSlab(page);
> > + __ClearPageSlabPfmemalloc(page);
> > reset_page_mapcount(page);
> > if (current->reclaim_state)
> > current->reclaim_state->reclaimed_slab += pages;
>
> So you mention a change here in v11's changelog but I don't see it.
>

Because I'm an idiot and send out the wrong branch and then was rude
enough to not include you on the CC. I have resent the series, correctly
this time I hope. Sorry about that.

--
Mel Gorman
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/