* [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 @ 2022-10-19 14:21 Gustavo Sousa 2022-10-19 15:48 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork ` (2 more replies) 0 siblings, 3 replies; 5+ messages in thread From: Gustavo Sousa @ 2022-10-19 14:21 UTC (permalink / raw) To: intel-gfx Workaround to be applied to platforms using XE_LP graphics. BSpec: 52890 Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com> --- drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + drivers/gpu/drm/i915/gt/intel_workarounds.c | 7 +++++++ 2 files changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h b/drivers/gpu/drm/i915/gt/intel_gt_regs.h index 36d95b79022c..23844ba7e824 100644 --- a/drivers/gpu/drm/i915/gt/intel_gt_regs.h +++ b/drivers/gpu/drm/i915/gt/intel_gt_regs.h @@ -443,6 +443,7 @@ #define HIZ_CHICKEN _MMIO(0x7018) #define CHV_HZ_8X8_MODE_IN_1X REG_BIT(15) #define DG1_HZ_READ_SUPPRESSION_OPTIMIZATION_DISABLE REG_BIT(14) +#define HZ_DEPTH_TEST_LE_GE_OPT_DISABLE REG_BIT(13) #define BDW_HIZ_POWER_COMPILER_CLOCK_GATING_DISABLE REG_BIT(3) #define GEN8_L3CNTLREG _MMIO(0x7034) diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c index 63e1e6becf34..5cdec699ae04 100644 --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c @@ -660,6 +660,8 @@ static void gen12_ctx_gt_tuning_init(struct intel_engine_cs *engine, static void gen12_ctx_workarounds_init(struct intel_engine_cs *engine, struct i915_wa_list *wal) { + struct drm_i915_private *i915 = engine->i915; + gen12_ctx_gt_tuning_init(engine, wal); /* @@ -693,6 +695,11 @@ static void gen12_ctx_workarounds_init(struct intel_engine_cs *engine, FF_MODE2_GS_TIMER_MASK, FF_MODE2_GS_TIMER_224, 0, false); + + if (!IS_DG1(i915)) + /* Wa_1806527549 */ + wa_masked_en(wal, HIZ_CHICKEN, HZ_DEPTH_TEST_LE_GE_OPT_DISABLE); + } static void dg1_ctx_workarounds_init(struct intel_engine_cs *engine, -- 2.38.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/xelp: Add Wa_1806527549 2022-10-19 14:21 [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 Gustavo Sousa @ 2022-10-19 15:48 ` Patchwork 2022-10-19 16:09 ` [Intel-gfx] [PATCH] " Gustavo Sousa 2022-10-21 4:03 ` Summers, Stuart 2 siblings, 0 replies; 5+ messages in thread From: Patchwork @ 2022-10-19 15:48 UTC (permalink / raw) To: Gustavo Sousa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 13556 bytes --] == Series Details == Series: drm/i915/xelp: Add Wa_1806527549 URL : https://patchwork.freedesktop.org/series/109885/ State : failure == Summary == CI Bug Log - changes from CI_DRM_12261 -> Patchwork_109885v1 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_109885v1 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_109885v1, 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_109885v1/index.html Participating hosts (42 -> 42) ------------------------------ Additional (3): fi-rkl-11600 fi-icl-u2 fi-tgl-dsi Missing (3): fi-kbl-soraka fi-hsw-4770 bat-adlp-6 Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_109885v1: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live@gem_contexts: - fi-kbl-7567u: [PASS][1] -> [DMESG-FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-kbl-7567u/igt@i915_selftest@live@gem_contexts.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-kbl-7567u/igt@i915_selftest@live@gem_contexts.html Known issues ------------ Here are the changes found in Patchwork_109885v1 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_suspend@basic-s3@smem: - fi-bdw-5557u: NOTRUN -> [INCOMPLETE][3] ([i915#146]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html * igt@gem_huc_copy@huc-copy: - fi-icl-u2: NOTRUN -> [SKIP][4] ([i915#2190]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-icl-u2/igt@gem_huc_copy@huc-copy.html - fi-rkl-11600: NOTRUN -> [SKIP][5] ([i915#2190]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@basic: - fi-rkl-11600: NOTRUN -> [SKIP][6] ([i915#4613]) +3 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@gem_lmem_swapping@basic.html * igt@gem_tiled_pread_basic: - fi-rkl-11600: NOTRUN -> [SKIP][7] ([i915#3282]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@gem_tiled_pread_basic.html * igt@i915_pm_backlight@basic-brightness: - fi-rkl-11600: NOTRUN -> [SKIP][8] ([i915#3012]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@i915_pm_backlight@basic-brightness.html * igt@i915_suspend@basic-s3-without-i915: - fi-rkl-11600: NOTRUN -> [INCOMPLETE][9] ([i915#5982]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-ivb-3770: NOTRUN -> [SKIP][10] ([fdo#109271] / [fdo#111827]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-ivb-3770/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_chamelium@hdmi-edid-read: - fi-rkl-11600: NOTRUN -> [SKIP][11] ([fdo#111827]) +7 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@kms_chamelium@hdmi-edid-read.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-icl-u2: NOTRUN -> [SKIP][12] ([fdo#111827]) +7 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor: - fi-rkl-11600: NOTRUN -> [SKIP][13] ([i915#4103]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html - fi-icl-u2: NOTRUN -> [SKIP][14] ([i915#4103]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html * igt@kms_flip@basic-flip-vs-modeset@a-edp1: - fi-icl-u2: NOTRUN -> [DMESG-WARN][15] ([i915#4890]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-icl-u2/igt@kms_flip@basic-flip-vs-modeset@a-edp1.html * igt@kms_force_connector_basic@force-load-detect: - fi-rkl-11600: NOTRUN -> [SKIP][16] ([fdo#109285] / [i915#4098]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@kms_force_connector_basic@force-load-detect.html * igt@kms_psr@primary_page_flip: - fi-rkl-11600: NOTRUN -> [SKIP][17] ([i915#1072]) +3 similar issues [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@kms_psr@primary_page_flip.html * igt@kms_setmode@basic-clone-single-crtc: - fi-rkl-11600: NOTRUN -> [SKIP][18] ([i915#3555] / [i915#4098]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@kms_setmode@basic-clone-single-crtc.html * igt@prime_vgem@basic-read: - fi-rkl-11600: NOTRUN -> [SKIP][19] ([fdo#109295] / [i915#3291] / [i915#3708]) +2 similar issues [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@prime_vgem@basic-read.html * igt@prime_vgem@basic-userptr: - fi-rkl-11600: NOTRUN -> [SKIP][20] ([fdo#109295] / [i915#3301] / [i915#3708]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-rkl-11600/igt@prime_vgem@basic-userptr.html * igt@runner@aborted: - fi-icl-u2: NOTRUN -> [FAIL][21] ([i915#4312]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-icl-u2/igt@runner@aborted.html #### Possible fixes #### * igt@gem_exec_gttfill@basic: - fi-pnv-d510: [SKIP][22] ([fdo#109271]) -> [PASS][23] +1 similar issue [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-pnv-d510/igt@gem_exec_gttfill@basic.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-pnv-d510/igt@gem_exec_gttfill@basic.html * igt@gem_exec_suspend@basic-s0@smem: - {bat-adlm-1}: [DMESG-WARN][24] ([i915#2867]) -> [PASS][25] +1 similar issue [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-adlm-1/igt@gem_exec_suspend@basic-s0@smem.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-adlm-1/igt@gem_exec_suspend@basic-s0@smem.html * igt@gem_huc_copy@huc-copy: - {bat-dg2-8}: [FAIL][26] ([i915#7029]) -> [PASS][27] [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-dg2-8/igt@gem_huc_copy@huc-copy.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-dg2-8/igt@gem_huc_copy@huc-copy.html * igt@i915_pm_rpm@basic-rte: - {bat-rplp-1}: [DMESG-WARN][28] -> [PASS][29] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-rplp-1/igt@i915_pm_rpm@basic-rte.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-rplp-1/igt@i915_pm_rpm@basic-rte.html * igt@i915_pm_rpm@module-reload: - {bat-rpls-2}: [DMESG-WARN][30] ([i915#5537]) -> [PASS][31] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-rpls-2/igt@i915_pm_rpm@module-reload.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-rpls-2/igt@i915_pm_rpm@module-reload.html * igt@i915_selftest@live@gem_contexts: - fi-glk-j4005: [DMESG-FAIL][32] ([i915#7270]) -> [PASS][33] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-glk-j4005/igt@i915_selftest@live@gem_contexts.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-glk-j4005/igt@i915_selftest@live@gem_contexts.html - fi-skl-guc: [DMESG-FAIL][34] ([i915#7270]) -> [PASS][35] [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-skl-guc/igt@i915_selftest@live@gem_contexts.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-skl-guc/igt@i915_selftest@live@gem_contexts.html - {bat-jsl-1}: [DMESG-FAIL][36] ([i915#7270]) -> [PASS][37] [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-jsl-1/igt@i915_selftest@live@gem_contexts.html [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-jsl-1/igt@i915_selftest@live@gem_contexts.html - {fi-jsl-1}: [DMESG-FAIL][38] ([i915#7270]) -> [PASS][39] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-jsl-1/igt@i915_selftest@live@gem_contexts.html [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-jsl-1/igt@i915_selftest@live@gem_contexts.html * igt@i915_selftest@live@gt_heartbeat: - {bat-jsl-1}: [DMESG-FAIL][40] -> [PASS][41] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-jsl-1/igt@i915_selftest@live@gt_heartbeat.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-jsl-1/igt@i915_selftest@live@gt_heartbeat.html * igt@i915_selftest@live@hangcheck: - fi-ivb-3770: [INCOMPLETE][42] ([i915#3303] / [i915#7122]) -> [PASS][43] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-ivb-3770/igt@i915_selftest@live@hangcheck.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-ivb-3770/igt@i915_selftest@live@hangcheck.html * igt@i915_suspend@basic-s3-without-i915: - fi-bdw-5557u: [INCOMPLETE][44] ([i915#146] / [i915#6712]) -> [PASS][45] [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/fi-bdw-5557u/igt@i915_suspend@basic-s3-without-i915.html [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/fi-bdw-5557u/igt@i915_suspend@basic-s3-without-i915.html - {bat-rpls-2}: [DMESG-WARN][46] ([i915#6434]) -> [PASS][47] [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12261/bat-rpls-2/igt@i915_suspend@basic-s3-without-i915.html [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/bat-rpls-2/igt@i915_suspend@basic-s3-without-i915.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#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146 [i915#1759]: https://gitlab.freedesktop.org/drm/intel/issues/1759 [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2867]: https://gitlab.freedesktop.org/drm/intel/issues/2867 [i915#3012]: https://gitlab.freedesktop.org/drm/intel/issues/3012 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291 [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301 [i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4890]: https://gitlab.freedesktop.org/drm/intel/issues/4890 [i915#5537]: https://gitlab.freedesktop.org/drm/intel/issues/5537 [i915#5982]: https://gitlab.freedesktop.org/drm/intel/issues/5982 [i915#6434]: https://gitlab.freedesktop.org/drm/intel/issues/6434 [i915#6480]: https://gitlab.freedesktop.org/drm/intel/issues/6480 [i915#6559]: https://gitlab.freedesktop.org/drm/intel/issues/6559 [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621 [i915#6712]: https://gitlab.freedesktop.org/drm/intel/issues/6712 [i915#6794]: https://gitlab.freedesktop.org/drm/intel/issues/6794 [i915#6818]: https://gitlab.freedesktop.org/drm/intel/issues/6818 [i915#6939]: https://gitlab.freedesktop.org/drm/intel/issues/6939 [i915#7029]: https://gitlab.freedesktop.org/drm/intel/issues/7029 [i915#7122]: https://gitlab.freedesktop.org/drm/intel/issues/7122 [i915#7269]: https://gitlab.freedesktop.org/drm/intel/issues/7269 [i915#7270]: https://gitlab.freedesktop.org/drm/intel/issues/7270 Build changes ------------- * Linux: CI_DRM_12261 -> Patchwork_109885v1 CI-20190529: 20190529 CI_DRM_12261: 41447224fdfbfbfd1e9ffa5fabc9d277f9c02f8a @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7019: fdbafce2b74e84739bb1d81223ae6f01fb442980 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_109885v1: 41447224fdfbfbfd1e9ffa5fabc9d277f9c02f8a @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 8a620d05ef82 drm/i915/xelp: Add Wa_1806527549 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_109885v1/index.html [-- Attachment #2: Type: text/html, Size: 15012 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 2022-10-19 14:21 [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 Gustavo Sousa 2022-10-19 15:48 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork @ 2022-10-19 16:09 ` Gustavo Sousa 2022-10-21 4:03 ` Summers, Stuart 2 siblings, 0 replies; 5+ messages in thread From: Gustavo Sousa @ 2022-10-19 16:09 UTC (permalink / raw) To: intel-gfx On Wed, Oct 19, 2022 at 11:21:45AM -0300, Gustavo Sousa wrote: > Workaround to be applied to platforms using XE_LP graphics. > > BSpec: 52890 > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com> > --- > drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + > drivers/gpu/drm/i915/gt/intel_workarounds.c | 7 +++++++ > 2 files changed, 8 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h b/drivers/gpu/drm/i915/gt/intel_gt_regs.h > index 36d95b79022c..23844ba7e824 100644 > --- a/drivers/gpu/drm/i915/gt/intel_gt_regs.h > +++ b/drivers/gpu/drm/i915/gt/intel_gt_regs.h > @@ -443,6 +443,7 @@ > #define HIZ_CHICKEN _MMIO(0x7018) > #define CHV_HZ_8X8_MODE_IN_1X REG_BIT(15) > #define DG1_HZ_READ_SUPPRESSION_OPTIMIZATION_DISABLE REG_BIT(14) > +#define HZ_DEPTH_TEST_LE_GE_OPT_DISABLE REG_BIT(13) > #define BDW_HIZ_POWER_COMPILER_CLOCK_GATING_DISABLE REG_BIT(3) > > #define GEN8_L3CNTLREG _MMIO(0x7034) > diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c > index 63e1e6becf34..5cdec699ae04 100644 > --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c > +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c > @@ -660,6 +660,8 @@ static void gen12_ctx_gt_tuning_init(struct intel_engine_cs *engine, > static void gen12_ctx_workarounds_init(struct intel_engine_cs *engine, > struct i915_wa_list *wal) > { > + struct drm_i915_private *i915 = engine->i915; > + > gen12_ctx_gt_tuning_init(engine, wal); > > /* > @@ -693,6 +695,11 @@ static void gen12_ctx_workarounds_init(struct intel_engine_cs *engine, > FF_MODE2_GS_TIMER_MASK, > FF_MODE2_GS_TIMER_224, > 0, false); > + > + if (!IS_DG1(i915)) > + /* Wa_1806527549 */ > + wa_masked_en(wal, HIZ_CHICKEN, HZ_DEPTH_TEST_LE_GE_OPT_DISABLE); > + Just realized this extra blank line... I will sent a v2 without it. > } > > static void dg1_ctx_workarounds_init(struct intel_engine_cs *engine, > -- > 2.38.0 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 2022-10-19 14:21 [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 Gustavo Sousa 2022-10-19 15:48 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork 2022-10-19 16:09 ` [Intel-gfx] [PATCH] " Gustavo Sousa @ 2022-10-21 4:03 ` Summers, Stuart 2022-10-21 5:28 ` Lucas De Marchi 2 siblings, 1 reply; 5+ messages in thread From: Summers, Stuart @ 2022-10-21 4:03 UTC (permalink / raw) To: Sousa, Gustavo, intel-gfx On Wed, 2022-10-19 at 11:21 -0300, Gustavo Sousa wrote: > Workaround to be applied to platforms using XE_LP graphics. > > BSpec: 52890 > Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com> > --- > drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + > drivers/gpu/drm/i915/gt/intel_workarounds.c | 7 +++++++ > 2 files changed, 8 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h > b/drivers/gpu/drm/i915/gt/intel_gt_regs.h > index 36d95b79022c..23844ba7e824 100644 > --- a/drivers/gpu/drm/i915/gt/intel_gt_regs.h > +++ b/drivers/gpu/drm/i915/gt/intel_gt_regs.h > @@ -443,6 +443,7 @@ > #define HIZ_CHICKEN _MMIO(0x7018) > #define CHV_HZ_8X8_MODE_IN_1X REG_BIT(15) > #define DG1_HZ_READ_SUPPRESSION_OPTIMIZATION_DISABLE REG_BIT > (14) > +#define HZ_DEPTH_TEST_LE_GE_OPT_DISABLE REG_BIT(13) > #define BDW_HIZ_POWER_COMPILER_CLOCK_GATING_DISABLE REG_BIT > (3) > > #define GEN8_L3CNTLREG _MMIO(0x7034) > diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c > b/drivers/gpu/drm/i915/gt/intel_workarounds.c > index 63e1e6becf34..5cdec699ae04 100644 > --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c > +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c > @@ -660,6 +660,8 @@ static void gen12_ctx_gt_tuning_init(struct > intel_engine_cs *engine, > static void gen12_ctx_workarounds_init(struct intel_engine_cs > *engine, > struct i915_wa_list *wal) > { > + struct drm_i915_private *i915 = engine->i915; > + > gen12_ctx_gt_tuning_init(engine, wal); > > /* > @@ -693,6 +695,11 @@ static void gen12_ctx_workarounds_init(struct > intel_engine_cs *engine, > FF_MODE2_GS_TIMER_MASK, > FF_MODE2_GS_TIMER_224, > 0, false); > + > + if (!IS_DG1(i915)) My reading of the spec here is that we should be applying this to DG1 as well. Why are you removing it there? Also, it looks like this is conditional on a specific depth buffer, do we need to take that into account here? Thanks, Stuart > + /* Wa_1806527549 */ > + wa_masked_en(wal, HIZ_CHICKEN, > HZ_DEPTH_TEST_LE_GE_OPT_DISABLE); > + > } > > static void dg1_ctx_workarounds_init(struct intel_engine_cs *engine, ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 2022-10-21 4:03 ` Summers, Stuart @ 2022-10-21 5:28 ` Lucas De Marchi 0 siblings, 0 replies; 5+ messages in thread From: Lucas De Marchi @ 2022-10-21 5:28 UTC (permalink / raw) To: Summers, Stuart; +Cc: intel-gfx On Fri, Oct 21, 2022 at 04:03:20AM +0000, Stuart Summers wrote: >On Wed, 2022-10-19 at 11:21 -0300, Gustavo Sousa wrote: >> Workaround to be applied to platforms using XE_LP graphics. >> >> BSpec: 52890 >> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com> >> --- >> drivers/gpu/drm/i915/gt/intel_gt_regs.h | 1 + >> drivers/gpu/drm/i915/gt/intel_workarounds.c | 7 +++++++ >> 2 files changed, 8 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h >> b/drivers/gpu/drm/i915/gt/intel_gt_regs.h >> index 36d95b79022c..23844ba7e824 100644 >> --- a/drivers/gpu/drm/i915/gt/intel_gt_regs.h >> +++ b/drivers/gpu/drm/i915/gt/intel_gt_regs.h >> @@ -443,6 +443,7 @@ >> #define HIZ_CHICKEN _MMIO(0x7018) >> #define CHV_HZ_8X8_MODE_IN_1X REG_BIT(15) >> #define DG1_HZ_READ_SUPPRESSION_OPTIMIZATION_DISABLE REG_BIT >> (14) >> +#define HZ_DEPTH_TEST_LE_GE_OPT_DISABLE REG_BIT(13) >> #define BDW_HIZ_POWER_COMPILER_CLOCK_GATING_DISABLE REG_BIT >> (3) >> >> #define GEN8_L3CNTLREG _MMIO(0x7034) >> diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c >> b/drivers/gpu/drm/i915/gt/intel_workarounds.c >> index 63e1e6becf34..5cdec699ae04 100644 >> --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c >> +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c >> @@ -660,6 +660,8 @@ static void gen12_ctx_gt_tuning_init(struct >> intel_engine_cs *engine, >> static void gen12_ctx_workarounds_init(struct intel_engine_cs >> *engine, >> struct i915_wa_list *wal) >> { >> + struct drm_i915_private *i915 = engine->i915; >> + >> gen12_ctx_gt_tuning_init(engine, wal); >> >> /* >> @@ -693,6 +695,11 @@ static void gen12_ctx_workarounds_init(struct >> intel_engine_cs *engine, >> FF_MODE2_GS_TIMER_MASK, >> FF_MODE2_GS_TIMER_224, >> 0, false); >> + >> + if (!IS_DG1(i915)) > >My reading of the spec here is that we should be applying this to DG1 >as well. Why are you removing it there? for me DG1 is marked as "no impact" >Also, it looks like this is conditional on a specific depth buffer, do >we need to take that into account here? it was agreed with mesa to do this unconditionally, so it can be done in i915 rather than in mesa as it doesn't appear to have side effects. Lucas De Marchi > >Thanks, >Stuart > >> + /* Wa_1806527549 */ >> + wa_masked_en(wal, HIZ_CHICKEN, >> HZ_DEPTH_TEST_LE_GE_OPT_DISABLE); >> + >> } >> >> static void dg1_ctx_workarounds_init(struct intel_engine_cs *engine, ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-10-21 5:29 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-10-19 14:21 [Intel-gfx] [PATCH] drm/i915/xelp: Add Wa_1806527549 Gustavo Sousa 2022-10-19 15:48 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for " Patchwork 2022-10-19 16:09 ` [Intel-gfx] [PATCH] " Gustavo Sousa 2022-10-21 4:03 ` Summers, Stuart 2022-10-21 5:28 ` Lucas De Marchi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.