Re: [PATCH V5 2/6] mm: don't assume anonymous pages have SwapBacked flag

2017-02-27 Thread Michal Hocko
On Mon 27-02-17 08:10:24, Shaohua Li wrote: > On Mon, Feb 27, 2017 at 03:35:34PM +0100, Michal Hocko wrote: > > On Fri 24-02-17 13:31:45, Shaohua Li wrote: > > > There are a few places the code assumes anonymous pages should have > > > SwapBacked flag set. MADV_FREE pages are anonymous pages but we

Re: [PATCH V5 2/6] mm: don't assume anonymous pages have SwapBacked flag

2017-02-27 Thread Shaohua Li
On Mon, Feb 27, 2017 at 03:35:34PM +0100, Michal Hocko wrote: > On Fri 24-02-17 13:31:45, Shaohua Li wrote: > > There are a few places the code assumes anonymous pages should have > > SwapBacked flag set. MADV_FREE pages are anonymous pages but we are > > going to add them to LRU_INACTIVE_FILE list

Re: [PATCH V5 2/6] mm: don't assume anonymous pages have SwapBacked flag

2017-02-27 Thread Michal Hocko
On Fri 24-02-17 13:31:45, Shaohua Li wrote: > There are a few places the code assumes anonymous pages should have > SwapBacked flag set. MADV_FREE pages are anonymous pages but we are > going to add them to LRU_INACTIVE_FILE list and clear SwapBacked flag > for them. The assumption doesn't hold any

Re: [PATCH V5 2/6] mm: don't assume anonymous pages have SwapBacked flag

2017-02-26 Thread Hillf Danton
On February 25, 2017 5:32 AM Shaohua Li wrote: > > There are a few places the code assumes anonymous pages should have > SwapBacked flag set. MADV_FREE pages are anonymous pages but we are > going to add them to LRU_INACTIVE_FILE list and clear SwapBacked flag > for them. The assumption doesn't

[PATCH V5 2/6] mm: don't assume anonymous pages have SwapBacked flag

2017-02-24 Thread Shaohua Li
There are a few places the code assumes anonymous pages should have SwapBacked flag set. MADV_FREE pages are anonymous pages but we are going to add them to LRU_INACTIVE_FILE list and clear SwapBacked flag for them. The assumption doesn't hold any more, so fix them. Cc: Michal Hocko Cc: Minchan K