[Intel-gfx] [CI v2 0/4] drm/i915/guc: Improve CTB error handling
There was a gap in handling MMIO result from CTB (de)registration and while fixing it improve some other error reports. Signed-off-by: Michal Wajdeczko Reviewed-by: Daniel Vetter v2: collect latest CI results Michal Wajdeczko (4): drm/i915/guc: Verify result from CTB (de)register action drm/i915/guc: Print error name on CTB (de)registration failure drm/i915/guc: Print error name on CTB send failure drm/i915/guc: Move and improve error message for missed CTB reply drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 30 ++- 1 file changed, 18 insertions(+), 12 deletions(-) -- 2.25.1
[Intel-gfx] [CI v2 3/4] drm/i915/guc: Print error name on CTB send failure
Instead of plain error value (%d) print more user friendly error
name (%pe).
Signed-off-by: Michal Wajdeczko
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
index b6c2f2a11dc5..e03f86d3b0b9 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
@@ -781,8 +781,8 @@ int intel_guc_ct_send(struct intel_guc_ct *ct, const u32
*action, u32 len,
ret = ct_send(ct, action, len, response_buf, response_buf_size,
&status);
if (unlikely(ret < 0)) {
- CT_ERROR(ct, "Sending action %#x failed (err=%d status=%#X)\n",
-action[0], ret, status);
+ CT_ERROR(ct, "Sending action %#x failed (%pe) status=%#X\n",
+action[0], ERR_PTR(ret), status);
} else if (unlikely(ret)) {
CT_DEBUG(ct, "send action %#x returned %d (%#x)\n",
action[0], ret, ret);
--
2.25.1
[Intel-gfx] [CI v2 1/4] drm/i915/guc: Verify result from CTB (de)register action
In commit b839a869dfc9 ("drm/i915/guc: Add support for data
reporting in GuC responses") we missed the hypothetical case
that GuC might return positive non-zero value as success data.
While that would be lucky treated as error case, and at the
end will result in reporting valid -EIO, in the meantime this
value will be passed to ERR_PTR that could be misleading.
v2: rebased
Reported-by: Dan Carpenter
Signed-off-by: Michal Wajdeczko
Cc: Dan Carpenter
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
index 20c710a74498..c39abb010181 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
@@ -168,12 +168,15 @@ static int guc_action_register_ct_buffer(struct intel_guc
*guc, u32 type,
FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_2_DESC_ADDR,
desc_addr),
FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_3_BUFF_ADDR,
buff_addr),
};
+ int ret;
GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type !=
GUC_CTB_TYPE_GUC2HOST);
GEM_BUG_ON(size % SZ_4K);
/* CT registration must go over MMIO */
- return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0);
+ ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0);
+
+ return ret > 0 ? -EPROTO : ret;
}
static int ct_register_buffer(struct intel_guc_ct *ct, u32 type,
@@ -201,11 +204,14 @@ static int guc_action_deregister_ct_buffer(struct
intel_guc *guc, u32 type)
FIELD_PREP(GUC_HXG_REQUEST_MSG_0_ACTION,
GUC_ACTION_HOST2GUC_DEREGISTER_CTB),
FIELD_PREP(HOST2GUC_DEREGISTER_CTB_REQUEST_MSG_1_TYPE, type),
};
+ int ret;
GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type !=
GUC_CTB_TYPE_GUC2HOST);
/* CT deregistration must go over MMIO */
- return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0);
+ ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0);
+
+ return ret > 0 ? -EPROTO : ret;
}
static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type)
--
2.25.1
[Intel-gfx] [CI v2 4/4] drm/i915/guc: Move and improve error message for missed CTB reply
If we timeout waiting for a CT reply we print very simple error
message. Improve that and by moving error reporting to the caller
we can use CT_ERROR instead of DRM_ERROR and report just fence
as error code will be reported later anyway.
Signed-off-by: Michal Wajdeczko
Reviewed-by: Daniel Vetter
---
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
index e03f86d3b0b9..0a3504bc0b61 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c
@@ -528,9 +528,6 @@ static int wait_for_ct_request_update(struct ct_request
*req, u32 *status)
err = wait_for(done, GUC_CTB_RESPONSE_TIMEOUT_LONG_MS);
#undef done
- if (unlikely(err))
- DRM_ERROR("CT: fence %u err %d\n", req->fence, err);
-
*status = req->status;
return err;
}
@@ -728,8 +725,11 @@ static int ct_send(struct intel_guc_ct *ct,
err = wait_for_ct_request_update(&request, status);
g2h_release_space(ct, GUC_CTB_HXG_MSG_MAX_LEN);
- if (unlikely(err))
+ if (unlikely(err)) {
+ CT_ERROR(ct, "No response for request %#x (fence %u)\n",
+action[0], request.fence);
goto unlink;
+ }
if (FIELD_GET(GUC_HXG_MSG_0_TYPE, *status) !=
GUC_HXG_TYPE_RESPONSE_SUCCESS) {
err = -EIO;
--
2.25.1
[Intel-gfx] [CI v2 2/4] drm/i915/guc: Print error name on CTB (de)registration failure
Instead of plain error value (%d) print more user friendly error name (%pe). Signed-off-by: Michal Wajdeczko Reviewed-by: Daniel Vetter --- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c index c39abb010181..b6c2f2a11dc5 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c @@ -191,8 +191,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, err = guc_action_register_ct_buffer(ct_to_guc(ct), type, desc_addr, buff_addr, size); if (unlikely(err)) - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", -guc_ct_buffer_type_to_str(type), err); + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", +guc_ct_buffer_type_to_str(type), ERR_PTR(err)); return err; } @@ -219,8 +219,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); if (unlikely(err)) - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", -guc_ct_buffer_type_to_str(type), err); + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", +guc_ct_buffer_type_to_str(type), ERR_PTR(err)); return err; } -- 2.25.1
[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Improve CTB error handling (rev3)
== Series Details == Series: drm/i915/guc: Improve CTB error handling (rev3) URL : https://patchwork.freedesktop.org/series/92118/ State : success == Summary == CI Bug Log - changes from CI_DRM_10644 -> Patchwork_21161 Summary --- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/index.html Known issues Here are the changes found in Patchwork_21161 that come from known issues: ### IGT changes ### Issues hit * igt@gem_huc_copy@huc-copy: - fi-tgl-1115g4: NOTRUN -> [SKIP][1] ([i915#2190]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@gem_huc_c...@huc-copy.html * igt@i915_pm_backlight@basic-brightness: - fi-tgl-1115g4: NOTRUN -> [SKIP][2] ([i915#1155]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@i915_pm_backli...@basic-brightness.html * igt@i915_pm_rpm@module-reload: - fi-tgl-1115g4: NOTRUN -> [INCOMPLETE][3] ([i915#4006] / [i915#4193]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@i915_pm_...@module-reload.html * igt@kms_addfb_basic@too-wide: - fi-tgl-1115g4: NOTRUN -> [DMESG-WARN][4] ([i915#4002]) +87 similar issues [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_addfb_ba...@too-wide.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-tgl-1115g4: NOTRUN -> [SKIP][5] ([fdo#111827]) +8 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_chamel...@common-hpd-after-suspend.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic: - fi-tgl-1115g4: NOTRUN -> [SKIP][6] ([i915#4103]) +1 similar issue [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_cursor_leg...@basic-busy-flip-before-cursor-atomic.html * igt@kms_force_connector_basic@force-load-detect: - fi-tgl-1115g4: NOTRUN -> [SKIP][7] ([fdo#109285]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_force_connector_ba...@force-load-detect.html * igt@kms_psr@primary_mmap_gtt: - fi-tgl-1115g4: NOTRUN -> [SKIP][8] ([i915#1072]) +2 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html * igt@kms_psr@primary_page_flip: - fi-tgl-1115g4: NOTRUN -> [SKIP][9] ([i915#1072] / [i915#1385]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@kms_psr@primary_page_flip.html * igt@prime_vgem@basic-userptr: - fi-tgl-1115g4: NOTRUN -> [SKIP][10] ([i915#3301]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@prime_v...@basic-userptr.html * igt@runner@aborted: - fi-tgl-1115g4: NOTRUN -> [FAIL][11] ([i915#2722]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/fi-tgl-1115g4/igt@run...@aborted.html [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#1385]: https://gitlab.freedesktop.org/drm/intel/issues/1385 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722 [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301 [i915#4002]: https://gitlab.freedesktop.org/drm/intel/issues/4002 [i915#4006]: https://gitlab.freedesktop.org/drm/intel/issues/4006 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4193]: https://gitlab.freedesktop.org/drm/intel/issues/4193 Participating hosts (38 -> 33) -- Additional (1): fi-tgl-1115g4 Missing(6): bat-dg1-6 fi-bsw-cyan bat-adlp-4 fi-kbl-7500u fi-bdw-samus bat-jsl-1 Build changes - * Linux: CI_DRM_10644 -> Patchwork_21161 CI-20190529: 20190529 CI_DRM_10644: ca294d706a72bd502f7e36b988c57fd634880b00 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6218: 8d4169d9543d8e5c01f0c746f603801a4d65ead0 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_21161: fe02b7c0835428d2b14ebbb4d5a9bea6aa9db253 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == fe02b7c08354 drm/i915/guc: Move and improve error message for missed CTB reply 4e2f5273bef4 drm/i915/guc: Print error name on CTB send failure 2bd46fb94fab drm/i915/guc: Print error name on CTB (de)registration failure 937f0903b643 drm/i915/guc: Verify result from CTB (de)register action == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/index.html
[Intel-gfx] [PATCH] drm/i915: Use fixed offset for PTEs location
We assumed that for all modern GENs the PTEs and register space are
split in the GTTMMADR BAR, but while it is true, we should rather use
fixed offset as it is defined in the specification.
Bspec: 4409, 4457, 4604, 11181, 9027, 13246, 13321, 44980
Signed-off-by: Michal Wajdeczko
Cc: CQ Tang
Cc: Matt Roper
---
drivers/gpu/drm/i915/gt/intel_ggtt.c | 19 +--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c
b/drivers/gpu/drm/i915/gt/intel_ggtt.c
index ba7c7ed89fa8..f17383e76eb7 100644
--- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
@@ -813,6 +813,21 @@ static unsigned int chv_get_total_gtt_size(u16 gmch_ctrl)
return 0;
}
+static unsigned int gen6_gttmmadr_size(struct drm_i915_private *i915)
+{
+ /*
+* GEN6: GTTMMADR size is 4MB and GTTADR starts at 2MB offset
+* GEN8: GTTMMADR size is 16MB and GTTADR starts at 8MB offset
+*/
+ GEM_BUG_ON(GRAPHICS_VER(i915) < 6);
+ return (GRAPHICS_VER(i915) < 8) ? SZ_4M : SZ_16M;
+}
+
+static unsigned int gen6_gttadr_offset(struct drm_i915_private *i915)
+{
+ return gen6_gttmmadr_size(i915) / 2;
+}
+
static int ggtt_probe_common(struct i915_ggtt *ggtt, u64 size)
{
struct drm_i915_private *i915 = ggtt->vm.i915;
@@ -821,8 +836,8 @@ static int ggtt_probe_common(struct i915_ggtt *ggtt, u64
size)
u32 pte_flags;
int ret;
- /* For Modern GENs the PTEs and register space are split in the BAR */
- phys_addr = pci_resource_start(pdev, 0) + pci_resource_len(pdev, 0) / 2;
+ GEM_WARN_ON(pci_resource_len(pdev, 0) != gen6_gttmmadr_size(i915));
+ phys_addr = pci_resource_start(pdev, 0) + gen6_gttadr_offset(i915);
/*
* On BXT+/ICL+ writes larger than 64 bit to the GTT pagetable range
--
2.25.1
[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/guc: Improve CTB error handling (rev3)
== Series Details == Series: drm/i915/guc: Improve CTB error handling (rev3) URL : https://patchwork.freedesktop.org/series/92118/ State : success == Summary == CI Bug Log - changes from CI_DRM_10644_full -> Patchwork_21161_full Summary --- **SUCCESS** No regressions found. Known issues Here are the changes found in Patchwork_21161_full that come from known issues: ### IGT changes ### Issues hit * igt@gem_ctx_exec@basic-close-race: - shard-iclb: [PASS][1] -> [INCOMPLETE][2] ([i915#1895]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-iclb8/igt@gem_ctx_e...@basic-close-race.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-iclb2/igt@gem_ctx_e...@basic-close-race.html * igt@gem_ctx_isolation@preservation-s3@rcs0: - shard-tglb: [PASS][3] -> [INCOMPLETE][4] ([i915#1373]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb8/igt@gem_ctx_isolation@preservation...@rcs0.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-tglb7/igt@gem_ctx_isolation@preservation...@rcs0.html * igt@gem_eio@unwedge-stress: - shard-tglb: [PASS][5] -> [TIMEOUT][6] ([i915#2369] / [i915#3063] / [i915#3648]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb7/igt@gem_...@unwedge-stress.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-tglb3/igt@gem_...@unwedge-stress.html * igt@gem_exec_fair@basic-deadline: - shard-apl: NOTRUN -> [FAIL][7] ([i915#2846]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-apl3/igt@gem_exec_f...@basic-deadline.html * igt@gem_exec_fair@basic-none@vcs1: - shard-iclb: NOTRUN -> [FAIL][8] ([i915#2842]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-iclb2/igt@gem_exec_fair@basic-n...@vcs1.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [PASS][9] -> [FAIL][10] ([i915#2842]) +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-glk4/igt@gem_exec_fair@basic-pace-s...@rcs0.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-glk9/igt@gem_exec_fair@basic-pace-s...@rcs0.html - shard-kbl: [PASS][11] -> [FAIL][12] ([i915#2842]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-kbl3/igt@gem_exec_fair@basic-pace-s...@rcs0.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-kbl7/igt@gem_exec_fair@basic-pace-s...@rcs0.html * igt@gem_huc_copy@huc-copy: - shard-tglb: [PASS][13] -> [SKIP][14] ([i915#2190]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb3/igt@gem_huc_c...@huc-copy.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-tglb7/igt@gem_huc_c...@huc-copy.html * igt@gem_userptr_blits@dmabuf-sync: - shard-apl: NOTRUN -> [SKIP][15] ([fdo#109271] / [i915#3323]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-apl3/igt@gem_userptr_bl...@dmabuf-sync.html * igt@gen7_exec_parse@basic-offset: - shard-apl: NOTRUN -> [SKIP][16] ([fdo#109271]) +299 similar issues [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-apl7/igt@gen7_exec_pa...@basic-offset.html * igt@i915_pm_dc@dc3co-vpb-simulation: - shard-kbl: NOTRUN -> [SKIP][17] ([fdo#109271] / [i915#658]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-kbl1/igt@i915_pm...@dc3co-vpb-simulation.html * igt@i915_pm_dc@dc6-psr: - shard-skl: [PASS][18] -> [FAIL][19] ([i915#454]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-skl6/igt@i915_pm...@dc6-psr.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-skl2/igt@i915_pm...@dc6-psr.html * igt@i915_selftest@live@gt_pm: - shard-skl: NOTRUN -> [DMESG-FAIL][20] ([i915#1886] / [i915#2291]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-skl1/igt@i915_selftest@live@gt_pm.html * igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-hflip: - shard-apl: NOTRUN -> [SKIP][21] ([fdo#109271] / [i915#3777]) +3 similar issues [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-apl2/igt@kms_big...@x-tiled-max-hw-stride-32bpp-rotate-180-hflip.html * igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc: - shard-skl: NOTRUN -> [SKIP][22] ([fdo#109271] / [i915#3886]) +1 similar issue [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21161/shard-skl1/igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html * igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc: - shard-apl: NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#3886]) +17 similar issues [23]: https://
[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Use fixed offset for PTEs location
== Series Details == Series: drm/i915: Use fixed offset for PTEs location URL : https://patchwork.freedesktop.org/series/95074/ State : success == Summary == CI Bug Log - changes from CI_DRM_10644 -> Patchwork_21162 Summary --- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/index.html Known issues Here are the changes found in Patchwork_21162 that come from known issues: ### IGT changes ### Issues hit * igt@core_hotunplug@unbind-rebind: - fi-tgl-1115g4: NOTRUN -> [DMESG-WARN][1] ([i915#1982] / [i915#4002]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@core_hotunp...@unbind-rebind.html * igt@gem_huc_copy@huc-copy: - fi-tgl-1115g4: NOTRUN -> [SKIP][2] ([i915#2190]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@gem_huc_c...@huc-copy.html * igt@i915_pm_backlight@basic-brightness: - fi-tgl-1115g4: NOTRUN -> [SKIP][3] ([i915#1155]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@i915_pm_backli...@basic-brightness.html * igt@i915_pm_rpm@module-reload: - fi-tgl-1115g4: NOTRUN -> [INCOMPLETE][4] ([i915#4006] / [i915#4193]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@i915_pm_...@module-reload.html - fi-kbl-r: [PASS][5] -> [INCOMPLETE][6] ([i915#151] / [i915#4193] / [i915#92]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/fi-kbl-r/igt@i915_pm_...@module-reload.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-kbl-r/igt@i915_pm_...@module-reload.html * igt@kms_addfb_basic@too-wide: - fi-tgl-1115g4: NOTRUN -> [DMESG-WARN][7] ([i915#4002]) +88 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_addfb_ba...@too-wide.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-tgl-1115g4: NOTRUN -> [SKIP][8] ([fdo#111827]) +8 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_chamel...@common-hpd-after-suspend.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic: - fi-tgl-1115g4: NOTRUN -> [SKIP][9] ([i915#4103]) +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_cursor_leg...@basic-busy-flip-before-cursor-atomic.html * igt@kms_flip@basic-plain-flip@c-dp1: - fi-cfl-8109u: [PASS][10] -> [FAIL][11] ([i915#4165]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/fi-cfl-8109u/igt@kms_flip@basic-plain-f...@c-dp1.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-cfl-8109u/igt@kms_flip@basic-plain-f...@c-dp1.html * igt@kms_force_connector_basic@force-load-detect: - fi-tgl-1115g4: NOTRUN -> [SKIP][12] ([fdo#109285]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_force_connector_ba...@force-load-detect.html * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b: - fi-cfl-8109u: [PASS][13] -> [DMESG-WARN][14] ([i915#295]) +14 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/fi-cfl-8109u/igt@kms_pipe_crc_ba...@compare-crc-sanitycheck-pipe-b.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-cfl-8109u/igt@kms_pipe_crc_ba...@compare-crc-sanitycheck-pipe-b.html * igt@kms_psr@primary_mmap_gtt: - fi-tgl-1115g4: NOTRUN -> [SKIP][15] ([i915#1072]) +2 similar issues [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_psr@primary_mmap_gtt.html * igt@kms_psr@primary_page_flip: - fi-tgl-1115g4: NOTRUN -> [SKIP][16] ([i915#1072] / [i915#1385]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@kms_psr@primary_page_flip.html * igt@prime_vgem@basic-userptr: - fi-tgl-1115g4: NOTRUN -> [SKIP][17] ([i915#3301]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@prime_v...@basic-userptr.html * igt@runner@aborted: - fi-kbl-r: NOTRUN -> [FAIL][18] ([fdo#109271] / [i915#1814] / [i915#2722] / [i915#3363]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-kbl-r/igt@run...@aborted.html - fi-tgl-1115g4: NOTRUN -> [FAIL][19] ([i915#2722]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/fi-tgl-1115g4/igt@run...@aborted.html [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [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#1385]: ht
[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Use fixed offset for PTEs location
== Series Details == Series: drm/i915: Use fixed offset for PTEs location URL : https://patchwork.freedesktop.org/series/95074/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10644_full -> Patchwork_21162_full Summary --- **FAILURE** Serious unknown changes coming with Patchwork_21162_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_21162_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_21162_full: ### IGT changes ### Possible regressions * igt@i915_selftest@live@gem_migrate: - shard-skl: NOTRUN -> [INCOMPLETE][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-skl8/igt@i915_selftest@live@gem_migrate.html Known issues Here are the changes found in Patchwork_21162_full that come from known issues: ### IGT changes ### Issues hit * igt@gem_create@create-massive: - shard-snb: NOTRUN -> [DMESG-WARN][2] ([i915#3002]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-snb5/igt@gem_cre...@create-massive.html * igt@gem_ctx_persistence@legacy-engines-mixed: - shard-snb: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099]) +2 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-snb5/igt@gem_ctx_persiste...@legacy-engines-mixed.html * igt@gem_eio@unwedge-stress: - shard-tglb: [PASS][4] -> [TIMEOUT][5] ([i915#2369] / [i915#3063] / [i915#3648]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb7/igt@gem_...@unwedge-stress.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-tglb3/igt@gem_...@unwedge-stress.html * igt@gem_exec_endless@dispatch@vecs0: - shard-iclb: [PASS][6] -> [INCOMPLETE][7] ([i915#3778]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-iclb3/igt@gem_exec_endless@dispa...@vecs0.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-iclb3/igt@gem_exec_endless@dispa...@vecs0.html * igt@gem_exec_fair@basic-flow@rcs0: - shard-tglb: [PASS][8] -> [FAIL][9] ([i915#2842]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb7/igt@gem_exec_fair@basic-f...@rcs0.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-tglb7/igt@gem_exec_fair@basic-f...@rcs0.html * igt@gem_exec_fair@basic-none@vcs0: - shard-kbl: NOTRUN -> [FAIL][10] ([i915#2842]) +2 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-kbl1/igt@gem_exec_fair@basic-n...@vcs0.html * igt@gem_exec_fair@basic-none@vcs1: - shard-iclb: NOTRUN -> [FAIL][11] ([i915#2842]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-iclb4/igt@gem_exec_fair@basic-n...@vcs1.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [PASS][12] -> [FAIL][13] ([i915#2842]) +2 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-glk4/igt@gem_exec_fair@basic-pace-s...@rcs0.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-glk9/igt@gem_exec_fair@basic-pace-s...@rcs0.html * igt@gem_exec_suspend@basic-s0: - shard-tglb: [PASS][14] -> [INCOMPLETE][15] ([i915#456]) +1 similar issue [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-tglb3/igt@gem_exec_susp...@basic-s0.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-tglb7/igt@gem_exec_susp...@basic-s0.html * igt@gem_pwrite@basic-exhaustion: - shard-snb: NOTRUN -> [WARN][16] ([i915#2658]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-snb5/igt@gem_pwr...@basic-exhaustion.html * igt@gen7_exec_parse@basic-offset: - shard-apl: NOTRUN -> [SKIP][17] ([fdo#109271]) +204 similar issues [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-apl6/igt@gen7_exec_pa...@basic-offset.html * igt@i915_pm_dc@dc3co-vpb-simulation: - shard-kbl: NOTRUN -> [SKIP][18] ([fdo#109271] / [i915#658]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-kbl1/igt@i915_pm...@dc3co-vpb-simulation.html * igt@i915_selftest@live@gt_pm: - shard-skl: NOTRUN -> [DMESG-FAIL][19] ([i915#1886] / [i915#2291]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21162/shard-skl8/igt@i915_selftest@live@gt_pm.html * igt@i915_suspend@forcewake: - shard-kbl: [PASS][20] -> [DMESG-WARN][21] ([i915#180]) +2 similar issues [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10644/shard-k
[Intel-gfx] linux-next: build warning after merge of the drm-misc tree
Hi all,
After merging the drm-misc tree, today's linux-next build (htmldocs)
produced these warnings:
include/drm/drm_edid.h:530: warning: Function parameter or member 'vend_chr_1'
not described in 'drm_edid_encode_panel_id'
include/drm/drm_edid.h:530: warning: Excess function parameter 'vend_chr_3'
description in 'drm_edid_encode_panel_id'
Introduced by commit
7d1be0a09fa6 ("drm/edid: Fix EDID quirk compile error on older compilers")
--
Cheers,
Stephen Rothwell
pgpvLFaE1h1O5.pgp
Description: OpenPGP digital signature
