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