[Intel-gfx] [PATCH i-g-t] intel-ci: Drop gem_ctx_switch/heavy
Mika Kuoppala
mika.kuoppala at linux.intel.com
Tue Jan 15 09:08:05 UTC 2019
Chris Wilson <chris at chris-wilson.co.uk> writes:
> Th heavy variant of gem_ctx_switch does little more than provide an
> alternate timing for the basic gem_ctx_switch; the timing only effects
> the HW and does not stress the driver any differently. As such,
> including gem_ctx_switch/heavy provides no more basic coverage for BAT
> over and above the default gem_ctx_switch and
> i915_selftests/live_contexts.
>
> It takes around 45s, of a 600s total target time for BAT.
I looked back drm-tip results and also did bugzilla searches.
drm-tip is all green for recent past and those 2 bugzilla
entries this test was in any way mentioned, are issues
where this test didn't play any particular role itself.
Heavy seems to indicate only bigger contexts for switching
so I agree with the timing argument.
These seconds are more well spent elsewhere.
Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> ---
> tests/intel-ci/fast-feedback.testlist | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index 6d42792c6..da3c4c8ed 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -16,7 +16,6 @@ igt at gem_ctx_exec@basic
> igt at gem_ctx_param@basic
> igt at gem_ctx_param@basic-default
> igt at gem_ctx_switch@basic-default
> -igt at gem_ctx_switch@basic-default-heavy
> igt at gem_exec_basic@basic-blt
> igt at gem_exec_basic@basic-bsd
> igt at gem_exec_basic@basic-bsd1
> --
> 2.20.1
More information about the Intel-gfx
mailing list