* [Intel-gfx] [PATCH] drm/i915/gt: Warn more clearly if the context state is still pinned
@ 2020-04-22 20:07 Chris Wilson
2020-04-22 21:09 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork
0 siblings, 1 reply; 2+ messages in thread
From: Chris Wilson @ 2020-04-22 20:07 UTC (permalink / raw)
To: intel-gfx; +Cc: Chris Wilson
When recording the default context state, we submit an ordinary context
and then steal the context image for our defaults. To be able to steal
the state, we must have total ownership of the context. During CI we
want to make this error extremely obvious, as otherwise we will fail the
user's module load.
References: https://gitlab.freedesktop.org/drm/intel/-/issues/1763
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/gt/intel_gt.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c b/drivers/gpu/drm/i915/gt/intel_gt.c
index 1c99cc72305a..c263f2e94703 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt.c
@@ -379,6 +379,11 @@ static int __intel_context_flush_retire(struct intel_context *ce)
return PTR_ERR(tl);
intel_context_timeline_unlock(tl);
+
+ /* Wait for the barrier */
+ if (i915_active_wait(&ce->active))
+ return -EINTR;
+
return 0;
}
@@ -472,6 +477,7 @@ static int __engines_record_defaults(struct intel_gt *gt)
/* We want to be able to unbind the state from the GGTT */
GEM_BUG_ON(intel_context_is_pinned(rq->context));
+ GEM_BUG_ON(i915_vma_is_pinned(state));
/*
* As we will hold a reference to the logical state, it will
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gt: Warn more clearly if the context state is still pinned
2020-04-22 20:07 [Intel-gfx] [PATCH] drm/i915/gt: Warn more clearly if the context state is still pinned Chris Wilson
@ 2020-04-22 21:09 ` Patchwork
0 siblings, 0 replies; 2+ messages in thread
From: Patchwork @ 2020-04-22 21:09 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/gt: Warn more clearly if the context state is still pinned
URL : https://patchwork.freedesktop.org/series/76351/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8350 -> Patchwork_17431
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_17431 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_17431, 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_17431/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_17431:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live@active:
- fi-snb-2520m: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-snb-2520m/igt@i915_selftest@live@active.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-snb-2520m/igt@i915_selftest@live@active.html
* igt@i915_selftest@live@late_gt_pm:
- fi-bwr-2160: [PASS][3] -> [INCOMPLETE][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-bwr-2160/igt@i915_selftest@live@late_gt_pm.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-bwr-2160/igt@i915_selftest@live@late_gt_pm.html
* igt@runner@aborted:
- fi-snb-2520m: NOTRUN -> [FAIL][5]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-snb-2520m/igt@runner@aborted.html
- fi-elk-e7500: NOTRUN -> [FAIL][6]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-elk-e7500/igt@runner@aborted.html
Known issues
------------
Here are the changes found in Patchwork_17431 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@gt_mocs:
- fi-snb-2600: [PASS][7] -> [INCOMPLETE][8] ([i915#82])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-snb-2600/igt@i915_selftest@live@gt_mocs.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-snb-2600/igt@i915_selftest@live@gt_mocs.html
* igt@i915_selftest@live@sanitycheck:
- fi-byt-j1900: [PASS][9] -> [INCOMPLETE][10] ([i915#45])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-byt-j1900/igt@i915_selftest@live@sanitycheck.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-byt-j1900/igt@i915_selftest@live@sanitycheck.html
* igt@i915_selftest@live@workarounds:
- fi-elk-e7500: [PASS][11] -> [INCOMPLETE][12] ([i915#66])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-elk-e7500/igt@i915_selftest@live@workarounds.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-elk-e7500/igt@i915_selftest@live@workarounds.html
#### Warnings ####
* igt@i915_pm_rpm@module-reload:
- fi-kbl-x1275: [FAIL][13] ([i915#62]) -> [SKIP][14] ([fdo#109271])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_8350/fi-kbl-x1275/igt@i915_pm_rpm@module-reload.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/fi-kbl-x1275/igt@i915_pm_rpm@module-reload.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#66]: https://gitlab.freedesktop.org/drm/intel/issues/66
[i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82
Participating hosts (48 -> 41)
------------------------------
Additional (1): fi-icl-dsi
Missing (8): fi-cml-u2 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-kbl-7500u fi-ivb-3770 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_8350 -> Patchwork_17431
CI-20190529: 20190529
CI_DRM_8350: 018bab6d1c4ac37bff9306384383fab59750e140 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5606: 678afb3954bec6227c8762756a0ad6d9946d49b2 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_17431: f3b4cb2337241ed72073d06d4bf92b89f99c5acd @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
f3b4cb233724 drm/i915/gt: Warn more clearly if the context state is still pinned
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_17431/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-22 21:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-22 20:07 [Intel-gfx] [PATCH] drm/i915/gt: Warn more clearly if the context state is still pinned Chris Wilson
2020-04-22 21:09 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork
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.