[Ocfs2-devel] [PATCH V8 4/8] mm/fs: add hooks to support cleancache
Dan Magenheimer
dan.magenheimer at oracle.com
Fri Apr 15 07:47:57 PDT 2011
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 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 "invalidate" are defined
precisely or used consistently everywhere in computer
science, but I think that "invalidate" is to destroy
a "pointer" to some data, but not necessarily destroy the
data itself. And "flush" means to actually remove
the data. So one would "invalidate a mapping" but one
would "flush a cache".
Since cleancache_flush_page and cleancache_flush_inode
semantically remove data from cleancache, I think flush
is a better name than invalidate.
Does that make sense?
Thanks,
Dan
More information about the Ocfs2-devel
mailing list