On Tue, 2007-06-12 at 20:07 +0800, Fengguang Wu wrote:
> Hi Rusty,

Hi Fengguang,

> or preferably:
> 
>       pgoff_t start;                 /* where readahead started */
>         unsigned long size;            /* # of readahead pages */
>       unsigned long async_size;      /* do asynchronous readahead when there 
> are only # of pages ahead */
> 
>       unsigned long async_size_old;  /* TODO: this one is not needed for now 
> */
> 
> Any opinions? Thanks.

These names and comments are really nice.  I think the code will become
more readable after this, too.

Did you want me to try to make this patch, or did you want to do it?

Thanks,
Rusty.


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

Reply via email to