* [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
@ 2021-09-14 19:51 ` Lucas De Marchi
0 siblings, 0 replies; 15+ messages in thread
From: Lucas De Marchi @ 2021-09-14 19:51 UTC (permalink / raw)
To: intel-gfx
Cc: dri-devel, Vinay Belgaumkar, John Harrison, Matthew Brost,
Daniele Ceraolo Spurio
The clflush calls here aren't doing anything since we are not writting
something and flushing the cache lines to be visible to GuC. Here the
intention seems to be to make sure whatever GuC has written is visible
to the CPU before we read them. However a clflush from the CPU side is
the wrong instruction to use.
From code inspection on the other clflush() calls in i915/gt/uc/ these
are the only ones with this behavrior. The others are apparently making
sure what we write is visible to GuC.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
index 65a3e7fdb2b2..2e996b77df80 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
@@ -108,7 +108,6 @@ static u32 slpc_get_state(struct intel_guc_slpc *slpc)
GEM_BUG_ON(!slpc->vma);
- drm_clflush_virt_range(slpc->vaddr, sizeof(u32));
data = slpc->vaddr;
return data->header.global_state;
@@ -172,8 +171,6 @@ static int slpc_query_task_state(struct intel_guc_slpc *slpc)
drm_err(&i915->drm, "Failed to query task state (%pe)\n",
ERR_PTR(ret));
- drm_clflush_virt_range(slpc->vaddr, SLPC_PAGE_SIZE_BYTES);
-
return ret;
}
--
2.32.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
(?)
@ 2021-09-14 20:10 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2021-09-14 20:10 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/guc/slpc: remove unneeded clflush calls
URL : https://patchwork.freedesktop.org/series/94668/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
a021bf892e8d drm/i915/guc/slpc: remove unneeded clflush calls
-:6: WARNING:TYPO_SPELLING: 'writting' may be misspelled - perhaps 'writing'?
#6:
The clflush calls here aren't doing anything since we are not writting
^^^^^^^^
total: 0 errors, 1 warnings, 0 checks, 15 lines checked
^ permalink raw reply [flat|nested] 15+ messages in thread* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
(?)
(?)
@ 2021-09-14 20:35 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2021-09-14 20:35 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 7186 bytes --]
== Series Details ==
Series: drm/i915/guc/slpc: remove unneeded clflush calls
URL : https://patchwork.freedesktop.org/series/94668/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10584 -> Patchwork_21047
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_21047 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_21047, 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_21047/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_21047:
### IGT changes ###
#### Possible regressions ####
* igt@i915_module_load@reload:
- fi-kbl-guc: NOTRUN -> [INCOMPLETE][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-guc/igt@i915_module_load@reload.html
Known issues
------------
Here are the changes found in Patchwork_21047 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@core_hotunplug@unbind-rebind:
- fi-cfl-guc: [PASS][2] -> [INCOMPLETE][3] ([i915#4130])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-cfl-guc/igt@core_hotunplug@unbind-rebind.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-cfl-guc/igt@core_hotunplug@unbind-rebind.html
- fi-icl-u2: [PASS][4] -> [INCOMPLETE][5] ([i915#4130])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-icl-u2/igt@core_hotunplug@unbind-rebind.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-icl-u2/igt@core_hotunplug@unbind-rebind.html
- fi-cfl-8109u: [PASS][6] -> [INCOMPLETE][7] ([i915#4130])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-cfl-8109u/igt@core_hotunplug@unbind-rebind.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-cfl-8109u/igt@core_hotunplug@unbind-rebind.html
- fi-skl-guc: [PASS][8] -> [INCOMPLETE][9] ([i915#4130])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-skl-guc/igt@core_hotunplug@unbind-rebind.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-skl-guc/igt@core_hotunplug@unbind-rebind.html
- fi-kbl-7567u: [PASS][10] -> [INCOMPLETE][11] ([i915#4130])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-kbl-7567u/igt@core_hotunplug@unbind-rebind.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-7567u/igt@core_hotunplug@unbind-rebind.html
* igt@gem_exec_fence@basic-busy@bcs0:
- fi-kbl-soraka: NOTRUN -> [SKIP][12] ([fdo#109271]) +2 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-soraka/igt@gem_exec_fence@basic-busy@bcs0.html
* igt@gem_exec_suspend@basic-s0:
- fi-kbl-soraka: NOTRUN -> [INCOMPLETE][13] ([i915#155])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-soraka/igt@gem_exec_suspend@basic-s0.html
* igt@i915_selftest@live@mman:
- fi-rkl-guc: NOTRUN -> [INCOMPLETE][14] ([i915#4129])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-rkl-guc/igt@i915_selftest@live@mman.html
- fi-kbl-7500u: NOTRUN -> [INCOMPLETE][15] ([i915#4129])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-7500u/igt@i915_selftest@live@mman.html
* igt@kms_chamelium@vga-hpd-fast:
- fi-kbl-guc: NOTRUN -> [SKIP][16] ([fdo#109271] / [fdo#111827]) +8 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-guc/igt@kms_chamelium@vga-hpd-fast.html
* igt@kms_flip@basic-flip-vs-modeset:
- fi-rkl-guc: NOTRUN -> [SKIP][17] ([i915#3669])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-rkl-guc/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- fi-kbl-guc: NOTRUN -> [SKIP][18] ([fdo#109271] / [i915#533])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-guc/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@kms_pipe_crc_basic@read-crc-pipe-c:
- fi-kbl-guc: NOTRUN -> [SKIP][19] ([fdo#109271]) +41 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-guc/igt@kms_pipe_crc_basic@read-crc-pipe-c.html
* igt@runner@aborted:
- fi-kbl-7500u: NOTRUN -> [FAIL][20] ([i915#1436] / [i915#2426] / [i915#3363])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-7500u/igt@runner@aborted.html
- fi-rkl-guc: NOTRUN -> [FAIL][21] ([i915#2426] / [i915#3928])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-rkl-guc/igt@runner@aborted.html
#### Possible fixes ####
* igt@core_hotunplug@unbind-rebind:
- fi-rkl-guc: [INCOMPLETE][22] ([i915#4130]) -> [PASS][23]
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-rkl-guc/igt@core_hotunplug@unbind-rebind.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-rkl-guc/igt@core_hotunplug@unbind-rebind.html
- fi-kbl-7500u: [INCOMPLETE][24] ([i915#4130]) -> [PASS][25]
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10584/fi-kbl-7500u/igt@core_hotunplug@unbind-rebind.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/fi-kbl-7500u/igt@core_hotunplug@unbind-rebind.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#155]: https://gitlab.freedesktop.org/drm/intel/issues/155
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3669]: https://gitlab.freedesktop.org/drm/intel/issues/3669
[i915#3928]: https://gitlab.freedesktop.org/drm/intel/issues/3928
[i915#4129]: https://gitlab.freedesktop.org/drm/intel/issues/4129
[i915#4130]: https://gitlab.freedesktop.org/drm/intel/issues/4130
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
Participating hosts (39 -> 36)
------------------------------
Additional (2): fi-kbl-soraka fi-kbl-guc
Missing (5): bat-dg1-6 bat-dg1-5 fi-bsw-cyan bat-jsl-1 fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_10584 -> Patchwork_21047
CI-20190529: 20190529
CI_DRM_10584: 7fa18a9066276df7e4c9ffe45c98f4604bd92bdd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6209: 07d6594ed02f55b68d64fa6dd7f80cfbc1ce4ef8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_21047: a021bf892e8def3ee508222e43246e1aad562ac7 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
a021bf892e8d drm/i915/guc/slpc: remove unneeded clflush calls
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21047/index.html
[-- Attachment #2: Type: text/html, Size: 8922 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
` (2 preceding siblings ...)
(?)
@ 2021-09-14 23:57 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2021-09-14 23:57 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
URL : https://patchwork.freedesktop.org/series/94668/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
eeef49b011c8 drm/i915/guc/slpc: remove unneeded clflush calls
-:6: WARNING:TYPO_SPELLING: 'writting' may be misspelled - perhaps 'writing'?
#6:
The clflush calls here aren't doing anything since we are not writting
^^^^^^^^
total: 0 errors, 1 warnings, 0 checks, 15 lines checked
^ permalink raw reply [flat|nested] 15+ messages in thread* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
` (3 preceding siblings ...)
(?)
@ 2021-09-15 0:28 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2021-09-15 0:28 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 7979 bytes --]
== Series Details ==
Series: drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
URL : https://patchwork.freedesktop.org/series/94668/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10586 -> Patchwork_21050
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_21050:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@i915_module_load@reload:
- {fi-jsl-1}: [TIMEOUT][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-jsl-1/igt@i915_module_load@reload.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-jsl-1/igt@i915_module_load@reload.html
* igt@runner@aborted:
- {fi-ehl-2}: NOTRUN -> [FAIL][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-ehl-2/igt@runner@aborted.html
Known issues
------------
Here are the changes found in Patchwork_21050 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@core_hotunplug@unbind-rebind:
- fi-skl-6700k2: [PASS][4] -> [INCOMPLETE][5] ([i915#4130])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-skl-6700k2/igt@core_hotunplug@unbind-rebind.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-skl-6700k2/igt@core_hotunplug@unbind-rebind.html
- fi-cfl-8109u: [PASS][6] -> [INCOMPLETE][7] ([i915#4130])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-cfl-8109u/igt@core_hotunplug@unbind-rebind.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-cfl-8109u/igt@core_hotunplug@unbind-rebind.html
- fi-kbl-7567u: [PASS][8] -> [INCOMPLETE][9] ([i915#4130])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-kbl-7567u/igt@core_hotunplug@unbind-rebind.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-7567u/igt@core_hotunplug@unbind-rebind.html
* igt@i915_selftest@live@gt_pm:
- fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][10] ([i915#1886] / [i915#2291])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
* igt@i915_selftest@live@mman:
- fi-kbl-soraka: NOTRUN -> [INCOMPLETE][11] ([i915#4129])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-soraka/igt@i915_selftest@live@mman.html
- fi-rkl-guc: NOTRUN -> [INCOMPLETE][12] ([i915#4129])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-rkl-guc/igt@i915_selftest@live@mman.html
- fi-cfl-guc: NOTRUN -> [INCOMPLETE][13] ([i915#4129])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-cfl-guc/igt@i915_selftest@live@mman.html
- fi-skl-guc: NOTRUN -> [INCOMPLETE][14] ([i915#3796] / [i915#4129])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-skl-guc/igt@i915_selftest@live@mman.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-legacy:
- fi-rkl-11600: [PASS][15] -> [SKIP][16] ([fdo#111825])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-rkl-11600/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-rkl-11600/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
* igt@runner@aborted:
- fi-cml-u2: NOTRUN -> [FAIL][17] ([i915#2722] / [i915#3363])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-cml-u2/igt@runner@aborted.html
- fi-rkl-guc: NOTRUN -> [FAIL][18] ([i915#2426] / [i915#3928])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-rkl-guc/igt@runner@aborted.html
- fi-cfl-guc: NOTRUN -> [FAIL][19] ([i915#2426] / [i915#3363])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-cfl-guc/igt@runner@aborted.html
- fi-skl-guc: NOTRUN -> [FAIL][20] ([i915#1436] / [i915#2426] / [i915#3363])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-skl-guc/igt@runner@aborted.html
#### Possible fixes ####
* igt@core_hotunplug@unbind-rebind:
- fi-rkl-guc: [INCOMPLETE][21] ([i915#4130]) -> [PASS][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-rkl-guc/igt@core_hotunplug@unbind-rebind.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-rkl-guc/igt@core_hotunplug@unbind-rebind.html
- fi-cfl-guc: [INCOMPLETE][23] ([i915#4130]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-cfl-guc/igt@core_hotunplug@unbind-rebind.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-cfl-guc/igt@core_hotunplug@unbind-rebind.html
- fi-skl-guc: [INCOMPLETE][25] ([i915#4130]) -> [PASS][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-skl-guc/igt@core_hotunplug@unbind-rebind.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-skl-guc/igt@core_hotunplug@unbind-rebind.html
* igt@i915_module_load@reload:
- fi-kbl-soraka: [INCOMPLETE][27] -> [PASS][28]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-kbl-soraka/igt@i915_module_load@reload.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-soraka/igt@i915_module_load@reload.html
- fi-kbl-guc: [INCOMPLETE][29] -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-kbl-guc/igt@i915_module_load@reload.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-guc/igt@i915_module_load@reload.html
#### Warnings ####
* igt@runner@aborted:
- fi-kbl-soraka: [FAIL][31] ([i915#2426] / [i915#3363]) -> [FAIL][32] ([i915#1436] / [i915#2426] / [i915#3363])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/fi-kbl-soraka/igt@runner@aborted.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/fi-kbl-soraka/igt@runner@aborted.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3796]: https://gitlab.freedesktop.org/drm/intel/issues/3796
[i915#3928]: https://gitlab.freedesktop.org/drm/intel/issues/3928
[i915#4129]: https://gitlab.freedesktop.org/drm/intel/issues/4129
[i915#4130]: https://gitlab.freedesktop.org/drm/intel/issues/4130
Participating hosts (41 -> 35)
------------------------------
Missing (6): fi-bxt-dsi bat-dg1-6 fi-tgl-1115g4 fi-bsw-cyan bat-jsl-2 fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_10586 -> Patchwork_21050
CI-20190529: 20190529
CI_DRM_10586: 84dcd8a1b2a281b296db22fac5f7ffe52dd7c501 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6209: 07d6594ed02f55b68d64fa6dd7f80cfbc1ce4ef8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_21050: eeef49b011c880e0dd049755466059338aca30a1 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
eeef49b011c8 drm/i915/guc/slpc: remove unneeded clflush calls
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/index.html
[-- Attachment #2: Type: text/html, Size: 10139 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
` (4 preceding siblings ...)
(?)
@ 2021-09-15 2:39 ` Patchwork
-1 siblings, 0 replies; 15+ messages in thread
From: Patchwork @ 2021-09-15 2:39 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 30278 bytes --]
== Series Details ==
Series: drm/i915/guc/slpc: remove unneeded clflush calls (rev2)
URL : https://patchwork.freedesktop.org/series/94668/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10586_full -> Patchwork_21050_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_21050_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_21050_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_21050_full:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live:
- shard-skl: NOTRUN -> [INCOMPLETE][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl6/igt@i915_selftest@live.html
Known issues
------------
Here are the changes found in Patchwork_21050_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@core_hotunplug@unbind-rebind:
- shard-iclb: [PASS][2] -> [INCOMPLETE][3] ([i915#4130])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb1/igt@core_hotunplug@unbind-rebind.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb2/igt@core_hotunplug@unbind-rebind.html
* igt@feature_discovery@display-2x:
- shard-tglb: NOTRUN -> [SKIP][4] ([i915#1839])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb2/igt@feature_discovery@display-2x.html
* igt@gem_exec_fair@basic-none@rcs0:
- shard-kbl: [PASS][5] -> [FAIL][6] ([i915#2842]) +1 similar issue
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl6/igt@gem_exec_fair@basic-none@rcs0.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@gem_exec_fair@basic-none@rcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-tglb: NOTRUN -> [FAIL][7] ([i915#2842])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_mmap_gtt@coherency:
- shard-tglb: NOTRUN -> [SKIP][8] ([fdo#111656])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@gem_mmap_gtt@coherency.html
* igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled:
- shard-iclb: NOTRUN -> [SKIP][9] ([i915#768])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled.html
* igt@gem_userptr_blits@unsync-unmap-after-close:
- shard-tglb: NOTRUN -> [SKIP][10] ([i915#3297])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb5/igt@gem_userptr_blits@unsync-unmap-after-close.html
* igt@gen9_exec_parse@bb-secure:
- shard-tglb: NOTRUN -> [SKIP][11] ([i915#2856])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@gen9_exec_parse@bb-secure.html
- shard-iclb: NOTRUN -> [SKIP][12] ([i915#2856])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@gen9_exec_parse@bb-secure.html
* igt@i915_pm_dc@dc6-dpms:
- shard-skl: NOTRUN -> [FAIL][13] ([i915#454])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@i915_pm_dc@dc6-dpms.html
* igt@i915_pm_rpm@system-suspend:
- shard-tglb: [PASS][14] -> [INCOMPLETE][15] ([i915#2411] / [i915#456])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb3/igt@i915_pm_rpm@system-suspend.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb7/igt@i915_pm_rpm@system-suspend.html
* igt@kms_async_flips@alternate-sync-async-flip:
- shard-skl: [PASS][16] -> [FAIL][17] ([i915#2521])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl4/igt@kms_async_flips@alternate-sync-async-flip.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl5/igt@kms_async_flips@alternate-sync-async-flip.html
* igt@kms_big_fb@x-tiled-8bpp-rotate-90:
- shard-tglb: NOTRUN -> [SKIP][18] ([fdo#111614]) +1 similar issue
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@kms_big_fb@x-tiled-8bpp-rotate-90.html
* igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-kbl: NOTRUN -> [SKIP][19] ([fdo#109271] / [i915#3777])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-tglb: NOTRUN -> [SKIP][20] ([fdo#111615])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb2/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
* igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc:
- shard-iclb: NOTRUN -> [SKIP][21] ([fdo#109278] / [i915#3886]) +2 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][22] ([i915#3689]) +1 similar issue
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_ccs.html
* igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_rc_ccs_cc:
- shard-kbl: NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#3886]) +3 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl2/igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_rc_ccs_cc.html
- shard-skl: NOTRUN -> [SKIP][24] ([fdo#109271] / [i915#3886]) +2 similar issues
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs:
- shard-tglb: NOTRUN -> [SKIP][25] ([i915#3689] / [i915#3886]) +2 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb2/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs.html
* igt@kms_chamelium@hdmi-audio-edid:
- shard-kbl: NOTRUN -> [SKIP][26] ([fdo#109271] / [fdo#111827]) +5 similar issues
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_chamelium@hdmi-audio-edid.html
* igt@kms_chamelium@vga-hpd-after-suspend:
- shard-skl: NOTRUN -> [SKIP][27] ([fdo#109271] / [fdo#111827]) +4 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_chamelium@vga-hpd-after-suspend.html
- shard-iclb: NOTRUN -> [SKIP][28] ([fdo#109284] / [fdo#111827]) +1 similar issue
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_chamelium@vga-hpd-after-suspend.html
* igt@kms_color@pipe-a-ctm-0-75:
- shard-skl: [PASS][29] -> [DMESG-WARN][30] ([i915#1982])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl7/igt@kms_color@pipe-a-ctm-0-75.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl8/igt@kms_color@pipe-a-ctm-0-75.html
* igt@kms_color_chamelium@pipe-invalid-degamma-lut-sizes:
- shard-tglb: NOTRUN -> [SKIP][31] ([fdo#109284] / [fdo#111827]) +4 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@kms_color_chamelium@pipe-invalid-degamma-lut-sizes.html
* igt@kms_cursor_crc@pipe-a-cursor-size-change:
- shard-kbl: NOTRUN -> [FAIL][32] ([i915#3444])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
* igt@kms_cursor_crc@pipe-b-cursor-32x10-random:
- shard-tglb: NOTRUN -> [SKIP][33] ([i915#3359]) +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb5/igt@kms_cursor_crc@pipe-b-cursor-32x10-random.html
* igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen:
- shard-tglb: NOTRUN -> [SKIP][34] ([i915#3319]) +1 similar issue
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb2/igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen:
- shard-iclb: NOTRUN -> [SKIP][35] ([fdo#109278] / [fdo#109279])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_cursor_crc@pipe-b-cursor-512x512-offscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-512x170-onscreen:
- shard-tglb: NOTRUN -> [SKIP][36] ([fdo#109279] / [i915#3359]) +3 similar issues
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@kms_cursor_crc@pipe-c-cursor-512x170-onscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-suspend:
- shard-kbl: NOTRUN -> [DMESG-WARN][37] ([i915#180])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_cursor_crc@pipe-c-cursor-suspend.html
* igt@kms_cursor_crc@pipe-d-cursor-32x32-onscreen:
- shard-iclb: NOTRUN -> [SKIP][38] ([fdo#109278]) +3 similar issues
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_cursor_crc@pipe-d-cursor-32x32-onscreen.html
* igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1:
- shard-skl: [PASS][39] -> [FAIL][40] ([i915#2122]) +1 similar issue
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl1/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl5/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs:
- shard-kbl: NOTRUN -> [SKIP][41] ([fdo#109271] / [i915#2672])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-onoff:
- shard-tglb: NOTRUN -> [SKIP][42] ([fdo#111825]) +17 similar issues
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-onoff.html
* igt@kms_frontbuffer_tracking@fbc-suspend:
- shard-tglb: [PASS][43] -> [INCOMPLETE][44] ([i915#456])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb3/igt@kms_frontbuffer_tracking@fbc-suspend.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb7/igt@kms_frontbuffer_tracking@fbc-suspend.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-shrfb-draw-blt:
- shard-iclb: NOTRUN -> [SKIP][45] ([fdo#109280]) +5 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-shrfb-draw-blt.html
* igt@kms_hdr@bpc-switch-suspend:
- shard-kbl: [PASS][46] -> [DMESG-WARN][47] ([i915#180]) +5 similar issues
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl3/igt@kms_hdr@bpc-switch-suspend.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@kms_hdr@bpc-switch-suspend.html
- shard-skl: [PASS][48] -> [FAIL][49] ([i915#1188])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl5/igt@kms_hdr@bpc-switch-suspend.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl3/igt@kms_hdr@bpc-switch-suspend.html
* igt@kms_pipe_crc_basic@read-crc-pipe-d:
- shard-kbl: NOTRUN -> [SKIP][50] ([fdo#109271] / [i915#533])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@kms_pipe_crc_basic@read-crc-pipe-d.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
- shard-skl: NOTRUN -> [FAIL][51] ([fdo#108145] / [i915#265])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_plane_alpha_blend@pipe-a-alpha-7efc.html
* igt@kms_plane_lowres@pipe-d-tiling-yf:
- shard-tglb: NOTRUN -> [SKIP][52] ([fdo#112054]) +2 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@kms_plane_lowres@pipe-d-tiling-yf.html
- shard-skl: NOTRUN -> [SKIP][53] ([fdo#109271]) +45 similar issues
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_plane_lowres@pipe-d-tiling-yf.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2:
- shard-tglb: NOTRUN -> [SKIP][54] ([i915#2920])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-5:
- shard-skl: NOTRUN -> [SKIP][55] ([fdo#109271] / [i915#658])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-5.html
- shard-kbl: NOTRUN -> [SKIP][56] ([fdo#109271] / [i915#658])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-5.html
- shard-iclb: NOTRUN -> [SKIP][57] ([i915#658])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-5.html
* igt@kms_psr@psr2_primary_mmap_gtt:
- shard-tglb: NOTRUN -> [FAIL][58] ([i915#132] / [i915#3467])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@kms_psr@psr2_primary_mmap_gtt.html
- shard-iclb: NOTRUN -> [SKIP][59] ([fdo#109441])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_psr@psr2_primary_mmap_gtt.html
* igt@kms_psr@psr2_sprite_mmap_gtt:
- shard-iclb: [PASS][60] -> [SKIP][61] ([fdo#109441])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb1/igt@kms_psr@psr2_sprite_mmap_gtt.html
* igt@kms_sysfs_edid_timing:
- shard-skl: NOTRUN -> [FAIL][62] ([IGT#2])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_sysfs_edid_timing.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-kbl: [PASS][63] -> [DMESG-WARN][64] ([i915#180] / [i915#295])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl6/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- shard-skl: [PASS][65] -> [INCOMPLETE][66] ([i915#146] / [i915#198] / [i915#2828])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl3/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl4/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
* igt@kms_writeback@writeback-check-output:
- shard-iclb: NOTRUN -> [SKIP][67] ([i915#2437])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb4/igt@kms_writeback@writeback-check-output.html
- shard-skl: NOTRUN -> [SKIP][68] ([fdo#109271] / [i915#2437])
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_writeback@writeback-check-output.html
- shard-tglb: NOTRUN -> [SKIP][69] ([i915#2437])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@kms_writeback@writeback-check-output.html
* igt@nouveau_crc@pipe-c-source-rg:
- shard-tglb: NOTRUN -> [SKIP][70] ([i915#2530]) +1 similar issue
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@nouveau_crc@pipe-c-source-rg.html
* igt@perf@gen12-oa-tlb-invalidate:
- shard-kbl: NOTRUN -> [SKIP][71] ([fdo#109271]) +28 similar issues
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@perf@gen12-oa-tlb-invalidate.html
* igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name:
- shard-tglb: NOTRUN -> [SKIP][72] ([fdo#109291])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name.html
* igt@sysfs_clients@fair-1:
- shard-tglb: NOTRUN -> [SKIP][73] ([i915#2994])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb5/igt@sysfs_clients@fair-1.html
* igt@sysfs_clients@sema-50:
- shard-kbl: NOTRUN -> [SKIP][74] ([fdo#109271] / [i915#2994])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@sysfs_clients@sema-50.html
#### Possible fixes ####
* igt@gem_ctx_isolation@preservation-s3@bcs0:
- shard-tglb: [INCOMPLETE][75] -> [PASS][76]
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb7/igt@gem_ctx_isolation@preservation-s3@bcs0.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb2/igt@gem_ctx_isolation@preservation-s3@bcs0.html
* igt@gem_eio@in-flight-contexts-10ms:
- shard-iclb: [TIMEOUT][77] ([i915#3070]) -> [PASS][78]
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb6/igt@gem_eio@in-flight-contexts-10ms.html
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb3/igt@gem_eio@in-flight-contexts-10ms.html
* igt@gem_eio@unwedge-stress:
- shard-tglb: [TIMEOUT][79] ([i915#2369] / [i915#3063] / [i915#3648]) -> [PASS][80]
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb7/igt@gem_eio@unwedge-stress.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb1/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_endless@dispatch@vcs0:
- shard-iclb: [INCOMPLETE][81] ([i915#3778]) -> [PASS][82]
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb8/igt@gem_exec_endless@dispatch@vcs0.html
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb3/igt@gem_exec_endless@dispatch@vcs0.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [FAIL][83] ([i915#2842]) -> [PASS][84]
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb1/igt@gem_exec_fair@basic-none-share@rcs0.html
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb7/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-pace@vecs0:
- shard-kbl: [FAIL][85] ([i915#2842]) -> [PASS][86]
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl6/igt@gem_exec_fair@basic-pace@vecs0.html
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl6/igt@gem_exec_fair@basic-pace@vecs0.html
- shard-tglb: [FAIL][87] ([i915#2842]) -> [PASS][88] +1 similar issue
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb6/igt@gem_exec_fair@basic-pace@vecs0.html
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb6/igt@gem_exec_fair@basic-pace@vecs0.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-iclb: [FAIL][89] ([i915#2849]) -> [PASS][90]
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb3/igt@gem_exec_fair@basic-throttle@rcs0.html
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb5/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_workarounds@suspend-resume-context:
- shard-skl: [INCOMPLETE][91] ([i915#198]) -> [PASS][92]
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl2/igt@gem_workarounds@suspend-resume-context.html
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl6/igt@gem_workarounds@suspend-resume-context.html
* igt@i915_suspend@debugfs-reader:
- shard-tglb: [INCOMPLETE][93] ([i915#456]) -> [PASS][94]
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-tglb7/igt@i915_suspend@debugfs-reader.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-tglb8/igt@i915_suspend@debugfs-reader.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-kbl: [INCOMPLETE][95] ([i915#155] / [i915#180] / [i915#636]) -> [PASS][96]
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl1/igt@kms_fbcon_fbt@fbc-suspend.html
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl2/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: [FAIL][97] ([fdo#108145] / [i915#265]) -> [PASS][98] +1 similar issue
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl6/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl4/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
* igt@kms_psr@psr2_primary_page_flip:
- shard-iclb: [SKIP][99] ([fdo#109441]) -> [PASS][100] +1 similar issue
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb1/igt@kms_psr@psr2_primary_page_flip.html
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html
* igt@kms_vblank@pipe-b-ts-continuation-suspend:
- shard-kbl: [DMESG-WARN][101] ([i915#180]) -> [PASS][102] +1 similar issue
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl4/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl2/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
#### Warnings ####
* igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
- shard-skl: [FAIL][103] ([i915#3743]) -> [FAIL][104] ([i915#3722])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl8/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl2/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
* igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1:
- shard-iclb: [SKIP][105] ([i915#2920]) -> [SKIP][106] ([i915#658])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb1/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html
* igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5:
- shard-iclb: [SKIP][107] ([i915#658]) -> [SKIP][108] ([i915#2920]) +1 similar issue
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-iclb1/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
* igt@runner@aborted:
- shard-kbl: ([FAIL][109], [FAIL][110], [FAIL][111], [FAIL][112], [FAIL][113], [FAIL][114], [FAIL][115], [FAIL][116], [FAIL][117], [FAIL][118]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#602] / [i915#92]) -> ([FAIL][119], [FAIL][120], [FAIL][121], [FAIL][122], [FAIL][123], [FAIL][124], [FAIL][125], [FAIL][126], [FAIL][127], [FAIL][128], [FAIL][129]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#602])
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl7/igt@runner@aborted.html
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl3/igt@runner@aborted.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl1/igt@runner@aborted.html
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl3/igt@runner@aborted.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl4/igt@runner@aborted.html
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl6/igt@runner@aborted.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl4/igt@runner@aborted.html
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl4/igt@runner@aborted.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl4/igt@runner@aborted.html
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-kbl1/igt@runner@aborted.html
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl7/igt@runner@aborted.html
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl7/igt@runner@aborted.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl6/igt@runner@aborted.html
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@runner@aborted.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@runner@aborted.html
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@runner@aborted.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@runner@aborted.html
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@runner@aborted.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl1/igt@runner@aborted.html
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl3/igt@runner@aborted.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-kbl4/igt@runner@aborted.html
- shard-skl: ([FAIL][130], [FAIL][131]) ([i915#1436] / [i915#3002] / [i915#3363]) -> ([FAIL][132], [FAIL][133]) ([i915#1814] / [i915#2029] / [i915#3002] / [i915#3363])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl10/igt@runner@aborted.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10586/shard-skl5/igt@runner@aborted.html
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl3/igt@runner@aborted.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/shard-skl3/igt@runner@aborted.html
[IGT#2]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/2
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279
[fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
[fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
[fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
[fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
[fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[fdo#112054]: https://bugs.freedesktop.org/show_bug.cgi?id=112054
[i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188
[i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146
[i915#155]: https://gitlab.freedesktop.org/drm/intel/issues/155
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
[i915#1839]: https://gitlab.freedesktop.org/drm/intel/issues/1839
[i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029
[i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
[i915#2369]: https://gitlab.freedesktop.org/drm/intel/issues/2369
[i915#2411]: https://gitlab.freedesktop.org/drm/intel/issues/2411
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521
[i915#2530]: https://gitlab.freedesktop.org/drm/intel/issues/2530
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#2828]: https://gitlab.freedesktop.org/drm/intel/issues/2828
[i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
[i915#2849]: https://gitlab.freedesktop.org/drm/intel/issues/2849
[i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
[i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
[i915#295]: https://gitlab.freedesktop.org/drm/intel/issues/295
[i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
[i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002
[i915#3063]: https://gitlab.freedesktop.org/drm/intel/issues/3063
[i915#3070]: https://gitlab.freedesktop.org/drm/intel/issues/3070
[i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297
[i915#3319]: https://gitlab.freedesktop.org/drm/intel/issues/3319
[i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3444]: https://gitlab.freedesktop.org/drm/intel/issues/3444
[i915#3467]: https
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21050/index.html
[-- Attachment #2: Type: text/html, Size: 36646 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-14 19:51 ` [Intel-gfx] " Lucas De Marchi
@ 2021-09-15 19:24 ` Belgaumkar, Vinay
-1 siblings, 0 replies; 15+ messages in thread
From: Belgaumkar, Vinay @ 2021-09-15 19:24 UTC (permalink / raw)
To: Lucas De Marchi, intel-gfx
Cc: dri-devel, John Harrison, Matthew Brost, Daniele Ceraolo Spurio
On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
> The clflush calls here aren't doing anything since we are not writting
> something and flushing the cache lines to be visible to GuC. Here the
> intention seems to be to make sure whatever GuC has written is visible
> to the CPU before we read them. However a clflush from the CPU side is
> the wrong instruction to use.
>
> From code inspection on the other clflush() calls in i915/gt/uc/ these
> are the only ones with this behavrior. The others are apparently making
> sure what we write is visible to GuC.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> index 65a3e7fdb2b2..2e996b77df80 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> @@ -108,7 +108,6 @@ static u32 slpc_get_state(struct intel_guc_slpc *slpc)
>
> GEM_BUG_ON(!slpc->vma);
>
> - drm_clflush_virt_range(slpc->vaddr, sizeof(u32));
> data = slpc->vaddr;
>
> return data->header.global_state;
> @@ -172,8 +171,6 @@ static int slpc_query_task_state(struct intel_guc_slpc *slpc)
> drm_err(&i915->drm, "Failed to query task state (%pe)\n",
> ERR_PTR(ret));
>
> - drm_clflush_virt_range(slpc->vaddr, SLPC_PAGE_SIZE_BYTES);
> -
LGTM.
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> return ret;
> }
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
@ 2021-09-15 19:24 ` Belgaumkar, Vinay
0 siblings, 0 replies; 15+ messages in thread
From: Belgaumkar, Vinay @ 2021-09-15 19:24 UTC (permalink / raw)
To: Lucas De Marchi, intel-gfx
Cc: dri-devel, John Harrison, Matthew Brost, Daniele Ceraolo Spurio
On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
> The clflush calls here aren't doing anything since we are not writting
> something and flushing the cache lines to be visible to GuC. Here the
> intention seems to be to make sure whatever GuC has written is visible
> to the CPU before we read them. However a clflush from the CPU side is
> the wrong instruction to use.
>
> From code inspection on the other clflush() calls in i915/gt/uc/ these
> are the only ones with this behavrior. The others are apparently making
> sure what we write is visible to GuC.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> index 65a3e7fdb2b2..2e996b77df80 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> @@ -108,7 +108,6 @@ static u32 slpc_get_state(struct intel_guc_slpc *slpc)
>
> GEM_BUG_ON(!slpc->vma);
>
> - drm_clflush_virt_range(slpc->vaddr, sizeof(u32));
> data = slpc->vaddr;
>
> return data->header.global_state;
> @@ -172,8 +171,6 @@ static int slpc_query_task_state(struct intel_guc_slpc *slpc)
> drm_err(&i915->drm, "Failed to query task state (%pe)\n",
> ERR_PTR(ret));
>
> - drm_clflush_virt_range(slpc->vaddr, SLPC_PAGE_SIZE_BYTES);
> -
LGTM.
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> return ret;
> }
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-15 19:24 ` [Intel-gfx] " Belgaumkar, Vinay
@ 2021-09-15 19:29 ` John Harrison
-1 siblings, 0 replies; 15+ messages in thread
From: John Harrison @ 2021-09-15 19:29 UTC (permalink / raw)
To: Belgaumkar, Vinay, Lucas De Marchi, intel-gfx
Cc: dri-devel, Matthew Brost, Daniele Ceraolo Spurio
On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
> On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
>> The clflush calls here aren't doing anything since we are not writting
>> something and flushing the cache lines to be visible to GuC. Here the
>> intention seems to be to make sure whatever GuC has written is visible
>> to the CPU before we read them. However a clflush from the CPU side is
>> the wrong instruction to use.
Is there a right instruction to use? Either we need to verify that no
flush/invalidate is required or we need to add in a replacement that
does the correct thing?
John.
>>
>> From code inspection on the other clflush() calls in i915/gt/uc/ these
>> are the only ones with this behavrior. The others are apparently making
>> sure what we write is visible to GuC.
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> ---
>> drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> index 65a3e7fdb2b2..2e996b77df80 100644
>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> @@ -108,7 +108,6 @@ static u32 slpc_get_state(struct intel_guc_slpc
>> *slpc)
>> GEM_BUG_ON(!slpc->vma);
>> - drm_clflush_virt_range(slpc->vaddr, sizeof(u32));
>> data = slpc->vaddr;
>> return data->header.global_state;
>> @@ -172,8 +171,6 @@ static int slpc_query_task_state(struct
>> intel_guc_slpc *slpc)
>> drm_err(&i915->drm, "Failed to query task state (%pe)\n",
>> ERR_PTR(ret));
>> - drm_clflush_virt_range(slpc->vaddr, SLPC_PAGE_SIZE_BYTES);
>> -
>
> LGTM.
> Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>
>> return ret;
>> }
>>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
@ 2021-09-15 19:29 ` John Harrison
0 siblings, 0 replies; 15+ messages in thread
From: John Harrison @ 2021-09-15 19:29 UTC (permalink / raw)
To: Belgaumkar, Vinay, Lucas De Marchi, intel-gfx
Cc: dri-devel, Matthew Brost, Daniele Ceraolo Spurio
On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
> On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
>> The clflush calls here aren't doing anything since we are not writting
>> something and flushing the cache lines to be visible to GuC. Here the
>> intention seems to be to make sure whatever GuC has written is visible
>> to the CPU before we read them. However a clflush from the CPU side is
>> the wrong instruction to use.
Is there a right instruction to use? Either we need to verify that no
flush/invalidate is required or we need to add in a replacement that
does the correct thing?
John.
>>
>> From code inspection on the other clflush() calls in i915/gt/uc/ these
>> are the only ones with this behavrior. The others are apparently making
>> sure what we write is visible to GuC.
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> ---
>> drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c | 3 ---
>> 1 file changed, 3 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> index 65a3e7fdb2b2..2e996b77df80 100644
>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
>> @@ -108,7 +108,6 @@ static u32 slpc_get_state(struct intel_guc_slpc
>> *slpc)
>> GEM_BUG_ON(!slpc->vma);
>> - drm_clflush_virt_range(slpc->vaddr, sizeof(u32));
>> data = slpc->vaddr;
>> return data->header.global_state;
>> @@ -172,8 +171,6 @@ static int slpc_query_task_state(struct
>> intel_guc_slpc *slpc)
>> drm_err(&i915->drm, "Failed to query task state (%pe)\n",
>> ERR_PTR(ret));
>> - drm_clflush_virt_range(slpc->vaddr, SLPC_PAGE_SIZE_BYTES);
>> -
>
> LGTM.
> Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>
>> return ret;
>> }
>>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-15 19:29 ` [Intel-gfx] " John Harrison
@ 2021-09-21 5:47 ` Lucas De Marchi
-1 siblings, 0 replies; 15+ messages in thread
From: Lucas De Marchi @ 2021-09-21 5:47 UTC (permalink / raw)
To: John Harrison
Cc: Belgaumkar, Vinay, intel-gfx, dri-devel, Matthew Brost,
Daniele Ceraolo Spurio
On Wed, Sep 15, 2021 at 12:29:12PM -0700, John Harrison wrote:
>On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
>>On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
>>>The clflush calls here aren't doing anything since we are not writting
>>>something and flushing the cache lines to be visible to GuC. Here the
>>>intention seems to be to make sure whatever GuC has written is visible
>>>to the CPU before we read them. However a clflush from the CPU side is
>>>the wrong instruction to use.
>Is there a right instruction to use? Either we need to verify that no
how can there be a right instruction? If the GuC needs to flush, then
the GuC needs to do it, nothing to be done by the CPU.
Flushing the CPU cache line here is doing nothing to guarantee that what
was written by GuC hit the memory and we are reading it. Not sure why it
was actually added, but since it was added by Vinay and he reviewed this
patch, I'm assuming he also agrees
Lucas De Marchi
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
@ 2021-09-21 5:47 ` Lucas De Marchi
0 siblings, 0 replies; 15+ messages in thread
From: Lucas De Marchi @ 2021-09-21 5:47 UTC (permalink / raw)
To: John Harrison
Cc: Belgaumkar, Vinay, intel-gfx, dri-devel, Matthew Brost,
Daniele Ceraolo Spurio
On Wed, Sep 15, 2021 at 12:29:12PM -0700, John Harrison wrote:
>On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
>>On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
>>>The clflush calls here aren't doing anything since we are not writting
>>>something and flushing the cache lines to be visible to GuC. Here the
>>>intention seems to be to make sure whatever GuC has written is visible
>>>to the CPU before we read them. However a clflush from the CPU side is
>>>the wrong instruction to use.
>Is there a right instruction to use? Either we need to verify that no
how can there be a right instruction? If the GuC needs to flush, then
the GuC needs to do it, nothing to be done by the CPU.
Flushing the CPU cache line here is doing nothing to guarantee that what
was written by GuC hit the memory and we are reading it. Not sure why it
was actually added, but since it was added by Vinay and he reviewed this
patch, I'm assuming he also agrees
Lucas De Marchi
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-21 5:47 ` [Intel-gfx] " Lucas De Marchi
(?)
@ 2021-09-21 13:06 ` Ville Syrjälä
2021-09-23 5:37 ` Lucas De Marchi
-1 siblings, 1 reply; 15+ messages in thread
From: Ville Syrjälä @ 2021-09-21 13:06 UTC (permalink / raw)
To: Lucas De Marchi
Cc: John Harrison, Belgaumkar, Vinay, intel-gfx, dri-devel,
Matthew Brost, Daniele Ceraolo Spurio
On Mon, Sep 20, 2021 at 10:47:08PM -0700, Lucas De Marchi wrote:
> On Wed, Sep 15, 2021 at 12:29:12PM -0700, John Harrison wrote:
> >On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
> >>On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
> >>>The clflush calls here aren't doing anything since we are not writting
> >>>something and flushing the cache lines to be visible to GuC. Here the
> >>>intention seems to be to make sure whatever GuC has written is visible
> >>>to the CPU before we read them. However a clflush from the CPU side is
> >>>the wrong instruction to use.
> >Is there a right instruction to use? Either we need to verify that no
>
> how can there be a right instruction? If the GuC needs to flush, then
> the GuC needs to do it, nothing to be done by the CPU.
>
> Flushing the CPU cache line here is doing nothing to guarantee that what
> was written by GuC hit the memory and we are reading it. Not sure why it
> was actually added, but since it was added by Vinay and he reviewed this
> patch, I'm assuming he also agrees
clflush == writeback + invalidate. The invalidate is the important part
when the CPU has to read something written by something else that's not
cache coherent.
Now, I have no idea if the guc has its own (CPU invisible) caches or not.
If it does then it will need to trigger a writeback. But regardless, if
the guc bypasses the CPU caches the CPU will need to invalidate before
it reads anything in case it has stale data sitting in its cache.
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/guc/slpc: remove unneeded clflush calls
2021-09-21 13:06 ` Ville Syrjälä
@ 2021-09-23 5:37 ` Lucas De Marchi
0 siblings, 0 replies; 15+ messages in thread
From: Lucas De Marchi @ 2021-09-23 5:37 UTC (permalink / raw)
To: Ville Syrjälä
Cc: John Harrison, Belgaumkar, Vinay, intel-gfx, dri-devel,
Matthew Brost, Daniele Ceraolo Spurio
On Tue, Sep 21, 2021 at 04:06:00PM +0300, Ville Syrjälä wrote:
>On Mon, Sep 20, 2021 at 10:47:08PM -0700, Lucas De Marchi wrote:
>> On Wed, Sep 15, 2021 at 12:29:12PM -0700, John Harrison wrote:
>> >On 9/15/2021 12:24, Belgaumkar, Vinay wrote:
>> >>On 9/14/2021 12:51 PM, Lucas De Marchi wrote:
>> >>>The clflush calls here aren't doing anything since we are not writting
>> >>>something and flushing the cache lines to be visible to GuC. Here the
>> >>>intention seems to be to make sure whatever GuC has written is visible
>> >>>to the CPU before we read them. However a clflush from the CPU side is
>> >>>the wrong instruction to use.
>> >Is there a right instruction to use? Either we need to verify that no
>>
>> how can there be a right instruction? If the GuC needs to flush, then
>> the GuC needs to do it, nothing to be done by the CPU.
>>
>> Flushing the CPU cache line here is doing nothing to guarantee that what
>> was written by GuC hit the memory and we are reading it. Not sure why it
>> was actually added, but since it was added by Vinay and he reviewed this
>> patch, I'm assuming he also agrees
>
>clflush == writeback + invalidate. The invalidate is the important part
>when the CPU has to read something written by something else that's not
>cache coherent.
Although the invalidate would be the important part, how would that work
if there is still a flush? Wouldn't we be overriding whatever
was written by the other side? Or are we using the fact that we
shouldn't be writting to this cacheline so we know it's not dirty?
>
>Now, I have no idea if the guc has its own (CPU invisible) caches or not.
>If it does then it will need to trigger a writeback. But regardless, if
>the guc bypasses the CPU caches the CPU will need to invalidate before
>it reads anything in case it has stale data sitting in its cache.
Indeed, thanks... but another case would be if caches are coherent
through snoop. Do you know what is the cache architecture with GuC
and CPU?
Another question comes to mind, but first some context: I'm looking
at this in order to support other archs besides x86... the only
platforms in which this would be relevant would be on the discrete ones
(I'm currently running an arm64 guest on qemu and using pci
passthrough). I see that for dgfx intel_guc_allocate_vma() uses
i915_gem_object_create_lmem() instead of i915_gem_object_create_shmem().
Would that make a difference?
thanks
Lucas De Marchi
^ permalink raw reply [flat|nested] 15+ messages in thread