From c551087d967643b9983057a2bf0813205f433120 Mon Sep 17 00:00:00 2001 From: Guillaume Tucker Date: Wed, 13 Feb 2019 09:31:37 +0000 Subject: i915/gem_ctx_sseu: Fix 32-bit build MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-by: Chris Wilson --- tests/i915/gem_ctx_sseu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests/i915/gem_ctx_sseu.c') diff --git a/tests/i915/gem_ctx_sseu.c b/tests/i915/gem_ctx_sseu.c index 16609bee..3afa5c15 100644 --- a/tests/i915/gem_ctx_sseu.c +++ b/tests/i915/gem_ctx_sseu.c @@ -381,7 +381,7 @@ test_ggtt_args(int fd) igt_assert_eq(__gem_context_get_param(fd, &arg), 0); igt_assert_eq(__gem_context_set_param(fd, &arg), 0); - munmap((void *)arg.value, 4096); + munmap((void *)(uintptr_t)arg.value, 4096); gem_close(fd, bo); gem_context_destroy(fd, arg.ctx_id); } -- cgit v1.2.3