On Mon, Oct 12, 2009 at 05:37:27AM +0800, Roel Kluin wrote:
> If not signed, testing of the read() return value in this function
> will not work.
Good catch! "long" is large enough since we do read in batches.
However the latest linux tree should have fixed this problem.
Thank you anyway~
Fengguang
> Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx>
> ---
> Or is long too small?
>
> diff --git a/Documentation/vm/page-types.c b/Documentation/vm/page-types.c
> index fa1a30d..d8a799a 100644
> --- a/Documentation/vm/page-types.c
> +++ b/Documentation/vm/page-types.c
> @@ -416,7 +416,7 @@ static void add_page(unsigned long offset, uint64_t flags)
> static void walk_pfn(unsigned long index, unsigned long count)
> {
> unsigned long batch;
> - unsigned long n;
> + long n;
> unsigned long i;
>
> if (index > ULONG_MAX / KPF_BYTES)
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
