[PATCH i-g-t 2/3] tests/intel/xe_eudebug_online: Fix 32b compilation warnings

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jan 24 14:00:13 UTC 2025


Hi Dominik,
On 2025-01-22 at 15:44:18 +0100, Dominik Grzegorzek wrote:
> Fix all debug prints which were causing compilation warnings for 32b
> archs.
> 
> Signed-off-by: Dominik Grzegorzek <dominik.grzegorzek at intel.com>

LGTM
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/intel/xe_eudebug_online.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel/xe_eudebug_online.c b/tests/intel/xe_eudebug_online.c
> index f532387af..1a146d8c4 100644
> --- a/tests/intel/xe_eudebug_online.c
> +++ b/tests/intel/xe_eudebug_online.c
> @@ -569,7 +569,7 @@ static void get_aips_offset_table(struct online_debug_data *data, int threads)
>  
>  	igt_debug("AIPs offset table:\n");
>  	for (int i = 0; i < threads; i++)
> -		igt_debug("%lx\n", data->aips_offset_table[i]);
> +		igt_debug("%" PRIx64 "\n", data->aips_offset_table[i]);
>  }
>  
>  static int get_stepped_threads_count(struct online_debug_data *data, int threads)
> @@ -2328,11 +2328,11 @@ static void test_many_sessions_on_tiles(int fd, bool multi_tile)
>  
>  		if (multi_tile)
>  			igt_assert_f(diff < WORKLOAD_DELAY_US,
> -				     "Expected to execute workloads concurrently. Actual delay: %lu ms\n",
> +				     "Expected to execute workloads concurrently. Actual delay: %" PRIu64 " ms\n",
>  				     diff);
>  		else
>  			igt_assert_f(diff >= WORKLOAD_DELAY_US,
> -				     "Expected a serialization of workloads. Actual delay: %lu ms\n",
> +				     "Expected a serialization of workloads. Actual delay: %" PRIu64 " ms\n",
>  				     diff);
>  	}
>  
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list