Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-02 Thread Yisheng Xie
Hi Minchan Thanks for reviewing. On 2017/2/2 15:28, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 11:00:23AM +0100, Michal Hocko wrote: >> On Wed 01-02-17 18:46:36, Minchan Kim wrote: >>> On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: On Wed 01-02-17 15:48:21, Minchan Kim wrote:

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-02 Thread Yisheng Xie
Hi Minchan Thanks for reviewing. On 2017/2/2 15:28, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 11:00:23AM +0100, Michal Hocko wrote: >> On Wed 01-02-17 18:46:36, Minchan Kim wrote: >>> On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: On Wed 01-02-17 15:48:21, Minchan Kim wrote:

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-02 Thread Yisheng Xie
hi Minchan, Thanks for your reviewing. On 2017/2/1 17:46, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: >> On Wed 01-02-17 15:48:21, Minchan Kim wrote: >>> Hi Yisheng, >>> >>> On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: From:

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-02 Thread Yisheng Xie
hi Minchan, Thanks for your reviewing. On 2017/2/1 17:46, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: >> On Wed 01-02-17 15:48:21, Minchan Kim wrote: >>> Hi Yisheng, >>> >>> On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: From:

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Minchan Kim
On Wed, Feb 01, 2017 at 11:00:23AM +0100, Michal Hocko wrote: > On Wed 01-02-17 18:46:36, Minchan Kim wrote: > > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > > > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > > > Hi Yisheng, > > > > > > > > On Tue, Jan 31, 2017 at 09:06:18PM

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Minchan Kim
On Wed, Feb 01, 2017 at 11:00:23AM +0100, Michal Hocko wrote: > On Wed 01-02-17 18:46:36, Minchan Kim wrote: > > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > > > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > > > Hi Yisheng, > > > > > > > > On Tue, Jan 31, 2017 at 09:06:18PM

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Michal Hocko
On Wed 01-02-17 18:46:36, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > > Hi Yisheng, > > > > > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > > > From: Yisheng Xie

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Michal Hocko
On Wed 01-02-17 18:46:36, Minchan Kim wrote: > On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > > Hi Yisheng, > > > > > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > > > From: Yisheng Xie > > > > > > >

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Minchan Kim
On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > Hi Yisheng, > > > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > > From: Yisheng Xie > > > > > > This patch changes the return type

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Minchan Kim
On Wed, Feb 01, 2017 at 08:59:24AM +0100, Michal Hocko wrote: > On Wed 01-02-17 15:48:21, Minchan Kim wrote: > > Hi Yisheng, > > > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > > From: Yisheng Xie > > > > > > This patch changes the return type of

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Michal Hocko
On Wed 01-02-17 15:48:21, Minchan Kim wrote: > Hi Yisheng, > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > From: Yisheng Xie > > > > This patch changes the return type of isolate_movable_page() > > from bool to int. It will return 0 when

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-02-01 Thread Michal Hocko
On Wed 01-02-17 15:48:21, Minchan Kim wrote: > Hi Yisheng, > > On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > > From: Yisheng Xie > > > > This patch changes the return type of isolate_movable_page() > > from bool to int. It will return 0 when isolate movable page > >

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-01-31 Thread Minchan Kim
Hi Yisheng, On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > From: Yisheng Xie > > This patch changes the return type of isolate_movable_page() > from bool to int. It will return 0 when isolate movable page > successfully, return -EINVAL when the page

Re: [PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-01-31 Thread Minchan Kim
Hi Yisheng, On Tue, Jan 31, 2017 at 09:06:18PM +0800, ys...@foxmail.com wrote: > From: Yisheng Xie > > This patch changes the return type of isolate_movable_page() > from bool to int. It will return 0 when isolate movable page > successfully, return -EINVAL when the page is not a non-lru

[PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-01-31 Thread ysxie
From: Yisheng Xie This patch changes the return type of isolate_movable_page() from bool to int. It will return 0 when isolate movable page successfully, return -EINVAL when the page is not a non-lru movable page, and for other cases it will return -EBUSY. There is no

[PATCH v5 1/4] mm/migration: make isolate_movable_page() return int type

2017-01-31 Thread ysxie
From: Yisheng Xie This patch changes the return type of isolate_movable_page() from bool to int. It will return 0 when isolate movable page successfully, return -EINVAL when the page is not a non-lru movable page, and for other cases it will return -EBUSY. There is no functional change within