* [igt-dev] [PATCH 0/2] HAX add test description to gem_exec_suspend @ 2022-06-09 5:28 janga.rahul.kumar 2022-06-09 5:28 ` [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description janga.rahul.kumar ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: janga.rahul.kumar @ 2022-06-09 5:28 UTC (permalink / raw) To: igt-dev, ramadevi.gandi, sai.gowtham.ch, Priyanka.Dandamudi From: Janga Rahul Kumar <janga.rahul.kumar@intel.com> Add test description to gem_exec_suspend Janga Rahul Kumar (2): tests/i915/gem_exec_suspend : Added subtests description HAX: don't do full run tests/i915/gem_exec_suspend.c | 33 +++-- tests/intel-ci/fast-feedback.testlist | 186 +------------------------- 2 files changed, 23 insertions(+), 196 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description 2022-06-09 5:28 [igt-dev] [PATCH 0/2] HAX add test description to gem_exec_suspend janga.rahul.kumar @ 2022-06-09 5:28 ` janga.rahul.kumar 2022-06-09 10:40 ` Kamil Konieczny 2022-06-09 5:28 ` [igt-dev] [PATCH 2/2] HAX: don't do full run janga.rahul.kumar 2022-06-09 6:25 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add test description to gem_exec_suspend Patchwork 2 siblings, 1 reply; 6+ messages in thread From: janga.rahul.kumar @ 2022-06-09 5:28 UTC (permalink / raw) To: igt-dev, ramadevi.gandi, sai.gowtham.ch, Priyanka.Dandamudi From: Janga Rahul Kumar <janga.rahul.kumar@intel.com> Added test description to basic subtests. Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com> --- tests/i915/gem_exec_suspend.c | 33 ++++++++++++++++++++++----------- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c index 401026ef..495748ec 100644 --- a/tests/i915/gem_exec_suspend.c +++ b/tests/i915/gem_exec_suspend.c @@ -37,6 +37,8 @@ #include "igt_gt.h" #include "igt_sysfs.h" +IGT_TEST_DESCRIPTION("Tests execute batches using execbuf ioctl across various suspend resume " + "states and validates the execution result by comparing with expected ones."); #define NOSLEEP 0 #define IDLE 1 #define SUSPEND_DEVICES 2 @@ -296,19 +298,26 @@ igt_main const char *name; unsigned int flags; void (*fn)(int, const intel_ctx_t *, unsigned, unsigned, uint32_t); + const char *describe; } *test, tests_all_engines[] = { - { "basic", NOSLEEP, run_test }, - { "basic-S0", IDLE, run_test }, - { "basic-S3-devices", SUSPEND_DEVICES, run_test }, - { "basic-S3", SUSPEND, run_test }, - { "basic-S4-devices", HIBERNATE_DEVICES, run_test }, - { "basic-S4", HIBERNATE, run_test }, + { "basic", NOSLEEP, run_test, "Check basic functionality." }, + { "basic-S0", IDLE, run_test, "Check with suspend-to-idle target state." }, + { "basic-S3-devices", SUSPEND_DEVICES, run_test, "Check with suspend-to-mem target " + "state and suspend sequence termination point set " + "to device suspend state." }, + { "basic-S3", SUSPEND, run_test, "Check with suspend-to-mem target state, perform " + "a full suspend/resume cycle." }, + { "basic-S4-devices", HIBERNATE_DEVICES, run_test, "Check with suspend-to-disk " + "target state and suspend sequence termination " + "point set to device suspend state." }, + { "basic-S4", HIBERNATE, run_test, "Check with suspend-to-disk target state, " + "perform a full suspend/resume cycle." }, { } }, tests_power_hang[] = { - { "hang-S3", SUSPEND | HANG, run_test }, - { "hang-S4", HIBERNATE | HANG, run_test }, - { "power-S0", IDLE, power_test }, - { "power-S3", SUSPEND, power_test }, + { "hang-S3", SUSPEND | HANG, run_test, "" }, + { "hang-S4", HIBERNATE | HANG, run_test, "" }, + { "power-S0", IDLE, power_test, "" }, + { "power-S3", SUSPEND, power_test, "" }, { } }; const struct intel_execution_engine2 *e; @@ -359,8 +368,10 @@ igt_main } \ } - for (test = tests_all_engines; test->name; test++) + for (test = tests_all_engines; test->name; test++) { + igt_describe(test->describe); subtest_for_each_combination(test->name, intel_ctx_0(fd), test->flags, test->fn); + } for (m = modes; m->suffix; m++) { igt_subtest_with_dynamic_f("fixed%s", m->suffix) { -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description 2022-06-09 5:28 ` [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description janga.rahul.kumar @ 2022-06-09 10:40 ` Kamil Konieczny 2022-06-22 9:40 ` Kumar, Janga Rahul 0 siblings, 1 reply; 6+ messages in thread From: Kamil Konieczny @ 2022-06-09 10:40 UTC (permalink / raw) To: igt-dev; +Cc: Janga Rahul Kumar Hi Janga, On 2022-06-09 at 10:58:18 +0530, janga.rahul.kumar@intel.com wrote: > From: Janga Rahul Kumar <janga.rahul.kumar@intel.com> > > Added test description to basic subtests. > > Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> > Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com> > --- > tests/i915/gem_exec_suspend.c | 33 ++++++++++++++++++++++----------- > 1 file changed, 22 insertions(+), 11 deletions(-) > > diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c > index 401026ef..495748ec 100644 > --- a/tests/i915/gem_exec_suspend.c > +++ b/tests/i915/gem_exec_suspend.c > @@ -37,6 +37,8 @@ > #include "igt_gt.h" > #include "igt_sysfs.h" > > +IGT_TEST_DESCRIPTION("Tests execute batches using execbuf ioctl across various suspend resume " > + "states and validates the execution result by comparing with expected ones."); Please change this description, you do not need to write down what exact step there is in code source (which we can read it), write what is purpose of these tests here. I suggest reading commit descriptions with: git log tests/i915/gem_exec_suspend.c git log 741bf7064c467d -- tests/gem_exec_suspend.c > #define NOSLEEP 0 > #define IDLE 1 > #define SUSPEND_DEVICES 2 > @@ -296,19 +298,26 @@ igt_main > const char *name; > unsigned int flags; > void (*fn)(int, const intel_ctx_t *, unsigned, unsigned, uint32_t); > + const char *describe; > } *test, tests_all_engines[] = { > - { "basic", NOSLEEP, run_test }, > - { "basic-S0", IDLE, run_test }, > - { "basic-S3-devices", SUSPEND_DEVICES, run_test }, > - { "basic-S3", SUSPEND, run_test }, > - { "basic-S4-devices", HIBERNATE_DEVICES, run_test }, > - { "basic-S4", HIBERNATE, run_test }, > + { "basic", NOSLEEP, run_test, "Check basic functionality." }, ----------------------------------------------------------------------- ^ imho you can add "without s/r" here, s/functionality./functionality without any suspend/resume cycle./ > + { "basic-S0", IDLE, run_test, "Check with suspend-to-idle target state." }, > + { "basic-S3-devices", SUSPEND_DEVICES, run_test, "Check with suspend-to-mem target " > + "state and suspend sequence termination point set " > + "to device suspend state." }, Please improve this description, see commit bbb950302f7c8405faa9c8018541501ee7bd335f tests/gem_exec_suspend: Add basic S3/S4-devices subtests > + { "basic-S3", SUSPEND, run_test, "Check with suspend-to-mem target state, perform " > + "a full suspend/resume cycle." }, These looks a little redundant, maybe something like: Check full cycle of suspend-to-mem. > + { "basic-S4-devices", HIBERNATE_DEVICES, run_test, "Check with suspend-to-disk " > + "target state and suspend sequence termination " > + "point set to device suspend state." }, Change this, maybe Check suspend-to-disk with device only. > + { "basic-S4", HIBERNATE, run_test, "Check with suspend-to-disk target state, " > + "perform a full suspend/resume cycle." }, Please improve this one. > { } > }, tests_power_hang[] = { > - { "hang-S3", SUSPEND | HANG, run_test }, > - { "hang-S4", HIBERNATE | HANG, run_test }, > - { "power-S0", IDLE, power_test }, > - { "power-S3", SUSPEND, power_test }, > + { "hang-S3", SUSPEND | HANG, run_test, "" }, > + { "hang-S4", HIBERNATE | HANG, run_test, "" }, > + { "power-S0", IDLE, power_test, "" }, > + { "power-S3", SUSPEND, power_test, "" }, Please also fill these descriptions here. Run test with --describe option, e.g. sudo ./gem_exec_suspend --describe to see what other descriptions are missing. Regards, Kamil > { } > }; > const struct intel_execution_engine2 *e; > @@ -359,8 +368,10 @@ igt_main > } \ > } > > - for (test = tests_all_engines; test->name; test++) > + for (test = tests_all_engines; test->name; test++) { > + igt_describe(test->describe); > subtest_for_each_combination(test->name, intel_ctx_0(fd), test->flags, test->fn); > + } > > for (m = modes; m->suffix; m++) { > igt_subtest_with_dynamic_f("fixed%s", m->suffix) { > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description 2022-06-09 10:40 ` Kamil Konieczny @ 2022-06-22 9:40 ` Kumar, Janga Rahul 0 siblings, 0 replies; 6+ messages in thread From: Kumar, Janga Rahul @ 2022-06-22 9:40 UTC (permalink / raw) To: Kamil Konieczny, igt-dev > -----Original Message----- > From: Kamil Konieczny <kamil.konieczny@linux.intel.com> > Sent: 09 June 2022 16:11 > To: igt-dev@lists.freedesktop.org > Cc: Kumar, Janga Rahul <janga.rahul.kumar@intel.com> > Subject: Re: [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests > description > > Hi Janga, > > On 2022-06-09 at 10:58:18 +0530, janga.rahul.kumar@intel.com wrote: > > From: Janga Rahul Kumar <janga.rahul.kumar@intel.com> > > > > Added test description to basic subtests. > > > > Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> > > Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar@intel.com> > > --- > > tests/i915/gem_exec_suspend.c | 33 ++++++++++++++++++++++----------- > > 1 file changed, 22 insertions(+), 11 deletions(-) > > > > diff --git a/tests/i915/gem_exec_suspend.c > > b/tests/i915/gem_exec_suspend.c index 401026ef..495748ec 100644 > > --- a/tests/i915/gem_exec_suspend.c > > +++ b/tests/i915/gem_exec_suspend.c > > @@ -37,6 +37,8 @@ > > #include "igt_gt.h" > > #include "igt_sysfs.h" > > > > +IGT_TEST_DESCRIPTION("Tests execute batches using execbuf ioctl across > various suspend resume " > > + "states and validates the execution result by comparing with > > +expected ones."); > > Please change this description, you do not need to write down what exact step > there is in code source (which we can read it), write what is purpose of these > tests here. > > I suggest reading commit descriptions with: > > git log tests/i915/gem_exec_suspend.c > > git log 741bf7064c467d -- tests/gem_exec_suspend.c > > > #define NOSLEEP 0 > > #define IDLE 1 > > #define SUSPEND_DEVICES 2 > > @@ -296,19 +298,26 @@ igt_main > > const char *name; > > unsigned int flags; > > void (*fn)(int, const intel_ctx_t *, unsigned, unsigned, uint32_t); > > + const char *describe; > > } *test, tests_all_engines[] = { > > - { "basic", NOSLEEP, run_test }, > > - { "basic-S0", IDLE, run_test }, > > - { "basic-S3-devices", SUSPEND_DEVICES, run_test }, > > - { "basic-S3", SUSPEND, run_test }, > > - { "basic-S4-devices", HIBERNATE_DEVICES, run_test }, > > - { "basic-S4", HIBERNATE, run_test }, > > + { "basic", NOSLEEP, run_test, "Check basic functionality." }, > ----------------------------------------------------------------------- ^ imho you can add > "without s/r" here, s/functionality./functionality without any suspend/resume > cycle./ > > > + { "basic-S0", IDLE, run_test, "Check with suspend-to-idle target > state." }, > > + { "basic-S3-devices", SUSPEND_DEVICES, run_test, "Check with > suspend-to-mem target " > > + "state and suspend sequence > termination point set " > > + "to device suspend state." }, > > Please improve this description, see commit > bbb950302f7c8405faa9c8018541501ee7bd335f > tests/gem_exec_suspend: Add basic S3/S4-devices subtests > > > + { "basic-S3", SUSPEND, run_test, "Check with suspend-to-mem > target state, perform " > > + "a full suspend/resume cycle." > }, > > These looks a little redundant, maybe something like: > Check full cycle of suspend-to-mem. > > > + { "basic-S4-devices", HIBERNATE_DEVICES, run_test, "Check > with suspend-to-disk " > > + "target state and suspend > sequence termination " > > + "point set to device suspend > state." }, > > Change this, maybe > Check suspend-to-disk with device only. > > > + { "basic-S4", HIBERNATE, run_test, "Check with suspend-to-disk > target state, " > > + "perform a full > suspend/resume cycle." }, > > Please improve this one. > > > { } > > }, tests_power_hang[] = { > > - { "hang-S3", SUSPEND | HANG, run_test }, > > - { "hang-S4", HIBERNATE | HANG, run_test }, > > - { "power-S0", IDLE, power_test }, > > - { "power-S3", SUSPEND, power_test }, > > + { "hang-S3", SUSPEND | HANG, run_test, "" }, > > + { "hang-S4", HIBERNATE | HANG, run_test, "" }, > > + { "power-S0", IDLE, power_test, "" }, > > + { "power-S3", SUSPEND, power_test, "" }, > > Please also fill these descriptions here. [Janga Rahul Kumar] Added descriptions for above tests in rev 2. > > Run test with --describe option, e.g. > sudo ./gem_exec_suspend --describe > to see what other descriptions are missing. > > Regards, > Kamil > > { } > > }; > > const struct intel_execution_engine2 *e; @@ -359,8 +368,10 @@ > > igt_main > > } \ > > } > > > > - for (test = tests_all_engines; test->name; test++) > > + for (test = tests_all_engines; test->name; test++) { > > + igt_describe(test->describe); > > subtest_for_each_combination(test->name, intel_ctx_0(fd), > > test->flags, test->fn); > > + } > > > > for (m = modes; m->suffix; m++) { > > igt_subtest_with_dynamic_f("fixed%s", m->suffix) { > > -- > > 2.25.1 > > [Janga Rahul Kumar] Regards, Rahul ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH 2/2] HAX: don't do full run 2022-06-09 5:28 [igt-dev] [PATCH 0/2] HAX add test description to gem_exec_suspend janga.rahul.kumar 2022-06-09 5:28 ` [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description janga.rahul.kumar @ 2022-06-09 5:28 ` janga.rahul.kumar 2022-06-09 6:25 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add test description to gem_exec_suspend Patchwork 2 siblings, 0 replies; 6+ messages in thread From: janga.rahul.kumar @ 2022-06-09 5:28 UTC (permalink / raw) To: igt-dev, ramadevi.gandi, sai.gowtham.ch, Priyanka.Dandamudi From: Janga Rahul Kumar <janga.rahul.kumar@intel.com> Just for test description, no full run is neccessary. --- tests/intel-ci/fast-feedback.testlist | 186 +------------------------- 1 file changed, 1 insertion(+), 185 deletions(-) diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist index 14cad7c0..9f8e18ce 100644 --- a/tests/intel-ci/fast-feedback.testlist +++ b/tests/intel-ci/fast-feedback.testlist @@ -1,185 +1 @@ -# Keep alphabetically sorted by default - -igt@core_auth@basic-auth -igt@debugfs_test@read_all_entries -igt@fbdev@eof -igt@fbdev@info -igt@fbdev@nullptr -igt@fbdev@read -igt@fbdev@write -igt@gem_basic@bad-close -igt@gem_basic@create-close -igt@gem_basic@create-fd-close -igt@gem_busy@busy@all -igt@gem_close_race@basic-process -igt@gem_close_race@basic-threads -igt@gem_ctx_create@basic -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-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 -igt@gem_exec_store@basic -igt@gem_flink_basic@bad-flink -igt@gem_flink_basic@bad-open -igt@gem_flink_basic@basic -igt@gem_flink_basic@double-flink -igt@gem_flink_basic@flink-lifetime -igt@gem_huc_copy@huc-copy -igt@gem_linear_blits@basic -igt@gem_mmap@basic -igt@gem_mmap_gtt@basic -igt@gem_render_linear_blits@basic -igt@gem_render_tiled_blits@basic -igt@gem_ringfill@basic-all -igt@gem_softpin@allocator-basic -igt@gem_softpin@allocator-basic-reserve -igt@gem_softpin@safe-alignment -igt@gem_sync@basic-all -igt@gem_sync@basic-each -igt@gem_tiled_blits@basic -igt@gem_tiled_fence_blits@basic -igt@gem_tiled_pread_basic -igt@gem_wait@busy@all -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_pciid -igt@kms_addfb_basic@addfb25-bad-modifier -igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling -igt@kms_addfb_basic@addfb25-modifier-no-flag -igt@kms_addfb_basic@addfb25-x-tiled-legacy -igt@kms_addfb_basic@addfb25-x-tiled-mismatch-legacy -igt@kms_addfb_basic@addfb25-yf-tiled-legacy -igt@kms_addfb_basic@addfb25-y-tiled-legacy -igt@kms_addfb_basic@addfb25-y-tiled-small-legacy -igt@kms_addfb_basic@bad-pitch-0 -igt@kms_addfb_basic@bad-pitch-1024 -igt@kms_addfb_basic@bad-pitch-128 -igt@kms_addfb_basic@bad-pitch-256 -igt@kms_addfb_basic@bad-pitch-32 -igt@kms_addfb_basic@bad-pitch-63 -igt@kms_addfb_basic@bad-pitch-65536 -igt@kms_addfb_basic@bad-pitch-999 -igt@kms_addfb_basic@basic -igt@kms_addfb_basic@basic-x-tiled-legacy -igt@kms_addfb_basic@basic-y-tiled-legacy -igt@kms_addfb_basic@bo-too-small -igt@kms_addfb_basic@bo-too-small-due-to-tiling -igt@kms_addfb_basic@clobberred-modifier -igt@kms_addfb_basic@framebuffer-vs-set-tiling -igt@kms_addfb_basic@invalid-get-prop -igt@kms_addfb_basic@invalid-get-prop-any -igt@kms_addfb_basic@invalid-set-prop -igt@kms_addfb_basic@invalid-set-prop-any -igt@kms_addfb_basic@no-handle -igt@kms_addfb_basic@size-max -igt@kms_addfb_basic@small-bo -igt@kms_addfb_basic@tile-pitch-mismatch -igt@kms_addfb_basic@too-high -igt@kms_addfb_basic@too-wide -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_busy@basic -igt@kms_chamelium@dp-hpd-fast -igt@kms_chamelium@dp-edid-read -igt@kms_chamelium@dp-crc-fast -igt@kms_chamelium@hdmi-hpd-fast -igt@kms_chamelium@hdmi-edid-read -igt@kms_chamelium@hdmi-crc-fast -igt@kms_chamelium@vga-hpd-fast -igt@kms_chamelium@vga-edid-read -igt@kms_prop_blob@basic -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 -igt@kms_cursor_legacy@basic-flip-after-cursor-legacy -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_flip@basic-flip-vs-dpms -igt@kms_flip@basic-flip-vs-modeset -igt@kms_flip@basic-flip-vs-wf_vblank -igt@kms_flip@basic-plain-flip -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@basic -igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-a -igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b -igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-c -igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d -igt@kms_pipe_crc_basic@hang-read-crc-pipe-a -igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a -igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence -igt@kms_pipe_crc_basic@read-crc-pipe-a -igt@kms_pipe_crc_basic@read-crc-pipe-b -igt@kms_pipe_crc_basic@read-crc-pipe-c -igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence -igt@kms_psr@primary_page_flip -igt@kms_psr@cursor_plane_move -igt@kms_psr@sprite_plane_onoff -igt@kms_psr@primary_mmap_gtt -igt@kms_setmode@basic-clone-single-crtc -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@prime_self_import@basic-llseek-bad -igt@prime_self_import@basic-llseek-size -igt@prime_self_import@basic-with_fd_dup -igt@prime_self_import@basic-with_one_bo -igt@prime_self_import@basic-with_one_bo_two_files -igt@prime_self_import@basic-with_two_bos -igt@prime_vgem@basic-fence-flip -igt@prime_vgem@basic-fence-mmap -igt@prime_vgem@basic-fence-read -igt@prime_vgem@basic-gtt -igt@prime_vgem@basic-read -igt@prime_vgem@basic-write -igt@prime_vgem@basic-userptr -igt@vgem_basic@setversion -igt@vgem_basic@create -igt@vgem_basic@debugfs -igt@vgem_basic@dmabuf-export -igt@vgem_basic@dmabuf-fence -igt@vgem_basic@dmabuf-fence-before -igt@vgem_basic@dmabuf-mmap -igt@vgem_basic@mmap -igt@vgem_basic@second-client -igt@vgem_basic@sysfs - -# All tests that do module unloading and reloading are executed last. -# They will sometimes reveal issues of earlier tests leaving the -# driver in a broken state that is not otherwise noticed in that test. - -igt@core_hotunplug@unbind-rebind -igt@vgem_basic@unload -igt@i915_module_load@reload -igt@gem_lmem_swapping@basic -igt@gem_lmem_swapping@parallel-random-engines -igt@gem_lmem_swapping@random-engines -igt@gem_lmem_swapping@verify-random -igt@i915_pm_rpm@module-reload - -# Kernel selftests -igt@i915_selftest@live -igt@dmabuf@all - -# System wide suspend tests -igt@i915_suspend@basic-s2idle-without-i915 -igt@i915_suspend@basic-s3-without-i915 -igt@gem_exec_suspend@basic-s0 -igt@gem_exec_suspend@basic-s3 -igt@kms_chamelium@common-hpd-after-suspend -igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a +igt@meta_test@fail-result -- 2.25.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for HAX add test description to gem_exec_suspend 2022-06-09 5:28 [igt-dev] [PATCH 0/2] HAX add test description to gem_exec_suspend janga.rahul.kumar 2022-06-09 5:28 ` [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description janga.rahul.kumar 2022-06-09 5:28 ` [igt-dev] [PATCH 2/2] HAX: don't do full run janga.rahul.kumar @ 2022-06-09 6:25 ` Patchwork 2 siblings, 0 replies; 6+ messages in thread From: Patchwork @ 2022-06-09 6:25 UTC (permalink / raw) To: janga.rahul.kumar; +Cc: igt-dev [-- Attachment #1: Type: text/plain, Size: 8104 bytes --] == Series Details == Series: HAX add test description to gem_exec_suspend URL : https://patchwork.freedesktop.org/series/104915/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11743 -> IGTPW_7264 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_7264 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_7264, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/index.html Participating hosts (44 -> 41) ------------------------------ Additional (1): bat-dg2-9 Missing (4): fi-kbl-soraka bat-adlm-1 fi-bdw-samus fi-pnv-d510 Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_7264: ### IGT changes ### #### Possible regressions #### * igt@meta_test@fail-result (NEW): - {bat-jsl-2}: NOTRUN -> [FAIL][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-jsl-2/igt@meta_test@fail-result.html - fi-ilk-650: NOTRUN -> [FAIL][2] [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-ilk-650/igt@meta_test@fail-result.html - {bat-jsl-1}: NOTRUN -> [FAIL][3] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-jsl-1/igt@meta_test@fail-result.html - fi-tgl-1115g4: NOTRUN -> [FAIL][4] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-tgl-1115g4/igt@meta_test@fail-result.html - fi-bxt-dsi: NOTRUN -> [FAIL][5] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-bxt-dsi/igt@meta_test@fail-result.html - {fi-jsl-1}: NOTRUN -> [FAIL][6] [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-jsl-1/igt@meta_test@fail-result.html - fi-blb-e6850: NOTRUN -> [FAIL][7] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-blb-e6850/igt@meta_test@fail-result.html - fi-bsw-n3050: NOTRUN -> [FAIL][8] [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-bsw-n3050/igt@meta_test@fail-result.html - {bat-adlp-6}: NOTRUN -> [FAIL][9] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-adlp-6/igt@meta_test@fail-result.html - fi-skl-6600u: NOTRUN -> [FAIL][10] [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-skl-6600u/igt@meta_test@fail-result.html - fi-glk-dsi: NOTRUN -> [FAIL][11] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-glk-dsi/igt@meta_test@fail-result.html - {fi-ehl-2}: NOTRUN -> [FAIL][12] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-ehl-2/igt@meta_test@fail-result.html - fi-apl-guc: NOTRUN -> [FAIL][13] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-apl-guc/igt@meta_test@fail-result.html - fi-snb-2520m: NOTRUN -> [FAIL][14] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-snb-2520m/igt@meta_test@fail-result.html - fi-glk-j4005: NOTRUN -> [FAIL][15] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-glk-j4005/igt@meta_test@fail-result.html - fi-rkl-guc: NOTRUN -> [FAIL][16] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-rkl-guc/igt@meta_test@fail-result.html - fi-skl-guc: NOTRUN -> [FAIL][17] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-skl-guc/igt@meta_test@fail-result.html - bat-dg1-6: NOTRUN -> [FAIL][18] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-dg1-6/igt@meta_test@fail-result.html - fi-kbl-7567u: NOTRUN -> [FAIL][19] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-kbl-7567u/igt@meta_test@fail-result.html - fi-cfl-8700k: NOTRUN -> [FAIL][20] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-cfl-8700k/igt@meta_test@fail-result.html - {fi-tgl-dsi}: NOTRUN -> [FAIL][21] [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-tgl-dsi/igt@meta_test@fail-result.html - fi-bsw-nick: NOTRUN -> [FAIL][22] [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-bsw-nick/igt@meta_test@fail-result.html - fi-rkl-11600: NOTRUN -> [FAIL][23] [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-rkl-11600/igt@meta_test@fail-result.html - fi-hsw-g3258: NOTRUN -> [FAIL][24] [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-hsw-g3258/igt@meta_test@fail-result.html - fi-bdw-gvtdvm: NOTRUN -> [FAIL][25] [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-bdw-gvtdvm/igt@meta_test@fail-result.html - fi-bsw-kefka: NOTRUN -> [FAIL][26] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-bsw-kefka/igt@meta_test@fail-result.html - fi-adl-ddr5: NOTRUN -> [FAIL][27] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-adl-ddr5/igt@meta_test@fail-result.html - fi-cfl-guc: NOTRUN -> [FAIL][28] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-cfl-guc/igt@meta_test@fail-result.html - {bat-dg2-9}: NOTRUN -> [FAIL][29] [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-dg2-9/igt@meta_test@fail-result.html - fi-kbl-x1275: NOTRUN -> [FAIL][30] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-kbl-x1275/igt@meta_test@fail-result.html - fi-hsw-4770: NOTRUN -> [FAIL][31] [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-hsw-4770/igt@meta_test@fail-result.html - fi-skl-6700k2: NOTRUN -> [FAIL][32] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-skl-6700k2/igt@meta_test@fail-result.html - fi-cfl-8109u: NOTRUN -> [FAIL][33] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-cfl-8109u/igt@meta_test@fail-result.html - bat-adlp-4: NOTRUN -> [FAIL][34] [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-adlp-4/igt@meta_test@fail-result.html - fi-kbl-8809g: NOTRUN -> [FAIL][35] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-kbl-8809g/igt@meta_test@fail-result.html - {bat-rpls-2}: NOTRUN -> [FAIL][36] [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-rpls-2/igt@meta_test@fail-result.html - fi-ivb-3770: NOTRUN -> [FAIL][37] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-ivb-3770/igt@meta_test@fail-result.html - fi-elk-e7500: NOTRUN -> [FAIL][38] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-elk-e7500/igt@meta_test@fail-result.html - {bat-dg2-8}: NOTRUN -> [FAIL][39] [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/bat-dg2-8/igt@meta_test@fail-result.html - fi-kbl-guc: NOTRUN -> [FAIL][40] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-kbl-guc/igt@meta_test@fail-result.html - fi-snb-2600: NOTRUN -> [FAIL][41] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/fi-snb-2600/igt@meta_test@fail-result.html New tests --------- New tests have been introduced between CI_DRM_11743 and IGTPW_7264: ### New IGT tests (1) ### * igt@meta_test@fail-result: - Statuses : 41 fail(s) - Exec time: [0.00, 0.21] s {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_6515 -> IGTPW_7264 CI-20190529: 20190529 CI_DRM_11743: 2d47fcd64571aa5e6b9ff875ddf061f372ed19a3 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_7264: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/index.html IGT_6515: 547f7c12716fe8e39573f32585d0377bdc8e7ff8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7264/index.html [-- Attachment #2: Type: text/html, Size: 9345 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-06-22 9:40 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-09 5:28 [igt-dev] [PATCH 0/2] HAX add test description to gem_exec_suspend janga.rahul.kumar 2022-06-09 5:28 ` [igt-dev] [PATCH 1/2] tests/i915/gem_exec_suspend : Added subtests description janga.rahul.kumar 2022-06-09 10:40 ` Kamil Konieczny 2022-06-22 9:40 ` Kumar, Janga Rahul 2022-06-09 5:28 ` [igt-dev] [PATCH 2/2] HAX: don't do full run janga.rahul.kumar 2022-06-09 6:25 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add test description to gem_exec_suspend Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.