Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-16 Thread kbuild test robot
Hi Minchan, [auto build test ERROR on mmotm/master] [also build test ERROR on next-20170310] [cannot apply to v4.11-rc2] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Minchan-Kim/make-try_to_un

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-15 Thread Minchan Kim
On Thu, Mar 16, 2017 at 02:44:30PM +0900, Sergey Senozhatsky wrote: > On (03/16/17 14:33), Minchan Kim wrote: > [..] > > "There is no user for it" > > > > I was liar so need to be a honest guy. > > ha-ha-ha. I didn't say that :) > > [..] > > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(str

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-15 Thread Sergey Senozhatsky
On (03/16/17 14:51), Minchan Kim wrote: [..] > > > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, > > > struct vm_area_struct *vma, > > >*/ > > > if (unlikely(PageSwapBacked(page) != > > > PageSwapCache(page))) { > > >

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-15 Thread Sergey Senozhatsky
On (03/16/17 14:33), Minchan Kim wrote: [..] > "There is no user for it" > > I was liar so need to be a honest guy. ha-ha-ha. I didn't say that :) [..] > @@ -1414,7 +1414,7 @@ static bool try_to_unmap_one(struct page *page, struct > vm_area_struct *vma, >*/ >

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-15 Thread Minchan Kim
Hey, Sergey, On Thu, Mar 16, 2017 at 01:40:23PM +0900, Sergey Senozhatsky wrote: > Hello, > > > On (03/15/17 14:24), Minchan Kim wrote: > > There is no user for it. Remove it. > > > > there is one. > > mm/rmap.c > > try_to_unmap_one() > ... > if (unlikely(PageSwapBacked(page) != PageSw

Re: [PATCH v2 10/10] mm: remove SWAP_[SUCCESS|AGAIN|FAIL]

2017-03-15 Thread Sergey Senozhatsky
Hello, On (03/15/17 14:24), Minchan Kim wrote: > There is no user for it. Remove it. > there is one. mm/rmap.c try_to_unmap_one() ... if (unlikely(PageSwapBacked(page) != PageSwapCache(page))) { WARN_ON_ONCE(1); ret = SWAP_FAIL; page_vma