Re: [Mesa-dev] [PATCH] i965: fix cast in OA performance query code

[Tapani Pälli]

On 05/11/2017 02:23 PM, Emil Velikov wrote:
On 1 May 2017 at 12:09, Mauro Rossi <issor.or...@gmail.com> wrote:
Fixes the following building error happening on marshmallow-x86
with Ubuntu 16.04 LTS stock build essentials package
and by the book Android build system.

external/mesa/src/mesa/drivers/dri/i965/brw_performance_query.c: In function 
'open_i915_perf_oa_stream':
external/mesa/src/mesa/drivers/dri/i965/brw_performance_query.c:836:25:
error: cast from pointer to integer of different size 
[-Werror=pointer-to-int-cast]
        .properties_ptr = (uint64_t)properties
                          ^

Fixes: 458468c "i965: Expose OA counters via INTEL_performance_query"
---
  src/mesa/drivers/dri/i965/brw_performance_query.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_performance_query.c 
b/src/mesa/drivers/dri/i965/brw_performance_query.c
index 95f112e..7436b39 100644
--- a/src/mesa/drivers/dri/i965/brw_performance_query.c
+++ b/src/mesa/drivers/dri/i965/brw_performance_query.c
@@ -833,7 +833,7 @@ open_i915_perf_oa_stream(struct brw_context *brw,
                 I915_PERF_FLAG_FD_NONBLOCK |
                 I915_PERF_FLAG_DISABLED,
        .num_properties = ARRAY_SIZE(properties) / 2,
-      .properties_ptr = (uint64_t)properties
+      .properties_ptr = (uint64_t *)properties
Rob, others - do we still need this?

I did brief look here but not 100% sure if this is right fix, 
properties_ptr is uint64_t so I guess you might end up with another 
warning/error here, should uintptr_t be used instead?

Thanks
Emil

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev