Re: [PATCH 2/6] fs: Take mapping lock in generic read paths

From: Andrew Morton
Date: Thu Jan 31 2013 - 18:59:47 EST


On Thu, 31 Jan 2013 22:49:50 +0100
Jan Kara <jack@xxxxxxx> wrote:

> Add mapping lock to struct address_space and grab it in all paths
> creating pages in page cache to read data into them. That means buffered
> read, readahead, and page fault code.

Boy, this does look expensive in both speed and space.

As you pointed out in [0/n], it's 2-3%. As always with pagecache
stuff, the cost of filling the page generally swamps any inefficiencies
in preparing that page.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/