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

[Zbigniew Kempczyński]

On Tue, Jul 28, 2020 at 01:49:49PM +0300, Petri Latvala wrote:
> 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 at api_intel_bb
> >  igt at core_auth@basic-auth
> >  igt at debugfs_test@read_all_entries
> >  igt at fbdev@mmap
> > @@ -7,6 +8,7 @@ igt at gem_basic@bad-close
> >  igt at gem_basic@create-close
> >  igt at gem_basic@create-fd-close
> >  igt at gem_busy@busy at all
> > +igt at gem_caching
> >  igt at gem_close_race@basic-process
> >  igt at gem_close_race@basic-threads
> >  igt at gem_ctx_create@basic
> > @@ -14,9 +16,9 @@ igt at gem_ctx_create@basic-files
> >  igt at gem_ctx_exec@basic
> >  igt at gem_exec_basic@basic
> >  igt at gem_exec_create@basic
> > +igt at gem_exec_fence@basic-await
> >  igt at gem_exec_fence@basic-busy
> >  igt at gem_exec_fence@basic-wait
> > -igt at gem_exec_fence@basic-await
> >  igt at gem_exec_fence@nb-await
> >  igt at gem_exec_gttfill@basic
> >  igt at gem_exec_parallel@engines
> > @@ -32,6 +34,9 @@ igt at gem_huc_copy@huc-copy
> >  igt at gem_linear_blits@basic
> >  igt at gem_mmap@basic
> >  igt at gem_mmap_gtt@basic
> > +igt at gem_ppgtt
> > +igt at gem_read_read_speed
> > +igt at gem_render_copy
> >  igt at gem_render_linear_blits@basic
> >  igt at gem_render_tiled_blits@basic
> >  igt at gem_ringfill@basic-all
> > @@ -45,6 +50,10 @@ igt at gem_wait@wait at all
> >  igt at i915_getparams_basic@basic-eu-total
> >  igt at i915_getparams_basic@basic-subslice-total
> >  igt at i915_hangman@error-state-basic
> > +igt at i915_pm_backlight@basic-brightness
> > +igt at i915_pm_rpm@basic-pci-d3-state
> > +igt at i915_pm_rpm@basic-rte
> > +igt at i915_pm_rps@basic-api
> >  igt at kms_addfb_basic@addfb25-bad-modifier
> >  igt at kms_addfb_basic@addfb25-framebuffer-vs-set-tiling
> >  igt at kms_addfb_basic@addfb25-modifier-no-flag
> > @@ -82,17 +91,18 @@ igt at kms_addfb_basic@unused-handle
> >  igt at kms_addfb_basic@unused-modifier
> >  igt at kms_addfb_basic@unused-offsets
> >  igt at kms_addfb_basic@unused-pitches
> > +igt at kms_big_fb
> >  igt at kms_busy@basic
> > -igt at kms_chamelium@dp-hpd-fast
> > -igt at kms_chamelium@dp-edid-read
> > +igt at kms_chamelium@common-hpd-after-suspend
> >  igt at kms_chamelium@dp-crc-fast
> > -igt at kms_chamelium@hdmi-hpd-fast
> > -igt at kms_chamelium@hdmi-edid-read
> > +igt at kms_chamelium@dp-edid-read
> > +igt at kms_chamelium@dp-hpd-fast
> >  igt at kms_chamelium@hdmi-crc-fast
> > -igt at kms_chamelium@vga-hpd-fast
> > +igt at kms_chamelium@hdmi-edid-read
> > +igt at kms_chamelium@hdmi-hpd-fast
> >  igt at kms_chamelium@vga-edid-read
> > -igt at kms_chamelium@common-hpd-after-suspend
> > -igt at kms_prop_blob@basic
> > +igt at kms_chamelium@vga-hpd-fast
> > +igt at kms_cursor_crc
> >  igt at kms_cursor_legacy@basic-busy-flip-before-cursor-atomic
> >  igt at kms_cursor_legacy@basic-busy-flip-before-cursor-legacy
> >  igt at kms_cursor_legacy@basic-flip-after-cursor-atomic
> > @@ -101,6 +111,7 @@ igt at kms_cursor_legacy@basic-flip-after-cursor-varying-size
> >  igt at kms_cursor_legacy@basic-flip-before-cursor-atomic
> >  igt at kms_cursor_legacy@basic-flip-before-cursor-legacy
> >  igt at kms_cursor_legacy@basic-flip-before-cursor-varying-size
> > +igt at kms_draw_crc
> >  igt at kms_flip@basic-flip-vs-dpms
> >  igt at kms_flip@basic-flip-vs-modeset
> >  igt at kms_flip@basic-flip-vs-wf_vblank
> > @@ -109,24 +120,23 @@ igt at kms_force_connector_basic@force-connector-state
> >  igt at kms_force_connector_basic@force-edid
> >  igt at kms_force_connector_basic@force-load-detect
> >  igt at kms_force_connector_basic@prune-stale-modes
> > +igt at kms_frontbuffer_tracking
> >  igt at 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.

I constructed by adding binaries we've changed in the series. 
I didn't know about technique to generate such list so I've only
added full binaries. Thanks for hint, I'm going to use it in the future
I think such info should be added to CI info 

https://intel-gfx-ci.01.org/#pre-merge-testing
(Forcing Tests In BAT And Changing Configuration)

--
Zbigniew
> 
> 
> -- 
> Petri Latvala