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

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

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

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: snip Performance, Kernel Building: Setup Gentoo w/ kernel v3.7-rc7

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: snip Performance, Kernel Building: Setup Gentoo w/ kernel v3.7-rc7

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 integrate

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 @

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: snip Performance, Kernel Building: Setup Gentoo w/ kernel v3.7-rc7 Quad-core i5-2500 @ 3.3GHz 512MB DDR3

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 @

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: snip Performance, Kernel Building: Setup Gentoo w/ kernel v3.7-rc7 Quad-core i5-2500 @ 3.3GHz 512MB DDR3

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

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: snip 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 and swap on

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-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 into

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

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. What tree

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? > >

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

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

[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

[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

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

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 letter:

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? Both -next and