On 3/4/2020 1:29 AM, Tvrtko Ursulin wrote:
> 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>
Reviewed-by: 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. */
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
