Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-21 Thread Konrad Rzeszutek Wilk
On Fri, Jun 21, 2013 at 08:20:34AM -0700, Dan Magenheimer wrote: > > From: Bob Liu [mailto:lliu...@gmail.com] > Subject: Re: [PATCHv13 3/4] zswap: add to mm/ > > > > On Thu, Jun 20, 2013 at 10:23 PM, Seth Jennings > > wrote: > > > On Thu, Jun 20, 201

RE: [PATCHv13 3/4] zswap: add to mm/

2013-06-21 Thread Dan Magenheimer
> From: Bob Liu [mailto:lliu...@gmail.com] Subject: Re: [PATCHv13 3/4] zswap: add to mm/ > > On Thu, Jun 20, 2013 at 10:23 PM, Seth Jennings > wrote: > > On Thu, Jun 20, 2013 at 05:42:04PM +0800, Bob Liu wrote: > >> > Just made a mmtests run of my own

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-20 Thread Bob Liu
On Thu, Jun 20, 2013 at 10:23 PM, Seth Jennings wrote: > On Thu, Jun 20, 2013 at 05:42:04PM +0800, Bob Liu wrote: >> > Just made a mmtests run of my own and got very different results: >> > >> >> It's strange, I'll update to rc6 and try again. >> By the way, are you using 824 hardware compressor i

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-20 Thread Seth Jennings
On Thu, Jun 20, 2013 at 05:42:04PM +0800, Bob Liu wrote: > > Just made a mmtests run of my own and got very different results: > > > > It's strange, I'll update to rc6 and try again. > By the way, are you using 824 hardware compressor instead of lzo? My results where using lzo software compressio

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-20 Thread Bob Liu
On Thu, Jun 20, 2013 at 10:37 AM, Seth Jennings wrote: > On Mon, Jun 17, 2013 at 02:20:05PM +0800, Bob Liu wrote: >> Hi Seth, >> >> On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings >> wrote: >> > zswap is a thin backend for frontswap that takes pages that are in the >> > process >> > of being swapp

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-19 Thread Seth Jennings
On Mon, Jun 17, 2013 at 02:20:05PM +0800, Bob Liu wrote: > Hi Seth, > > On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings > wrote: > > zswap is a thin backend for frontswap that takes pages that are in the > > process > > of being swapped out and attempts to compress them and store them in a > > RAM

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-19 Thread Bob Liu
On Wed, Jun 19, 2013 at 10:09 PM, Seth Jennings wrote: > On Mon, Jun 17, 2013 at 02:20:05PM +0800, Bob Liu wrote: >> Hi Seth, >> >> On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings >> wrote: >> > zswap is a thin backend for frontswap that takes pages that are in the >> > process >> > of being swapp

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-19 Thread Seth Jennings
On Mon, Jun 17, 2013 at 02:20:05PM +0800, Bob Liu wrote: > Hi Seth, > > On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings > wrote: > > zswap is a thin backend for frontswap that takes pages that are in the > > process > > of being swapped out and attempts to compress them and store them in a > > RAM

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-18 Thread Bob Liu
> > I'm not sure how representative this is of real workloads, but it does > look rather fatal for zswap. The differences are so large, I wonder if > it's just some silly bug or config issue. > In my observation, zswap_pool_pages always close to zswap_stored_pages in this testing. I think it mean

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-18 Thread Bob Liu
> > So the minor fault rate improved and everything else got worse? I did the test again, in a new clean environment. I'm sure the config files are the same except enabled zswap. v3.10-rc4 v3.10-rc4 2G-paralli

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-17 Thread Andrew Morton
On Mon, 17 Jun 2013 14:20:05 +0800 Bob Liu wrote: > Hi Seth, > > On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings > wrote: > > zswap is a thin backend for frontswap that takes pages that are in the > > process > > of being swapped out and attempts to compress them and store them in a > > RAM-base

Re: [PATCHv13 3/4] zswap: add to mm/

2013-06-16 Thread Bob Liu
Hi Seth, On Tue, Jun 4, 2013 at 4:33 AM, Seth Jennings wrote: > zswap is a thin backend for frontswap that takes pages that are in the process > of being swapped out and attempts to compress them and store them in a > RAM-based memory pool. This can result in a significant I/O reduction on the >