* [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill
@ 2022-06-27 11:08 priyanka.dandamudi
2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: priyanka.dandamudi @ 2022-06-27 11:08 UTC (permalink / raw)
To: kamil.konieczny, igt-dev, priyanka.dandamudi
From: Priyanka Dandamudi <priyanka.dandamudi@intel.com>
Add test description to gem_ringfill.
Priyanka Dandamudi (2):
tests/i915/gem_ringfill : Added test description
HAX: don't do full run
tests/i915/gem_ringfill.c | 28 ++--
tests/intel-ci/fast-feedback.testlist | 176 +-------------------------
2 files changed, 20 insertions(+), 184 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 8+ messages in thread* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description 2022-06-27 11:08 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi @ 2022-06-27 11:08 ` priyanka.dandamudi 2022-06-27 17:09 ` Kamil Konieczny 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 2/2] HAX: don't do full run priyanka.dandamudi 2022-06-27 14:04 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add description to gem_ringfill Patchwork 2 siblings, 1 reply; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-27 11:08 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Added global description and description to all the available subtests. Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> --- tests/i915/gem_ringfill.c | 28 +++++++++++++++++++--------- 1 file changed, 19 insertions(+), 9 deletions(-) diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c index ecfa0bea..c51b32a2 100644 --- a/tests/i915/gem_ringfill.c +++ b/tests/i915/gem_ringfill.c @@ -51,6 +51,8 @@ #define HIBERNATE 0x40 #define NEWFD 0x80 +IGT_TEST_DESCRIPTION("Exercise many tiny batchbuffer operations, in the hope of catching" + " failure to manage the ring properly near full."); static unsigned int ring_size; static void check_bo(int fd, uint32_t handle) @@ -291,16 +293,21 @@ igt_main const char *suffix; unsigned flags; unsigned timeout; + const char *description; } modes[] = { - { "basic", 0, 0 }, - { "interruptible", INTERRUPTIBLE, 1 }, - { "hang", HANG, 10 }, - { "child", CHILD, 0 }, - { "forked", FORKED, 0 }, - { "fd", FORKED | NEWFD, 0 }, - { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150 }, - { "S3", BOMB | SUSPEND, 30 }, - { "S4", BOMB | HIBERNATE, 30 }, + { "basic", 0, 0, "Basic check how the driver handles a full ring"}, + { "interruptible", INTERRUPTIBLE, 1, "Exercise all potential injection sites by " + "using igt_sigiter interface to repeat the ringfill testing"}, + { "hang", HANG, 10, "Exercise many batchbuffer operations along with" + " a hang batch until ring is full"}, + { "child", CHILD, 0, "Check to fill the ring parallely using fork" }, + { "forked", FORKED, 0, "Check to fill the ring parallely using fork" }, + { "fd", FORKED | NEWFD, 0, "Fills the ring upto maximim parallely using" + " fork with different fd's" }, + { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150, "Fills the ring upto maximim parallely" + " using fork with different fd's along with interruptions" }, + { "S3", BOMB | SUSPEND, 30, "Handle a full ring across suspend cycle"}, + { "S4", BOMB | HIBERNATE, 30, "Handle a full ring across hibernate cycle"}, { NULL } }, *m; bool master = false; @@ -330,6 +337,7 @@ igt_main /* Legacy path for selecting "rings". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s - on legacy ring.", m->description); igt_subtest_with_dynamic_f("legacy-%s", m->suffix) { igt_skip_on(m->flags & NEWFD && master); @@ -350,6 +358,7 @@ igt_main /* New interface for selecting "engines". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s - engines.", m->description); igt_subtest_with_dynamic_f("engines-%s", m->suffix) { const struct intel_execution_engine2 *e; @@ -371,6 +380,7 @@ igt_main } } + igt_describe("Basic check to fill the ring upto maximum on all engines simultaneously."); igt_subtest("basic-all") { const struct intel_execution_engine2 *e; intel_allocator_multiprocess_start(); -- 2.25.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi @ 2022-06-27 17:09 ` Kamil Konieczny 0 siblings, 0 replies; 8+ messages in thread From: Kamil Konieczny @ 2022-06-27 17:09 UTC (permalink / raw) To: igt-dev Hi Priyanka, On 2022-06-27 at 16:38:08 +0530, priyanka.dandamudi@intel.com wrote: > From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> > > Added global description and description to all the available subtests. > > Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> > Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> > --- > tests/i915/gem_ringfill.c | 28 +++++++++++++++++++--------- > 1 file changed, 19 insertions(+), 9 deletions(-) > > diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c > index ecfa0bea..c51b32a2 100644 > --- a/tests/i915/gem_ringfill.c > +++ b/tests/i915/gem_ringfill.c > @@ -51,6 +51,8 @@ > #define HIBERNATE 0x40 > #define NEWFD 0x80 > > +IGT_TEST_DESCRIPTION("Exercise many tiny batchbuffer operations, in the hope of catching" ----------------------- ^ > + " failure to manage the ring properly near full."); -------------- ^ Align to above. > static unsigned int ring_size; > > static void check_bo(int fd, uint32_t handle) > @@ -291,16 +293,21 @@ igt_main > const char *suffix; > unsigned flags; > unsigned timeout; > + const char *description; > } modes[] = { > - { "basic", 0, 0 }, > - { "interruptible", INTERRUPTIBLE, 1 }, > - { "hang", HANG, 10 }, > - { "child", CHILD, 0 }, > - { "forked", FORKED, 0 }, > - { "fd", FORKED | NEWFD, 0 }, > - { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150 }, > - { "S3", BOMB | SUSPEND, 30 }, > - { "S4", BOMB | HIBERNATE, 30 }, > + { "basic", 0, 0, "Basic check how the driver handles a full ring"}, > + { "interruptible", INTERRUPTIBLE, 1, "Exercise all potential injection sites by " ---------------------------------------------------- ^ > + "using igt_sigiter interface to repeat the ringfill testing"}, ----------------------- ^ Align this string to one above (imho you can split if you want). > + { "hang", HANG, 10, "Exercise many batchbuffer operations along with" > + " a hang batch until ring is full"}, Same here, align. > + { "child", CHILD, 0, "Check to fill the ring parallely using fork" }, > + { "forked", FORKED, 0, "Check to fill the ring parallely using fork" }, > + { "fd", FORKED | NEWFD, 0, "Fills the ring upto maximim parallely using" > + " fork with different fd's" }, Same here. > + { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150, "Fills the ring upto maximim parallely" > + " using fork with different fd's along with interruptions" }, > + { "S3", BOMB | SUSPEND, 30, "Handle a full ring across suspend cycle"}, ------------------------------------------- ^ Delete one space from here. > + { "S4", BOMB | HIBERNATE, 30, "Handle a full ring across hibernate cycle"}, - ------------------------------------------- ^ Same here. > { NULL } > }, *m; > bool master = false; > @@ -330,6 +337,7 @@ igt_main > > /* Legacy path for selecting "rings". */ > for (m = modes; m->suffix; m++) { > + igt_describe_f("%s - on legacy ring.", m->description); > igt_subtest_with_dynamic_f("legacy-%s", m->suffix) { > igt_skip_on(m->flags & NEWFD && master); > > @@ -350,6 +358,7 @@ igt_main > > /* New interface for selecting "engines". */ > for (m = modes; m->suffix; m++) { > + igt_describe_f("%s - engines.", m->description); --------------------------------- ^ imho here we can drop this, so s/ - engines././ Regards, Kamil > igt_subtest_with_dynamic_f("engines-%s", m->suffix) { > const struct intel_execution_engine2 *e; > > @@ -371,6 +380,7 @@ igt_main > } > } > > + igt_describe("Basic check to fill the ring upto maximum on all engines simultaneously."); > igt_subtest("basic-all") { > const struct intel_execution_engine2 *e; > intel_allocator_multiprocess_start(); > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] HAX: don't do full run 2022-06-27 11:08 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi @ 2022-06-27 11:08 ` priyanka.dandamudi 2022-06-27 14:04 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add description to gem_ringfill Patchwork 2 siblings, 0 replies; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-27 11:08 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Just for test description, no full run is necessary. Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> --- tests/intel-ci/fast-feedback.testlist | 176 +------------------------- 1 file changed, 1 insertion(+), 175 deletions(-) diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist index bd5538a0..1d57372d 100644 --- a/tests/intel-ci/fast-feedback.testlist +++ b/tests/intel-ci/fast-feedback.testlist @@ -1,177 +1,3 @@ -# Try to load the driver if it's not available yet. -igt@i915_module_load@load - # 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 -igt@kms_cursor_legacy@basic-flip-after-cursor -igt@kms_cursor_legacy@basic-flip-before-cursor -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 -igt@kms_pipe_crc_basic@hang-read-crc -igt@kms_pipe_crc_basic@nonblocking-crc -igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence -igt@kms_pipe_crc_basic@read-crc -igt@kms_pipe_crc_basic@read-crc-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 +igt@meta_test@fail-result -- 2.25.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for HAX add description to gem_ringfill 2022-06-27 11:08 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 2/2] HAX: don't do full run priyanka.dandamudi @ 2022-06-27 14:04 ` Patchwork 2 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2022-06-27 14:04 UTC (permalink / raw) To: priyanka.dandamudi; +Cc: igt-dev [-- Attachment #1: Type: text/plain, Size: 8356 bytes --] == Series Details == Series: HAX add description to gem_ringfill URL : https://patchwork.freedesktop.org/series/105662/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11810 -> IGTPW_7397 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_7397 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_7397, 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_7397/index.html Participating hosts (43 -> 43) ------------------------------ Additional (1): bat-adls-5 Missing (1): fi-bdw-samus Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_7397: ### IGT changes ### #### Possible regressions #### * igt@meta_test@fail-result (NEW): - fi-ilk-650: NOTRUN -> [FAIL][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-ilk-650/igt@meta_test@fail-result.html - fi-tgl-u2: NOTRUN -> [FAIL][2] [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-tgl-u2/igt@meta_test@fail-result.html - {bat-jsl-1}: NOTRUN -> [FAIL][3] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-jsl-1/igt@meta_test@fail-result.html - fi-bxt-dsi: NOTRUN -> [FAIL][4] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-bxt-dsi/igt@meta_test@fail-result.html - {fi-jsl-1}: NOTRUN -> [FAIL][5] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-jsl-1/igt@meta_test@fail-result.html - fi-blb-e6850: NOTRUN -> [FAIL][6] [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-blb-e6850/igt@meta_test@fail-result.html - {bat-adlp-6}: NOTRUN -> [FAIL][7] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-adlp-6/igt@meta_test@fail-result.html - fi-skl-6600u: NOTRUN -> [FAIL][8] [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-skl-6600u/igt@meta_test@fail-result.html - fi-glk-dsi: NOTRUN -> [FAIL][9] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-glk-dsi/igt@meta_test@fail-result.html - {fi-ehl-2}: NOTRUN -> [FAIL][10] [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-ehl-2/igt@meta_test@fail-result.html - fi-icl-u2: NOTRUN -> [FAIL][11] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-icl-u2/igt@meta_test@fail-result.html - fi-apl-guc: NOTRUN -> [FAIL][12] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-apl-guc/igt@meta_test@fail-result.html - bat-dg1-5: NOTRUN -> [FAIL][13] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-dg1-5/igt@meta_test@fail-result.html - fi-pnv-d510: NOTRUN -> [FAIL][14] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-pnv-d510/igt@meta_test@fail-result.html - fi-bdw-5557u: NOTRUN -> [FAIL][15] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-bdw-5557u/igt@meta_test@fail-result.html - fi-snb-2520m: NOTRUN -> [FAIL][16] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-snb-2520m/igt@meta_test@fail-result.html - fi-glk-j4005: NOTRUN -> [FAIL][17] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-glk-j4005/igt@meta_test@fail-result.html - fi-rkl-guc: NOTRUN -> [FAIL][18] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-rkl-guc/igt@meta_test@fail-result.html - fi-skl-guc: NOTRUN -> [FAIL][19] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-skl-guc/igt@meta_test@fail-result.html - fi-kbl-soraka: NOTRUN -> [FAIL][20] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-kbl-soraka/igt@meta_test@fail-result.html - bat-dg1-6: NOTRUN -> [FAIL][21] [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-dg1-6/igt@meta_test@fail-result.html - fi-kbl-7567u: NOTRUN -> [FAIL][22] [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-kbl-7567u/igt@meta_test@fail-result.html - fi-cfl-8700k: NOTRUN -> [FAIL][23] [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-cfl-8700k/igt@meta_test@fail-result.html - {fi-tgl-dsi}: NOTRUN -> [FAIL][24] [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-tgl-dsi/igt@meta_test@fail-result.html - fi-bsw-nick: NOTRUN -> [FAIL][25] [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-bsw-nick/igt@meta_test@fail-result.html - fi-rkl-11600: NOTRUN -> [FAIL][26] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-rkl-11600/igt@meta_test@fail-result.html - fi-hsw-g3258: NOTRUN -> [FAIL][27] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-hsw-g3258/igt@meta_test@fail-result.html - {bat-adls-5}: NOTRUN -> [FAIL][28] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-adls-5/igt@meta_test@fail-result.html - fi-bdw-gvtdvm: NOTRUN -> [FAIL][29] [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-bdw-gvtdvm/igt@meta_test@fail-result.html - fi-bsw-kefka: NOTRUN -> [FAIL][30] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-bsw-kefka/igt@meta_test@fail-result.html - fi-adl-ddr5: NOTRUN -> [FAIL][31] [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-adl-ddr5/igt@meta_test@fail-result.html - {bat-dg2-9}: NOTRUN -> [FAIL][32] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-dg2-9/igt@meta_test@fail-result.html - fi-hsw-4770: NOTRUN -> [FAIL][33] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-hsw-4770/igt@meta_test@fail-result.html - fi-skl-6700k2: NOTRUN -> [FAIL][34] [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-skl-6700k2/igt@meta_test@fail-result.html - fi-cfl-8109u: NOTRUN -> [FAIL][35] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-cfl-8109u/igt@meta_test@fail-result.html - bat-adlp-4: NOTRUN -> [FAIL][36] [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-adlp-4/igt@meta_test@fail-result.html - {bat-adln-1}: NOTRUN -> [FAIL][37] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-adln-1/igt@meta_test@fail-result.html - fi-kbl-8809g: NOTRUN -> [FAIL][38] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-kbl-8809g/igt@meta_test@fail-result.html - fi-ivb-3770: NOTRUN -> [FAIL][39] [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-ivb-3770/igt@meta_test@fail-result.html - fi-elk-e7500: NOTRUN -> [FAIL][40] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-elk-e7500/igt@meta_test@fail-result.html - {bat-dg2-8}: NOTRUN -> [FAIL][41] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/bat-dg2-8/igt@meta_test@fail-result.html - fi-kbl-guc: NOTRUN -> [FAIL][42] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-kbl-guc/igt@meta_test@fail-result.html - fi-snb-2600: NOTRUN -> [FAIL][43] [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/fi-snb-2600/igt@meta_test@fail-result.html New tests --------- New tests have been introduced between CI_DRM_11810 and IGTPW_7397: ### New IGT tests (1) ### * igt@meta_test@fail-result: - Statuses : 43 fail(s) - Exec time: [0.00, 0.42] 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_6542 -> IGTPW_7397 CI-20190529: 20190529 CI_DRM_11810: 3c2c4f54d75ac013f8d395c71d4ccb94d662a481 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_7397: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/index.html IGT_6542: d38a476ee4b9f9a95d8f452de0d66cc52f7f079b @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7397/index.html [-- Attachment #2: Type: text/html, Size: 9625 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill @ 2022-06-28 4:51 priyanka.dandamudi 2022-06-28 4:51 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi 0 siblings, 1 reply; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-28 4:51 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Add test description to gem_ringfill. Priyanka Dandamudi (2): tests/i915/gem_ringfill : Added test description HAX: don't do full run tests/i915/gem_ringfill.c | 30 +++-- tests/intel-ci/fast-feedback.testlist | 176 +------------------------- 2 files changed, 22 insertions(+), 184 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description 2022-06-28 4:51 [igt-dev] [PATCH i-g-t 0/2] " priyanka.dandamudi @ 2022-06-28 4:51 ` priyanka.dandamudi 2022-06-28 7:50 ` Kamil Konieczny 0 siblings, 1 reply; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-28 4:51 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Added global description and description to all the available subtests. v2: Minor corrections. Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> --- tests/i915/gem_ringfill.c | 30 +++++++++++++++++++++--------- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c index ecfa0bea..e097c874 100644 --- a/tests/i915/gem_ringfill.c +++ b/tests/i915/gem_ringfill.c @@ -51,6 +51,8 @@ #define HIBERNATE 0x40 #define NEWFD 0x80 +IGT_TEST_DESCRIPTION("Exercise many tiny batchbuffer operations, in the hope of catching" + " failure to manage the ring properly near full."); static unsigned int ring_size; static void check_bo(int fd, uint32_t handle) @@ -291,16 +293,23 @@ igt_main const char *suffix; unsigned flags; unsigned timeout; + const char *description; } modes[] = { - { "basic", 0, 0 }, - { "interruptible", INTERRUPTIBLE, 1 }, - { "hang", HANG, 10 }, - { "child", CHILD, 0 }, - { "forked", FORKED, 0 }, - { "fd", FORKED | NEWFD, 0 }, - { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150 }, - { "S3", BOMB | SUSPEND, 30 }, - { "S4", BOMB | HIBERNATE, 30 }, + { "basic", 0, 0, "Basic check how the driver handles a full ring"}, + { "interruptible", INTERRUPTIBLE, 1, "Exercise all potential injection sites by " + "using igt_sigiter interface to repeat the " + "ringfill testing"}, + { "hang", HANG, 10, "Exercise many batchbuffer operations along with" + " a hang batch until ring is full"}, + { "child", CHILD, 0, "Check to fill the ring parallely using fork" }, + { "forked", FORKED, 0, "Check to fill the ring parallely using fork" }, + { "fd", FORKED | NEWFD, 0, "Fills the ring upto maximim parallely using" + " fork with different fd's" }, + { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150, "Fills the ring upto maximim parallely" + " using fork with different fd's along " + "with interruptions" }, + { "S3", BOMB | SUSPEND, 30, "Handle a full ring across suspend cycle"}, + { "S4", BOMB | HIBERNATE, 30, "Handle a full ring across hibernate cycle"}, { NULL } }, *m; bool master = false; @@ -330,6 +339,7 @@ igt_main /* Legacy path for selecting "rings". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s - on legacy ring.", m->description); igt_subtest_with_dynamic_f("legacy-%s", m->suffix) { igt_skip_on(m->flags & NEWFD && master); @@ -350,6 +360,7 @@ igt_main /* New interface for selecting "engines". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s.", m->description); igt_subtest_with_dynamic_f("engines-%s", m->suffix) { const struct intel_execution_engine2 *e; @@ -371,6 +382,7 @@ igt_main } } + igt_describe("Basic check to fill the ring upto maximum on all engines simultaneously."); igt_subtest("basic-all") { const struct intel_execution_engine2 *e; intel_allocator_multiprocess_start(); -- 2.25.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description 2022-06-28 4:51 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi @ 2022-06-28 7:50 ` Kamil Konieczny 0 siblings, 0 replies; 8+ messages in thread From: Kamil Konieczny @ 2022-06-28 7:50 UTC (permalink / raw) To: igt-dev Hi Priyanka, On 2022-06-28 at 10:21:22 +0530, priyanka.dandamudi@intel.com wrote: > From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> > > Added global description and description to all the available subtests. > > v2: Minor corrections. > > Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> > Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> > --- > tests/i915/gem_ringfill.c | 30 +++++++++++++++++++++--------- > 1 file changed, 21 insertions(+), 9 deletions(-) > > diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c > index ecfa0bea..e097c874 100644 > --- a/tests/i915/gem_ringfill.c > +++ b/tests/i915/gem_ringfill.c > @@ -51,6 +51,8 @@ > #define HIBERNATE 0x40 > #define NEWFD 0x80 > > +IGT_TEST_DESCRIPTION("Exercise many tiny batchbuffer operations, in the hope of catching" > + " failure to manage the ring properly near full."); > static unsigned int ring_size; > > static void check_bo(int fd, uint32_t handle) > @@ -291,16 +293,23 @@ igt_main > const char *suffix; > unsigned flags; > unsigned timeout; > + const char *description; > } modes[] = { > - { "basic", 0, 0 }, > - { "interruptible", INTERRUPTIBLE, 1 }, > - { "hang", HANG, 10 }, > - { "child", CHILD, 0 }, > - { "forked", FORKED, 0 }, > - { "fd", FORKED | NEWFD, 0 }, > - { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150 }, > - { "S3", BOMB | SUSPEND, 30 }, > - { "S4", BOMB | HIBERNATE, 30 }, > + { "basic", 0, 0, "Basic check how the driver handles a full ring"}, > + { "interruptible", INTERRUPTIBLE, 1, "Exercise all potential injection sites by " > + "using igt_sigiter interface to repeat the " > + "ringfill testing"}, > + { "hang", HANG, 10, "Exercise many batchbuffer operations along with" > + " a hang batch until ring is full"}, ------------------------------------ ^ Please be consistent, use either space at end of string or at beginning. See your description above on interruptible. I would prefer at beginning but it is your choice. > + { "child", CHILD, 0, "Check to fill the ring parallely using fork" }, > + { "forked", FORKED, 0, "Check to fill the ring parallely using fork" }, > + { "fd", FORKED | NEWFD, 0, "Fills the ring upto maximim parallely using" > + " fork with different fd's" }, > + { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150, "Fills the ring upto maximim parallely" > + " using fork with different fd's along " Same here, use one at begin or one at end, not both. Rest looks good, after corrections you can add my r-b tag. Regards, Kamil > + "with interruptions" }, > + { "S3", BOMB | SUSPEND, 30, "Handle a full ring across suspend cycle"}, > + { "S4", BOMB | HIBERNATE, 30, "Handle a full ring across hibernate cycle"}, > { NULL } > }, *m; > bool master = false; > @@ -330,6 +339,7 @@ igt_main > > /* Legacy path for selecting "rings". */ > for (m = modes; m->suffix; m++) { > + igt_describe_f("%s - on legacy ring.", m->description); > igt_subtest_with_dynamic_f("legacy-%s", m->suffix) { > igt_skip_on(m->flags & NEWFD && master); > > @@ -350,6 +360,7 @@ igt_main > > /* New interface for selecting "engines". */ > for (m = modes; m->suffix; m++) { > + igt_describe_f("%s.", m->description); > igt_subtest_with_dynamic_f("engines-%s", m->suffix) { > const struct intel_execution_engine2 *e; > > @@ -371,6 +382,7 @@ igt_main > } > } > > + igt_describe("Basic check to fill the ring upto maximum on all engines simultaneously."); > igt_subtest("basic-all") { > const struct intel_execution_engine2 *e; > intel_allocator_multiprocess_start(); > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill @ 2022-06-28 9:07 priyanka.dandamudi 2022-06-28 9:07 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi 0 siblings, 1 reply; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-28 9:07 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Add test description to gem_ringfill. Priyanka Dandamudi (2): tests/i915/gem_ringfill : Added test description HAX: don't do full run tests/i915/gem_ringfill.c | 30 +++-- tests/intel-ci/fast-feedback.testlist | 176 +------------------------- 2 files changed, 22 insertions(+), 184 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description 2022-06-28 9:07 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi @ 2022-06-28 9:07 ` priyanka.dandamudi 0 siblings, 0 replies; 8+ messages in thread From: priyanka.dandamudi @ 2022-06-28 9:07 UTC (permalink / raw) To: kamil.konieczny, igt-dev, priyanka.dandamudi From: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Added global description and description to all the available subtests. v2: Minor corrections. Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com> Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi@intel.com> Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com> --- tests/i915/gem_ringfill.c | 30 +++++++++++++++++++++--------- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c index ecfa0bea..b68c5ae6 100644 --- a/tests/i915/gem_ringfill.c +++ b/tests/i915/gem_ringfill.c @@ -51,6 +51,8 @@ #define HIBERNATE 0x40 #define NEWFD 0x80 +IGT_TEST_DESCRIPTION("Exercise many tiny batchbuffer operations, in the hope of catching" + " failure to manage the ring properly near full."); static unsigned int ring_size; static void check_bo(int fd, uint32_t handle) @@ -291,16 +293,23 @@ igt_main const char *suffix; unsigned flags; unsigned timeout; + const char *description; } modes[] = { - { "basic", 0, 0 }, - { "interruptible", INTERRUPTIBLE, 1 }, - { "hang", HANG, 10 }, - { "child", CHILD, 0 }, - { "forked", FORKED, 0 }, - { "fd", FORKED | NEWFD, 0 }, - { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150 }, - { "S3", BOMB | SUSPEND, 30 }, - { "S4", BOMB | HIBERNATE, 30 }, + { "basic", 0, 0, "Basic check how the driver handles a full ring"}, + { "interruptible", INTERRUPTIBLE, 1, "Exercise all potential injection sites by" + " using igt_sigiter interface to repeat the" + " ringfill testing"}, + { "hang", HANG, 10, "Exercise many batchbuffer operations along with" + " a hang batch until ring is full"}, + { "child", CHILD, 0, "Check to fill the ring parallely using fork" }, + { "forked", FORKED, 0, "Check to fill the ring parallely using fork" }, + { "fd", FORKED | NEWFD, 0, "Fills the ring upto maximim parallely using" + " fork with different fd's" }, + { "bomb", BOMB | NEWFD | INTERRUPTIBLE, 150, "Fills the ring upto maximim parallely" + " using fork with different fd's along" + " with interruptions" }, + { "S3", BOMB | SUSPEND, 30, "Handle a full ring across suspend cycle"}, + { "S4", BOMB | HIBERNATE, 30, "Handle a full ring across hibernate cycle"}, { NULL } }, *m; bool master = false; @@ -330,6 +339,7 @@ igt_main /* Legacy path for selecting "rings". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s - on legacy ring.", m->description); igt_subtest_with_dynamic_f("legacy-%s", m->suffix) { igt_skip_on(m->flags & NEWFD && master); @@ -350,6 +360,7 @@ igt_main /* New interface for selecting "engines". */ for (m = modes; m->suffix; m++) { + igt_describe_f("%s.", m->description); igt_subtest_with_dynamic_f("engines-%s", m->suffix) { const struct intel_execution_engine2 *e; @@ -371,6 +382,7 @@ igt_main } } + igt_describe("Basic check to fill the ring upto maximum on all engines simultaneously."); igt_subtest("basic-all") { const struct intel_execution_engine2 *e; intel_allocator_multiprocess_start(); -- 2.25.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-06-28 9:18 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-27 11:08 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi 2022-06-27 17:09 ` Kamil Konieczny 2022-06-27 11:08 ` [igt-dev] [PATCH i-g-t 2/2] HAX: don't do full run priyanka.dandamudi 2022-06-27 14:04 ` [igt-dev] ✗ Fi.CI.BAT: failure for HAX add description to gem_ringfill Patchwork 2022-06-28 4:51 [igt-dev] [PATCH i-g-t 0/2] " priyanka.dandamudi 2022-06-28 4:51 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi 2022-06-28 7:50 ` Kamil Konieczny 2022-06-28 9:07 [igt-dev] [PATCH i-g-t 0/2] HAX add description to gem_ringfill priyanka.dandamudi 2022-06-28 9:07 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ringfill : Added test description priyanka.dandamudi
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.