Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-04 Thread Minchan Kim
Hi, Nitin. I am happy to hear you started this work. On Fri, Jun 04, 2010 at 03:06:49PM +0530, Nitin Gupta wrote: > On 06/03/2010 09:13 PM, Dan Magenheimer wrote: > >> On 06/03/2010 10:23 AM, Andreas Dilger wrote: > >>> On 2010-06-02, at 20:46, Nitin Gupta wrote: > >> > >>> I was thinking it wo

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-04 Thread Nitin Gupta
On 06/03/2010 09:13 PM, Dan Magenheimer wrote: >> On 06/03/2010 10:23 AM, Andreas Dilger wrote: >>> On 2010-06-02, at 20:46, Nitin Gupta wrote: >> >>> I was thinking it would be quite clever to do compression in, say, >>> 64kB or 128kB chunks in a mapping (to get decent compression) and >>> then wr

RE: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-03 Thread Dan Magenheimer
> On 06/03/2010 10:23 AM, Andreas Dilger wrote: > > On 2010-06-02, at 20:46, Nitin Gupta wrote: > > > I was thinking it would be quite clever to do compression in, say, > > 64kB or 128kB chunks in a mapping (to get decent compression) and > > then write these compressed chunks directly from the pa

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Nitin Gupta
On 06/03/2010 10:23 AM, Andreas Dilger wrote: > On 2010-06-02, at 20:46, Nitin Gupta wrote: >> On 06/03/2010 04:32 AM, Dan Magenheimer wrote: From: Minchan Kim [mailto:minchan@gmail.com] >>> > I am also eagerly awaiting Nitin Gupta's cleancache backend > and implementation to do in

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Andreas Dilger
On 2010-06-02, at 20:46, Nitin Gupta wrote: > On 06/03/2010 04:32 AM, Dan Magenheimer wrote: >>> From: Minchan Kim [mailto:minchan@gmail.com] >> I am also eagerly awaiting Nitin Gupta's cleancache backend and implementation to do in-kernel page cache compression. >>> >>> Do Nitin sa

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Nitin Gupta
On 06/03/2010 04:32 AM, Dan Magenheimer wrote: >> From: Minchan Kim [mailto:minchan@gmail.com] > >>> I am also eagerly awaiting Nitin Gupta's cleancache backend >>> and implementation to do in-kernel page cache compression. >> >> Do Nitin say he will make backend of cleancache for >> page cach

RE: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Dan Magenheimer
> From: Minchan Kim [mailto:minchan@gmail.com] > > I am also eagerly awaiting Nitin Gupta's cleancache backend > > and implementation to do in-kernel page cache compression. > > Do Nitin say he will make backend of cleancache for > page cache compression? > > It would be good feature. > I ha

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Minchan Kim
Hi, Dan. On Wed, Jun 02, 2010 at 08:27:48AM -0700, Dan Magenheimer wrote: > Hi Minchan -- > > > I think cleancache approach is cool. :) > > I have some suggestions and questions. > > Thanks for your interest! > > > > If a get_page is successful on a non-shared pool, the page is flushed > > (th

RE: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Dan Magenheimer
> From: Christoph Hellwig [mailto:h...@infradead.org] > Subject: Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): > overview Hi Christophe -- Thanks for your feedback! > > fs/btrfs/super.c |2 > > fs/buffer.c|5 + > > fs/ext

RE: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Dan Magenheimer
> From: Jamie Lokier [mailto:ja...@shareable.org] > Subject: Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): > overview > > Dan Magenheimer wrote: > > Most important, cleancache is "ephemeral". Pages which are copied > into > > cleancache have an

RE: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Dan Magenheimer
Hi Minchan -- > I think cleancache approach is cool. :) > I have some suggestions and questions. Thanks for your interest! > > If a get_page is successful on a non-shared pool, the page is flushed > (thus > > making cleancache an "exclusive" cache).  On a shared pool, the page > > Do you have a

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Christoph Hellwig
Please give your patches some semi-resonable subject line. > fs/btrfs/super.c |2 > fs/buffer.c|5 + > fs/ext3/super.c|2 > fs/ext4/super.c|2 > fs/mpage.c |7 + > fs/ocfs2/super.c |3 > fs/sup

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-02 Thread Jamie Lokier
Dan Magenheimer wrote: > Most important, cleancache is "ephemeral". Pages which are copied into > cleancache have an indefinite lifetime which is completely unknowable > by the kernel and so may or may not still be in cleancache at any later time. > Thus, as its name implies, cleancache is not sui

Re: [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-06-01 Thread Minchan Kim
Hello. I think cleancache approach is cool. :) I have some suggestions and questions. On Sat, May 29, 2010 at 2:35 AM, Dan Magenheimer wrote: > [PATCH V2 0/7] Cleancache (was Transcendent Memory): overview > > Changes since V1: > - Rebased to 2.6.34 (no functional changes) > -

[PATCH V2 0/7] Cleancache (was Transcendent Memory): overview

2010-05-28 Thread Dan Magenheimer
[PATCH V2 0/7] Cleancache (was Transcendent Memory): overview Changes since V1: - Rebased to 2.6.34 (no functional changes) - Convert to sane types (Al Viro) - Define some raw constants (Konrad Wilk) - Add ack from Andreas Dilger In previous patch postings, cleancache was part of the