Re: [PATCH 2/2] mm: swap: move anonymous THP split logic to vmscan

2017-05-12 Thread Johannes Weiner
On Fri, May 12, 2017 at 11:21:24AM +0900, Minchan Kim wrote: > The add_to_swap aims to allocate swap_space(ie, swap slot and > swapcache) so if it fails due to lack of space in case of THP > or something(hdd swap but tries THP swapout) *caller* rather > than add_to_swap itself should split the THP

Re: [PATCH 2/2] mm: swap: move anonymous THP split logic to vmscan

2017-05-12 Thread Johannes Weiner
On Fri, May 12, 2017 at 11:21:24AM +0900, Minchan Kim wrote: > The add_to_swap aims to allocate swap_space(ie, swap slot and > swapcache) so if it fails due to lack of space in case of THP > or something(hdd swap but tries THP swapout) *caller* rather > than add_to_swap itself should split the THP

[PATCH 2/2] mm: swap: move anonymous THP split logic to vmscan

2017-05-11 Thread Minchan Kim
The add_to_swap aims to allocate swap_space(ie, swap slot and swapcache) so if it fails due to lack of space in case of THP or something(hdd swap but tries THP swapout) *caller* rather than add_to_swap itself should split the THP page and retry it with base page which is more natural. Cc:

[PATCH 2/2] mm: swap: move anonymous THP split logic to vmscan

2017-05-11 Thread Minchan Kim
The add_to_swap aims to allocate swap_space(ie, swap slot and swapcache) so if it fails due to lack of space in case of THP or something(hdd swap but tries THP swapout) *caller* rather than add_to_swap itself should split the THP page and retry it with base page which is more natural. Cc: