lkml.org 
[lkml]   [2009]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: linux-next: Tree for October 2: percpu compile warnings (i386)
Hello,

Randy Dunlap wrote:
> mm/percpu.c:1873: warning: comparison of distinct pointer types lacks a cast
> mm/percpu.c:1879: warning: format '%lx' expects type 'long unsigned int', but argument 2 has type 'size_t'
>
> The second one is easily fixed (s/%lx/%zu/), but is that the correct fix?

I think so. Will fix it up.

> The first one is a max(size_t, unsigned long).
> The C99 spec says that the max value of a size_t is:
> limit of size_t
> SIZE_MAX 65535

The same section also says

limits of ptrdiff_t
PTRDIFF_MIN -65535
PTRDIFF_MAX +65535

and ptrdiff_t is defined as

ptrdiff_t
which is the signed integer type of the result of subtracting two
pointers;

Also, later in that section.

The types used for size_t and ptrdiff_t should not have an integer
conversion rank greater than that of signed long int unless the
implementation supports objects large enough to make this necessary.

So, it looks like the limits there are examples from 16bit systems
although it's not stated anywhere. Strange choice...

Anyways, at least in kernel, it's assumed to be large enough to carry
any supported memory size and in that sense it should be as large as
ptrdiff_t or intptr_t. Otherwise, we're already deeply screwed (no
allocation larger than 64k or no vmalloc area larger than 64k!).

Thanks.

--
tejun


\
 
 \ /
  Last update: 2009-10-04 02:31    [from the cache]
©2003-2011 Jasper Spaans