* [Intel-gfx] [PATCH] drm/i915/gt: Trim default context register state for gen9 @ 2020-01-28 23:07 Chris Wilson 2020-01-29 2:34 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2020-01-29 2:56 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork 0 siblings, 2 replies; 4+ messages in thread From: Chris Wilson @ 2020-01-28 23:07 UTC (permalink / raw) To: intel-gfx Now that it appears we have tamed the mystery hang on module reload, we can relax some of the counter-measures we employed to help stabilise and debug gen9. As we have aggressive poisoning of the context image, we can confidently reduce the default context state in gen9 to match the limits of the ring regs as laid out on other platforms. References: 9c6527116e8f ("drm/i915: Restore the kernel context after verifying the w/a") Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> --- drivers/gpu/drm/i915/gt/intel_lrc.c | 98 +---------------------------- 1 file changed, 2 insertions(+), 96 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c index cf6c43bd540a..7470dad9afff 100644 --- a/drivers/gpu/drm/i915/gt/intel_lrc.c +++ b/drivers/gpu/drm/i915/gt/intel_lrc.c @@ -631,54 +631,7 @@ static const u8 gen9_xcs_offsets[] = { LRI(1, POSTED), REG16(0x200), - NOP(13), - LRI(44, POSTED), - REG(0x028), - REG(0x09c), - REG(0x0c0), - REG(0x178), - REG(0x17c), - REG16(0x358), - REG(0x170), - REG(0x150), - REG(0x154), - REG(0x158), - REG16(0x41c), - REG16(0x600), - REG16(0x604), - REG16(0x608), - REG16(0x60c), - REG16(0x610), - REG16(0x614), - REG16(0x618), - REG16(0x61c), - REG16(0x620), - REG16(0x624), - REG16(0x628), - REG16(0x62c), - REG16(0x630), - REG16(0x634), - REG16(0x638), - REG16(0x63c), - REG16(0x640), - REG16(0x644), - REG16(0x648), - REG16(0x64c), - REG16(0x650), - REG16(0x654), - REG16(0x658), - REG16(0x65c), - REG16(0x660), - REG16(0x664), - REG16(0x668), - REG16(0x66c), - REG16(0x670), - REG16(0x674), - REG16(0x678), - REG16(0x67c), - REG(0x068), - - END(176) + END(80) }; static const u8 gen12_xcs_offsets[] = { @@ -784,54 +737,7 @@ static const u8 gen9_rcs_offsets[] = { LRI(1, 0), REG(0xc8), - NOP(13), - LRI(44, POSTED), - REG(0x28), - REG(0x9c), - REG(0xc0), - REG(0x178), - REG(0x17c), - REG16(0x358), - REG(0x170), - REG(0x150), - REG(0x154), - REG(0x158), - REG16(0x41c), - REG16(0x600), - REG16(0x604), - REG16(0x608), - REG16(0x60c), - REG16(0x610), - REG16(0x614), - REG16(0x618), - REG16(0x61c), - REG16(0x620), - REG16(0x624), - REG16(0x628), - REG16(0x62c), - REG16(0x630), - REG16(0x634), - REG16(0x638), - REG16(0x63c), - REG16(0x640), - REG16(0x644), - REG16(0x648), - REG16(0x64c), - REG16(0x650), - REG16(0x654), - REG16(0x658), - REG16(0x65c), - REG16(0x660), - REG16(0x664), - REG16(0x668), - REG16(0x66c), - REG16(0x670), - REG16(0x674), - REG16(0x678), - REG16(0x67c), - REG(0x68), - - END(176) + END(80) }; static const u8 gen11_rcs_offsets[] = { -- 2.25.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Trim default context register state for gen9 2020-01-28 23:07 [Intel-gfx] [PATCH] drm/i915/gt: Trim default context register state for gen9 Chris Wilson @ 2020-01-29 2:34 ` Patchwork 2020-01-29 2:56 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork 1 sibling, 0 replies; 4+ messages in thread From: Patchwork @ 2020-01-29 2:34 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/gt: Trim default context register state for gen9 URL : https://patchwork.freedesktop.org/series/72697/ State : warning == Summary == $ dim checkpatch origin/drm-tip 01bc69209e01 drm/i915/gt: Trim default context register state for gen9 -:12: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #12: References: 9c6527116e8f ("drm/i915: Restore the kernel context after verifying the w/a") -:12: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9c6527116e8f ("drm/i915: Restore the kernel context after verifying the w/a")' #12: References: 9c6527116e8f ("drm/i915: Restore the kernel context after verifying the w/a") total: 1 errors, 1 warnings, 0 checks, 110 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gt: Trim default context register state for gen9 2020-01-28 23:07 [Intel-gfx] [PATCH] drm/i915/gt: Trim default context register state for gen9 Chris Wilson 2020-01-29 2:34 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork @ 2020-01-29 2:56 ` Patchwork 2020-01-29 8:04 ` Chris Wilson 1 sibling, 1 reply; 4+ messages in thread From: Patchwork @ 2020-01-29 2:56 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/gt: Trim default context register state for gen9 URL : https://patchwork.freedesktop.org/series/72697/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7833 -> Patchwork_16308 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_16308 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_16308, 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/Patchwork_16308/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_16308: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live_execlists: - fi-skl-lmem: [PASS][1] -> [DMESG-FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-skl-lmem/igt@i915_selftest@live_execlists.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-skl-lmem/igt@i915_selftest@live_execlists.html - fi-cfl-8109u: [PASS][3] -> [DMESG-FAIL][4] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-cfl-8109u/igt@i915_selftest@live_execlists.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-cfl-8109u/igt@i915_selftest@live_execlists.html Known issues ------------ Here are the changes found in Patchwork_16308 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_close_race@basic-threads: - fi-byt-n2820: [PASS][5] -> [TIMEOUT][6] ([fdo#112271] / [i915#816]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-byt-n2820/igt@gem_close_race@basic-threads.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-byt-n2820/igt@gem_close_race@basic-threads.html * igt@gem_exec_parallel@fds: - fi-byt-n2820: [PASS][7] -> [TIMEOUT][8] ([fdo#112271]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-byt-n2820/igt@gem_exec_parallel@fds.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-byt-n2820/igt@gem_exec_parallel@fds.html * igt@i915_selftest@live_gem_contexts: - fi-byt-n2820: [PASS][9] -> [DMESG-FAIL][10] ([i915#1052]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-byt-n2820/igt@i915_selftest@live_gem_contexts.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-byt-n2820/igt@i915_selftest@live_gem_contexts.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: [PASS][11] -> [FAIL][12] ([i915#217]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html #### Possible fixes #### * igt@gem_close_race@basic-threads: - fi-hsw-peppy: [INCOMPLETE][13] ([i915#816]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-hsw-peppy/igt@gem_close_race@basic-threads.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-hsw-peppy/igt@gem_close_race@basic-threads.html - fi-byt-j1900: [INCOMPLETE][15] ([i915#45]) -> [PASS][16] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-byt-j1900/igt@gem_close_race@basic-threads.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-byt-j1900/igt@gem_close_race@basic-threads.html * igt@gem_exec_suspend@basic-s3: - fi-icl-u2: [FAIL][17] ([fdo#103375]) -> [PASS][18] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-icl-u2/igt@gem_exec_suspend@basic-s3.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-icl-u2/igt@gem_exec_suspend@basic-s3.html * igt@gem_exec_suspend@basic-s4-devices: - fi-icl-u2: [FAIL][19] ([fdo#111550]) -> [PASS][20] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-icl-u2/igt@gem_exec_suspend@basic-s4-devices.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-icl-u2/igt@gem_exec_suspend@basic-s4-devices.html * igt@i915_selftest@live_blt: - fi-hsw-4770r: [DMESG-FAIL][21] ([i915#563]) -> [PASS][22] [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-hsw-4770r/igt@i915_selftest@live_blt.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-hsw-4770r/igt@i915_selftest@live_blt.html - fi-hsw-4770: [DMESG-FAIL][23] ([i915#725]) -> [PASS][24] [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-hsw-4770/igt@i915_selftest@live_blt.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-hsw-4770/igt@i915_selftest@live_blt.html * igt@i915_selftest@live_gem_contexts: - fi-icl-guc: [INCOMPLETE][25] ([i915#140]) -> [PASS][26] [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-icl-guc/igt@i915_selftest@live_gem_contexts.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-icl-guc/igt@i915_selftest@live_gem_contexts.html #### Warnings #### * igt@gem_exec_parallel@contexts: - fi-byt-n2820: [TIMEOUT][27] ([fdo#112271]) -> [FAIL][28] ([i915#694]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-byt-n2820/igt@gem_exec_parallel@contexts.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-byt-n2820/igt@gem_exec_parallel@contexts.html * igt@i915_pm_rpm@basic-rte: - fi-kbl-guc: [FAIL][29] ([i915#579]) -> [SKIP][30] ([fdo#109271]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-kbl-guc/igt@i915_pm_rpm@basic-rte.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-kbl-guc/igt@i915_pm_rpm@basic-rte.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-icl-u2: [FAIL][31] ([fdo#103375]) -> [DMESG-WARN][32] ([IGT#4] / [i915#263]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html [IGT#4]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/4 [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#111550]: https://bugs.freedesktop.org/show_bug.cgi?id=111550 [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#1052]: https://gitlab.freedesktop.org/drm/intel/issues/1052 [i915#140]: https://gitlab.freedesktop.org/drm/intel/issues/140 [i915#217]: https://gitlab.freedesktop.org/drm/intel/issues/217 [i915#263]: https://gitlab.freedesktop.org/drm/intel/issues/263 [i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45 [i915#563]: https://gitlab.freedesktop.org/drm/intel/issues/563 [i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579 [i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 Participating hosts (50 -> 37) ------------------------------ Additional (1): fi-snb-2520m Missing (14): fi-ilk-m540 fi-bdw-samus fi-bdw-5557u fi-kbl-7560u fi-byt-squawks fi-bsw-cyan fi-ilk-650 fi-kbl-7500u fi-ctg-p8600 fi-elk-e7500 fi-blb-e6850 fi-byt-clapper fi-bsw-nick fi-skl-6600u Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7833 -> Patchwork_16308 CI-20190529: 20190529 CI_DRM_7833: 8210f0f999e2d396a8611e0cabc2f6c6a52468de @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5394: 991fd07bcd7add7a5beca2c95b72a994e62fbb75 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16308: 01bc69209e0127b5e6f52590bcff309fb98c5a81 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 01bc69209e01 drm/i915/gt: Trim default context register state for gen9 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gt: Trim default context register state for gen9 2020-01-29 2:56 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork @ 2020-01-29 8:04 ` Chris Wilson 0 siblings, 0 replies; 4+ messages in thread From: Chris Wilson @ 2020-01-29 8:04 UTC (permalink / raw) To: Patchwork, intel-gfx; +Cc: intel-gfx Quoting Patchwork (2020-01-29 02:56:00) > * igt@i915_selftest@live_execlists: > - fi-skl-lmem: [PASS][1] -> [DMESG-FAIL][2] > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-skl-lmem/igt@i915_selftest@live_execlists.html > [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-skl-lmem/igt@i915_selftest@live_execlists.html > - fi-cfl-8109u: [PASS][3] -> [DMESG-FAIL][4] > [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7833/fi-cfl-8109u/igt@i915_selftest@live_execlists.html > [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16308/fi-cfl-8109u/igt@i915_selftest@live_execlists.html Hey, selftests proving useful! -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-29 8:04 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-01-28 23:07 [Intel-gfx] [PATCH] drm/i915/gt: Trim default context register state for gen9 Chris Wilson 2020-01-29 2:34 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2020-01-29 2:56 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork 2020-01-29 8:04 ` Chris Wilson
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.