Re: invalidate_inode_pages in 2.5.32/3

From: Andrew Morton (akpm@digeo.com)
Date: Tue Sep 10 2002 - 19:07:18 EST


Daniel Phillips wrote:
>
> ...
> Andrew, did I miss something, or does the current code really ignore
> the pte dirty bits?

Sure. pte_dirty -> PageDirty propagation happens in page reclaim,
and in msync.

We _could_ walk the pte chain in writeback. But that would involve
visiting every page in the mapping, basically. That could hurt.

But if a page is already dirty, and we're going to write it anyway,
it makes tons of sense to run around and clean all the ptes which
point at it.

It especially makes sense for fielmap_sync() to do that. (quickly
edits the todo file).

I'm not sure that MAP_SHARED is a good way of performing file writing,
really. And not many things seem to use it for that. It's more there
as a way for unrelated processes to find a chunk of common memory via
the usual namespace. Not sure about that, but I am sure that it's a
damn pest.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Sep 15 2002 - 22:00:23 EST