[PATCH i-g-t 1/3] tests/intel: replace div64_u64_round_up() with DIV_ROUND_UP

Shuicheng Lin shuicheng.lin at intel.com
Wed Apr 16 16:30:51 UTC 2025


DIV_ROUND_UP is defined in igt_aux.h with the same function as
div64_u64_round_up().

Cc: Matt Roper <matthew.d.roper at intel.com>
Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>
---
 tests/intel/gem_exec_balancer.c |  9 ++-------
 tests/intel/gem_exec_fair.c     | 11 +++--------
 tests/intel/gem_exec_schedule.c |  9 ++-------
 tests/intel/gem_watchdog.c      |  9 ++-------
 4 files changed, 9 insertions(+), 29 deletions(-)

diff --git a/tests/intel/gem_exec_balancer.c b/tests/intel/gem_exec_balancer.c
index 46ea19a2a..1ad87036b 100644
--- a/tests/intel/gem_exec_balancer.c
+++ b/tests/intel/gem_exec_balancer.c
@@ -2636,15 +2636,10 @@ static int read_ctx_timestamp_frequency(int i915)
 	return value;
 }
 
-static uint64_t div64_u64_round_up(uint64_t x, uint64_t y)
-{
-	return (x + y - 1) / y;
-}
-
 static uint64_t ticks_to_ns(int i915, uint64_t ticks)
 {
-	return div64_u64_round_up(ticks * NSEC_PER_SEC,
-				  read_ctx_timestamp_frequency(i915));
+	int f = read_ctx_timestamp_frequency(i915);
+	return DIV_ROUND_UP(ticks * NSEC_PER_SEC, f);
 }
 
 static void __fairslice(int i915,
diff --git a/tests/intel/gem_exec_fair.c b/tests/intel/gem_exec_fair.c
index 19df66921..cdceb8927 100644
--- a/tests/intel/gem_exec_fair.c
+++ b/tests/intel/gem_exec_fair.c
@@ -187,23 +187,18 @@ static int read_timestamp_frequency(int i915)
 	return value;
 }
 
-static uint64_t div64_u64_round_up(uint64_t x, uint64_t y)
-{
-	return (x + y - 1) / y;
-}
-
 static uint64_t ns_to_ctx_ticks(int i915, uint64_t ns)
 {
 	int f = read_timestamp_frequency(i915);
 	if (intel_gen(intel_get_drm_devid(i915)) == 11)
 		f = 12500000; /* gen11!!! are you feeling alright? CTX vs CS */
-	return div64_u64_round_up(ns * f, NSEC64);
+	return DIV_ROUND_UP(ns * f, NSEC64);
 }
 
 static uint64_t ticks_to_ns(int i915, uint64_t ticks)
 {
-	return div64_u64_round_up(ticks * NSEC64,
-				  read_timestamp_frequency(i915));
+	int f = read_timestamp_frequency(i915);
+	return DIV_ROUND_UP(ticks * NSEC64, f);
 }
 
 static void delay(int i915,
diff --git a/tests/intel/gem_exec_schedule.c b/tests/intel/gem_exec_schedule.c
index 84219b4cf..1055e8501 100644
--- a/tests/intel/gem_exec_schedule.c
+++ b/tests/intel/gem_exec_schedule.c
@@ -3031,15 +3031,10 @@ static int read_timestamp_frequency(int i915)
 	return value;
 }
 
-static uint64_t div64_u64_round_up(uint64_t x, uint64_t y)
-{
-	return (x + y - 1) / y;
-}
-
 static uint64_t ticks_to_ns(int i915, uint64_t ticks)
 {
-	return div64_u64_round_up(ticks * NSEC_PER_SEC,
-				  read_timestamp_frequency(i915));
+	int f = read_timestamp_frequency(i915);
+	return DIV_ROUND_UP(ticks * NSEC_PER_SEC, f);
 }
 
 static int cmp_u32(const void *A, const void *B)
diff --git a/tests/intel/gem_watchdog.c b/tests/intel/gem_watchdog.c
index 1edde8b3c..4cb92c68e 100644
--- a/tests/intel/gem_watchdog.c
+++ b/tests/intel/gem_watchdog.c
@@ -289,11 +289,6 @@ static unsigned int offset_in_page(void *addr)
 	return (uintptr_t)addr & 4095;
 }
 
-static uint64_t div64_u64_round_up(uint64_t x, uint64_t y)
-{
-	return (x + y - 1) / y;
-}
-
 static int read_timestamp_frequency(int i915)
 {
 	int value = 0;
@@ -307,8 +302,8 @@ static int read_timestamp_frequency(int i915)
 
 static uint64_t ns_to_ticks(int i915, uint64_t ns)
 {
-	return div64_u64_round_up(ns * read_timestamp_frequency(i915),
-				  NSEC_PER_SEC);
+	int f = read_timestamp_frequency(i915);
+	return DIV_ROUND_UP(ns * f, NSEC_PER_SEC);
 }
 
 static uint32_t __batch_create(int i915, uint32_t offset)
-- 
2.25.1



More information about the igt-dev mailing list