diff options
| author | Gustavo Sousa <gustavo.sousa@intel.com> | 2025-07-10 17:53:10 -0300 |
|---|---|---|
| committer | Kamil Konieczny <kamil.konieczny@linux.intel.com> | 2025-07-11 15:40:11 +0200 |
| commit | 67f8a0cc3a159bf6f912c08c030dfed5a85af328 (patch) | |
| tree | a8c3e16a7df84f88388c59fb03d0ba5cef4e5f68 | |
| parent | 64c0be2da10d441f4054f139d6ff14d2d5c2c3e3 (diff) | |
The function xe_dev_max_gt() returns the maximum GT ID. Because the ID
is zero-based, if we want to allocate an array that can hold positions
for all possible GTs, we need to take xe_dev_max_gt() + 1.
Fix the current allocations to conform to that.
Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Fixes: edd059dcd652 ("tests/xe: Don't assume max GT ID is num_gt - 1")
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
| -rw-r--r-- | tests/intel/xe_gt_freq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/intel/xe_gt_freq.c b/tests/intel/xe_gt_freq.c index 2c9080428..d0f35b10a 100644 --- a/tests/intel/xe_gt_freq.c +++ b/tests/intel/xe_gt_freq.c @@ -408,8 +408,8 @@ igt_main max_gt = xe_dev_max_gt(fd); /* The defaults are the same. Stashing the gt0 is enough */ - stash_min = (uint32_t *) malloc(sizeof(uint32_t) * max_gt); - stash_max = (uint32_t *) malloc(sizeof(uint32_t) * max_gt); + stash_min = (uint32_t *) malloc(sizeof(uint32_t) * (max_gt + 1)); + stash_max = (uint32_t *) malloc(sizeof(uint32_t) * (max_gt + 1)); xe_for_each_gt(fd, gt) { stash_min[gt] = xe_gt_get_freq(fd, gt, "min"); |