* [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines @ 2019-09-25 19:34 Chris Wilson 2019-09-25 20:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork ` (3 more replies) 0 siblings, 4 replies; 5+ messages in thread From: Chris Wilson @ 2019-09-25 19:34 UTC (permalink / raw) To: intel-gfx The simplest and most maximal submission we can do, a thread to submit requests unto each engine. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> --- drivers/gpu/drm/i915/selftests/i915_request.c | 125 ++++++++++++++++++ 1 file changed, 125 insertions(+) diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c index b3688543ed7d..57cd4180d06c 100644 --- a/drivers/gpu/drm/i915/selftests/i915_request.c +++ b/drivers/gpu/drm/i915/selftests/i915_request.c @@ -1062,6 +1062,130 @@ static int live_sequential_engines(void *arg) return err; } +static int __live_parallel_engine1(void *arg) +{ + struct intel_engine_cs *engine = arg; + IGT_TIMEOUT(end_time); + unsigned long count; + + count = 0; + do { + struct i915_request *rq; + int err; + + mutex_lock(&engine->i915->drm.struct_mutex); + rq = i915_request_create(engine->kernel_context); + if (IS_ERR(rq)) { + mutex_unlock(&engine->i915->drm.struct_mutex); + return PTR_ERR(rq); + } + + i915_request_get(rq); + i915_request_add(rq); + mutex_unlock(&engine->i915->drm.struct_mutex); + + err = 0; + if (i915_request_wait(rq, 0, HZ / 5) < 0) + err = -ETIME; + i915_request_put(rq); + if (err) + return err; + + count++; + } while (!__igt_timeout(end_time, NULL)); + + pr_info("%s: %lu request + sync\n", engine->name, count); + return 0; +} + +static int __live_parallel_engineN(void *arg) +{ + struct intel_engine_cs *engine = arg; + IGT_TIMEOUT(end_time); + unsigned long count; + + count = 0; + do { + struct i915_request *rq; + + mutex_lock(&engine->i915->drm.struct_mutex); + rq = i915_request_create(engine->kernel_context); + if (IS_ERR(rq)) { + mutex_unlock(&engine->i915->drm.struct_mutex); + return PTR_ERR(rq); + } + + i915_request_add(rq); + mutex_unlock(&engine->i915->drm.struct_mutex); + + count++; + } while (!__igt_timeout(end_time, NULL)); + + pr_info("%s: %lu requests\n", engine->name, count); + return 0; +} + +static int live_parallel_engines(void *arg) +{ + struct drm_i915_private *i915 = arg; + static int (* const func[])(void *arg) = { + __live_parallel_engine1, + __live_parallel_engineN, + NULL, + }; + struct intel_engine_cs *engine; + enum intel_engine_id id; + int (* const *fn)(void *arg); + int err = 0; + + /* + * Check we can submit requests to all engines concurrently. This + * tests that we load up the system maximally. + */ + + for (fn = func; !err && *fn; fn++) { + struct task_struct *tsk[I915_NUM_ENGINES] = {}; + struct igt_live_test t; + + mutex_lock(&i915->drm.struct_mutex); + err = igt_live_test_begin(&t, i915, __func__, ""); + mutex_unlock(&i915->drm.struct_mutex); + if (err) + break; + + for_each_engine(engine, i915, id) { + tsk[id] = kthread_run(*fn, engine, + "igt/parallel:%s", + engine->name); + if (IS_ERR(tsk[id])) { + err = PTR_ERR(tsk[id]); + break; + } + get_task_struct(tsk[id]); + } + + for_each_engine(engine, i915, id) { + int status; + + if (IS_ERR_OR_NULL(tsk[id])) + continue; + + status = kthread_stop(tsk[id]); + if (status && !err) + err = status; + + put_task_struct(tsk[id]); + } + + mutex_lock(&i915->drm.struct_mutex); + if (igt_live_test_end(&t)) + err = -EIO; + mutex_unlock(&i915->drm.struct_mutex); + } + + return err; +} + static int max_batches(struct i915_gem_context *ctx, struct intel_engine_cs *engine) { @@ -1240,6 +1364,7 @@ int i915_request_live_selftests(struct drm_i915_private *i915) SUBTEST(live_nop_request), SUBTEST(live_all_engines), SUBTEST(live_sequential_engines), + SUBTEST(live_parallel_engines), SUBTEST(live_empty_request), SUBTEST(live_breadcrumbs_smoketest), }; -- 2.23.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 5+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Exercise concurrent submission to all engines 2019-09-25 19:34 [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines Chris Wilson @ 2019-09-25 20:09 ` Patchwork 2019-09-25 20:32 ` ✓ Fi.CI.BAT: success " Patchwork ` (2 subsequent siblings) 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2019-09-25 20:09 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Exercise concurrent submission to all engines URL : https://patchwork.freedesktop.org/series/67237/ State : warning == Summary == $ dim checkpatch origin/drm-tip 92cc309e8be5 drm/i915/selftests: Exercise concurrent submission to all engines -:24: WARNING:LINE_SPACING: Missing a blank line after declarations #24: FILE: drivers/gpu/drm/i915/selftests/i915_request.c:1068: + struct intel_engine_cs *engine = arg; + IGT_TIMEOUT(end_time); -:60: WARNING:LINE_SPACING: Missing a blank line after declarations #60: FILE: drivers/gpu/drm/i915/selftests/i915_request.c:1104: + struct intel_engine_cs *engine = arg; + IGT_TIMEOUT(end_time); -:87: WARNING:LINE_SPACING: Missing a blank line after declarations #87: FILE: drivers/gpu/drm/i915/selftests/i915_request.c:1131: + struct drm_i915_private *i915 = arg; + static int (* const func[])(void *arg) = { -:94: WARNING:LINE_SPACING: Missing a blank line after declarations #94: FILE: drivers/gpu/drm/i915/selftests/i915_request.c:1138: + enum intel_engine_id id; + int (* const *fn)(void *arg); total: 0 errors, 4 warnings, 0 checks, 137 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915/selftests: Exercise concurrent submission to all engines 2019-09-25 19:34 [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines Chris Wilson 2019-09-25 20:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork @ 2019-09-25 20:32 ` Patchwork 2019-09-26 14:45 ` ✓ Fi.CI.IGT: " Patchwork 2019-09-27 10:38 ` [PATCH] " Mika Kuoppala 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2019-09-25 20:32 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Exercise concurrent submission to all engines URL : https://patchwork.freedesktop.org/series/67237/ State : success == Summary == CI Bug Log - changes from CI_DRM_6958 -> Patchwork_14537 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_14537: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@i915_selftest@live_requests: - {fi-tgl-u2}: NOTRUN -> [INCOMPLETE][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/fi-tgl-u2/igt@i915_selftest@live_requests.html Known issues ------------ Here are the changes found in Patchwork_14537 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: [PASS][2] -> [FAIL][3] ([fdo#109483]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html #### Possible fixes #### * igt@gem_ctx_switch@rcs0: - {fi-icl-guc}: [INCOMPLETE][4] ([fdo#107713]) -> [PASS][5] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/fi-icl-guc/igt@gem_ctx_switch@rcs0.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/fi-icl-guc/igt@gem_ctx_switch@rcs0.html * igt@kms_frontbuffer_tracking@basic: - fi-hsw-peppy: [DMESG-WARN][6] ([fdo#102614]) -> [PASS][7] [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614 [fdo#106107]: https://bugs.freedesktop.org/show_bug.cgi?id=106107 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#109483]: https://bugs.freedesktop.org/show_bug.cgi?id=109483 Participating hosts (50 -> 43) ------------------------------ Additional (1): fi-tgl-u2 Missing (8): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-kbl-7500u fi-icl-y fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_6958 -> Patchwork_14537 CI-20190529: 20190529 CI_DRM_6958: d5c4d0bc1c0570ba9128e7afb419d5d5a8ebd4bc @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5203: 82326332f7af336d390e00ae87187bc207fd33dd @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14537: 92cc309e8be552039795fc00b1182c8fac10fdf3 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 92cc309e8be5 drm/i915/selftests: Exercise concurrent submission to all engines == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* ✓ Fi.CI.IGT: success for drm/i915/selftests: Exercise concurrent submission to all engines 2019-09-25 19:34 [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines Chris Wilson 2019-09-25 20:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2019-09-25 20:32 ` ✓ Fi.CI.BAT: success " Patchwork @ 2019-09-26 14:45 ` Patchwork 2019-09-27 10:38 ` [PATCH] " Mika Kuoppala 3 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2019-09-26 14:45 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Exercise concurrent submission to all engines URL : https://patchwork.freedesktop.org/series/67237/ State : success == Summary == CI Bug Log - changes from CI_DRM_6958_full -> Patchwork_14537_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_14537_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_eio@reset-stress: - shard-snb: [PASS][1] -> [FAIL][2] ([fdo#109661]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-snb6/igt@gem_eio@reset-stress.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-snb4/igt@gem_eio@reset-stress.html * igt@gem_exec_schedule@preempt-contexts-bsd: - shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#111325]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb3/igt@gem_exec_schedule@preempt-contexts-bsd.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb1/igt@gem_exec_schedule@preempt-contexts-bsd.html * igt@gem_pwrite@small-cpu-backwards: - shard-apl: [PASS][5] -> [INCOMPLETE][6] ([fdo#103927]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-apl7/igt@gem_pwrite@small-cpu-backwards.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-apl3/igt@gem_pwrite@small-cpu-backwards.html * igt@i915_pm_rpm@system-suspend: - shard-skl: [PASS][7] -> [INCOMPLETE][8] ([fdo#104108] / [fdo#107807]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl3/igt@i915_pm_rpm@system-suspend.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl4/igt@i915_pm_rpm@system-suspend.html * igt@kms_frontbuffer_tracking@fbc-stridechange: - shard-iclb: [PASS][9] -> [FAIL][10] ([fdo#103167]) +6 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-stridechange.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-stridechange.html * igt@kms_frontbuffer_tracking@fbc-suspend: - shard-apl: [PASS][11] -> [DMESG-WARN][12] ([fdo#108566]) +2 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-apl7/igt@kms_frontbuffer_tracking@fbc-suspend.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-apl1/igt@kms_frontbuffer_tracking@fbc-suspend.html * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a: - shard-skl: [PASS][13] -> [FAIL][14] ([fdo#103191]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl9/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl1/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c: - shard-kbl: [PASS][15] -> [DMESG-WARN][16] ([fdo#108566]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-kbl3/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-kbl3/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes: - shard-skl: [PASS][17] -> [INCOMPLETE][18] ([fdo#104108]) +1 similar issue [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min: - shard-skl: [PASS][19] -> [FAIL][20] ([fdo#108145]) +2 similar issues [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl1/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl3/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html * igt@perf@blocking: - shard-skl: [PASS][21] -> [FAIL][22] ([fdo#110728]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl7/igt@perf@blocking.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl1/igt@perf@blocking.html * igt@perf_pmu@enable-race-vcs0: - shard-iclb: [PASS][23] -> [INCOMPLETE][24] ([fdo#107713]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb5/igt@perf_pmu@enable-race-vcs0.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb7/igt@perf_pmu@enable-race-vcs0.html * igt@prime_busy@hang-bsd2: - shard-iclb: [PASS][25] -> [SKIP][26] ([fdo#109276]) +7 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb1/igt@prime_busy@hang-bsd2.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb7/igt@prime_busy@hang-bsd2.html #### Possible fixes #### * igt@gem_ctx_isolation@rcs0-s3: - shard-skl: [INCOMPLETE][27] ([fdo#104108]) -> [PASS][28] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-skl4/igt@gem_ctx_isolation@rcs0-s3.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-skl6/igt@gem_ctx_isolation@rcs0-s3.html * igt@gem_exec_schedule@in-order-bsd: - shard-iclb: [SKIP][29] ([fdo#111325]) -> [PASS][30] +3 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb2/igt@gem_exec_schedule@in-order-bsd.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb5/igt@gem_exec_schedule@in-order-bsd.html * igt@gem_exec_schedule@promotion-bsd1: - shard-iclb: [SKIP][31] ([fdo#109276]) -> [PASS][32] +9 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb8/igt@gem_exec_schedule@promotion-bsd1.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb4/igt@gem_exec_schedule@promotion-bsd1.html * igt@gem_softpin@noreloc-s3: - shard-apl: [DMESG-WARN][33] ([fdo#108566]) -> [PASS][34] +4 similar issues [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-apl8/igt@gem_softpin@noreloc-s3.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-apl8/igt@gem_softpin@noreloc-s3.html * igt@gem_workarounds@suspend-resume-context: - shard-apl: [INCOMPLETE][35] ([fdo#103927]) -> [PASS][36] +2 similar issues [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-apl6/igt@gem_workarounds@suspend-resume-context.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-apl2/igt@gem_workarounds@suspend-resume-context.html * igt@kms_flip@flip-vs-expired-vblank-interruptible: - shard-glk: [FAIL][37] ([fdo#105363]) -> [PASS][38] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-glk1/igt@kms_flip@flip-vs-expired-vblank-interruptible.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-glk9/igt@kms_flip@flip-vs-expired-vblank-interruptible.html * igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw: - shard-iclb: [FAIL][39] ([fdo#103167]) -> [PASS][40] +2 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-iclb: [INCOMPLETE][41] ([fdo#107713]) -> [PASS][42] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb8/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_psr@psr2_no_drrs: - shard-iclb: [SKIP][43] ([fdo#109441]) -> [PASS][44] [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb4/igt@kms_psr@psr2_no_drrs.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb2/igt@kms_psr@psr2_no_drrs.html * igt@kms_rotation_crc@sprite-rotation-90: - shard-iclb: [INCOMPLETE][45] ([fdo#107713] / [fdo#110026]) -> [PASS][46] [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-iclb7/igt@kms_rotation_crc@sprite-rotation-90.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-iclb1/igt@kms_rotation_crc@sprite-rotation-90.html * igt@kms_vblank@pipe-a-ts-continuation-suspend: - shard-kbl: [DMESG-WARN][47] ([fdo#108566]) -> [PASS][48] +1 similar issue [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6958/shard-kbl3/igt@kms_vblank@pipe-a-ts-continuation-suspend.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/shard-kbl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661 [fdo#110026]: https://bugs.freedesktop.org/show_bug.cgi?id=110026 [fdo#110548]: https://bugs.freedesktop.org/show_bug.cgi?id=110548 [fdo#110728]: https://bugs.freedesktop.org/show_bug.cgi?id=110728 [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325 Participating hosts (9 -> 9) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_6958 -> Patchwork_14537 CI-20190529: 20190529 CI_DRM_6958: d5c4d0bc1c0570ba9128e7afb419d5d5a8ebd4bc @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5203: 82326332f7af336d390e00ae87187bc207fd33dd @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14537: 92cc309e8be552039795fc00b1182c8fac10fdf3 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14537/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines 2019-09-25 19:34 [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines Chris Wilson ` (2 preceding siblings ...) 2019-09-26 14:45 ` ✓ Fi.CI.IGT: " Patchwork @ 2019-09-27 10:38 ` Mika Kuoppala 3 siblings, 0 replies; 5+ messages in thread From: Mika Kuoppala @ 2019-09-27 10:38 UTC (permalink / raw) To: Chris Wilson, intel-gfx Chris Wilson <chris@chris-wilson.co.uk> writes: > The simplest and most maximal submission we can do, a thread to submit > requests unto each engine. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> > --- > drivers/gpu/drm/i915/selftests/i915_request.c | 125 ++++++++++++++++++ > 1 file changed, 125 insertions(+) > > diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c > index b3688543ed7d..57cd4180d06c 100644 > --- a/drivers/gpu/drm/i915/selftests/i915_request.c > +++ b/drivers/gpu/drm/i915/selftests/i915_request.c > @@ -1062,6 +1062,130 @@ static int live_sequential_engines(void *arg) > return err; > } > > +static int __live_parallel_engine1(void *arg) > +{ > + struct intel_engine_cs *engine = arg; > + IGT_TIMEOUT(end_time); > + unsigned long count; > + > + count = 0; > + do { > + struct i915_request *rq; > + int err; > + > + mutex_lock(&engine->i915->drm.struct_mutex); > + rq = i915_request_create(engine->kernel_context); > + if (IS_ERR(rq)) { > + mutex_unlock(&engine->i915->drm.struct_mutex); > + return PTR_ERR(rq); > + } > + > + i915_request_get(rq); > + i915_request_add(rq); > + mutex_unlock(&engine->i915->drm.struct_mutex); > + > + err = 0; > + if (i915_request_wait(rq, 0, HZ / 5) < 0) > + err = -ETIME; > + i915_request_put(rq); > + if (err) > + return err; > + > + count++; > + } while (!__igt_timeout(end_time, NULL)); > + > + pr_info("%s: %lu request + sync\n", engine->name, count); > + return 0; > +} > + > +static int __live_parallel_engineN(void *arg) > +{ > + struct intel_engine_cs *engine = arg; > + IGT_TIMEOUT(end_time); > + unsigned long count; > + > + count = 0; > + do { > + struct i915_request *rq; > + > + mutex_lock(&engine->i915->drm.struct_mutex); > + rq = i915_request_create(engine->kernel_context); > + if (IS_ERR(rq)) { > + mutex_unlock(&engine->i915->drm.struct_mutex); > + return PTR_ERR(rq); > + } > + > + i915_request_add(rq); > + mutex_unlock(&engine->i915->drm.struct_mutex); > + > + count++; > + } while (!__igt_timeout(end_time, NULL)); > + > + pr_info("%s: %lu requests\n", engine->name, count); > + return 0; > +} > + > +static int live_parallel_engines(void *arg) > +{ > + struct drm_i915_private *i915 = arg; > + static int (* const func[])(void *arg) = { > + __live_parallel_engine1, > + __live_parallel_engineN, The ratio of waited vs nonwaited ones is now up to their competition for access which did concern me. But then on the other hand, due to wait, that thread will end up with lot less access. So, I think in the end the pattern will be mostly unwaited ones with a wait now and then. Which was kind of what I was yearning for so, Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com> > + NULL, > + }; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + int (* const *fn)(void *arg); > + int err = 0; > + > + /* > + * Check we can submit requests to all engines concurrently. This > + * tests that we load up the system maximally. > + */ > + > + for (fn = func; !err && *fn; fn++) { > + struct task_struct *tsk[I915_NUM_ENGINES] = {}; > + struct igt_live_test t; > + > + mutex_lock(&i915->drm.struct_mutex); > + err = igt_live_test_begin(&t, i915, __func__, ""); > + mutex_unlock(&i915->drm.struct_mutex); > + if (err) > + break; > + > + for_each_engine(engine, i915, id) { > + tsk[id] = kthread_run(*fn, engine, > + "igt/parallel:%s", > + engine->name); > + if (IS_ERR(tsk[id])) { > + err = PTR_ERR(tsk[id]); > + break; > + } > + get_task_struct(tsk[id]); > + } > + > + for_each_engine(engine, i915, id) { > + int status; > + > + if (IS_ERR_OR_NULL(tsk[id])) > + continue; > + > + status = kthread_stop(tsk[id]); > + if (status && !err) > + err = status; > + > + put_task_struct(tsk[id]); > + } > + > + mutex_lock(&i915->drm.struct_mutex); > + if (igt_live_test_end(&t)) > + err = -EIO; > + mutex_unlock(&i915->drm.struct_mutex); > + } > + > + return err; > +} > + > static int > max_batches(struct i915_gem_context *ctx, struct intel_engine_cs *engine) > { > @@ -1240,6 +1364,7 @@ int i915_request_live_selftests(struct drm_i915_private *i915) > SUBTEST(live_nop_request), > SUBTEST(live_all_engines), > SUBTEST(live_sequential_engines), > + SUBTEST(live_parallel_engines), > SUBTEST(live_empty_request), > SUBTEST(live_breadcrumbs_smoketest), > }; > -- > 2.23.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-27 10:38 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-09-25 19:34 [PATCH] drm/i915/selftests: Exercise concurrent submission to all engines Chris Wilson 2019-09-25 20:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2019-09-25 20:32 ` ✓ Fi.CI.BAT: success " Patchwork 2019-09-26 14:45 ` ✓ Fi.CI.IGT: " Patchwork 2019-09-27 10:38 ` [PATCH] " Mika Kuoppala
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.