[PATCH i-g-t v3 8/8] tests/intel: fixup print/scan formatting for files in intel tests
Jan Sokolowski
jan.sokolowski at intel.com
Mon May 26 08:07:40 UTC 2025
Compiling on other architectures than x86-64 causes a lot
of print/scan formatting warnings.
Fix formatting by using proper formatters, eg. PRIu64.
Signed-off-by: Jan Sokolowski <jan.sokolowski at intel.com>
---
tests/intel/xe_compute_preempt.c | 5 +++--
tests/intel/xe_eu_stall.c | 4 ++--
tests/intel/xe_eudebug.c | 4 ++--
tests/intel/xe_exec_capture.c | 9 +++++----
tests/intel/xe_exec_system_allocator.c | 8 ++++----
tests/intel/xe_oa.c | 6 +++---
tests/intel/xe_pmu.c | 16 ++++++++--------
tests/intel/xe_render_copy.c | 2 +-
8 files changed, 28 insertions(+), 26 deletions(-)
diff --git a/tests/intel/xe_compute_preempt.c b/tests/intel/xe_compute_preempt.c
index 43ffb8dba..f8f08c664 100644
--- a/tests/intel/xe_compute_preempt.c
+++ b/tests/intel/xe_compute_preempt.c
@@ -13,6 +13,7 @@
*/
#include <string.h>
+#include <inttypes.h>
#include "igt.h"
#include "intel_compute.h"
@@ -84,7 +85,7 @@ igt_main
*/
child_count = ram_mb / 2 / CONTEXT_MB / 2;
- igt_debug("RAM: %zd, child count: %d\n",
+ igt_debug("RAM: %" PRIu64 ", child count: %d\n",
ram_mb, child_count);
test_compute_preempt(xe, hwe, false);
@@ -111,7 +112,7 @@ igt_main
*/
child_count = ram_mb / CONTEXT_MB / 2;
- igt_debug("RAM: %zd, child count: %d\n",
+ igt_debug("RAM: %" PRIu64 ", child count: %d\n",
ram_mb, child_count);
test_compute_preempt(xe, hwe, false);
diff --git a/tests/intel/xe_eu_stall.c b/tests/intel/xe_eu_stall.c
index 411c30871..b27f5685e 100644
--- a/tests/intel/xe_eu_stall.c
+++ b/tests/intel/xe_eu_stall.c
@@ -516,7 +516,7 @@ static void test_eustall(int drm_fd, uint32_t devid, bool blocking_read, int ite
igt_assert(query_eu_stall_data->num_sampling_rates > 0);
if (p_rate == 0)
properties[3] = query_eu_stall_data->sampling_rates[0];
- igt_info("Sampling Rate: %lu\n", properties[3]);
+ igt_info("Sampling Rate: %" PRIu64 "\n", properties[3]);
stream_fd = xe_eu_stall_ioctl(drm_fd, DRM_XE_OBSERVATION_OP_STREAM_OPEN, &props);
igt_require_fd(stream_fd);
@@ -580,7 +580,7 @@ enable:
}
} while (child_is_running);
- igt_info("Total size read: %lu\n", total_size);
+ igt_info("Total size read: %" PRIu64 "\n", total_size);
igt_info("Number of samples: %u\n", num_samples);
igt_info("Number of drops reported: %u\n", num_drops);
diff --git a/tests/intel/xe_eudebug.c b/tests/intel/xe_eudebug.c
index 60700a8bb..bdf73c4e9 100644
--- a/tests/intel/xe_eudebug.c
+++ b/tests/intel/xe_eudebug.c
@@ -1156,7 +1156,7 @@ static void *discovery_race_thread(void *data)
xe_eudebug_debugger_stop_worker(s->debugger, 1);
- igt_debug("Resources discovered: %lu\n", s->debugger->event_count);
+ igt_debug("Resources discovered: %" PRIu64 "\n", s->debugger->event_count);
if (!s->client->done) {
xe_eudebug_for_each_event(e, s->debugger->log) {
if (e->type == DRM_XE_EUDEBUG_EVENT_OPEN) {
@@ -2669,7 +2669,7 @@ static void vma_ufence_trigger(struct xe_eudebug_debugger *d,
ef->vm_bind_ref_seqno);
xe_eudebug_assert_f(d, eb->flags & DRM_XE_EUDEBUG_EVENT_VM_BIND_FLAG_UFENCE,
"vm bind event does not have ufence: %s\n", event_str);
- igt_debug("vm bind ufence event received with ref %lld, addr 0x%lx, range 0x%lx\n",
+ igt_debug("vm bind ufence event received with ref %lld, addr 0x%" PRIu64 ", range 0x%" PRIu64 "\n",
ef->vm_bind_ref_seqno,
addr,
range);
diff --git a/tests/intel/xe_exec_capture.c b/tests/intel/xe_exec_capture.c
index 92113cdb1..35f1222db 100644
--- a/tests/intel/xe_exec_capture.c
+++ b/tests/intel/xe_exec_capture.c
@@ -17,6 +17,7 @@
#include <regex.h>
#include <stdio.h>
#include <string.h>
+#include <inttypes.h>
#include "igt.h"
#include "igt_device.h"
@@ -290,10 +291,10 @@ check_item_u64(regex_t *regex, char **lines, const char *tag, u64 addr_lo,
"Target not found:%s\n", tag);
result = compare_hex_value(output);
- igt_debug("Compare %s %s vs [0x%lX-0x%lX] result %lX\n", tag, output,
+ igt_debug("Compare %s %s vs [0x%" PRIX64 "-0x%" PRIX64 "] result %" PRIX64 "\n", tag, output,
addr_lo, addr_hi, result);
igt_assert_f((addr_lo <= result) && (result <= addr_hi),
- "value %lX out of range[0x%lX-0x%lX]\n", result, addr_lo, addr_hi);
+ "value %" PRIX64 " out of range[0x%" PRIX64 "-0x%" PRIX64 "]\n", result, addr_lo, addr_hi);
}
static void
@@ -423,7 +424,7 @@ igt_main
/* Reduce timeout value to speedup test */
xe_sysfs_set_job_timeout_ms(xe, hwe, CAPTURE_JOB_TIMEOUT);
- igt_debug("Reduced %s class timeout from %ld to %d\n",
+ igt_debug("Reduced %s class timeout from %" PRIu64 " to %d\n",
xe_engine_class_name(hwe->engine_class),
timeouts[hwe->engine_class], CAPTURE_JOB_TIMEOUT);
}
@@ -452,7 +453,7 @@ igt_main
store = xe_sysfs_get_job_timeout_ms(xe, hwe);
igt_abort_on_f(timeout != store, "job_timeout_ms not restored!\n");
- igt_debug("Restored %s class timeout to %ld\n",
+ igt_debug("Restored %s class timeout to %" PRIu64 "\n",
xe_engine_class_name(hwe->engine_class),
timeouts[hwe->engine_class]);
diff --git a/tests/intel/xe_exec_system_allocator.c b/tests/intel/xe_exec_system_allocator.c
index 06daac8c2..564ea22b8 100644
--- a/tests/intel/xe_exec_system_allocator.c
+++ b/tests/intel/xe_exec_system_allocator.c
@@ -325,7 +325,7 @@ static void touch_all_pages(int fd, uint32_t exec_queue, void *ptr,
&timeout);
if (ret) {
igt_info("FAIL EXEC_UFENCE_ADDR: 0x%016llx\n", sync[0].addr);
- igt_info("FAIL EXEC_UFENCE: EXPECTED=0x%016llx, ACTUAL=0x%016lx\n",
+ igt_info("FAIL EXEC_UFENCE: EXPECTED=0x%016llx, ACTUAL=0x%016" PRIx64 "\n",
USER_FENCE_VALUE, exec_ufence[0]);
addr = to_user_pointer(ptr);
@@ -336,9 +336,9 @@ static void touch_all_pages(int fd, uint32_t exec_queue, void *ptr,
uint64_t sdi_offset = (char *)&data->data - (char *)data;
uint64_t sdi_addr = addr + sdi_offset;
- igt_info("FAIL BATCH_ADDR: 0x%016lx\n", batch_addr);
- igt_info("FAIL SDI_ADDR: 0x%016lx\n", sdi_addr);
- igt_info("FAIL SDI_ADDR (in batch): 0x%016lx\n",
+ igt_info("FAIL BATCH_ADDR: 0x%016" PRIx64 "\n", batch_addr);
+ igt_info("FAIL SDI_ADDR: 0x%016" PRIx64 "\n", sdi_addr);
+ igt_info("FAIL SDI_ADDR (in batch): 0x%016" PRIx64 "\n",
(((u64)data->batch[2]) << 32) | data->batch[1]);
igt_info("FAIL DATA: EXPECTED=0x%08x, ACTUAL=0x%08x\n",
data->expected_data, data->data);
diff --git a/tests/intel/xe_oa.c b/tests/intel/xe_oa.c
index 2022c5cb3..cb59918ef 100644
--- a/tests/intel/xe_oa.c
+++ b/tests/intel/xe_oa.c
@@ -1015,13 +1015,13 @@ static void pec_sanity_check(const u32 *report0, const u32 *report1,
return;
}
- igt_debug("tick delta = %#lx\n", tick_delta);
+ igt_debug("tick delta = %#" PRIx64 "\n", tick_delta);
/* Difference in test_event1_cycles_xecore* values should be close to tick_delta */
for (int i = 0; i < ARRAY_SIZE(xecore_idx); i++) {
int n = xecore_idx[i];
- igt_debug("n %d: pec1[n] - pec0[n] %#lx, tick delta %#lx\n",
+ igt_debug("n %d: pec1[n] - pec0[n] %#" PRIx64 ", tick delta %#" PRIx64 "\n",
n, pec1[n] - pec0[n], tick_delta);
/* 0 value for pec[xecore_idx[i]] indicates missing xecore */
if (pec1[n] && pec0[n])
@@ -1031,7 +1031,7 @@ static void pec_sanity_check(const u32 *report0, const u32 *report1,
igt_assert(pec0[n]);
}
- igt_debug("pec1[2] - pec0[2] %#lx, tick_delta * num_xecores: %#lx\n",
+ igt_debug("pec1[2] - pec0[2] %#" PRIx64 ", tick_delta * num_xecores: %#" PRIx64 "\n",
pec1[2] - pec0[2], tick_delta * intel_xe_perf->devinfo.n_eu_sub_slices);
/* Difference in test_event1_cycles should be close to (tick_delta * num_xecores) */
assert_within_epsilon(pec1[2] - pec0[2],
diff --git a/tests/intel/xe_pmu.c b/tests/intel/xe_pmu.c
index 02f23bd7d..8ed2fc3c0 100644
--- a/tests/intel/xe_pmu.c
+++ b/tests/intel/xe_pmu.c
@@ -200,9 +200,9 @@ static void check_all_engines(int num_engines, int *flag, uint64_t *before, uint
engine_active_ticks = after[idx] - before[idx];
engine_total_ticks = after[idx + 1] - before[idx + 1];
- igt_debug("[%d] Engine active ticks: after %ld, before %ld delta %ld\n", engine_idx,
+ igt_debug("[%d] Engine active ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", engine_idx,
after[idx], before[idx], engine_active_ticks);
- igt_debug("[%d] Engine total ticks: after %ld, before %ld delta %ld\n", engine_idx,
+ igt_debug("[%d] Engine total ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", engine_idx,
after[idx + 1], before[idx + 1], engine_total_ticks);
if (flag[engine_idx] == TEST_LOAD)
@@ -243,9 +243,9 @@ static void engine_activity(int fd, struct drm_xe_engine_class_instance *eci, un
engine_active_ticks = after[0] - before[0];
engine_total_ticks = after[1] - before[1];
- igt_debug("Engine active ticks: after %ld, before %ld delta %ld\n", after[0], before[0],
+ igt_debug("Engine active ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", after[0], before[0],
engine_active_ticks);
- igt_debug("Engine total ticks: after %ld, before %ld delta %ld\n", after[1], before[1],
+ igt_debug("Engine total ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", after[1], before[1],
engine_total_ticks);
if (cork)
@@ -471,9 +471,9 @@ static void engine_activity_all_fn(int fd, struct drm_xe_engine_class_instance *
engine_active_ticks = after[idx] - before[idx];
engine_total_ticks = after[idx + 1] - before[idx + 1];
- igt_debug("[%d] Engine active ticks: after %ld, before %ld delta %ld\n", i,
+ igt_debug("[%d] Engine active ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", i,
after[idx], before[idx], engine_active_ticks);
- igt_debug("[%d] Engine total ticks: after %ld, before %ld delta %ld\n", i,
+ igt_debug("[%d] Engine total ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", i,
after[idx + 1], before[idx + 1], engine_total_ticks);
if (f->cork)
@@ -526,9 +526,9 @@ static void engine_activity_fn(int fd, struct drm_xe_engine_class_instance *eci,
engine_active_ticks = after[0] - before[0];
engine_total_ticks = after[1] - before[1];
- igt_debug("[%d] Engine active ticks: after %ld, before %ld delta %ld\n", function,
+ igt_debug("[%d] Engine active ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", function,
after[0], before[0], engine_active_ticks);
- igt_debug("[%d] Engine total ticks: after %ld, before %ld delta %ld\n", function,
+ igt_debug("[%d] Engine total ticks: after %" PRIu64 ", before %" PRIu64 " delta %" PRIu64 "\n", function,
after[1], before[1], engine_total_ticks);
busy_percent = (double)engine_active_ticks / engine_total_ticks;
diff --git a/tests/intel/xe_render_copy.c b/tests/intel/xe_render_copy.c
index 60cd0fb45..19fdd43b0 100644
--- a/tests/intel/xe_render_copy.c
+++ b/tests/intel/xe_render_copy.c
@@ -629,7 +629,7 @@ static void render_stress_copy(int fd, struct igt_collection *set,
render_duration_max = duration;
}
igt_info("%d render() loops in %d seconds\n", render_count, render_timeout);
- igt_info("Render duration: avg = %ld ns, min = %ld ns, max = %ld ns\n",
+ igt_info("Render duration: avg = %" PRIu64 " ns, min = %" PRIu64 " ns, max = %" PRIu64 " ns\n",
render_duration_total / render_count,
render_duration_min, render_duration_max);
--
2.34.1
More information about the igt-dev
mailing list