RE: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-26 Thread Dan Magenheimer
From: Minchan Kim [mailto:minchan@gmail.com] On Sat, Apr 16, 2011 at 3:53 AM, Dan Magenheimer dan.magenhei...@oracle.com wrote: From: OGAWA Hirofumi [mailto:hirof...@mail.parknet.co.jp] Andrew Morton a...@linux-foundation.org writes: Andrew, I wonder if you would be so kind as to

Re: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-17 Thread Minchan Kim
On Sat, Apr 16, 2011 at 3:53 AM, Dan Magenheimer dan.magenhei...@oracle.com wrote: From: OGAWA Hirofumi [mailto:hirof...@mail.parknet.co.jp] Andrew Morton a...@linux-foundation.org writes: Before I suggested a thing about cleancache_flush_page, cleancache_flush_inode. what's the

RE: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-15 Thread Dan Magenheimer
Hi Minchan -- First of all, thanks for resolving conflict with my patch. You're welcome! As I pointed out offlist, yours was the first change in MM that caused any semantic changes to the cleancache core hooks patch since before 2.6.18. Before I suggested a thing about

Re: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-15 Thread Andrew Morton
On Fri, 15 Apr 2011 07:47:57 -0700 (PDT) Dan Magenheimer dan.magenhei...@oracle.com wrote: Hi Minchan -- First of all, thanks for resolving conflict with my patch. You're welcome! As I pointed out offlist, yours was the first change in MM that caused any semantic changes to the

Re: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-15 Thread OGAWA Hirofumi
Andrew Morton a...@linux-foundation.org writes: Before I suggested a thing about cleancache_flush_page, cleancache_flush_inode. what's the meaning of flush's semantic? I thought it means invalidation. AFAIC, how about change flush with invalidate? I'm not sure the words flush and

RE: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-15 Thread Dan Magenheimer
From: OGAWA Hirofumi [mailto:hirof...@mail.parknet.co.jp] Andrew Morton a...@linux-foundation.org writes: Before I suggested a thing about cleancache_flush_page, cleancache_flush_inode. what's the meaning of flush's semantic? I thought it means invalidation. AFAIC, how

[PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-14 Thread Dan Magenheimer
[PATCH V8 4/8] mm/fs: add hooks to support cleancache This fourth patch of eight in this cleancache series provides the core hooks in VFS for: initializing cleancache per filesystem; capturing clean pages reclaimed by page cache; attempting to get pages from cleancache before filesystem read

Re: [PATCH V8 4/8] mm/fs: add hooks to support cleancache

2011-04-14 Thread Minchan Kim
Hi Dan, On Fri, Apr 15, 2011 at 6:17 AM, Dan Magenheimer dan.magenhei...@oracle.com wrote: [PATCH V8 4/8] mm/fs: add hooks to support cleancache This fourth patch of eight in this cleancache series provides the core hooks in VFS for: initializing cleancache per filesystem; capturing clean