Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-07-12 Thread Nitin Gupta
On 07/07/2010 02:21 AM, Konrad Rzeszutek Wilk wrote: On Tue, Jun 22, 2010 at 09:26:28AM -0700, Dave Hansen wrote: On Mon, 2010-06-21 at 16:19 -0700, Dan Magenheimer wrote: --- linux-2.6.35-rc2/include/linux/cleancache.h 1969-12-31 17:00:00.0 -0700 +++

Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-07-12 Thread Konrad Rzeszutek Wilk
Can you reference with a link or a git branch the patches that utilize this? And also mention that in the 0/X patch so that folks can reference your cleancache implementation? FYI. I am working on 'zcache' which uses cleancache_ops to provide page cache compression support.

Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-07-12 Thread Konrad Rzeszutek Wilk
On Tue, Jun 22, 2010 at 09:26:28AM -0700, Dave Hansen wrote: On Mon, 2010-06-21 at 16:19 -0700, Dan Magenheimer wrote: --- linux-2.6.35-rc2/include/linux/cleancache.h 1969-12-31 17:00:00.0 -0700 +++ linux-2.6.35-rc2-cleancache/include/linux/cleancache.h 2010-06-21

Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-06-28 Thread Nitin Gupta
On 06/22/2010 08:13 PM, Christoph Hellwig wrote: On Mon, Jun 21, 2010 at 04:19:39PM -0700, Dan Magenheimer wrote: [PATCH V3 3/8] Cleancache: core ops functions and configuration Cleancache core ops functions and configuration NACK for code that just adds random hooks all over VFS and even

Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-06-28 Thread Dave Hansen
On Mon, 2010-06-21 at 16:19 -0700, Dan Magenheimer wrote: --- linux-2.6.35-rc2/include/linux/cleancache.h 1969-12-31 17:00:00.0 -0700 +++ linux-2.6.35-rc2-cleancache/include/linux/cleancache.h 2010-06-21 14:45:18.0 -0600 @@ -0,0 +1,88 @@ +#ifndef _LINUX_CLEANCACHE_H

Re: [Ocfs2-devel] [PATCH V3 3/8] Cleancache: core ops functions and configuration

2010-06-28 Thread Dan Magenheimer
Hi Christoph -- Thanks for the comments... replying to both in one reply. Subject: Re: [PATCH V3 0/8] Cleancache: overview What all this fails to explain is that this actually is useful for? See FAQ #1 in patch 1/8 (and repeated in patch 0/8). But, in a few words, it's useful for