* [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines
@ 2020-01-28 9:12 Bommu Krishnaiah
2020-01-28 9:12 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Bommu Krishnaiah @ 2020-01-28 9:12 UTC (permalink / raw)
To: igt-dev; +Cc: Bommu Krishnaiah
Update the context with supported engines on the platform with set_property
I915_CONTEXT_PARAM_ENGINES to make sure the work load is submitted to
the available engines only.
v2: addressed the review comments.
v3: addressed v2 review comments.
V4: Re-introduced the gem_context_set_all_engines() API,test_process_mixed()
test creating new forked process, to set engine map to new forked process
I added gem_context_set_all_engines() API.
Bommu Krishnaiah (1):
tests/i915/gem_ctx_persistence: Set context with supported engines
lib/i915/gem_engine_topology.c | 11 +++++++
lib/i915/gem_engine_topology.h | 2 ++
tests/i915/gem_ctx_persistence.c | 52 ++++++++++++++++++++++++++------
3 files changed, 55 insertions(+), 10 deletions(-)
--
2.24.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 6+ messages in thread* [igt-dev] [PATCH i-g-t v4 1/1] tests/i915/gem_ctx_persistence: Set context with supported engines 2020-01-28 9:12 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah @ 2020-01-28 9:12 ` Bommu Krishnaiah 2020-01-28 9:41 ` Tvrtko Ursulin 2020-01-28 12:57 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) Patchwork 2020-01-29 16:00 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 2 siblings, 1 reply; 6+ messages in thread From: Bommu Krishnaiah @ 2020-01-28 9:12 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah, Tvrtko Ursulin Update the context with supported engines on the platform with set_property I915_CONTEXT_PARAM_ENGINES to make sure the work load is submitted to the available engines only. v2: addressed the review comments. v3: addressed v2 review comments. V4: Re-introduced the gem_context_set_all_engines() API,test_process_mixed() test creating new forked process, to set engine map to new forked process I added gem_context_set_all_engines() API. Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> --- lib/i915/gem_engine_topology.c | 11 +++++++ lib/i915/gem_engine_topology.h | 2 ++ tests/i915/gem_ctx_persistence.c | 52 ++++++++++++++++++++++++++------ 3 files changed, 55 insertions(+), 10 deletions(-) diff --git a/lib/i915/gem_engine_topology.c b/lib/i915/gem_engine_topology.c index 43a99e0f..ed6737ef 100644 --- a/lib/i915/gem_engine_topology.c +++ b/lib/i915/gem_engine_topology.c @@ -274,6 +274,17 @@ int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, return 0; } +void gem_context_set_all_engines(int fd, uint32_t ctx) +{ + DEFINE_CONTEXT_ENGINES_PARAM(engines, param, ctx, GEM_MAX_ENGINES); + struct intel_engine_data engine_data = { }; + + if (!gem_topology_get_param(fd, ¶m) && !param.size) { + query_engine_list(fd, &engine_data); + ctx_map_engines(fd, &engine_data, ¶m); + } +} + bool gem_has_engine_topology(int fd) { struct drm_i915_gem_context_param param = { diff --git a/lib/i915/gem_engine_topology.h b/lib/i915/gem_engine_topology.h index e40d7ec8..525741cc 100644 --- a/lib/i915/gem_engine_topology.h +++ b/lib/i915/gem_engine_topology.h @@ -51,6 +51,8 @@ void intel_next_engine(struct intel_engine_data *ed); int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, struct intel_execution_engine2 *e); +void gem_context_set_all_engines(int fd, uint32_t ctx); + bool gem_context_has_engine_map(int fd, uint32_t ctx); bool gem_engine_is_equal(const struct intel_execution_engine2 *e1, diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c index d68431ae..4e1004c8 100644 --- a/tests/i915/gem_ctx_persistence.c +++ b/tests/i915/gem_ctx_persistence.c @@ -156,7 +156,7 @@ static void test_persistence(int i915, unsigned int engine) * request is retired -- no early termination. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, true); spin = igt_spin_new(i915, ctx, @@ -188,7 +188,7 @@ static void test_nonpersistent_cleanup(int i915, unsigned int engine) * any inflight request is cancelled. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, @@ -217,7 +217,7 @@ static void test_nonpersistent_mixed(int i915, unsigned int engine) igt_spin_t *spin; uint32_t ctx; - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, i & 1); spin = igt_spin_new(i915, ctx, @@ -253,7 +253,7 @@ static void test_nonpersistent_hostile(int i915, unsigned int engine) * the requests and cleanup the context. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, @@ -284,7 +284,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) igt_require(gem_scheduler_has_preemption(i915)); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, true); gem_context_set_priority(i915, ctx, 0); spin[0] = igt_spin_new(i915, ctx, @@ -295,7 +295,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) igt_spin_busywait_until_started(spin[0]); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); gem_context_set_priority(i915, ctx, 1); /* higher priority than 0 */ spin[1] = igt_spin_new(i915, ctx, @@ -385,7 +385,7 @@ static void test_nonpersistent_queued(int i915, unsigned int engine) gem_quiescent_gpu(i915); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, .engine = engine, @@ -512,7 +512,8 @@ static void test_process_mixed(int i915, unsigned int engine) igt_spin_t *spin; uint32_t ctx; - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); + gem_context_set_all_engines(i915, ctx); gem_context_set_persistence(i915, ctx, persists); spin = igt_spin_new(i915, ctx, @@ -727,13 +728,44 @@ igt_main igt_subtest("hangcheck") test_nohangcheck_hostile(i915); - __for_each_static_engine(e) { + for_each_engine(e, i915) { igt_subtest_group { igt_fixture { gem_require_ring(i915, e->flags); gem_require_contexts(i915); } + igt_subtest_f("legacy-%s-persistence", e->name) + test_persistence(i915, e->flags); + + igt_subtest_f("legacy-%s-cleanup", e->name) + test_nonpersistent_cleanup(i915, e->flags); + + igt_subtest_f("legacy-%s-queued", e->name) + test_nonpersistent_queued(i915, e->flags); + + igt_subtest_f("legacy-%s-mixed", e->name) + test_nonpersistent_mixed(i915, e->flags); + + igt_subtest_f("legacy-%s-mixed-process", e->name) + test_process_mixed(i915, e->flags); + + igt_subtest_f("legacy-%s-hostile", e->name) + test_nonpersistent_hostile(i915, e->flags); + + igt_subtest_f("legacy-%s-hostile-preempt", e->name) + test_nonpersistent_hostile_preempt(i915, + e->flags); + } + } + + __for_each_physical_engine(i915, e) { + igt_subtest_group { + igt_fixture { + gem_require_ring(i915, e->flags); + gem_require_contexts(i915); + } + igt_subtest_f("%s-persistence", e->name) test_persistence(i915, e->flags); @@ -753,7 +785,7 @@ igt_main test_nonpersistent_hostile(i915, e->flags); igt_subtest_f("%s-hostile-preempt", e->name) - test_nonpersistent_hostile_preempt(i915, + test_nonpersistent_hostile_preempt(i915, e->flags); } } -- 2.24.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v4 1/1] tests/i915/gem_ctx_persistence: Set context with supported engines 2020-01-28 9:12 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah @ 2020-01-28 9:41 ` Tvrtko Ursulin 0 siblings, 0 replies; 6+ messages in thread From: Tvrtko Ursulin @ 2020-01-28 9:41 UTC (permalink / raw) To: Bommu Krishnaiah, igt-dev; +Cc: Tvrtko Ursulin On 28/01/2020 09:12, Bommu Krishnaiah wrote: > Update the context with supported engines on the platform with set_property > I915_CONTEXT_PARAM_ENGINES to make sure the work load is submitted to > the available engines only. > > v2: addressed the review comments. > v3: addressed v2 review comments. > V4: Re-introduced the gem_context_set_all_engines() API,test_process_mixed() > test creating new forked process, to set engine map to new forked process > I added gem_context_set_all_engines() API. I can remember two emails where I explained how we deliberately removed gem_context_set_all_engines and how gem_context_clone_with_engines should be used instead. Including a code sketch. Regards, Tvrtko > Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> > Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > --- > lib/i915/gem_engine_topology.c | 11 +++++++ > lib/i915/gem_engine_topology.h | 2 ++ > tests/i915/gem_ctx_persistence.c | 52 ++++++++++++++++++++++++++------ > 3 files changed, 55 insertions(+), 10 deletions(-) > > diff --git a/lib/i915/gem_engine_topology.c b/lib/i915/gem_engine_topology.c > index 43a99e0f..ed6737ef 100644 > --- a/lib/i915/gem_engine_topology.c > +++ b/lib/i915/gem_engine_topology.c > @@ -274,6 +274,17 @@ int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, > return 0; > } > > +void gem_context_set_all_engines(int fd, uint32_t ctx) > +{ > + DEFINE_CONTEXT_ENGINES_PARAM(engines, param, ctx, GEM_MAX_ENGINES); > + struct intel_engine_data engine_data = { }; > + > + if (!gem_topology_get_param(fd, ¶m) && !param.size) { > + query_engine_list(fd, &engine_data); > + ctx_map_engines(fd, &engine_data, ¶m); > + } > +} > + > bool gem_has_engine_topology(int fd) > { > struct drm_i915_gem_context_param param = { > diff --git a/lib/i915/gem_engine_topology.h b/lib/i915/gem_engine_topology.h > index e40d7ec8..525741cc 100644 > --- a/lib/i915/gem_engine_topology.h > +++ b/lib/i915/gem_engine_topology.h > @@ -51,6 +51,8 @@ void intel_next_engine(struct intel_engine_data *ed); > int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, > struct intel_execution_engine2 *e); > > +void gem_context_set_all_engines(int fd, uint32_t ctx); > + > bool gem_context_has_engine_map(int fd, uint32_t ctx); > > bool gem_engine_is_equal(const struct intel_execution_engine2 *e1, > diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c > index d68431ae..4e1004c8 100644 > --- a/tests/i915/gem_ctx_persistence.c > +++ b/tests/i915/gem_ctx_persistence.c > @@ -156,7 +156,7 @@ static void test_persistence(int i915, unsigned int engine) > * request is retired -- no early termination. > */ > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, true); > > spin = igt_spin_new(i915, ctx, > @@ -188,7 +188,7 @@ static void test_nonpersistent_cleanup(int i915, unsigned int engine) > * any inflight request is cancelled. > */ > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, false); > > spin = igt_spin_new(i915, ctx, > @@ -217,7 +217,7 @@ static void test_nonpersistent_mixed(int i915, unsigned int engine) > igt_spin_t *spin; > uint32_t ctx; > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, i & 1); > > spin = igt_spin_new(i915, ctx, > @@ -253,7 +253,7 @@ static void test_nonpersistent_hostile(int i915, unsigned int engine) > * the requests and cleanup the context. > */ > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, false); > > spin = igt_spin_new(i915, ctx, > @@ -284,7 +284,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) > > igt_require(gem_scheduler_has_preemption(i915)); > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, true); > gem_context_set_priority(i915, ctx, 0); > spin[0] = igt_spin_new(i915, ctx, > @@ -295,7 +295,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) > > igt_spin_busywait_until_started(spin[0]); > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, false); > gem_context_set_priority(i915, ctx, 1); /* higher priority than 0 */ > spin[1] = igt_spin_new(i915, ctx, > @@ -385,7 +385,7 @@ static void test_nonpersistent_queued(int i915, unsigned int engine) > > gem_quiescent_gpu(i915); > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > gem_context_set_persistence(i915, ctx, false); > spin = igt_spin_new(i915, ctx, > .engine = engine, > @@ -512,7 +512,8 @@ static void test_process_mixed(int i915, unsigned int engine) > igt_spin_t *spin; > uint32_t ctx; > > - ctx = gem_context_create(i915); > + ctx = gem_context_clone_with_engines(i915, 0); > + gem_context_set_all_engines(i915, ctx); > gem_context_set_persistence(i915, ctx, persists); > > spin = igt_spin_new(i915, ctx, > @@ -727,13 +728,44 @@ igt_main > igt_subtest("hangcheck") > test_nohangcheck_hostile(i915); > > - __for_each_static_engine(e) { > + for_each_engine(e, i915) { > igt_subtest_group { > igt_fixture { > gem_require_ring(i915, e->flags); > gem_require_contexts(i915); > } > > + igt_subtest_f("legacy-%s-persistence", e->name) > + test_persistence(i915, e->flags); > + > + igt_subtest_f("legacy-%s-cleanup", e->name) > + test_nonpersistent_cleanup(i915, e->flags); > + > + igt_subtest_f("legacy-%s-queued", e->name) > + test_nonpersistent_queued(i915, e->flags); > + > + igt_subtest_f("legacy-%s-mixed", e->name) > + test_nonpersistent_mixed(i915, e->flags); > + > + igt_subtest_f("legacy-%s-mixed-process", e->name) > + test_process_mixed(i915, e->flags); > + > + igt_subtest_f("legacy-%s-hostile", e->name) > + test_nonpersistent_hostile(i915, e->flags); > + > + igt_subtest_f("legacy-%s-hostile-preempt", e->name) > + test_nonpersistent_hostile_preempt(i915, > + e->flags); > + } > + } > + > + __for_each_physical_engine(i915, e) { > + igt_subtest_group { > + igt_fixture { > + gem_require_ring(i915, e->flags); > + gem_require_contexts(i915); > + } > + > igt_subtest_f("%s-persistence", e->name) > test_persistence(i915, e->flags); > > @@ -753,7 +785,7 @@ igt_main > test_nonpersistent_hostile(i915, e->flags); > > igt_subtest_f("%s-hostile-preempt", e->name) > - test_nonpersistent_hostile_preempt(i915, > + test_nonpersistent_hostile_preempt(i915, > e->flags); > } > } > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) 2020-01-28 9:12 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah 2020-01-28 9:12 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah @ 2020-01-28 12:57 ` Patchwork 2020-01-29 16:00 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 2 siblings, 0 replies; 6+ messages in thread From: Patchwork @ 2020-01-28 12:57 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev == Series Details == Series: tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) URL : https://patchwork.freedesktop.org/series/71810/ State : success == Summary == CI Bug Log - changes from CI_DRM_7829 -> IGTPW_4014 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/index.html Known issues ------------ Here are the changes found in IGTPW_4014 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_close_race@basic-threads: - fi-hsw-peppy: [PASS][1] -> [INCOMPLETE][2] ([i915#816]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-hsw-peppy/igt@gem_close_race@basic-threads.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-hsw-peppy/igt@gem_close_race@basic-threads.html * igt@i915_selftest@live_gem_contexts: - fi-cfl-guc: [PASS][3] -> [INCOMPLETE][4] ([fdo#106070] / [i915#424]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html #### Possible fixes #### * igt@gem_close_race@basic-threads: - fi-byt-n2820: [INCOMPLETE][5] ([i915#45]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-byt-n2820/igt@gem_close_race@basic-threads.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-byt-n2820/igt@gem_close_race@basic-threads.html * igt@gem_exec_suspend@basic-s0: - fi-skl-6700k2: [FAIL][7] ([fdo#103375]) -> [PASS][8] +1 similar issue [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-skl-6700k2/igt@gem_exec_suspend@basic-s0.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-skl-6700k2/igt@gem_exec_suspend@basic-s0.html * igt@gem_exec_suspend@basic-s4-devices: - fi-skl-6700k2: [INCOMPLETE][9] ([i915#69]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-skl-6700k2/igt@gem_exec_suspend@basic-s4-devices.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-skl-6700k2/igt@gem_exec_suspend@basic-s4-devices.html * igt@i915_selftest@live_blt: - fi-ivb-3770: [DMESG-FAIL][11] ([i915#725]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-ivb-3770/igt@i915_selftest@live_blt.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-ivb-3770/igt@i915_selftest@live_blt.html - fi-hsw-4770: [DMESG-FAIL][13] ([i915#770]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-hsw-4770/igt@i915_selftest@live_blt.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-hsw-4770/igt@i915_selftest@live_blt.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-7500u: [FAIL][15] ([fdo#111407]) -> [PASS][16] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html #### Warnings #### * igt@i915_selftest@live_blt: - fi-hsw-4770r: [DMESG-FAIL][17] ([i915#553] / [i915#725]) -> [DMESG-FAIL][18] ([i915#725]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/fi-hsw-4770r/igt@i915_selftest@live_blt.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/fi-hsw-4770r/igt@i915_selftest@live_blt.html [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375 [fdo#106070]: https://bugs.freedesktop.org/show_bug.cgi?id=106070 [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407 [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424 [i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45 [i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553 [i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#770]: https://gitlab.freedesktop.org/drm/intel/issues/770 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 Participating hosts (50 -> 43) ------------------------------ Missing (7): fi-ilk-m540 fi-byt-j1900 fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5392 -> IGTPW_4014 CI-20190529: 20190529 CI_DRM_7829: d7c77ce081cc74a8e9f14a68769509ebabab5aa5 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_4014: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/index.html IGT_5392: fb8487a63fcbe1c129206c9303bfcd5feb0e3ee1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/index.html _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] ✗ Fi.CI.IGT: failure for tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) 2020-01-28 9:12 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah 2020-01-28 9:12 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah 2020-01-28 12:57 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) Patchwork @ 2020-01-29 16:00 ` Patchwork 2 siblings, 0 replies; 6+ messages in thread From: Patchwork @ 2020-01-29 16:00 UTC (permalink / raw) To: Bommu Krishnaiah; +Cc: igt-dev == Series Details == Series: tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) URL : https://patchwork.freedesktop.org/series/71810/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7829_full -> IGTPW_4014_full ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_4014_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_4014_full, 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_4014/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_4014_full: ### IGT changes ### #### Possible regressions #### * igt@gem_partial_pwrite_pread@writes-after-reads: - shard-hsw: [PASS][1] -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw1/igt@gem_partial_pwrite_pread@writes-after-reads.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw8/igt@gem_partial_pwrite_pread@writes-after-reads.html Known issues ------------ Here are the changes found in IGTPW_4014_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@bcs0-s3: - shard-apl: [PASS][3] -> [DMESG-WARN][4] ([i915#180]) +2 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-apl7/igt@gem_ctx_isolation@bcs0-s3.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-apl6/igt@gem_ctx_isolation@bcs0-s3.html * igt@gem_ctx_isolation@vcs1-reset: - shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#109276] / [fdo#112080]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb4/igt@gem_ctx_isolation@vcs1-reset.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb8/igt@gem_ctx_isolation@vcs1-reset.html * igt@gem_ctx_persistence@vcs1-queued: - shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#112080]) +10 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb4/igt@gem_ctx_persistence@vcs1-queued.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb7/igt@gem_ctx_persistence@vcs1-queued.html * igt@gem_ctx_shared@exec-single-timeline-bsd: - shard-iclb: [PASS][9] -> [SKIP][10] ([fdo#110841]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb7/igt@gem_ctx_shared@exec-single-timeline-bsd.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb4/igt@gem_ctx_shared@exec-single-timeline-bsd.html * igt@gem_exec_schedule@preempt-contexts-bsd2: - shard-iclb: [PASS][11] -> [SKIP][12] ([fdo#109276]) +13 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb4/igt@gem_exec_schedule@preempt-contexts-bsd2.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb3/igt@gem_exec_schedule@preempt-contexts-bsd2.html * igt@gem_exec_schedule@preempt-other-chain-bsd: - shard-iclb: [PASS][13] -> [SKIP][14] ([fdo#112146]) +5 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb5/igt@gem_exec_schedule@preempt-other-chain-bsd.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb4/igt@gem_exec_schedule@preempt-other-chain-bsd.html * igt@gem_ppgtt@flink-and-close-vma-leak: - shard-glk: [PASS][15] -> [FAIL][16] ([i915#644]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk1/igt@gem_ppgtt@flink-and-close-vma-leak.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk9/igt@gem_ppgtt@flink-and-close-vma-leak.html * igt@gem_userptr_blits@sync-unmap-cycles: - shard-snb: [PASS][17] -> [DMESG-WARN][18] ([fdo#110789] / [fdo#111870] / [i915#478]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-snb1/igt@gem_userptr_blits@sync-unmap-cycles.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-snb4/igt@gem_userptr_blits@sync-unmap-cycles.html * igt@i915_pm_dc@dc6-psr: - shard-iclb: [PASS][19] -> [FAIL][20] ([i915#454]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb7/igt@i915_pm_dc@dc6-psr.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb2/igt@i915_pm_dc@dc6-psr.html * igt@i915_suspend@forcewake: - shard-kbl: [PASS][21] -> [DMESG-WARN][22] ([i915#180]) +5 similar issues [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-kbl7/igt@i915_suspend@forcewake.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-kbl1/igt@i915_suspend@forcewake.html * igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-pipe-c: - shard-glk: [PASS][23] -> [SKIP][24] ([fdo#109271]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk1/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-pipe-c.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk1/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-pipe-c.html - shard-kbl: [PASS][25] -> [SKIP][26] ([fdo#109271]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-kbl7/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-pipe-c.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-kbl1/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-pipe-c.html * igt@kms_flip@flip-vs-expired-vblank-interruptible: - shard-glk: [PASS][27] -> [FAIL][28] ([i915#79]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk8/igt@kms_flip@flip-vs-expired-vblank-interruptible.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk3/igt@kms_flip@flip-vs-expired-vblank-interruptible.html * igt@kms_flip@flip-vs-suspend: - shard-hsw: [PASS][29] -> [INCOMPLETE][30] ([i915#61]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw2/igt@kms_flip@flip-vs-suspend.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw2/igt@kms_flip@flip-vs-suspend.html * igt@kms_psr@psr2_basic: - shard-iclb: [PASS][31] -> [SKIP][32] ([fdo#109441]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb2/igt@kms_psr@psr2_basic.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb1/igt@kms_psr@psr2_basic.html * igt@kms_vblank@pipe-c-wait-busy-hang: - shard-hsw: [PASS][33] -> [INCOMPLETE][34] ([CI#80] / [i915#61]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw6/igt@kms_vblank@pipe-c-wait-busy-hang.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw4/igt@kms_vblank@pipe-c-wait-busy-hang.html * igt@perf@short-reads: - shard-apl: [PASS][35] -> [TIMEOUT][36] ([fdo#112271] / [i915#51]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-apl6/igt@perf@short-reads.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-apl8/igt@perf@short-reads.html * igt@prime_mmap_coherency@ioctl-errors: - shard-hsw: [PASS][37] -> [FAIL][38] ([i915#831]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw1/igt@prime_mmap_coherency@ioctl-errors.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw6/igt@prime_mmap_coherency@ioctl-errors.html #### Possible fixes #### * igt@gem_busy@busy-vcs1: - shard-iclb: [SKIP][39] ([fdo#112080]) -> [PASS][40] +14 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb5/igt@gem_busy@busy-vcs1.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb2/igt@gem_busy@busy-vcs1.html * igt@gem_caching@reads: - shard-hsw: [FAIL][41] ([i915#694]) -> [PASS][42] +2 similar issues [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw8/igt@gem_caching@reads.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw6/igt@gem_caching@reads.html * igt@gem_ctx_persistence@vcs1-persistence: - shard-iclb: [SKIP][43] ([fdo#109276] / [fdo#112080]) -> [PASS][44] [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb3/igt@gem_ctx_persistence@vcs1-persistence.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb2/igt@gem_ctx_persistence@vcs1-persistence.html * igt@gem_exec_schedule@independent-bsd2: - shard-iclb: [SKIP][45] ([fdo#109276]) -> [PASS][46] +24 similar issues [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb8/igt@gem_exec_schedule@independent-bsd2.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb1/igt@gem_exec_schedule@independent-bsd2.html * igt@gem_exec_schedule@pi-distinct-iova-bsd: - shard-iclb: [SKIP][47] ([i915#677]) -> [PASS][48] [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb4/igt@gem_exec_schedule@pi-distinct-iova-bsd.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb7/igt@gem_exec_schedule@pi-distinct-iova-bsd.html * igt@gem_exec_schedule@reorder-wide-bsd: - shard-iclb: [SKIP][49] ([fdo#112146]) -> [PASS][50] +8 similar issues [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb4/igt@gem_exec_schedule@reorder-wide-bsd.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb6/igt@gem_exec_schedule@reorder-wide-bsd.html * igt@gen9_exec_parse@allowed-all: - shard-glk: [INCOMPLETE][51] ([i915#58] / [k.org#198133]) -> [PASS][52] [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk9/igt@gen9_exec_parse@allowed-all.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk6/igt@gen9_exec_parse@allowed-all.html * igt@i915_pm_rps@reset: - shard-iclb: [FAIL][53] ([i915#413]) -> [PASS][54] +1 similar issue [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb3/igt@i915_pm_rps@reset.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb8/igt@i915_pm_rps@reset.html * igt@i915_selftest@live_blt: - shard-hsw: [DMESG-FAIL][55] ([i915#553] / [i915#725]) -> [PASS][56] [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw5/igt@i915_selftest@live_blt.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw2/igt@i915_selftest@live_blt.html * igt@kms_busy@extended-modeset-hang-newfb-render-pipe-c: - shard-hsw: [INCOMPLETE][57] ([CI#80] / [i915#61]) -> [PASS][58] [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-hsw7/igt@kms_busy@extended-modeset-hang-newfb-render-pipe-c.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-hsw2/igt@kms_busy@extended-modeset-hang-newfb-render-pipe-c.html * igt@kms_cursor_crc@pipe-a-cursor-suspend: - shard-kbl: [DMESG-WARN][59] ([i915#180]) -> [PASS][60] [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-kbl6/igt@kms_cursor_crc@pipe-a-cursor-suspend.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html * igt@kms_psr@psr2_cursor_mmap_cpu: - shard-iclb: [SKIP][61] ([fdo#109441]) -> [PASS][62] [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb7/igt@kms_psr@psr2_cursor_mmap_cpu.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html * igt@kms_vblank@pipe-c-ts-continuation-suspend: - shard-apl: [DMESG-WARN][63] ([i915#180]) -> [PASS][64] +2 similar issues [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-apl4/igt@kms_vblank@pipe-c-ts-continuation-suspend.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-apl3/igt@kms_vblank@pipe-c-ts-continuation-suspend.html #### Warnings #### * igt@gem_ctx_isolation@vcs1-nonpriv: - shard-iclb: [SKIP][65] ([fdo#109276] / [fdo#112080]) -> [FAIL][66] ([IGT#28]) [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb3/igt@gem_ctx_isolation@vcs1-nonpriv.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb2/igt@gem_ctx_isolation@vcs1-nonpriv.html * igt@gem_ctx_isolation@vcs1-nonpriv-switch: - shard-iclb: [FAIL][67] ([IGT#28]) -> [SKIP][68] ([fdo#109276] / [fdo#112080]) [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb2/igt@gem_ctx_isolation@vcs1-nonpriv-switch.html [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb7/igt@gem_ctx_isolation@vcs1-nonpriv-switch.html * igt@gem_ctx_persistence@vcs1-mixed-process: - shard-iclb: [SKIP][69] ([fdo#109276] / [fdo#112080]) -> [SKIP][70] ([fdo#112080]) +9 similar issues [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-iclb7/igt@gem_ctx_persistence@vcs1-mixed-process.html [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-iclb5/igt@gem_ctx_persistence@vcs1-mixed-process.html * igt@gem_eio@in-flight-10ms: - shard-glk: [TIMEOUT][71] ([fdo#112271]) -> [INCOMPLETE][72] ([CI#80] / [i915#58] / [k.org#198133]) [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk5/igt@gem_eio@in-flight-10ms.html [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk3/igt@gem_eio@in-flight-10ms.html * igt@gem_eio@in-flight-contexts-10ms: - shard-glk: [INCOMPLETE][73] ([CI#80] / [i915#58] / [k.org#198133]) -> [TIMEOUT][74] ([fdo#112271]) [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk8/igt@gem_eio@in-flight-contexts-10ms.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk9/igt@gem_eio@in-flight-contexts-10ms.html * igt@gem_eio@kms: - shard-glk: [INCOMPLETE][75] ([CI#80] / [i915#58] / [k.org#198133]) -> [INCOMPLETE][76] ([i915#58] / [k.org#198133]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk7/igt@gem_eio@kms.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk6/igt@gem_eio@kms.html * igt@gem_eio@unwedge-stress: - shard-glk: [TIMEOUT][77] ([fdo#112271]) -> [FAIL][78] ([i915#232]) [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-glk8/igt@gem_eio@unwedge-stress.html [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-glk4/igt@gem_eio@unwedge-stress.html - shard-apl: [TIMEOUT][79] ([fdo#112271]) -> [INCOMPLETE][80] ([CI#80] / [fdo#103927]) +1 similar issue [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-apl4/igt@gem_eio@unwedge-stress.html [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-apl6/igt@gem_eio@unwedge-stress.html * igt@gem_userptr_blits@dmabuf-unsync: - shard-snb: [DMESG-WARN][81] ([fdo#111870] / [i915#478]) -> [DMESG-WARN][82] ([fdo#110789] / [fdo#111870] / [i915#478]) [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-snb5/igt@gem_userptr_blits@dmabuf-unsync.html [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-snb1/igt@gem_userptr_blits@dmabuf-unsync.html * igt@i915_pm_dc@dc6-dpms: - shard-tglb: [SKIP][83] ([i915#468]) -> [FAIL][84] ([i915#454]) [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-tglb2/igt@i915_pm_dc@dc6-dpms.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-tglb3/igt@i915_pm_dc@dc6-dpms.html * igt@i915_pm_rpm@gem-idle: - shard-snb: [INCOMPLETE][85] ([i915#82]) -> [SKIP][86] ([fdo#109271]) [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7829/shard-snb2/igt@i915_pm_rpm@gem-idle.html [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/shard-snb2/igt@i915_pm_rpm@gem-idle.html [CI#80]: https://gitlab.freedesktop.org/gfx-ci/i915-infra/issues/80 [IGT#28]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/28 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#110789]: https://bugs.freedesktop.org/show_bug.cgi?id=110789 [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841 [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870 [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080 [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146 [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#232]: https://gitlab.freedesktop.org/drm/intel/issues/232 [i915#413]: https://gitlab.freedesktop.org/drm/intel/issues/413 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#468]: https://gitlab.freedesktop.org/drm/intel/issues/468 [i915#478]: https://gitlab.freedesktop.org/drm/intel/issues/478 [i915#51]: https://gitlab.freedesktop.org/drm/intel/issues/51 [i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553 [i915#58]: https://gitlab.freedesktop.org/drm/intel/issues/58 [i915#61]: https://gitlab.freedesktop.org/drm/intel/issues/61 [i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644 [i915#677]: https://gitlab.freedesktop.org/drm/intel/issues/677 [i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79 [i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82 [i915#831]: https://gitlab.freedesktop.org/drm/intel/issues/831 [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133 Participating hosts (10 -> 8) ------------------------------ Missing (2): pig-skl-6260u pig-glk-j5005 Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5392 -> IGTPW_4014 * Piglit: piglit_4509 -> None CI-20190529: 20190529 CI_DRM_7829: d7c77ce081cc74a8e9f14a68769509ebabab5aa5 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_4014: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/index.html IGT_5392: fb8487a63fcbe1c129206c9303bfcd5feb0e3ee1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4014/index.html _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines @ 2020-01-28 8:15 Bommu Krishnaiah 2020-01-28 8:15 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah 0 siblings, 1 reply; 6+ messages in thread From: Bommu Krishnaiah @ 2020-01-28 8:15 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah Update the context with supported engines on the platform with set_property I915_CONTEXT_PARAM_ENGINES to make sure the work load is submitted to the available engines only. v2: addressed the review comments. v3: addressed v2 review comments. V4: addressed v3 review comments. In v4 Re-introduced the gem_context_set_all_engines() API, test_process_mixed() test creating new forked process, to set engine map to new forked process I added gem_context_set_all_engines() API. Bommu Krishnaiah (1): tests/i915/gem_ctx_persistence: Set context with supported engines lib/i915/gem_engine_topology.c | 11 ++++++ lib/i915/gem_engine_topology.h | 2 + tests/i915/gem_ctx_persistence.c | 65 ++++++++++++++++++++++++-------- 3 files changed, 62 insertions(+), 16 deletions(-) -- 2.24.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] [PATCH i-g-t v4 1/1] tests/i915/gem_ctx_persistence: Set context with supported engines 2020-01-28 8:15 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah @ 2020-01-28 8:15 ` Bommu Krishnaiah 0 siblings, 0 replies; 6+ messages in thread From: Bommu Krishnaiah @ 2020-01-28 8:15 UTC (permalink / raw) To: igt-dev; +Cc: Bommu Krishnaiah, Tvrtko Ursulin Update the context with supported engines on the platform with set_property I915_CONTEXT_PARAM_ENGINES to make sure the work load is submitted to the available engines only. v2: addressed the review comments. v3: addressed v2 review comments. V4: addressed v3 review comments. Signed-off-by: Bommu Krishnaiah <krishnaiah.bommu@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> --- lib/i915/gem_engine_topology.c | 11 ++++++ lib/i915/gem_engine_topology.h | 2 + tests/i915/gem_ctx_persistence.c | 65 ++++++++++++++++++++++++-------- 3 files changed, 62 insertions(+), 16 deletions(-) diff --git a/lib/i915/gem_engine_topology.c b/lib/i915/gem_engine_topology.c index 43a99e0f..ed6737ef 100644 --- a/lib/i915/gem_engine_topology.c +++ b/lib/i915/gem_engine_topology.c @@ -274,6 +274,17 @@ int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, return 0; } +void gem_context_set_all_engines(int fd, uint32_t ctx) +{ + DEFINE_CONTEXT_ENGINES_PARAM(engines, param, ctx, GEM_MAX_ENGINES); + struct intel_engine_data engine_data = { }; + + if (!gem_topology_get_param(fd, ¶m) && !param.size) { + query_engine_list(fd, &engine_data); + ctx_map_engines(fd, &engine_data, ¶m); + } +} + bool gem_has_engine_topology(int fd) { struct drm_i915_gem_context_param param = { diff --git a/lib/i915/gem_engine_topology.h b/lib/i915/gem_engine_topology.h index e40d7ec8..525741cc 100644 --- a/lib/i915/gem_engine_topology.h +++ b/lib/i915/gem_engine_topology.h @@ -51,6 +51,8 @@ void intel_next_engine(struct intel_engine_data *ed); int gem_context_lookup_engine(int fd, uint64_t engine, uint32_t ctx_id, struct intel_execution_engine2 *e); +void gem_context_set_all_engines(int fd, uint32_t ctx); + bool gem_context_has_engine_map(int fd, uint32_t ctx); bool gem_engine_is_equal(const struct intel_execution_engine2 *e1, diff --git a/tests/i915/gem_ctx_persistence.c b/tests/i915/gem_ctx_persistence.c index d68431ae..e0c6965a 100644 --- a/tests/i915/gem_ctx_persistence.c +++ b/tests/i915/gem_ctx_persistence.c @@ -156,7 +156,7 @@ static void test_persistence(int i915, unsigned int engine) * request is retired -- no early termination. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, true); spin = igt_spin_new(i915, ctx, @@ -188,7 +188,7 @@ static void test_nonpersistent_cleanup(int i915, unsigned int engine) * any inflight request is cancelled. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, @@ -217,7 +217,7 @@ static void test_nonpersistent_mixed(int i915, unsigned int engine) igt_spin_t *spin; uint32_t ctx; - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, i & 1); spin = igt_spin_new(i915, ctx, @@ -253,7 +253,7 @@ static void test_nonpersistent_hostile(int i915, unsigned int engine) * the requests and cleanup the context. */ - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, @@ -284,7 +284,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) igt_require(gem_scheduler_has_preemption(i915)); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, true); gem_context_set_priority(i915, ctx, 0); spin[0] = igt_spin_new(i915, ctx, @@ -295,7 +295,7 @@ static void test_nonpersistent_hostile_preempt(int i915, unsigned int engine) igt_spin_busywait_until_started(spin[0]); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); gem_context_set_priority(i915, ctx, 1); /* higher priority than 0 */ spin[1] = igt_spin_new(i915, ctx, @@ -385,7 +385,7 @@ static void test_nonpersistent_queued(int i915, unsigned int engine) gem_quiescent_gpu(i915); - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); gem_context_set_persistence(i915, ctx, false); spin = igt_spin_new(i915, ctx, .engine = engine, @@ -512,7 +512,8 @@ static void test_process_mixed(int i915, unsigned int engine) igt_spin_t *spin; uint32_t ctx; - ctx = gem_context_create(i915); + ctx = gem_context_clone_with_engines(i915, 0); + gem_context_set_all_engines(i915, ctx); gem_context_set_persistence(i915, ctx, persists); spin = igt_spin_new(i915, ctx, @@ -727,37 +728,69 @@ igt_main igt_subtest("hangcheck") test_nohangcheck_hostile(i915); - __for_each_static_engine(e) { + for_each_engine(e, i915) { igt_subtest_group { igt_fixture { gem_require_ring(i915, e->flags); gem_require_contexts(i915); } - igt_subtest_f("%s-persistence", e->name) + igt_subtest_f("legacy-%s-persistence", e->name) test_persistence(i915, e->flags); - igt_subtest_f("%s-cleanup", e->name) + igt_subtest_f("legacy-%s-cleanup", e->name) test_nonpersistent_cleanup(i915, e->flags); - igt_subtest_f("%s-queued", e->name) + igt_subtest_f("legacy-%s-queued", e->name) test_nonpersistent_queued(i915, e->flags); - igt_subtest_f("%s-mixed", e->name) + igt_subtest_f("legacy-%s-mixed", e->name) test_nonpersistent_mixed(i915, e->flags); - igt_subtest_f("%s-mixed-process", e->name) + igt_subtest_f("legacy-%s-mixed-process", e->name) test_process_mixed(i915, e->flags); - igt_subtest_f("%s-hostile", e->name) + igt_subtest_f("legacy-%s-hostile", e->name) test_nonpersistent_hostile(i915, e->flags); - igt_subtest_f("%s-hostile-preempt", e->name) + igt_subtest_f("legacy-%s-hostile-preempt", e->name) test_nonpersistent_hostile_preempt(i915, e->flags); } } + __for_each_physical_engine(i915, e) { + igt_subtest_group { + igt_fixture { + gem_require_ring(i915, e->flags); + gem_require_contexts(i915); + } + + igt_subtest_f("%s-persistence", e->name) + test_persistence(i915, e->flags); + + igt_subtest_f("%s-cleanup", e->name) + test_nonpersistent_cleanup(i915, e->flags); + + igt_subtest_f("%s-queued", e->name) + test_nonpersistent_queued(i915, e->flags); + + igt_subtest_f("%s-mixed", e->name) + test_nonpersistent_mixed(i915, e->flags); + + igt_subtest_f("%s-mixed-process", e->name) + test_process_mixed(i915, e->flags); + + igt_subtest_f("%s-hostile", e->name) + test_nonpersistent_hostile(i915, e->flags); + + igt_subtest_f("%s-hostile-preempt", e->name) + test_nonpersistent_hostile_preempt(i915, + e->flags); + } + } + + igt_subtest("smoketest") smoketest(i915); -- 2.24.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-01-29 16:00 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-01-28 9:12 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah 2020-01-28 9:12 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah 2020-01-28 9:41 ` Tvrtko Ursulin 2020-01-28 12:57 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/gem_ctx_persistence: Set context with supported engines (rev5) Patchwork 2020-01-29 16:00 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork -- strict thread matches above, loose matches on Subject: below -- 2020-01-28 8:15 [igt-dev] [PATCH i-g-t v4 0/1] tests/i915/gem_ctx_persistence: Set context with supported engines Bommu Krishnaiah 2020-01-28 8:15 ` [igt-dev] [PATCH i-g-t v4 1/1] " Bommu Krishnaiah
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox