Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-18 Thread Dan Streetman
On Wed, Oct 15, 2014 at 6:00 AM, Heesub Shin wrote: > When zbud is initialized through the zpool wrapper, pool->ops which > points to user-defined operations is always set regardless of whether it > is specified from the upper layer. This causes zbud_reclaim_page() to > iterate its loop for

Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-18 Thread Dan Streetman
On Wed, Oct 15, 2014 at 6:00 AM, Heesub Shin heesub.s...@samsung.com wrote: When zbud is initialized through the zpool wrapper, pool-ops which points to user-defined operations is always set regardless of whether it is specified from the upper layer. This causes zbud_reclaim_page() to iterate

Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Heesub Shin
Hello, On 10/16/2014 05:17 AM, Andrew Morton wrote: On Wed, 15 Oct 2014 19:00:43 +0900 Heesub Shin wrote: When zbud is initialized through the zpool wrapper, pool->ops which points to user-defined operations is always set regardless of whether it is specified from the upper layer. This

Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Andrew Morton
On Wed, 15 Oct 2014 19:00:43 +0900 Heesub Shin wrote: > When zbud is initialized through the zpool wrapper, pool->ops which > points to user-defined operations is always set regardless of whether it > is specified from the upper layer. This causes zbud_reclaim_page() to > iterate its loop for

[PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Heesub Shin
When zbud is initialized through the zpool wrapper, pool->ops which points to user-defined operations is always set regardless of whether it is specified from the upper layer. This causes zbud_reclaim_page() to iterate its loop for evicting pool pages out without any gain. This patch sets the

[PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Heesub Shin
When zbud is initialized through the zpool wrapper, pool-ops which points to user-defined operations is always set regardless of whether it is specified from the upper layer. This causes zbud_reclaim_page() to iterate its loop for evicting pool pages out without any gain. This patch sets the

Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Andrew Morton
On Wed, 15 Oct 2014 19:00:43 +0900 Heesub Shin heesub.s...@samsung.com wrote: When zbud is initialized through the zpool wrapper, pool-ops which points to user-defined operations is always set regardless of whether it is specified from the upper layer. This causes zbud_reclaim_page() to

Re: [PATCH] mm/zbud: init user ops only when it is needed

2014-10-15 Thread Heesub Shin
Hello, On 10/16/2014 05:17 AM, Andrew Morton wrote: On Wed, 15 Oct 2014 19:00:43 +0900 Heesub Shin heesub.s...@samsung.com wrote: When zbud is initialized through the zpool wrapper, pool-ops which points to user-defined operations is always set regardless of whether it is specified from the