[Intel-gfx] [PATCH i-g-t 10/10] gem_wsim: Fix calibration handling
Tvrtko Ursulin
tvrtko.ursulin at linux.intel.com
Wed Jun 17 16:01:20 UTC 2020
From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Intended use case was that run without arguments prints out the
calibrations which can be simply copied and pasted to the -n argument and
things should just work.
Two problems we need to solve: If the print out loops shows zero
calibrations (engine not present) they are later rejected and also if some
calibration is not given it is only an error if it needs to be used
(engine present).
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
---
benchmarks/gem_wsim.c | 20 ++------------------
1 file changed, 2 insertions(+), 18 deletions(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index ca07b670bd42..3bbb8fcbe17c 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -296,8 +296,8 @@ print_engine_calibrations(void)
printf("Nop calibration for %uus delay is: ", nop_calibration_us);
for (int i = 0; i < NUM_ENGINES; i++) {
- /* skip DEFAULT and VCS engines */
- if (i != DEFAULT && i != VCS) {
+ /* skip engines not present and DEFAULT and VCS */
+ if (i != DEFAULT && i != VCS && engine_calib_map[i]) {
if (first_entry) {
printf("%s=%lu", ring_str_map[i], engine_calib_map[i]);
first_entry = false;
@@ -2840,22 +2840,6 @@ int main(int argc, char **argv)
if (verbose)
print_engine_calibrations();
goto out;
- } else {
- bool missing = false;
-
- for (i = 0; i < NUM_ENGINES; i++) {
- if (i == VCS)
- continue;
-
- if (!engine_calib_map[i]) {
- wsim_err("Missing calibration for '%s'!\n",
- ring_str_map[i]);
- missing = true;
- }
- }
-
- if (missing)
- goto err;
}
if (!nr_w_args) {
--
2.20.1
More information about the Intel-gfx
mailing list