Re: [PATCH 4/4] mm: remove free_page_mlock

2012-09-21 Thread Mel Gorman
On Tue, Sep 18, 2012 at 08:57:27PM -0700, Hugh Dickins wrote: > We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. > So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED > is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now > be checking it,

Re: [PATCH 4/4] mm: remove free_page_mlock

2012-09-21 Thread Mel Gorman
On Tue, Sep 18, 2012 at 08:57:27PM -0700, Hugh Dickins wrote: We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now be checking it,

Re: [PATCH 4/4] mm: remove free_page_mlock

2012-09-19 Thread Johannes Weiner
On Tue, Sep 18, 2012 at 08:57:27PM -0700, Hugh Dickins wrote: > We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. > So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED > is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now > be checking it,

Re: [PATCH 4/4] mm: remove free_page_mlock

2012-09-19 Thread Johannes Weiner
On Tue, Sep 18, 2012 at 08:57:27PM -0700, Hugh Dickins wrote: We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now be checking it,

[PATCH 4/4] mm: remove free_page_mlock

2012-09-18 Thread Hugh Dickins
We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now be checking it, reporting "BUG: Bad page state" if it's ever found set. Comment

[PATCH 4/4] mm: remove free_page_mlock

2012-09-18 Thread Hugh Dickins
We should not be seeing non-0 unevictable_pgs_mlockfreed any longer. So remove free_page_mlock() from the page freeing paths: __PG_MLOCKED is already in PAGE_FLAGS_CHECK_AT_FREE, so free_pages_check() will now be checking it, reporting BUG: Bad page state if it's ever found set. Comment