* [Intel-gfx] [PATCH] drm/i915: individualize fences before adding @ 2022-05-13 11:37 Nirmoy Das 2022-05-13 14:54 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork 2022-05-16 9:38 ` [Intel-gfx] [PATCH] " Hellstrom, Thomas 0 siblings, 2 replies; 4+ messages in thread From: Nirmoy Das @ 2022-05-13 11:37 UTC (permalink / raw) To: intel-gfx; +Cc: thomas.hellstrom, matthew.auld _i915_vma_move_to_active() can receive > 1 fence for multiple batch buffer submission so make sure to individualize fences before adding to dma_resv obj Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5614 Signed-off-by: Nirmoy Das <nirmoy.das@intel.com> --- drivers/gpu/drm/i915/i915_vma.c | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_vma.c b/drivers/gpu/drm/i915/i915_vma.c index 4f6db539571a..f987fc1264c0 100644 --- a/drivers/gpu/drm/i915/i915_vma.c +++ b/drivers/gpu/drm/i915/i915_vma.c @@ -23,6 +23,7 @@ */ #include <linux/sched/mm.h> +#include <linux/dma-fence-array.h> #include <drm/drm_gem.h> #include "display/intel_frontbuffer.h" @@ -1840,8 +1841,12 @@ int _i915_vma_move_to_active(struct i915_vma *vma, } if (fence) { - dma_resv_add_fence(vma->obj->base.resv, fence, - DMA_RESV_USAGE_WRITE); + int idx; + struct dma_fence *curr; + + dma_fence_array_for_each(curr, idx, fence) + dma_resv_add_fence(vma->obj->base.resv, curr, + DMA_RESV_USAGE_WRITE); obj->write_domain = I915_GEM_DOMAIN_RENDER; obj->read_domains = 0; } @@ -1853,8 +1858,12 @@ int _i915_vma_move_to_active(struct i915_vma *vma, } if (fence) { - dma_resv_add_fence(vma->obj->base.resv, fence, - DMA_RESV_USAGE_READ); + int idx; + struct dma_fence *curr; + + dma_fence_array_for_each(curr, idx, fence) + dma_resv_add_fence(vma->obj->base.resv, curr, + DMA_RESV_USAGE_READ); obj->write_domain = 0; } } -- 2.35.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: individualize fences before adding 2022-05-13 11:37 [Intel-gfx] [PATCH] drm/i915: individualize fences before adding Nirmoy Das @ 2022-05-13 14:54 ` Patchwork 2022-05-16 10:57 ` Das, Nirmoy 2022-05-16 9:38 ` [Intel-gfx] [PATCH] " Hellstrom, Thomas 1 sibling, 1 reply; 4+ messages in thread From: Patchwork @ 2022-05-13 14:54 UTC (permalink / raw) To: Nirmoy Das; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 13187 bytes --] == Series Details == Series: drm/i915: individualize fences before adding URL : https://patchwork.freedesktop.org/series/103981/ State : failure == Summary == CI Bug Log - changes from CI_DRM_11649 -> Patchwork_103981v1 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_103981v1 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_103981v1, 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_103981v1/index.html Participating hosts (39 -> 40) ------------------------------ Additional (3): fi-blb-e6850 fi-icl-u2 bat-dg1-5 Missing (2): bat-adlm-1 bat-jsl-2 Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_103981v1: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live@gem: - fi-blb-e6850: NOTRUN -> [INCOMPLETE][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@i915_selftest@live@gem.html Known issues ------------ Here are the changes found in Patchwork_103981v1 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@fbdev@write: - bat-dg1-5: NOTRUN -> [SKIP][2] ([i915#2582]) +4 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@fbdev@write.html * igt@gem_huc_copy@huc-copy: - fi-icl-u2: NOTRUN -> [SKIP][3] ([i915#2190]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@parallel-random-engines: - fi-icl-u2: NOTRUN -> [SKIP][4] ([i915#4613]) +3 similar issues [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@gem_lmem_swapping@parallel-random-engines.html * igt@gem_mmap@basic: - bat-dg1-5: NOTRUN -> [SKIP][5] ([i915#4083]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_mmap@basic.html * igt@gem_tiled_fence_blits@basic: - bat-dg1-5: NOTRUN -> [SKIP][6] ([i915#4077]) +2 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_tiled_fence_blits@basic.html * igt@gem_tiled_pread_basic: - bat-dg1-5: NOTRUN -> [SKIP][7] ([i915#4079]) +1 similar issue [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_tiled_pread_basic.html * igt@i915_pm_backlight@basic-brightness: - bat-dg1-5: NOTRUN -> [SKIP][8] ([i915#1155]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@i915_pm_backlight@basic-brightness.html * igt@i915_selftest@live@gt_engines: - bat-dg1-5: NOTRUN -> [INCOMPLETE][9] ([i915#4418]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@i915_selftest@live@gt_engines.html * igt@i915_selftest@live@gtt: - fi-bdw-5557u: [PASS][10] -> [INCOMPLETE][11] ([i915#5685]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-bdw-5557u/igt@i915_selftest@live@gtt.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-bdw-5557u/igt@i915_selftest@live@gtt.html * igt@kms_addfb_basic@basic-y-tiled-legacy: - bat-dg1-5: NOTRUN -> [SKIP][12] ([i915#4215]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_addfb_basic@basic-y-tiled-legacy.html * igt@kms_addfb_basic@tile-pitch-mismatch: - bat-dg1-5: NOTRUN -> [SKIP][13] ([i915#4212]) +7 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_addfb_basic@tile-pitch-mismatch.html * igt@kms_busy@basic: - bat-dg1-5: NOTRUN -> [SKIP][14] ([i915#4303]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_busy@basic.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-hsw-4770: NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-hsw-4770/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: NOTRUN -> [SKIP][16] ([fdo#111827]) +8 similar issues [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html - bat-dg1-5: NOTRUN -> [SKIP][17] ([fdo#111827]) +7 similar issues [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy: - fi-icl-u2: NOTRUN -> [SKIP][18] ([fdo#109278]) +2 similar issues [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html - bat-dg1-5: NOTRUN -> [SKIP][19] ([i915#4103] / [i915#4213]) +1 similar issue [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html * igt@kms_flip@basic-flip-vs-dpms: - bat-dg1-5: NOTRUN -> [SKIP][20] ([i915#4078]) +23 similar issues [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_flip@basic-flip-vs-dpms.html * igt@kms_force_connector_basic@force-load-detect: - bat-dg1-5: NOTRUN -> [SKIP][21] ([fdo#109285]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_force_connector_basic@force-load-detect.html - fi-icl-u2: NOTRUN -> [SKIP][22] ([fdo#109285]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d: - fi-blb-e6850: NOTRUN -> [SKIP][23] ([fdo#109271]) +39 similar issues [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html * igt@kms_psr@primary_page_flip: - bat-dg1-5: NOTRUN -> [SKIP][24] ([i915#1072] / [i915#4078]) +3 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_psr@primary_page_flip.html * igt@kms_setmode@basic-clone-single-crtc: - fi-icl-u2: NOTRUN -> [SKIP][25] ([i915#3555]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_setmode@basic-clone-single-crtc.html - bat-dg1-5: NOTRUN -> [SKIP][26] ([i915#3555]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html * igt@prime_vgem@basic-fence-mmap: - bat-dg1-5: NOTRUN -> [SKIP][27] ([i915#3708] / [i915#4077]) +1 similar issue [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-fence-mmap.html * igt@prime_vgem@basic-userptr: - fi-icl-u2: NOTRUN -> [SKIP][28] ([i915#3301]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@prime_vgem@basic-userptr.html - bat-dg1-5: NOTRUN -> [SKIP][29] ([i915#3708] / [i915#4873]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-userptr.html * igt@prime_vgem@basic-write: - bat-dg1-5: NOTRUN -> [SKIP][30] ([i915#3708]) +3 similar issues [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-write.html * igt@runner@aborted: - bat-dg1-5: NOTRUN -> [FAIL][31] ([i915#4312] / [i915#5257]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@runner@aborted.html - fi-blb-e6850: NOTRUN -> [FAIL][32] ([fdo#109271] / [i915#2403] / [i915#4312]) [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@runner@aborted.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s0@smem: - {fi-ehl-2}: [DMESG-WARN][33] ([i915#5122]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html * igt@i915_selftest@live@gt_heartbeat: - {fi-jsl-1}: [DMESG-FAIL][35] ([i915#5334]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-jsl-1/igt@i915_selftest@live@gt_heartbeat.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-jsl-1/igt@i915_selftest@live@gt_heartbeat.html * igt@i915_selftest@live@hangcheck: - fi-hsw-4770: [INCOMPLETE][37] ([i915#4785]) -> [PASS][38] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html * igt@i915_suspend@system-suspend-without-i915: - {bat-dg2-9}: [DMESG-WARN][39] ([i915#5763]) -> [PASS][40] [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/bat-dg2-9/igt@i915_suspend@system-suspend-without-i915.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg2-9/igt@i915_suspend@system-suspend-without-i915.html * igt@kms_flip@basic-flip-vs-modeset@a-edp1: - {bat-adlp-6}: [DMESG-WARN][41] ([i915#3576]) -> [PASS][42] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/bat-adlp-6/igt@kms_flip@basic-flip-vs-modeset@a-edp1.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-adlp-6/igt@kms_flip@basic-flip-vs-modeset@a-edp1.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2403]: https://gitlab.freedesktop.org/drm/intel/issues/2403 [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582 [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079 [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212 [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213 [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215 [i915#4303]: https://gitlab.freedesktop.org/drm/intel/issues/4303 [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312 [i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391 [i915#4418]: https://gitlab.freedesktop.org/drm/intel/issues/4418 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785 [i915#4873]: https://gitlab.freedesktop.org/drm/intel/issues/4873 [i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122 [i915#5257]: https://gitlab.freedesktop.org/drm/intel/issues/5257 [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334 [i915#5685]: https://gitlab.freedesktop.org/drm/intel/issues/5685 [i915#5763]: https://gitlab.freedesktop.org/drm/intel/issues/5763 [i915#5879]: https://gitlab.freedesktop.org/drm/intel/issues/5879 [i915#5950]: https://gitlab.freedesktop.org/drm/intel/issues/5950 Build changes ------------- * Linux: CI_DRM_11649 -> Patchwork_103981v1 CI-20190529: 20190529 CI_DRM_11649: 41053495dd77aa5604144477ef59334a9492a169 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6472: c815c94f0ceb33ae852622538f0136cf44c5725d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_103981v1: 41053495dd77aa5604144477ef59334a9492a169 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 4a604aeca9d8 drm/i915: individualize fences before adding == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/index.html [-- Attachment #2: Type: text/html, Size: 15219 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: individualize fences before adding 2022-05-13 14:54 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork @ 2022-05-16 10:57 ` Das, Nirmoy 0 siblings, 0 replies; 4+ messages in thread From: Das, Nirmoy @ 2022-05-16 10:57 UTC (permalink / raw) To: intel-gfx, Patchwork, Nirmoy Das, Hellstrom Thomas [-- Attachment #1: Type: text/plain, Size: 14287 bytes --] On 5/13/2022 4:54 PM, Patchwork wrote: > Project List - Patchwork *Patch Details* > *Series:* drm/i915: individualize fences before adding > *URL:* https://patchwork.freedesktop.org/series/103981/ > *State:* failure > *Details:* > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/index.html > > > CI Bug Log - changes from CI_DRM_11649 -> Patchwork_103981v1 > > > Summary > > *FAILURE* > > Serious unknown changes coming with Patchwork_103981v1 absolutely need > to be > verified manually. > > If you think the reported changes have nothing to do with the changes > introduced in Patchwork_103981v1, 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_103981v1/index.html > > > Participating hosts (39 -> 40) > > Additional (3): fi-blb-e6850 fi-icl-u2 bat-dg1-5 > Missing (2): bat-adlm-1 bat-jsl-2 > > > Possible new issues > > Here are the unknown changes that may have been introduced in > Patchwork_103981v1: > > > IGT changes > > > Possible regressions > > * igt@i915_selftest@live@gem: > o fi-blb-e6850: NOTRUN -> INCOMPLETE > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@i915_selftest@live@gem.html> > Unrelated and this test on blb is failing for a while https://intel-gfx-ci.01.org/tree/drm-tip/fi-blb-e6850.html Nirmoy > * > > > Known issues > > Here are the changes found in Patchwork_103981v1 that come from known > issues: > > > IGT changes > > > Issues hit > > * > > igt@fbdev@write: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@fbdev@write.html> > (i915#2582 > <https://gitlab.freedesktop.org/drm/intel/issues/2582>) +4 > similar issues > * > > igt@gem_huc_copy@huc-copy: > > o fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@gem_huc_copy@huc-copy.html> > (i915#2190 <https://gitlab.freedesktop.org/drm/intel/issues/2190>) > * > > igt@gem_lmem_swapping@parallel-random-engines: > > o fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@gem_lmem_swapping@parallel-random-engines.html> > (i915#4613 > <https://gitlab.freedesktop.org/drm/intel/issues/4613>) +3 > similar issues > * > > igt@gem_mmap@basic: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_mmap@basic.html> > (i915#4083 <https://gitlab.freedesktop.org/drm/intel/issues/4083>) > * > > igt@gem_tiled_fence_blits@basic: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_tiled_fence_blits@basic.html> > (i915#4077 > <https://gitlab.freedesktop.org/drm/intel/issues/4077>) +2 > similar issues > * > > igt@gem_tiled_pread_basic: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@gem_tiled_pread_basic.html> > (i915#4079 > <https://gitlab.freedesktop.org/drm/intel/issues/4079>) +1 > similar issue > * > > igt@i915_pm_backlight@basic-brightness: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@i915_pm_backlight@basic-brightness.html> > (i915#1155 <https://gitlab.freedesktop.org/drm/intel/issues/1155>) > * > > igt@i915_selftest@live@gt_engines: > > o bat-dg1-5: NOTRUN -> INCOMPLETE > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@i915_selftest@live@gt_engines.html> > (i915#4418 <https://gitlab.freedesktop.org/drm/intel/issues/4418>) > * > > igt@i915_selftest@live@gtt: > > o fi-bdw-5557u: PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-bdw-5557u/igt@i915_selftest@live@gtt.html> > -> INCOMPLETE > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-bdw-5557u/igt@i915_selftest@live@gtt.html> > (i915#5685 <https://gitlab.freedesktop.org/drm/intel/issues/5685>) > * > > igt@kms_addfb_basic@basic-y-tiled-legacy: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_addfb_basic@basic-y-tiled-legacy.html> > (i915#4215 <https://gitlab.freedesktop.org/drm/intel/issues/4215>) > * > > igt@kms_addfb_basic@tile-pitch-mismatch: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_addfb_basic@tile-pitch-mismatch.html> > (i915#4212 > <https://gitlab.freedesktop.org/drm/intel/issues/4212>) +7 > similar issues > * > > igt@kms_busy@basic: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_busy@basic.html> > (i915#4303 <https://gitlab.freedesktop.org/drm/intel/issues/4303>) > * > > igt@kms_chamelium@common-hpd-after-suspend: > > o fi-hsw-4770: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-hsw-4770/igt@kms_chamelium@common-hpd-after-suspend.html> > (fdo#109271 > <https://bugs.freedesktop.org/show_bug.cgi?id=109271> / > fdo#111827 <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) > * > > igt@kms_chamelium@hdmi-hpd-fast: > > o > > fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html> > (fdo#111827 > <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +8 > similar issues > > o > > bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_chamelium@hdmi-hpd-fast.html> > (fdo#111827 > <https://bugs.freedesktop.org/show_bug.cgi?id=111827>) +7 > similar issues > > * > > igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy: > > o > > fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html> > (fdo#109278 > <https://bugs.freedesktop.org/show_bug.cgi?id=109278>) +2 > similar issues > > o > > bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html> > (i915#4103 > <https://gitlab.freedesktop.org/drm/intel/issues/4103> / > i915#4213 > <https://gitlab.freedesktop.org/drm/intel/issues/4213>) +1 > similar issue > > * > > igt@kms_flip@basic-flip-vs-dpms: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_flip@basic-flip-vs-dpms.html> > (i915#4078 > <https://gitlab.freedesktop.org/drm/intel/issues/4078>) +23 > similar issues > * > > igt@kms_force_connector_basic@force-load-detect: > > o > > bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_force_connector_basic@force-load-detect.html> > (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>) > > o > > fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html> > (fdo#109285 <https://bugs.freedesktop.org/show_bug.cgi?id=109285>) > > * > > igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d: > > o fi-blb-e6850: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html> > (fdo#109271 > <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) +39 > similar issues > * > > igt@kms_psr@primary_page_flip: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_psr@primary_page_flip.html> > (i915#1072 > <https://gitlab.freedesktop.org/drm/intel/issues/1072> / > i915#4078 > <https://gitlab.freedesktop.org/drm/intel/issues/4078>) +3 > similar issues > * > > igt@kms_setmode@basic-clone-single-crtc: > > o > > fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@kms_setmode@basic-clone-single-crtc.html> > (i915#3555 <https://gitlab.freedesktop.org/drm/intel/issues/3555>) > > o > > bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html> > (i915#3555 <https://gitlab.freedesktop.org/drm/intel/issues/3555>) > > * > > igt@prime_vgem@basic-fence-mmap: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-fence-mmap.html> > (i915#3708 > <https://gitlab.freedesktop.org/drm/intel/issues/3708> / > i915#4077 > <https://gitlab.freedesktop.org/drm/intel/issues/4077>) +1 > similar issue > * > > igt@prime_vgem@basic-userptr: > > o > > fi-icl-u2: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-icl-u2/igt@prime_vgem@basic-userptr.html> > (i915#3301 <https://gitlab.freedesktop.org/drm/intel/issues/3301>) > > o > > bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-userptr.html> > (i915#3708 > <https://gitlab.freedesktop.org/drm/intel/issues/3708> / > i915#4873 <https://gitlab.freedesktop.org/drm/intel/issues/4873>) > > * > > igt@prime_vgem@basic-write: > > o bat-dg1-5: NOTRUN -> SKIP > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@prime_vgem@basic-write.html> > (i915#3708 > <https://gitlab.freedesktop.org/drm/intel/issues/3708>) +3 > similar issues > * > > igt@runner@aborted: > > o > > bat-dg1-5: NOTRUN -> FAIL > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg1-5/igt@runner@aborted.html> > (i915#4312 > <https://gitlab.freedesktop.org/drm/intel/issues/4312> / > i915#5257 <https://gitlab.freedesktop.org/drm/intel/issues/5257>) > > o > > fi-blb-e6850: NOTRUN -> FAIL > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-blb-e6850/igt@runner@aborted.html> > (fdo#109271 > <https://bugs.freedesktop.org/show_bug.cgi?id=109271> / > i915#2403 > <https://gitlab.freedesktop.org/drm/intel/issues/2403> / > i915#4312 <https://gitlab.freedesktop.org/drm/intel/issues/4312>) > > > Possible fixes > > * > > igt@gem_exec_suspend@basic-s0@smem: > > o {fi-ehl-2}: DMESG-WARN > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html> > (i915#5122 > <https://gitlab.freedesktop.org/drm/intel/issues/5122>) -> > PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html> > * > > igt@i915_selftest@live@gt_heartbeat: > > o {fi-jsl-1}: DMESG-FAIL > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-jsl-1/igt@i915_selftest@live@gt_heartbeat.html> > (i915#5334 > <https://gitlab.freedesktop.org/drm/intel/issues/5334>) -> > PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-jsl-1/igt@i915_selftest@live@gt_heartbeat.html> > * > > igt@i915_selftest@live@hangcheck: > > o fi-hsw-4770: INCOMPLETE > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html> > (i915#4785 > <https://gitlab.freedesktop.org/drm/intel/issues/4785>) -> > PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html> > * > > igt@i915_suspend@system-suspend-without-i915: > > o {bat-dg2-9}: DMESG-WARN > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/bat-dg2-9/igt@i915_suspend@system-suspend-without-i915.html> > (i915#5763 > <https://gitlab.freedesktop.org/drm/intel/issues/5763>) -> > PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-dg2-9/igt@i915_suspend@system-suspend-without-i915.html> > * > > igt@kms_flip@basic-flip-vs-modeset@a-edp1: > > o {bat-adlp-6}: DMESG-WARN > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11649/bat-adlp-6/igt@kms_flip@basic-flip-vs-modeset@a-edp1.html> > (i915#3576 > <https://gitlab.freedesktop.org/drm/intel/issues/3576>) -> > PASS > <https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_103981v1/bat-adlp-6/igt@kms_flip@basic-flip-vs-modeset@a-edp1.html> > > {name}: This element is suppressed. This means it is ignored when > computing > the status of the difference (SUCCESS, WARNING, or FAILURE). > > > Build changes > > * Linux: CI_DRM_11649 -> Patchwork_103981v1 > > CI-20190529: 20190529 > CI_DRM_11649: 41053495dd77aa5604144477ef59334a9492a169 @ > git://anongit.freedesktop.org/gfx-ci/linux > IGT_6472: c815c94f0ceb33ae852622538f0136cf44c5725d @ > https://gitlab.freedesktop.org/drm/igt-gpu-tools.git > Patchwork_103981v1: 41053495dd77aa5604144477ef59334a9492a169 @ > git://anongit.freedesktop.org/gfx-ci/linux > > > Linux commits > > 4a604aeca9d8 drm/i915: individualize fences before adding > [-- Attachment #2: Type: text/html, Size: 22899 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915: individualize fences before adding 2022-05-13 11:37 [Intel-gfx] [PATCH] drm/i915: individualize fences before adding Nirmoy Das 2022-05-13 14:54 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork @ 2022-05-16 9:38 ` Hellstrom, Thomas 1 sibling, 0 replies; 4+ messages in thread From: Hellstrom, Thomas @ 2022-05-16 9:38 UTC (permalink / raw) To: Das, Nirmoy, intel-gfx@lists.freedesktop.org; +Cc: Auld, Matthew Hi Nirmoy. On Fri, 2022-05-13 at 13:37 +0200, Nirmoy Das wrote: > _i915_vma_move_to_active() can receive > 1 fence for > multiple batch buffer submission so make sure to > individualize fences before adding to dma_resv obj > > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/5614 > Signed-off-by: Nirmoy Das <nirmoy.das@intel.com> LGTM. Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/i915_vma.c | 17 +++++++++++++---- > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_vma.c > b/drivers/gpu/drm/i915/i915_vma.c > index 4f6db539571a..f987fc1264c0 100644 > --- a/drivers/gpu/drm/i915/i915_vma.c > +++ b/drivers/gpu/drm/i915/i915_vma.c > @@ -23,6 +23,7 @@ > */ > > #include <linux/sched/mm.h> > +#include <linux/dma-fence-array.h> > #include <drm/drm_gem.h> > > #include "display/intel_frontbuffer.h" > @@ -1840,8 +1841,12 @@ int _i915_vma_move_to_active(struct i915_vma > *vma, > } > > if (fence) { > - dma_resv_add_fence(vma->obj->base.resv, > fence, > - DMA_RESV_USAGE_WRITE); > + int idx; > + struct dma_fence *curr; > + > + dma_fence_array_for_each(curr, idx, fence) > + dma_resv_add_fence(vma->obj- > >base.resv, curr, > + > DMA_RESV_USAGE_WRITE); > obj->write_domain = I915_GEM_DOMAIN_RENDER; > obj->read_domains = 0; > } > @@ -1853,8 +1858,12 @@ int _i915_vma_move_to_active(struct i915_vma > *vma, > } > > if (fence) { > - dma_resv_add_fence(vma->obj->base.resv, > fence, > - DMA_RESV_USAGE_READ); > + int idx; > + struct dma_fence *curr; > + > + dma_fence_array_for_each(curr, idx, fence) > + dma_resv_add_fence(vma->obj- > >base.resv, curr, > + > DMA_RESV_USAGE_READ); > obj->write_domain = 0; > } > } ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-05-16 10:57 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-05-13 11:37 [Intel-gfx] [PATCH] drm/i915: individualize fences before adding Nirmoy Das 2022-05-13 14:54 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork 2022-05-16 10:57 ` Das, Nirmoy 2022-05-16 9:38 ` [Intel-gfx] [PATCH] " Hellstrom, Thomas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox