From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> These are not input fields and i915 currently leaves them untouched. In the spirit of trusting the query as the authoritative source of information, stop asserting these output only reserved fields are zero. This should prevent the test from auto-failing if we extend the data in the future. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> Cc: Brian Welty <brian.welty@xxxxxxxxx> --- tests/i915/i915_query.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c index 71807425fbc4..e7c6fc91e32b 100644 --- a/tests/i915/i915_query.c +++ b/tests/i915/i915_query.c @@ -690,11 +690,6 @@ static void engines(int fd) engine->flags, engine->capabilities); - /* MBZ fields. */ - igt_assert_eq(engine->rsvd0, 0); - igt_assert_eq(engine->rsvd1[0], 0); - igt_assert_eq(engine->rsvd1[1], 0); - switch (engine->engine.engine_class) { case I915_ENGINE_CLASS_RENDER: /* Will be tested later. */ -- 2.20.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx