Re: [PATCH 1/2] mm: swap: unify swap slot free functions to put_swap_page

2017-05-12 Thread Johannes Weiner
On Fri, May 12, 2017 at 11:21:23AM +0900, Minchan Kim wrote: > Now, get_swap_page takes struct page and allocates swap space > according to page size(ie, normal or THP) so it would be more > cleaner to introduce put_swap_page which is a counter function > of get_swap_page. Then, it calls right

Re: [PATCH 1/2] mm: swap: unify swap slot free functions to put_swap_page

2017-05-12 Thread Johannes Weiner
On Fri, May 12, 2017 at 11:21:23AM +0900, Minchan Kim wrote: > Now, get_swap_page takes struct page and allocates swap space > according to page size(ie, normal or THP) so it would be more > cleaner to introduce put_swap_page which is a counter function > of get_swap_page. Then, it calls right

[PATCH 1/2] mm: swap: unify swap slot free functions to put_swap_page

2017-05-11 Thread Minchan Kim
Now, get_swap_page takes struct page and allocates swap space according to page size(ie, normal or THP) so it would be more cleaner to introduce put_swap_page which is a counter function of get_swap_page. Then, it calls right swap slot free function depending on page's size. Cc: Johannes Weiner

[PATCH 1/2] mm: swap: unify swap slot free functions to put_swap_page

2017-05-11 Thread Minchan Kim
Now, get_swap_page takes struct page and allocates swap space according to page size(ie, normal or THP) so it would be more cleaner to introduce put_swap_page which is a counter function of get_swap_page. Then, it calls right swap slot free function depending on page's size. Cc: Johannes Weiner