RE: [PATCHv2 8/9] zswap: add to mm/

2013-02-11 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: Re: [PATCHv2 8/9] zswap: add to mm/ > > On 01/29/2013 04:21 AM, Lord Glauber Costa of Sealand wrote: > > On 01/28/2013 07:27 PM, Seth Jennings wrote: > >> Yes, I prototyped a shrinker interface

RE: [PATCHv2 8/9] zswap: add to mm/

2013-02-11 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: Re: [PATCHv2 8/9] zswap: add to mm/ On 01/29/2013 04:21 AM, Lord Glauber Costa of Sealand wrote: On 01/28/2013 07:27 PM, Seth Jennings wrote: Yes, I prototyped a shrinker interface for zswap, but, as we both figured

Re: [PATCHv2 8/9] zswap: add to mm/

2013-02-07 Thread Seth Jennings
On 01/29/2013 04:21 AM, Lord Glauber Costa of Sealand wrote: > On 01/28/2013 07:27 PM, Seth Jennings wrote: >> Yes, I prototyped a shrinker interface for zswap, but, as we both >> figured, it shrinks the zswap compressed pool too aggressively to the >> point of being useless. > Can't you advertise

Re: [PATCHv2 8/9] zswap: add to mm/

2013-02-07 Thread Seth Jennings
On 01/29/2013 04:21 AM, Lord Glauber Costa of Sealand wrote: On 01/28/2013 07:27 PM, Seth Jennings wrote: Yes, I prototyped a shrinker interface for zswap, but, as we both figured, it shrinks the zswap compressed pool too aggressively to the point of being useless. Can't you advertise a

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-29 Thread Lord Glauber Costa of Sealand
On 01/28/2013 07:27 PM, Seth Jennings wrote: > Yes, I prototyped a shrinker interface for zswap, but, as we both > figured, it shrinks the zswap compressed pool too aggressively to the > point of being useless. Can't you advertise a smaller number of objects that you actively have? Since the

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-29 Thread Lord Glauber Costa of Sealand
On 01/28/2013 07:27 PM, Seth Jennings wrote: Yes, I prototyped a shrinker interface for zswap, but, as we both figured, it shrinks the zswap compressed pool too aggressively to the point of being useless. Can't you advertise a smaller number of objects that you actively have? Since the

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-28 Thread Seth Jennings
On 01/25/2013 04:44 PM, Rik van Riel wrote: > On 01/07/2013 03:24 PM, Seth Jennings wrote: >> zswap is a thin compression backend for frontswap. It receives >> pages from frontswap and attempts to store them in a compressed >> memory pool, resulting in an effective partial memory reclaim and >>

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-28 Thread Seth Jennings
On 01/25/2013 04:44 PM, Rik van Riel wrote: On 01/07/2013 03:24 PM, Seth Jennings wrote: zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim and

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-25 Thread Dan Magenheimer
> From: Rik van Riel [mailto:r...@redhat.com] > Subject: Re: [PATCHv2 8/9] zswap: add to mm/ > > On 01/07/2013 03:24 PM, Seth Jennings wrote: > > zswap is a thin compression backend for frontswap. It receives > > pages from frontswap and attempts to store them in a co

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-25 Thread Rik van Riel
On 01/07/2013 03:24 PM, Seth Jennings wrote: zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim and dramatically reduced swap device I/O. Additional, in most

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-25 Thread Rik van Riel
On 01/07/2013 03:24 PM, Seth Jennings wrote: zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim and dramatically reduced swap device I/O. Additional, in most

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-25 Thread Dan Magenheimer
From: Rik van Riel [mailto:r...@redhat.com] Subject: Re: [PATCHv2 8/9] zswap: add to mm/ On 01/07/2013 03:24 PM, Seth Jennings wrote: zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-10 Thread Dan Magenheimer
> From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] > Subject: [PATCHv2 8/9] zswap: add to mm/ > > zswap is a thin compression backend for frontswap. It receives > pages from frontswap and attempts to store them in a compressed > memory pool, resulting in an effect

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-10 Thread Dan Magenheimer
From: Seth Jennings [mailto:sjenn...@linux.vnet.ibm.com] Subject: [PATCHv2 8/9] zswap: add to mm/ zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-08 Thread Dan Magenheimer
nes Weiner; > Rik van Riel; Larry > Woodman; linux...@kvack.org; linux-kernel@vger.kernel.org; > de...@driverdev.osuosl.org > Subject: Re: [PATCHv2 8/9] zswap: add to mm/ > > On 01/07/2013 12:24 PM, Seth Jennings wrote: > > +struct zswap_tree { > > + str

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-08 Thread Dave Hansen
On 01/07/2013 12:24 PM, Seth Jennings wrote: > +struct zswap_tree { > + struct rb_root rbroot; > + struct list_head lru; > + spinlock_t lock; > + struct zs_pool *pool; > +}; BTW, I spent some time trying to get this lock contended. You thought the anon_vma locks would dominate

Re: [PATCHv2 8/9] zswap: add to mm/

2013-01-08 Thread Dave Hansen
On 01/07/2013 12:24 PM, Seth Jennings wrote: +struct zswap_tree { + struct rb_root rbroot; + struct list_head lru; + spinlock_t lock; + struct zs_pool *pool; +}; BTW, I spent some time trying to get this lock contended. You thought the anon_vma locks would dominate and this

RE: [PATCHv2 8/9] zswap: add to mm/

2013-01-08 Thread Dan Magenheimer
; Larry Woodman; linux...@kvack.org; linux-kernel@vger.kernel.org; de...@driverdev.osuosl.org Subject: Re: [PATCHv2 8/9] zswap: add to mm/ On 01/07/2013 12:24 PM, Seth Jennings wrote: +struct zswap_tree { + struct rb_root rbroot; + struct list_head lru; + spinlock_t lock

[PATCHv2 8/9] zswap: add to mm/

2013-01-07 Thread Seth Jennings
zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim and dramatically reduced swap device I/O. Additional, in most cases, pages can be retrieved from this

[PATCHv2 8/9] zswap: add to mm/

2013-01-07 Thread Seth Jennings
zswap is a thin compression backend for frontswap. It receives pages from frontswap and attempts to store them in a compressed memory pool, resulting in an effective partial memory reclaim and dramatically reduced swap device I/O. Additional, in most cases, pages can be retrieved from this