Re: [PATCH i-g-t] i915/gem_ctx_sseu: Fix 32-bit build

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Guillaume Tucker (2019-02-13 09:31:37)
> This fixes a compiler warning treated as an error when building for
> 32-bit architectures since their pointer size does not match the size
> of drm_i915_gem_context_param.value which is 64 bits:
> 
>   CC       i915/gem_ctx_sseu.o
>   i915/gem_ctx_sseu.c: In function ‘test_ggtt_args’:
>   i915/gem_ctx_sseu.c:384:9: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
>     munmap((void *)arg.value, 4096);
> 
> It was found while building for arm with gcc 6.3.0 and I suspect the
> same problem would arise for i386 or other 32-bit architectures.  The
> uintptr_t type is by definition an unsigned integer of the same length
> as a pointer on a given architecture, so this should fix the problem
> for all architectures up to 64 bits.
> 
> Signed-off-by: Guillaume Tucker <guillaume.tucker@xxxxxxxxxxxxx>

At some point, we should save our eyesight and do u64_to_pointer().
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux