Re: [PATCH] mm/zswap: add writethrough option

2014-01-23 Thread Dan Streetman
On Wed, Jan 22, 2014 at 3:33 PM, Andrew Morton wrote: > On Wed, 22 Jan 2014 09:19:58 -0500 Dan Streetman wrote: > >> >>> > Acutally, I really don't know how much benefit we have that in-memory >> >>> > swap overcomming to the real storage but if you want, zRAM with >> >>> > dm-cache >> >>> > is

Re: [PATCH] mm/zswap: add writethrough option

2014-01-23 Thread Dan Streetman
On Wed, Jan 22, 2014 at 7:18 PM, Minchan Kim wrote: > Hello all, > > On Wed, Jan 22, 2014 at 12:33:58PM -0800, Andrew Morton wrote: >> On Wed, 22 Jan 2014 09:19:58 -0500 Dan Streetman wrote: >> >> > >>> > Acutally, I really don't know how much benefit we have that in-memory >> > >>> > swap overco

Re: [PATCH] mm/zswap: add writethrough option

2014-01-23 Thread Austin S Hemmelgarn
On 2014-01-22 19:18, Minchan Kim wrote: > > From the beginning, zswap is for reducing swap I/O but if workingset > overflows, it should write back rather than OOM with expecting a small > number of writeback would make the system happy because the high memory > pressure is temporal so soon most of

Re: [PATCH] mm/zswap: add writethrough option

2014-01-22 Thread Bob Liu
On 01/23/2014 08:18 AM, Minchan Kim wrote: > Hello all, > > On Wed, Jan 22, 2014 at 12:33:58PM -0800, Andrew Morton wrote: >> On Wed, 22 Jan 2014 09:19:58 -0500 Dan Streetman wrote: >> >>> Acutally, I really don't know how much benefit we have that in-memory >>> swap overcomming to the r

Re: [PATCH] mm/zswap: add writethrough option

2014-01-22 Thread Minchan Kim
Hello all, On Wed, Jan 22, 2014 at 12:33:58PM -0800, Andrew Morton wrote: > On Wed, 22 Jan 2014 09:19:58 -0500 Dan Streetman wrote: > > > >>> > Acutally, I really don't know how much benefit we have that in-memory > > >>> > swap overcomming to the real storage but if you want, zRAM with > > >>>

Re: [PATCH] mm/zswap: add writethrough option

2014-01-22 Thread Andrew Morton
On Wed, 22 Jan 2014 09:19:58 -0500 Dan Streetman wrote: > >>> > Acutally, I really don't know how much benefit we have that in-memory > >>> > swap overcomming to the real storage but if you want, zRAM with dm-cache > >>> > is another option rather than invent new wheel by "just having is > >>> >

Re: [PATCH] mm/zswap: add writethrough option

2014-01-22 Thread Dan Streetman
On Fri, Jan 17, 2014 at 12:41 AM, Dan Streetman wrote: > On Wed, Jan 15, 2014 at 12:42 AM, Minchan Kim wrote: >> Hello, >> >> On Tue, Jan 14, 2014 at 10:10:44AM -0500, Dan Streetman wrote: >>> On Mon, Jan 13, 2014 at 7:11 PM, Minchan Kim wrote: >>> > Hello Dan, >>> > >>> > Sorry for the late res

Re: [PATCH] mm/zswap: add writethrough option

2014-01-16 Thread Dan Streetman
On Wed, Jan 15, 2014 at 12:42 AM, Minchan Kim wrote: > Hello, > > On Tue, Jan 14, 2014 at 10:10:44AM -0500, Dan Streetman wrote: >> On Mon, Jan 13, 2014 at 7:11 PM, Minchan Kim wrote: >> > Hello Dan, >> > >> > Sorry for the late response and I didn't look at the code yet >> > because I am not con

Re: [PATCH] mm/zswap: add writethrough option

2014-01-14 Thread Minchan Kim
Hello, On Tue, Jan 14, 2014 at 10:10:44AM -0500, Dan Streetman wrote: > On Mon, Jan 13, 2014 at 7:11 PM, Minchan Kim wrote: > > Hello Dan, > > > > Sorry for the late response and I didn't look at the code yet > > because I am not convinced. :( > > > > On Thu, Dec 19, 2013 at 08:23:27AM -0500, Dan

Re: [PATCH] mm/zswap: add writethrough option

2014-01-14 Thread Dan Streetman
On Mon, Jan 13, 2014 at 7:11 PM, Minchan Kim wrote: > Hello Dan, > > Sorry for the late response and I didn't look at the code yet > because I am not convinced. :( > > On Thu, Dec 19, 2013 at 08:23:27AM -0500, Dan Streetman wrote: >> Currently, zswap is writeback cache; stored pages are not sent >

Re: [PATCH] mm/zswap: add writethrough option

2014-01-13 Thread Minchan Kim
Hello Dan, Sorry for the late response and I didn't look at the code yet because I am not convinced. :( On Thu, Dec 19, 2013 at 08:23:27AM -0500, Dan Streetman wrote: > Currently, zswap is writeback cache; stored pages are not sent > to swap disk, and when zswap wants to evict old pages it must

Re: [PATCH] mm/zswap: add writethrough option

2014-01-13 Thread Dan Streetman
Ping to see if this patch can get picked up. On Fri, Jan 3, 2014 at 10:11 AM, Seth Jennings wrote: > On Thu, Dec 19, 2013 at 08:23:27AM -0500, Dan Streetman wrote: >> Currently, zswap is writeback cache; stored pages are not sent >> to swap disk, and when zswap wants to evict old pages it must >>

Re: [PATCH] mm/zswap: add writethrough option

2014-01-03 Thread Seth Jennings
On Thu, Dec 19, 2013 at 08:23:27AM -0500, Dan Streetman wrote: > Currently, zswap is writeback cache; stored pages are not sent > to swap disk, and when zswap wants to evict old pages it must > first write them back to swap cache/disk manually. This avoids > swap out disk I/O up front, but only mo

RE: [PATCH] mm/zswap: add writethrough option

2014-01-02 Thread Weijie Yang
On Thu, Jan 2, 2014 at 11:38 PM, Dan Streetman wrote: > Happy new year! > > Seth, just checking if you have had a chance yet to think about this one. > > > On Thu, Dec 19, 2013 at 8:23 AM, Dan Streetman wrote: > > Currently, zswap is writeback cache; stored pages are not sent > > to swap disk, an

Re: [PATCH] mm/zswap: add writethrough option

2014-01-02 Thread Dan Streetman
Happy new year! Seth, just checking if you have had a chance yet to think about this one. On Thu, Dec 19, 2013 at 8:23 AM, Dan Streetman wrote: > Currently, zswap is writeback cache; stored pages are not sent > to swap disk, and when zswap wants to evict old pages it must > first write them ba