Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-07 Thread Tim Chen
On Tue, 2016-06-07 at 17:21 +0900, Minchan Kim wrote: > On Wed, Jun 01, 2016 at 11:23:53AM -0700, Tim Chen wrote: > > > > On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: > > > > > >   > > > Hi Tim, > > > > > > To me, this reorganization is too limited and not good for me, > > > frankly

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-07 Thread Tim Chen
On Tue, 2016-06-07 at 17:21 +0900, Minchan Kim wrote: > On Wed, Jun 01, 2016 at 11:23:53AM -0700, Tim Chen wrote: > > > > On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: > > > > > >   > > > Hi Tim, > > > > > > To me, this reorganization is too limited and not good for me, > > > frankly

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-07 Thread Minchan Kim
On Wed, Jun 01, 2016 at 11:23:53AM -0700, Tim Chen wrote: > On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: > >  > > Hi Tim, > > > > To me, this reorganization is too limited and not good for me, > > frankly speaking. It works for only your goal which allocate batch > > swap slot, I guess.

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-07 Thread Minchan Kim
On Wed, Jun 01, 2016 at 11:23:53AM -0700, Tim Chen wrote: > On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: > >  > > Hi Tim, > > > > To me, this reorganization is too limited and not good for me, > > frankly speaking. It works for only your goal which allocate batch > > swap slot, I guess.

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-01 Thread Tim Chen
On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: >  > Hi Tim, > > To me, this reorganization is too limited and not good for me, > frankly speaking. It works for only your goal which allocate batch > swap slot, I guess. :) > > My goal is to make them work with batch page_check_references, >

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-06-01 Thread Tim Chen
On Wed, 2016-06-01 at 16:12 +0900, Minchan Kim wrote: >  > Hi Tim, > > To me, this reorganization is too limited and not good for me, > frankly speaking. It works for only your goal which allocate batch > swap slot, I guess. :) > > My goal is to make them work with batch page_check_references, >

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-05-31 Thread Tim Chen
gt; > Thanks. My mail client is misbehaving after a system upgrade. Here's the patch again. Subject: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions This patch consolidates the page out and the varous cleanup operations within shrink_page_list function into han

Re: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-05-31 Thread Tim Chen
gt; > Thanks. My mail client is misbehaving after a system upgrade. Here's the patch again. Subject: [PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions This patch consolidates the page out and the varous cleanup operations within shrink_page_list function into han

[PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-05-20 Thread Tim Chen
This patch consolidates the page out and the various cleanup operations within shrink_page_list function into handle_pgout and pg_finish functions. This makes the shrink_page_list function more concise and allows for the separation of page out and page scan operations. It paves the way to group

[PATCH] mm: Cleanup - Reorganize the shrink_page_list code into smaller functions

2016-05-20 Thread Tim Chen
This patch consolidates the page out and the various cleanup operations within shrink_page_list function into handle_pgout and pg_finish functions. This makes the shrink_page_list function more concise and allows for the separation of page out and page scan operations. It paves the way to group