[PATCH i-g-t 71/93] lib/i915: Use for_each_physical_ring for submission tests
Jason Ekstrand
jason at jlekstrand.net
Wed May 26 18:00:36 UTC 2021
This does make the assumption that ctx0 has the default set of engines
but, now that we've converted everything to intel_ctx_t, this assumption
should be ok.
Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
---
lib/i915/gem_submission.c | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/lib/i915/gem_submission.c b/lib/i915/gem_submission.c
index 051f9d04..c13dfdce 100644
--- a/lib/i915/gem_submission.c
+++ b/lib/i915/gem_submission.c
@@ -32,6 +32,7 @@
#include "i915/gem.h"
#include "i915/gem_engine_topology.h"
+#include "i915/gem_ring.h"
#include "i915/gem_submission.h"
#include "igt_core.h"
@@ -199,10 +200,8 @@ void gem_test_engine(int i915, unsigned int engine)
gem_write(i915, obj.handle, 0, &bbe, sizeof(bbe));
if (engine == ALL_ENGINES) {
- const struct intel_execution_engine2 *e2;
-
- __for_each_physical_engine(i915, e2) {
- execbuf.flags = e2->flags;
+ for_each_physical_ring(e, i915) {
+ execbuf.flags = eb_ring(e);
gem_execbuf(i915, &execbuf);
}
} else {
@@ -381,11 +380,9 @@ unsigned int gem_submission_measure(int i915, unsigned int engine)
fcntl(i915, F_SETFL, fcntl(i915, F_GETFL) | O_NONBLOCK);
if (engine == ALL_ENGINES) {
- struct intel_execution_engine2 *e;
-
size = -1;
- __for_each_physical_engine(i915, e) {
- unsigned int this = __measure_ringsize(i915, e->flags);
+ for_each_physical_ring(e, i915) {
+ unsigned int this = __measure_ringsize(i915, eb_ring(e));
if (this < size)
size = this;
}
--
2.31.1
More information about the Intel-gfx-trybot
mailing list