xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* xen fix radix tree build
@ 2011-05-11  9:45 Christoph Egger
  2011-05-11 11:56 ` Christoph Egger
  0 siblings, 1 reply; 2+ messages in thread
From: Christoph Egger @ 2011-05-11  9:45 UTC (permalink / raw)
  To: xen-devel@lists.xensource.com, Jan Beulich


Hi,

the radix tree headers don't compile:

xen/include/xen/radix-tree.h: In function 'radix_tree_int_to_ptr':
xen/include/xen/radix-tree.h:184: warning: comparison is always true due 
to limited range of data type
xen/include/xen/radix-tree.h:184: warning: comparison is always true due 
to limited range of data type

An 'int' is too small to hold a pointer.
C99 says 'uintptr_t' or 'intptr_t' is correct.

Christoph


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: xen fix radix tree build
  2011-05-11  9:45 xen fix radix tree build Christoph Egger
@ 2011-05-11 11:56 ` Christoph Egger
  0 siblings, 0 replies; 2+ messages in thread
From: Christoph Egger @ 2011-05-11 11:56 UTC (permalink / raw)
  To: xen-devel

On 05/11/11 11:45, Christoph Egger wrote:
>
> Hi,
>
> the radix tree headers don't compile:
>
> xen/include/xen/radix-tree.h: In function 'radix_tree_int_to_ptr':
> xen/include/xen/radix-tree.h:184: warning: comparison is always true due
> to limited range of data type
> xen/include/xen/radix-tree.h:184: warning: comparison is always true due
> to limited range of data type
>
> An 'int' is too small to hold a pointer.
> C99 says 'uintptr_t' or 'intptr_t' is correct.
>
> Christoph

Changeset 23324 fixes this. Thanks Keir.

Christoph


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-05-11 11:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-11  9:45 xen fix radix tree build Christoph Egger
2011-05-11 11:56 ` Christoph Egger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).