diff mbox series

[CI,1/3] drm/i915/guc: Enable guc logging on guc log relay write

Message ID 20191022163754.23870-1-robert.m.fosha@intel.com (mailing list archive)
State New, archived
Headers show
Series [CI,1/3] drm/i915/guc: Enable guc logging on guc log relay write | expand

Commit Message

Fosha, Robert M Oct. 22, 2019, 4:37 p.m. UTC
Creating and opening the GuC log relay file enables and starts
the relay potentially before the caller is ready to consume logs.
Change the behavior so that relay starts only on an explicit call
to the write function (with a value of '1'). Other values flush
the log relay as before.

v2: Style changes and fix typos. Add guc_log_relay_stop()
function. (Daniele)

Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Robert M. Fosha <robert.m.fosha@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
---
 drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 53 +++++++++++++++++-----
 drivers/gpu/drm/i915/gt/uc/intel_guc_log.h |  4 +-
 drivers/gpu/drm/i915/i915_debugfs.c        | 22 +++++++--
 3 files changed, 62 insertions(+), 17 deletions(-)

Comments

Daniele Ceraolo Spurio Oct. 23, 2019, 10:28 p.m. UTC | #1
On 10/23/19 8:14 AM, Patchwork wrote:
> == Series Details ==
> 
> Series: series starting with [CI,1/3] drm/i915/guc: Enable guc logging on guc log relay write
> URL   : https://patchwork.freedesktop.org/series/68406/
> State : success
> 
> == Summary ==
> 
> CI Bug Log - changes from CI_DRM_7155_full -> Patchwork_14932_full
> ====================================================
> 
> Summary
> -------
> 
>    **WARNING**
> 
>    Minor unknown changes coming with Patchwork_14932_full need to be verified
>    manually.
>    
>    If you think the reported changes have nothing to do with the changes
>    introduced in Patchwork_14932_full, please notify your bug team to allow them
>    to document this new failure mode, which will reduce false positives in CI.
> 
>    
> 
> Possible new issues
> -------------------
> 
>    Here are the unknown changes that may have been introduced in Patchwork_14932_full:
> 
> ### IGT changes ###
> 
> #### Warnings ####
> 
>    * igt@i915_pm_rc6_residency@media-rc6-accuracy:
>      - shard-iclb:         [SKIP][1] ([fdo#109289]) -> [SKIP][2]
>     [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb8/igt@i915_pm_rc6_residency@media-rc6-accuracy.html
>     [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb1/igt@i915_pm_rc6_residency@media-rc6-accuracy.html

This signature has shown up on other runs, so not caused by this series. 
Patches pushed.

Daniele

> 
>    
> #### Suppressed ####
> 
>    The following results come from untrusted machines, tests, or statuses.
>    They do not affect the overall result.
> 
>    * igt@gem_fence_thrash@bo-write-verify-threaded-y:
>      - {shard-tglb}:       [PASS][3] -> [INCOMPLETE][4]
>     [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html
>     [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html
> 
>    
> Known issues
> ------------
> 
>    Here are the changes found in Patchwork_14932_full that come from known issues:
> 
> ### IGT changes ###
> 
> #### Issues hit ####
> 
>    * igt@gem_ctx_isolation@vcs1-s3:
>      - shard-iclb:         [PASS][5] -> [SKIP][6] ([fdo#109276] / [fdo#112080])
>     [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb4/igt@gem_ctx_isolation@vcs1-s3.html
>     [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb3/igt@gem_ctx_isolation@vcs1-s3.html
> 
>    * igt@gem_eio@in-flight-contexts-immediate:
>      - shard-snb:          [PASS][7] -> [FAIL][8] ([fdo#111925])
>     [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-snb7/igt@gem_eio@in-flight-contexts-immediate.html
>     [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-snb2/igt@gem_eio@in-flight-contexts-immediate.html
> 
>    * igt@gem_exec_schedule@out-order-bsd2:
>      - shard-iclb:         [PASS][9] -> [SKIP][10] ([fdo#109276]) +8 similar issues
>     [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb4/igt@gem_exec_schedule@out-order-bsd2.html
>     [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb3/igt@gem_exec_schedule@out-order-bsd2.html
> 
>    * igt@gem_exec_schedule@preemptive-hang-bsd:
>      - shard-iclb:         [PASS][11] -> [SKIP][12] ([fdo#111325]) +3 similar issues
>     [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb8/igt@gem_exec_schedule@preemptive-hang-bsd.html
>     [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb1/igt@gem_exec_schedule@preemptive-hang-bsd.html
> 
>    * igt@gem_softpin@noreloc-s3:
>      - shard-skl:          [PASS][13] -> [INCOMPLETE][14] ([fdo#104108])
>     [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl10/igt@gem_softpin@noreloc-s3.html
>     [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl8/igt@gem_softpin@noreloc-s3.html
> 
>    * igt@gem_userptr_blits@map-fixed-invalidate-busy-gup:
>      - shard-snb:          [PASS][15] -> [DMESG-WARN][16] ([fdo#111870]) +2 similar issues
>     [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-snb4/igt@gem_userptr_blits@map-fixed-invalidate-busy-gup.html
>     [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-snb6/igt@gem_userptr_blits@map-fixed-invalidate-busy-gup.html
> 
>    * igt@gem_workarounds@suspend-resume-fd:
>      - shard-kbl:          [PASS][17] -> [INCOMPLETE][18] ([fdo#103665])
>     [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-kbl4/igt@gem_workarounds@suspend-resume-fd.html
>     [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-kbl4/igt@gem_workarounds@suspend-resume-fd.html
> 
>    * igt@i915_pm_rc6_residency@rc6-accuracy:
>      - shard-apl:          [PASS][19] -> [SKIP][20] ([fdo#109271])
>     [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-apl6/igt@i915_pm_rc6_residency@rc6-accuracy.html
>     [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-apl4/igt@i915_pm_rc6_residency@rc6-accuracy.html
>      - shard-skl:          [PASS][21] -> [SKIP][22] ([fdo#109271])
>     [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl9/igt@i915_pm_rc6_residency@rc6-accuracy.html
>     [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl8/igt@i915_pm_rc6_residency@rc6-accuracy.html
> 
>    * igt@i915_pm_rpm@system-suspend-execbuf:
>      - shard-skl:          [PASS][23] -> [INCOMPLETE][24] ([fdo#104108] / [fdo#107807])
>     [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl4/igt@i915_pm_rpm@system-suspend-execbuf.html
>     [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl1/igt@i915_pm_rpm@system-suspend-execbuf.html
> 
>    * igt@kms_flip@flip-vs-expired-vblank-interruptible:
>      - shard-skl:          [PASS][25] -> [FAIL][26] ([fdo#105363])
>     [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
>     [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
> 
>    * igt@kms_flip@modeset-vs-vblank-race-interruptible:
>      - shard-glk:          [PASS][27] -> [FAIL][28] ([fdo#111609])
>     [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-glk7/igt@kms_flip@modeset-vs-vblank-race-interruptible.html
>     [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-glk4/igt@kms_flip@modeset-vs-vblank-race-interruptible.html
> 
>    * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt:
>      - shard-iclb:         [PASS][29] -> [FAIL][30] ([fdo#103167]) +4 similar issues
>     [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt.html
>     [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb8/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt.html
> 
>    * igt@kms_frontbuffer_tracking@fbc-suspend:
>      - shard-kbl:          [PASS][31] -> [DMESG-WARN][32] ([fdo#108566]) +10 similar issues
>     [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-kbl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
>     [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-kbl2/igt@kms_frontbuffer_tracking@fbc-suspend.html
>      - shard-apl:          [PASS][33] -> [DMESG-WARN][34] ([fdo#108566]) +8 similar issues
>     [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-apl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
>     [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-apl7/igt@kms_frontbuffer_tracking@fbc-suspend.html
> 
>    * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min:
>      - shard-skl:          [PASS][35] -> [FAIL][36] ([fdo#108145])
>     [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
>     [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html
> 
>    * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
>      - shard-skl:          [PASS][37] -> [FAIL][38] ([fdo#108145] / [fdo#110403])
>     [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
>     [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl5/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
> 
>    * igt@kms_plane_multiple@atomic-pipe-a-tiling-yf:
>      - shard-skl:          [PASS][39] -> [DMESG-WARN][40] ([fdo#106885])
>     [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl3/igt@kms_plane_multiple@atomic-pipe-a-tiling-yf.html
>     [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl6/igt@kms_plane_multiple@atomic-pipe-a-tiling-yf.html
> 
>    * igt@perf@gen8-unprivileged-single-ctx-counters:
>      - shard-skl:          [PASS][41] -> [INCOMPLETE][42] ([fdo#111747])
>     [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl1/igt@perf@gen8-unprivileged-single-ctx-counters.html
>     [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl1/igt@perf@gen8-unprivileged-single-ctx-counters.html
> 
>    * igt@perf_pmu@busy-no-semaphores-vcs1:
>      - shard-iclb:         [PASS][43] -> [SKIP][44] ([fdo#112080]) +4 similar issues
>     [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb4/igt@perf_pmu@busy-no-semaphores-vcs1.html
>     [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb3/igt@perf_pmu@busy-no-semaphores-vcs1.html
> 
>    
> #### Possible fixes ####
> 
>    * igt@gem_ctx_isolation@vecs0-s3:
>      - shard-kbl:          [INCOMPLETE][45] ([fdo#103665]) -> [PASS][46]
>     [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-kbl4/igt@gem_ctx_isolation@vecs0-s3.html
>     [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-kbl1/igt@gem_ctx_isolation@vecs0-s3.html
> 
>    * igt@gem_exec_schedule@preempt-contexts-bsd2:
>      - shard-iclb:         [SKIP][47] ([fdo#109276]) -> [PASS][48] +8 similar issues
>     [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb7/igt@gem_exec_schedule@preempt-contexts-bsd2.html
>     [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb4/igt@gem_exec_schedule@preempt-contexts-bsd2.html
> 
>    * igt@gem_exec_schedule@smoketest-bsd:
>      - shard-iclb:         [SKIP][49] ([fdo#111325]) -> [PASS][50]
>     [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb2/igt@gem_exec_schedule@smoketest-bsd.html
>     [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb3/igt@gem_exec_schedule@smoketest-bsd.html
> 
>    * igt@gem_userptr_blits@map-fixed-invalidate-busy:
>      - shard-snb:          [DMESG-WARN][51] ([fdo#111870]) -> [PASS][52]
>     [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-snb5/igt@gem_userptr_blits@map-fixed-invalidate-busy.html
>     [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-snb7/igt@gem_userptr_blits@map-fixed-invalidate-busy.html
> 
>    * igt@gem_userptr_blits@sync-unmap-after-close:
>      - shard-hsw:          [DMESG-WARN][53] ([fdo#111870]) -> [PASS][54]
>     [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-hsw6/igt@gem_userptr_blits@sync-unmap-after-close.html
>     [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-hsw2/igt@gem_userptr_blits@sync-unmap-after-close.html
> 
>    * igt@gem_userptr_blits@unsync-unmap-cycles:
>      - shard-apl:          [INCOMPLETE][55] ([fdo#103927]) -> [PASS][56] +1 similar issue
>     [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-apl3/igt@gem_userptr_blits@unsync-unmap-cycles.html
>     [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-apl1/igt@gem_userptr_blits@unsync-unmap-cycles.html
> 
>    * igt@gem_workarounds@suspend-resume-context:
>      - shard-apl:          [DMESG-WARN][57] ([fdo#108566]) -> [PASS][58] +4 similar issues
>     [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-apl7/igt@gem_workarounds@suspend-resume-context.html
>     [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-apl2/igt@gem_workarounds@suspend-resume-context.html
> 
>    * igt@i915_pm_rpm@system-suspend:
>      - {shard-tglb}:       [INCOMPLETE][59] ([fdo#111747] / [fdo#111850]) -> [PASS][60]
>     [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb8/igt@i915_pm_rpm@system-suspend.html
>     [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb1/igt@i915_pm_rpm@system-suspend.html
> 
>    * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
>      - shard-snb:          [SKIP][61] ([fdo#109271]) -> [PASS][62] +3 similar issues
>     [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-snb6/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
>     [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-snb2/igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b.html
> 
>    * igt@kms_cursor_legacy@cursor-vs-flip-atomic:
>      - shard-iclb:         [INCOMPLETE][63] ([fdo#107713]) -> [PASS][64]
>     [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb1/igt@kms_cursor_legacy@cursor-vs-flip-atomic.html
>     [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb5/igt@kms_cursor_legacy@cursor-vs-flip-atomic.html
> 
>    * igt@kms_flip@dpms-vs-vblank-race:
>      - shard-glk:          [FAIL][65] ([fdo#111609]) -> [PASS][66]
>     [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-glk2/igt@kms_flip@dpms-vs-vblank-race.html
>     [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-glk5/igt@kms_flip@dpms-vs-vblank-race.html
> 
>    * igt@kms_flip@flip-vs-expired-vblank-interruptible:
>      - shard-glk:          [FAIL][67] ([fdo#105363]) -> [PASS][68] +1 similar issue
>     [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-glk1/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
>     [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-glk7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
> 
>    * igt@kms_flip_tiling@flip-changes-tiling-y:
>      - shard-skl:          [FAIL][69] ([fdo#107931] / [fdo#108303]) -> [PASS][70]
>     [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl3/igt@kms_flip_tiling@flip-changes-tiling-y.html
>     [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl6/igt@kms_flip_tiling@flip-changes-tiling-y.html
> 
>    * igt@kms_frontbuffer_tracking@fbc-tilingchange:
>      - shard-iclb:         [FAIL][71] ([fdo#103167]) -> [PASS][72] +2 similar issues
>     [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-tilingchange.html
>     [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-tilingchange.html
> 
>    * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-pgflip-blt:
>      - {shard-tglb}:       [FAIL][73] ([fdo#103167]) -> [PASS][74]
>     [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-pgflip-blt.html
>     [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-pgflip-blt.html
> 
>    * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
>      - {shard-tglb}:       [INCOMPLETE][75] ([fdo#111832] / [fdo#111850]) -> [PASS][76] +2 similar issues
>     [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb8/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
>     [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
> 
>    * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-d-planes:
>      - {shard-tglb}:       [INCOMPLETE][77] ([fdo#111850]) -> [PASS][78]
>     [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-d-planes.html
>     [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-d-planes.html
> 
>    * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
>      - shard-skl:          [FAIL][79] ([fdo#108145]) -> [PASS][80]
>     [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-skl3/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
>     [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-skl6/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
> 
>    * igt@kms_psr2_su@frontbuffer:
>      - shard-iclb:         [SKIP][81] ([fdo#109642] / [fdo#111068]) -> [PASS][82]
>     [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb3/igt@kms_psr2_su@frontbuffer.html
>     [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb2/igt@kms_psr2_su@frontbuffer.html
> 
>    * igt@kms_setmode@basic:
>      - shard-kbl:          [FAIL][83] ([fdo#99912]) -> [PASS][84]
>     [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-kbl2/igt@kms_setmode@basic.html
>     [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-kbl4/igt@kms_setmode@basic.html
> 
>    * igt@perf@short-reads:
>      - shard-glk:          [TIMEOUT][85] ([fdo#103183]) -> [PASS][86]
>     [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-glk9/igt@perf@short-reads.html
>     [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-glk3/igt@perf@short-reads.html
> 
>    * igt@perf_pmu@busy-accuracy-98-vcs1:
>      - shard-iclb:         [SKIP][87] ([fdo#112080]) -> [PASS][88] +7 similar issues
>     [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb7/igt@perf_pmu@busy-accuracy-98-vcs1.html
>     [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb4/igt@perf_pmu@busy-accuracy-98-vcs1.html
> 
>    * igt@perf_pmu@busy-start-vecs0:
>      - {shard-tglb}:       [INCOMPLETE][89] ([fdo#111747]) -> [PASS][90]
>     [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-tglb6/igt@perf_pmu@busy-start-vecs0.html
>     [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-tglb2/igt@perf_pmu@busy-start-vecs0.html
> 
>    
> #### Warnings ####
> 
>    * igt@gem_ctx_isolation@vcs1-nonpriv:
>      - shard-iclb:         [FAIL][91] ([fdo#111329]) -> [SKIP][92] ([fdo#109276] / [fdo#112080])
>     [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb1/igt@gem_ctx_isolation@vcs1-nonpriv.html
>     [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb5/igt@gem_ctx_isolation@vcs1-nonpriv.html
> 
>    * igt@gem_mocs_settings@mocs-settings-bsd2:
>      - shard-iclb:         [FAIL][93] ([fdo#111330]) -> [SKIP][94] ([fdo#109276])
>     [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7155/shard-iclb2/igt@gem_mocs_settings@mocs-settings-bsd2.html
>     [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/shard-iclb3/igt@gem_mocs_settings@mocs-settings-bsd2.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#103183]: https://bugs.freedesktop.org/show_bug.cgi?id=103183
>    [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
>    [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#106885]: https://bugs.freedesktop.org/show_bug.cgi?id=106885
>    [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
>    [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
>    [fdo#107931]: https://bugs.freedesktop.org/show_bug.cgi?id=107931
>    [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
>    [fdo#108303]: https://bugs.freedesktop.org/show_bug.cgi?id=108303
>    [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
>    [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
>    [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
>    [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
>    [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
>    [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403
>    [fdo#110548]: https://bugs.freedesktop.org/show_bug.cgi?id=110548
>    [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
>    [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325
>    [fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329
>    [fdo#
> 
> == Logs ==
> 
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14932/index.html
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
index 2cf2d3314f62..caed0d57e704 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c
@@ -226,7 +226,7 @@  static void guc_read_update_log_buffer(struct intel_guc_log *log)
 
 	mutex_lock(&log->relay.lock);
 
-	if (WARN_ON(!intel_guc_log_relay_enabled(log)))
+	if (WARN_ON(!intel_guc_log_relay_created(log)))
 		goto out_unlock;
 
 	/* Get the pointer to shared GuC log buffer */
@@ -361,6 +361,7 @@  void intel_guc_log_init_early(struct intel_guc_log *log)
 {
 	mutex_init(&log->relay.lock);
 	INIT_WORK(&log->relay.flush_work, capture_logs_work);
+	log->relay.started = false;
 }
 
 static int guc_log_relay_create(struct intel_guc_log *log)
@@ -546,7 +547,7 @@  int intel_guc_log_set_level(struct intel_guc_log *log, u32 level)
 	return ret;
 }
 
-bool intel_guc_log_relay_enabled(const struct intel_guc_log *log)
+bool intel_guc_log_relay_created(const struct intel_guc_log *log)
 {
 	return log->relay.buf_addr;
 }
@@ -560,7 +561,7 @@  int intel_guc_log_relay_open(struct intel_guc_log *log)
 
 	mutex_lock(&log->relay.lock);
 
-	if (intel_guc_log_relay_enabled(log)) {
+	if (intel_guc_log_relay_created(log)) {
 		ret = -EEXIST;
 		goto out_unlock;
 	}
@@ -585,6 +586,21 @@  int intel_guc_log_relay_open(struct intel_guc_log *log)
 
 	mutex_unlock(&log->relay.lock);
 
+	return 0;
+
+out_relay:
+	guc_log_relay_destroy(log);
+out_unlock:
+	mutex_unlock(&log->relay.lock);
+
+	return ret;
+}
+
+int intel_guc_log_relay_start(struct intel_guc_log *log)
+{
+	if (log->relay.started)
+		return -EEXIST;
+
 	guc_log_enable_flush_events(log);
 
 	/*
@@ -594,14 +610,9 @@  int intel_guc_log_relay_open(struct intel_guc_log *log)
 	 */
 	queue_work(system_highpri_wq, &log->relay.flush_work);
 
-	return 0;
-
-out_relay:
-	guc_log_relay_destroy(log);
-out_unlock:
-	mutex_unlock(&log->relay.lock);
+	log->relay.started = true;
 
-	return ret;
+	return 0;
 }
 
 void intel_guc_log_relay_flush(struct intel_guc_log *log)
@@ -609,6 +620,9 @@  void intel_guc_log_relay_flush(struct intel_guc_log *log)
 	struct intel_guc *guc = log_to_guc(log);
 	intel_wakeref_t wakeref;
 
+	if (!log->relay.started)
+		return;
+
 	/*
 	 * Before initiating the forceful flush, wait for any pending/ongoing
 	 * flush to complete otherwise forceful flush may not actually happen.
@@ -622,18 +636,33 @@  void intel_guc_log_relay_flush(struct intel_guc_log *log)
 	guc_log_capture_logs(log);
 }
 
-void intel_guc_log_relay_close(struct intel_guc_log *log)
+/*
+ * Stops the relay log. Called from intel_guc_log_relay_close(), so no
+ * possibility of race with start/flush since relay_write cannot race
+ * relay_close.
+ */
+static void guc_log_relay_stop(struct intel_guc_log *log)
 {
 	struct intel_guc *guc = log_to_guc(log);
 	struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
 
+	if (!log->relay.started)
+		return;
+
 	guc_log_disable_flush_events(log);
 	intel_synchronize_irq(i915);
 
 	flush_work(&log->relay.flush_work);
 
+	log->relay.started = false;
+}
+
+void intel_guc_log_relay_close(struct intel_guc_log *log)
+{
+	guc_log_relay_stop(log);
+
 	mutex_lock(&log->relay.lock);
-	GEM_BUG_ON(!intel_guc_log_relay_enabled(log));
+	GEM_BUG_ON(!intel_guc_log_relay_created(log));
 	guc_log_unmap(log);
 	guc_log_relay_destroy(log);
 	mutex_unlock(&log->relay.lock);
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
index 6f764879acb1..c252c022c5fc 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_log.h
@@ -47,6 +47,7 @@  struct intel_guc_log {
 	struct i915_vma *vma;
 	struct {
 		void *buf_addr;
+		bool started;
 		struct work_struct flush_work;
 		struct rchan *channel;
 		struct mutex lock;
@@ -65,8 +66,9 @@  int intel_guc_log_create(struct intel_guc_log *log);
 void intel_guc_log_destroy(struct intel_guc_log *log);
 
 int intel_guc_log_set_level(struct intel_guc_log *log, u32 level);
-bool intel_guc_log_relay_enabled(const struct intel_guc_log *log);
+bool intel_guc_log_relay_created(const struct intel_guc_log *log);
 int intel_guc_log_relay_open(struct intel_guc_log *log);
+int intel_guc_log_relay_start(struct intel_guc_log *log);
 void intel_guc_log_relay_flush(struct intel_guc_log *log);
 void intel_guc_log_relay_close(struct intel_guc_log *log);
 
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index ada57eee914a..ad536d82e60f 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -1866,8 +1866,8 @@  static void i915_guc_log_info(struct seq_file *m,
 	struct intel_guc_log *log = &dev_priv->gt.uc.guc.log;
 	enum guc_log_buffer_type type;
 
-	if (!intel_guc_log_relay_enabled(log)) {
-		seq_puts(m, "GuC log relay disabled\n");
+	if (!intel_guc_log_relay_created(log)) {
+		seq_puts(m, "GuC log relay not created\n");
 		return;
 	}
 
@@ -2054,9 +2054,23 @@  i915_guc_log_relay_write(struct file *filp,
 			 loff_t *ppos)
 {
 	struct intel_guc_log *log = filp->private_data;
+	int val;
+	int ret;
 
-	intel_guc_log_relay_flush(log);
-	return cnt;
+	ret = kstrtoint_from_user(ubuf, cnt, 0, &val);
+	if (ret < 0)
+		return ret;
+
+	/*
+	 * Enable and start the guc log relay on value of 1.
+	 * Flush log relay for any other value.
+	 */
+	if (val == 1)
+		ret = intel_guc_log_relay_start(log);
+	else
+		intel_guc_log_relay_flush(log);
+
+	return ret ?: cnt;
 }
 
 static int i915_guc_log_relay_release(struct inode *inode, struct file *file)