Re: [igt-dev] [PATCH i-g-t v10 17/17] HAX: run rendercopy tests

From: Petri Latvala <petri.latvala@intel.com>
To: "Zbigniew Kempczyński" <zbigniew.kempczynski@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t v10 17/17] HAX: run rendercopy tests
Date: Tue, 28 Jul 2020 13:49:49 +0300	[thread overview]
Message-ID: <20200728104949.GF20883@platvala-desk.ger.corp.intel.com> (raw)
In-Reply-To: <20200728093547.23682-18-zbigniew.kempczynski@intel.com>

On Tue, Jul 28, 2020 at 11:35:47AM +0200, Zbigniew Kempczyński wrote:
> ---
>  tests/intel-ci/fast-feedback.testlist | 42 +++++++++++++++++----------
>  1 file changed, 26 insertions(+), 16 deletions(-)
> 
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index b796b264..d45fd523 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -1,5 +1,6 @@
>  # Keep alphabetically sorted by default
>  
> +igt@api_intel_bb
>  igt@core_auth@basic-auth
>  igt@debugfs_test@read_all_entries
>  igt@fbdev@mmap
> @@ -7,6 +8,7 @@ igt@gem_basic@bad-close
>  igt@gem_basic@create-close
>  igt@gem_basic@create-fd-close
>  igt@gem_busy@busy@all
> +igt@gem_caching
>  igt@gem_close_race@basic-process
>  igt@gem_close_race@basic-threads
>  igt@gem_ctx_create@basic
> @@ -14,9 +16,9 @@ igt@gem_ctx_create@basic-files
>  igt@gem_ctx_exec@basic
>  igt@gem_exec_basic@basic
>  igt@gem_exec_create@basic
> +igt@gem_exec_fence@basic-await
>  igt@gem_exec_fence@basic-busy
>  igt@gem_exec_fence@basic-wait
> -igt@gem_exec_fence@basic-await
>  igt@gem_exec_fence@nb-await
>  igt@gem_exec_gttfill@basic
>  igt@gem_exec_parallel@engines
> @@ -32,6 +34,9 @@ igt@gem_huc_copy@huc-copy
>  igt@gem_linear_blits@basic
>  igt@gem_mmap@basic
>  igt@gem_mmap_gtt@basic
> +igt@gem_ppgtt
> +igt@gem_read_read_speed
> +igt@gem_render_copy
>  igt@gem_render_linear_blits@basic
>  igt@gem_render_tiled_blits@basic
>  igt@gem_ringfill@basic-all
> @@ -45,6 +50,10 @@ igt@gem_wait@wait@all
>  igt@i915_getparams_basic@basic-eu-total
>  igt@i915_getparams_basic@basic-subslice-total
>  igt@i915_hangman@error-state-basic
> +igt@i915_pm_backlight@basic-brightness
> +igt@i915_pm_rpm@basic-pci-d3-state
> +igt@i915_pm_rpm@basic-rte
> +igt@i915_pm_rps@basic-api
>  igt@kms_addfb_basic@addfb25-bad-modifier
>  igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling
>  igt@kms_addfb_basic@addfb25-modifier-no-flag
> @@ -82,17 +91,18 @@ igt@kms_addfb_basic@unused-handle
>  igt@kms_addfb_basic@unused-modifier
>  igt@kms_addfb_basic@unused-offsets
>  igt@kms_addfb_basic@unused-pitches
> +igt@kms_big_fb
>  igt@kms_busy@basic
> -igt@kms_chamelium@dp-hpd-fast
> -igt@kms_chamelium@dp-edid-read
> +igt@kms_chamelium@common-hpd-after-suspend
>  igt@kms_chamelium@dp-crc-fast
> -igt@kms_chamelium@hdmi-hpd-fast
> -igt@kms_chamelium@hdmi-edid-read
> +igt@kms_chamelium@dp-edid-read
> +igt@kms_chamelium@dp-hpd-fast
>  igt@kms_chamelium@hdmi-crc-fast
> -igt@kms_chamelium@vga-hpd-fast
> +igt@kms_chamelium@hdmi-edid-read
> +igt@kms_chamelium@hdmi-hpd-fast
>  igt@kms_chamelium@vga-edid-read
> -igt@kms_chamelium@common-hpd-after-suspend
> -igt@kms_prop_blob@basic
> +igt@kms_chamelium@vga-hpd-fast
> +igt@kms_cursor_crc
>  igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic
>  igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy
>  igt@kms_cursor_legacy@basic-flip-after-cursor-atomic
> @@ -101,6 +111,7 @@ igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size
>  igt@kms_cursor_legacy@basic-flip-before-cursor-atomic
>  igt@kms_cursor_legacy@basic-flip-before-cursor-legacy
>  igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size
> +igt@kms_draw_crc
>  igt@kms_flip@basic-flip-vs-dpms
>  igt@kms_flip@basic-flip-vs-modeset
>  igt@kms_flip@basic-flip-vs-wf_vblank
> @@ -109,24 +120,23 @@ igt@kms_force_connector_basic@force-connector-state
>  igt@kms_force_connector_basic@force-edid
>  igt@kms_force_connector_basic@force-load-detect
>  igt@kms_force_connector_basic@prune-stale-modes
> +igt@kms_frontbuffer_tracking
>  igt@kms_frontbuffer_tracking@basic

For the record, adding full binaries of tests with subtests on the
list like this, along with some subtests, causes Bad Things (tm) to
happen if some of them fail. The tests get run twice basically with
some fields getting dropped undeterministically.

How did you construct these testlist changes? Appending the output of
igt_runner -L -t searchregex to the testlist is the best practice.

-- 
Petri Latvala
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev