Re: [PATCH 1/9] readahead: introduce PG_readahead

From: Rusty Russell
Date: Mon Jun 11 2007 - 21:05:26 EST


On Thu, 2007-05-17 at 06:47 +0800, Fengguang Wu wrote:
> plain text document attachment (mm-introduce-pg_readahead.patch)
> Introduce a new page flag: PG_readahead.
>
> It acts as a look-ahead mark, which tells the page reader:
> Hey, it's time to invoke the read-ahead logic. For the sake of I/O pipelining,
> don't wait until it runs out of cached pages!

Hi Fengguang!

I've been reading your patches, and I have some (possibly dumb!)
questions.

For this patch: why set a bit in the page, rather than keep a value
inside the "struct file_ra_state"?

Thanks,
Rusty.


-
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/