On Fri 24-02-17 13:31:48, Shaohua Li wrote: > Now MADV_FREE pages can be easily reclaimed even for swapless system. We > can safely enable MADV_FREE for all systems. > > Cc: Michal Hocko <mho...@suse.com> > Cc: Minchan Kim <minc...@kernel.org> > Cc: Hugh Dickins <hu...@google.com> > Cc: Rik van Riel <r...@redhat.com> > Cc: Mel Gorman <mgor...@techsingularity.net> > Cc: Andrew Morton <a...@linux-foundation.org> > Acked-by: Johannes Weiner <han...@cmpxchg.org> > Signed-off-by: Shaohua Li <s...@fb.com>
Acked-by: Michal Hocko <mho...@suse.com> > --- > mm/madvise.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/mm/madvise.c b/mm/madvise.c > index 225af7d..5ab4b7b 100644 > --- a/mm/madvise.c > +++ b/mm/madvise.c > @@ -612,13 +612,7 @@ madvise_vma(struct vm_area_struct *vma, struct > vm_area_struct **prev, > case MADV_WILLNEED: > return madvise_willneed(vma, prev, start, end); > case MADV_FREE: > - /* > - * XXX: In this implementation, MADV_FREE works like > - * MADV_DONTNEED on swapless system or full swap. > - */ > - if (get_nr_swap_pages() > 0) > - return madvise_free(vma, prev, start, end); > - /* passthrough */ > + return madvise_free(vma, prev, start, end); > case MADV_DONTNEED: > return madvise_dontneed(vma, prev, start, end); > default: > -- > 2.9.3 > -- Michal Hocko SUSE Labs
