> > drm_handle_t appears to be assigned values from void*. As such unsigned > int is certainly not the same size on 64-bit. Convert to uintptr_t in > all cases as it is defined for this purpose.
No, its "designed" as is. We can't change it now as its ABI. We make sure we only use 32-bit handles anyways. Dave. > > I fear this patch changes ABI, but I am not sure. > > Matthew W.S. Bell > > --- > include/drm/drm.h | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/drm/drm.h b/include/drm/drm.h > index 4822159..95141a2 100644 > --- a/include/drm/drm.h > +++ b/include/drm/drm.h > @@ -40,7 +40,7 @@ > > #include <linux/types.h> > #include <asm/ioctl.h> > -typedef unsigned int drm_handle_t; > +typedef uintptr_t drm_handle_t; > > #else /* One of the BSDs */ > > @@ -54,7 +54,7 @@ typedef int32_t __s32; > typedef uint32_t __u32; > typedef int64_t __s64; > typedef uint64_t __u64; > -typedef unsigned long drm_handle_t; > +typedef uintptr_t drm_handle_t; > > #endif > > ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
