Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-04 Thread Seth Jennings
On 02/04/2013 03:30 PM, Seth Jennings wrote: > On 01/29/2013 03:40 PM, Seth Jennings wrote: >> Sorry for the churn but just this set might be easier to review. >> The code required for the flushing is in a separate patch now >> as requested. I've got a large and valuable body of feedback to integr

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-04 Thread Seth Jennings
On 02/03/2013 07:03 PM, Simon Jeons wrote: > On Fri, 2013-02-01 at 09:13 -0600, Seth Jennings wrote: >> On 01/31/2013 07:39 PM, Simon Jeons wrote: >>> Hi Seth, >>> On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: >> Performance, Kernel Building: Setup Gento

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-04 Thread Seth Jennings
On 02/02/2013 06:17 PM, Simon Jeons wrote: > On Fri, 2013-02-01 at 09:13 -0600, Seth Jennings wrote: >> On 01/31/2013 07:39 PM, Simon Jeons wrote: >>> Hi Seth, >>> On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: >> Performance, Kernel Building: Setup Gento

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-03 Thread Simon Jeons
On Fri, 2013-02-01 at 09:13 -0600, Seth Jennings wrote: > On 01/31/2013 07:39 PM, Simon Jeons wrote: > > Hi Seth, > > On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: > > >> Performance, Kernel Building: > >> > >> Setup > >> > >> Gentoo w/ kernel v3.7-rc7 > >> Quad-core i5-2500 @ 3

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-02 Thread Simon Jeons
On Fri, 2013-02-01 at 09:13 -0600, Seth Jennings wrote: > On 01/31/2013 07:39 PM, Simon Jeons wrote: > > Hi Seth, > > On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: > > >> Performance, Kernel Building: > >> > >> Setup > >> > >> Gentoo w/ kernel v3.7-rc7 > >> Quad-core i5-2500 @ 3

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-02-01 Thread Seth Jennings
On 01/31/2013 07:39 PM, Simon Jeons wrote: > Hi Seth, > On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: >> Performance, Kernel Building: >> >> Setup >> >> Gentoo w/ kernel v3.7-rc7 >> Quad-core i5-2500 @ 3.3GHz >> 512MB DDR3 1600MHz (limited with mem=512m on boot) >> Filesystem an

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-01-31 Thread Simon Jeons
Hi Seth, On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: > Sorry for the churn but just this set might be easier to review. > The code required for the flushing is in a separate patch now > as requested. > > Changelog: > > v4: > * Added Acks (Minchan) > * Separated flushing functionality

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-01-30 Thread Seth Jennings
On 01/29/2013 10:32 PM, Minchan Kim wrote: > On Tue, Jan 29, 2013 at 04:49:04PM -0600, Seth Jennings wrote: >> On 01/29/2013 04:14 PM, Joe Perches wrote: >>> On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: The code required for the flushing is in a separate patch now as requested.

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-01-29 Thread Minchan Kim
On Tue, Jan 29, 2013 at 04:49:04PM -0600, Seth Jennings wrote: > On 01/29/2013 04:14 PM, Joe Perches wrote: > > On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: > >> The code required for the flushing is in a separate patch now > >> as requested. > > > > What tree does this apply to? > > Bo

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-01-29 Thread Seth Jennings
On 01/29/2013 04:14 PM, Joe Perches wrote: > On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: >> The code required for the flushing is in a separate patch now >> as requested. > > What tree does this apply to? > Both -next and linus fail to compile. Link to build instruction in the cover l

Re: [PATCHv4 0/7] zswap: compressed swap caching

2013-01-29 Thread Joe Perches
On Tue, 2013-01-29 at 15:40 -0600, Seth Jennings wrote: > The code required for the flushing is in a separate patch now > as requested. What tree does this apply to? Both -next and linus fail to compile. There's a whitespace error applying 3/7 (line 543 in zswap.c) and on 3.8-rc5 (allyesconfig x8

[PATCHv4 0/7] zswap: compressed swap caching

2013-01-29 Thread Seth Jennings
Sorry for the churn but just this set might be easier to review. The code required for the flushing is in a separate patch now as requested. Changelog: v4: * Added Acks (Minchan) * Separated flushing functionality into standalone patch for easier review (Minchan) * fix comment on zswap enabled