Messages in this thread | | | Date | Thu, 12 Apr 2007 13:26:00 +0200 (CEST) | From | Roman Zippel <> | Subject | Re: [PATCH 1/17] cramfs: use read_mapping_page |
| |
Hi,
On Thu, 12 Apr 2007, Christoph Hellwig wrote:
> On Wed, Apr 11, 2007 at 07:49:38PM -0700, Nate Diller wrote: > > read_mapping_page_async() is going away, so convert its only user to > > read_mapping_page(). This change has not been benchmarked, however, in > > order to get real parallelism this wants something completely different, > > like __do_page_cache_readahead(), which is not currently exported. > > Why is read_mapping_page_async going away? This probably needs a lot more > testing, and I'd be much happier if you split it out of the series and > sent it separately at the end.
That function wasn't fully async anyway, as it would often sleep in lock_page(). AFAICT only in the special case of a partial written page would this function return a not yet uptodate page.
bye, Roman - 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/
| |