* [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
@ 2024-04-19 21:26 Jonathan Cavitt
2024-04-19 22:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Jonathan Cavitt @ 2024-04-19 21:26 UTC (permalink / raw
To: intel-gfx
Cc: saurabhg.gupta, jonathan.cavitt, John.C.Harrison, chris.p.wilson,
andi.shyti, nirmoy.das
In the case where lmem_size < dsm_base, hardware is reporting that
stolen lmem is unusable. In this case, instead of throwing a warning,
we can continue execution as normal by disabling stolen LMEM support.
For example, this change will allow the following error report from
ATS-M to no longer apply:
<6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
<6> [144.859900] pcieport 0000:4b:00.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
<6> [144.859917] pcieport 0000:4c:01.0: PCI bridge to [bus 4d-4e]
<6> [144.859932] pcieport 0000:4c:01.0: bridge window [mem 0xb1000000-0xb11fffff]
<6> [144.859945] pcieport 0000:4c:01.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
<6> [144.859984] i915 0000:4d:00.0: [drm] BAR2 resized to 256M
<6> [144.860640] i915 0000:4d:00.0: [drm] Using a reduced BAR size of 256MiB. Consider enabling 'Resizable BAR' or similar, if available in the BIOS.
<4> [144.860719] -----------[ cut here ]-----------
<4> [144.860727] WARNING: CPU: 17 PID: 1815 at drivers/gpu/drm/i915/gem/i915_gem_stolen.c:939 i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
<4> [144.861430] Modules linked in: i915 snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm vgem drm_shmem_helper prime_numbers i2c_algo_bit ttm video drm_display_helper drm_buddy fuse x86_pkg_temp_thermal coretemp kvm_intel kvm ixgbe mdio irqbypass ptp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pps_core i2c_i801 mei_me i2c_smbus mei wmi acpi_power_meter [last unloaded: i915]
<4> [144.861611] CPU: 17 PID: 1815 Comm: i915_module_loa Tainted: G U 6.8.0-rc5-drmtip_1515-g78f49af27723+ #1
<4> [144.861624] Hardware name: Intel Corporation WHITLEY/WHITLEY, BIOS SE5C6200.86B.0020.P41.2109300305 09/30/2021
<4> [144.861632] RIP: 0010:i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
<4> [144.862287] Code: ff 41 c1 e4 05 e9 ac fe ff ff 4d 63 e4 48 89 ef 48 85 ed 74 04 48 8b 7d 08 48 c7 c6 10 a3 7b a0 e8 e9 90 43 e1 e9 ee fd ff ff <0f> 0b 49 c7 c4 ed ff ff ff e9 e0 fd ff ff 0f b7 d2 48 c7 c6 00 d9
<4> [144.862299] RSP: 0018:ffffc90005607980 EFLAGS: 00010207
<4> [144.862315] RAX: fffffffffff00000 RBX: 0000000000000003 RCX: 0000000000000000
Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
Suggested-by: Chris Wilson <chris.p.wilson@linux.intel.com>
Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
index ad6dd7f3259bc..efa632a9e61c6 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
@@ -936,8 +936,12 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
} else {
/* Use DSM base address instead for stolen memory */
dsm_base = intel_uncore_read64(uncore, GEN6_DSMBASE) & GEN11_BDSM_MASK;
- if (WARN_ON(lmem_size < dsm_base))
+ if (lmem_size < dsm_base) {
+ drm_dbg(&i915->drm,
+ "Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
+ lmem_size, dsm_base);
return ERR_PTR(-ENODEV);
+ }
dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-19 21:26 [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning Jonathan Cavitt
@ 2024-04-19 22:09 ` Patchwork
2024-04-19 22:19 ` ✗ Fi.CI.BAT: failure " Patchwork
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2024-04-19 22:09 UTC (permalink / raw
To: Jonathan Cavitt; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/gem: Downgrade stolen lmem setup warning
URL : https://patchwork.freedesktop.org/series/132663/
State : warning
== Summary ==
Error: dim checkpatch failed
dc5960ccb539 drm/i915/gem: Downgrade stolen lmem setup warning
-:12: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#12:
<6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
-:29: WARNING:COMMIT_LOG_USE_LINK: Unknown link reference 'Fixes:', use 'Link:' or 'Closes:' instead
#29:
Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
total: 0 errors, 2 warnings, 0 checks, 13 lines checked
^ permalink raw reply [flat|nested] 7+ messages in thread
* ✗ Fi.CI.BAT: failure for drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-19 21:26 [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning Jonathan Cavitt
2024-04-19 22:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
@ 2024-04-19 22:19 ` Patchwork
2024-04-22 6:29 ` [PATCH] " Vivekanandan, Balasubramani
2024-04-22 8:28 ` Jani Nikula
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2024-04-19 22:19 UTC (permalink / raw
To: Jonathan Cavitt; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 9966 bytes --]
== Series Details ==
Series: drm/i915/gem: Downgrade stolen lmem setup warning
URL : https://patchwork.freedesktop.org/series/132663/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_14616 -> Patchwork_132663v1
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_132663v1 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_132663v1, please notify your bug team ("I915-ci-infra@lists.freedesktop.org") 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_132663v1/index.html
Participating hosts (34 -> 32)
------------------------------
Additional (4): bat-dg2-8 bat-kbl-2 fi-apl-guc bat-arls-3
Missing (6): fi-kbl-7567u bat-dg1-7 fi-bsw-n3050 fi-cfl-8109u fi-kbl-8809g bat-jsl-1
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_132663v1:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live:
- bat-arls-2: NOTRUN -> [FAIL][1] +1 other test fail
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-arls-2/igt@i915_selftest@live.html
* igt@vgem_basic@dmabuf-mmap:
- bat-arls-2: [PASS][2] -> [FAIL][3] +13 other tests fail
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14616/bat-arls-2/igt@vgem_basic@dmabuf-mmap.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-arls-2/igt@vgem_basic@dmabuf-mmap.html
#### Warnings ####
* igt@gem_lmem_swapping@verify-random:
- bat-arls-2: [SKIP][4] ([i915#10213]) -> [FAIL][5] +3 other tests fail
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_14616/bat-arls-2/igt@gem_lmem_swapping@verify-random.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-arls-2/igt@gem_lmem_swapping@verify-random.html
Known issues
------------
Here are the changes found in Patchwork_132663v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@debugfs_test@basic-hwmon:
- bat-arls-3: NOTRUN -> [SKIP][6] ([i915#9318])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-arls-3/igt@debugfs_test@basic-hwmon.html
* igt@fbdev@eof:
- bat-dg2-8: NOTRUN -> [SKIP][7] ([i915#2582]) +3 other tests skip
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@fbdev@eof.html
* igt@fbdev@info:
- bat-dg2-8: NOTRUN -> [SKIP][8] ([i915#1849] / [i915#2582])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@fbdev@info.html
- bat-kbl-2: NOTRUN -> [SKIP][9] ([i915#1849])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-kbl-2/igt@fbdev@info.html
* igt@gem_exec_parallel@engines@fds:
- bat-arls-3: NOTRUN -> [ABORT][10] ([i915#10698])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-arls-3/igt@gem_exec_parallel@engines@fds.html
* igt@gem_lmem_swapping@basic:
- fi-apl-guc: NOTRUN -> [SKIP][11] ([i915#4613]) +3 other tests skip
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@basic@lmem0:
- bat-dg2-8: NOTRUN -> [FAIL][12] ([i915#10378])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@gem_lmem_swapping@basic@lmem0.html
* igt@gem_lmem_swapping@parallel-random-engines:
- bat-kbl-2: NOTRUN -> [SKIP][13] +39 other tests skip
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-kbl-2/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@gem_mmap@basic:
- bat-dg2-8: NOTRUN -> [SKIP][14] ([i915#4083])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@gem_mmap@basic.html
* igt@gem_mmap_gtt@basic:
- bat-dg2-8: NOTRUN -> [SKIP][15] ([i915#4077]) +2 other tests skip
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@gem_mmap_gtt@basic.html
* igt@gem_tiled_pread_basic:
- bat-dg2-8: NOTRUN -> [SKIP][16] ([i915#4079]) +1 other test skip
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@gem_tiled_pread_basic.html
* igt@i915_pm_rps@basic-api:
- bat-dg2-8: NOTRUN -> [SKIP][17] ([i915#6621])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@i915_pm_rps@basic-api.html
* igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
- bat-dg2-8: NOTRUN -> [SKIP][18] ([i915#5190])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html
* igt@kms_addfb_basic@basic-y-tiled-legacy:
- bat-dg2-8: NOTRUN -> [SKIP][19] ([i915#4215] / [i915#5190])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_addfb_basic@basic-y-tiled-legacy.html
* igt@kms_addfb_basic@framebuffer-vs-set-tiling:
- bat-dg2-8: NOTRUN -> [SKIP][20] ([i915#4212]) +7 other tests skip
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_addfb_basic@framebuffer-vs-set-tiling.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size:
- bat-dg2-8: NOTRUN -> [SKIP][21] ([i915#9197]) +16 other tests skip
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size.html
* igt@kms_force_connector_basic@force-load-detect:
- bat-dg2-8: NOTRUN -> [SKIP][22]
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_force_connector_basic@prune-stale-modes:
- bat-dg2-8: NOTRUN -> [SKIP][23] ([i915#5274])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_force_connector_basic@prune-stale-modes.html
* igt@kms_frontbuffer_tracking@basic:
- bat-dg2-8: NOTRUN -> [SKIP][24] ([i915#5354]) +5 other tests skip
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_frontbuffer_tracking@basic.html
* igt@kms_hdmi_inject@inject-audio:
- fi-apl-guc: NOTRUN -> [SKIP][25] +17 other tests skip
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/fi-apl-guc/igt@kms_hdmi_inject@inject-audio.html
* igt@kms_psr@psr-sprite-plane-onoff:
- bat-dg2-8: NOTRUN -> [SKIP][26] ([i915#1072] / [i915#9673] / [i915#9732]) +3 other tests skip
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_psr@psr-sprite-plane-onoff.html
* igt@kms_setmode@basic-clone-single-crtc:
- bat-dg2-8: NOTRUN -> [SKIP][27] ([i915#3555])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-fence-flip:
- bat-dg2-8: NOTRUN -> [SKIP][28] ([i915#3708] / [i915#9197])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@prime_vgem@basic-fence-flip.html
* igt@prime_vgem@basic-fence-mmap:
- bat-dg2-8: NOTRUN -> [SKIP][29] ([i915#3708] / [i915#4077]) +1 other test skip
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@prime_vgem@basic-fence-mmap.html
* igt@prime_vgem@basic-write:
- bat-dg2-8: NOTRUN -> [SKIP][30] ([i915#3291] / [i915#3708]) +2 other tests skip
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/bat-dg2-8/igt@prime_vgem@basic-write.html
[i915#10213]: https://gitlab.freedesktop.org/drm/intel/issues/10213
[i915#10378]: https://gitlab.freedesktop.org/drm/intel/issues/10378
[i915#10698]: https://gitlab.freedesktop.org/drm/intel/issues/10698
[i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
[i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
[i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
[i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
[i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
[i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
[i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
[i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
[i915#9197]: https://gitlab.freedesktop.org/drm/intel/issues/9197
[i915#9318]: https://gitlab.freedesktop.org/drm/intel/issues/9318
[i915#9673]: https://gitlab.freedesktop.org/drm/intel/issues/9673
[i915#9732]: https://gitlab.freedesktop.org/drm/intel/issues/9732
Build changes
-------------
* Linux: CI_DRM_14616 -> Patchwork_132663v1
CI-20190529: 20190529
CI_DRM_14616: 53dd7a2909139fb469db4423b79d8a2888a734c2 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7814: 7814
Patchwork_132663v1: 53dd7a2909139fb469db4423b79d8a2888a734c2 @ git://anongit.freedesktop.org/gfx-ci/linux
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_132663v1/index.html
[-- Attachment #2: Type: text/html, Size: 11740 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-19 21:26 [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning Jonathan Cavitt
2024-04-19 22:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2024-04-19 22:19 ` ✗ Fi.CI.BAT: failure " Patchwork
@ 2024-04-22 6:29 ` Vivekanandan, Balasubramani
2024-04-25 21:09 ` Cavitt, Jonathan
2024-04-22 8:28 ` Jani Nikula
3 siblings, 1 reply; 7+ messages in thread
From: Vivekanandan, Balasubramani @ 2024-04-22 6:29 UTC (permalink / raw
To: Jonathan Cavitt, intel-gfx
Cc: saurabhg.gupta, John.C.Harrison, chris.p.wilson, andi.shyti,
nirmoy.das
On 19.04.2024 14:26, Jonathan Cavitt wrote:
> In the case where lmem_size < dsm_base, hardware is reporting that
> stolen lmem is unusable. In this case, instead of throwing a warning,
> we can continue execution as normal by disabling stolen LMEM support.
> For example, this change will allow the following error report from
> ATS-M to no longer apply:
>
> <6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
> <6> [144.859900] pcieport 0000:4b:00.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> <6> [144.859917] pcieport 0000:4c:01.0: PCI bridge to [bus 4d-4e]
> <6> [144.859932] pcieport 0000:4c:01.0: bridge window [mem 0xb1000000-0xb11fffff]
> <6> [144.859945] pcieport 0000:4c:01.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> <6> [144.859984] i915 0000:4d:00.0: [drm] BAR2 resized to 256M
> <6> [144.860640] i915 0000:4d:00.0: [drm] Using a reduced BAR size of 256MiB. Consider enabling 'Resizable BAR' or similar, if available in the BIOS.
> <4> [144.860719] -----------[ cut here ]-----------
> <4> [144.860727] WARNING: CPU: 17 PID: 1815 at drivers/gpu/drm/i915/gem/i915_gem_stolen.c:939 i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> <4> [144.861430] Modules linked in: i915 snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm vgem drm_shmem_helper prime_numbers i2c_algo_bit ttm video drm_display_helper drm_buddy fuse x86_pkg_temp_thermal coretemp kvm_intel kvm ixgbe mdio irqbypass ptp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pps_core i2c_i801 mei_me i2c_smbus mei wmi acpi_power_meter [last unloaded: i915]
> <4> [144.861611] CPU: 17 PID: 1815 Comm: i915_module_loa Tainted: G U 6.8.0-rc5-drmtip_1515-g78f49af27723+ #1
> <4> [144.861624] Hardware name: Intel Corporation WHITLEY/WHITLEY, BIOS SE5C6200.86B.0020.P41.2109300305 09/30/2021
> <4> [144.861632] RIP: 0010:i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> <4> [144.862287] Code: ff 41 c1 e4 05 e9 ac fe ff ff 4d 63 e4 48 89 ef 48 85 ed 74 04 48 8b 7d 08 48 c7 c6 10 a3 7b a0 e8 e9 90 43 e1 e9 ee fd ff ff <0f> 0b 49 c7 c4 ed ff ff ff e9 e0 fd ff ff 0f b7 d2 48 c7 c6 00 d9
> <4> [144.862299] RSP: 0018:ffffc90005607980 EFLAGS: 00010207
> <4> [144.862315] RAX: fffffffffff00000 RBX: 0000000000000003 RCX: 0000000000000000
>
> Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
>
> Suggested-by: Chris Wilson <chris.p.wilson@linux.intel.com>
> Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> ---
> drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> index ad6dd7f3259bc..efa632a9e61c6 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> @@ -936,8 +936,12 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
> } else {
> /* Use DSM base address instead for stolen memory */
> dsm_base = intel_uncore_read64(uncore, GEN6_DSMBASE) & GEN11_BDSM_MASK;
> - if (WARN_ON(lmem_size < dsm_base))
> + if (lmem_size < dsm_base) {
> + drm_dbg(&i915->drm,
> + "Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
> + lmem_size, dsm_base);
> return ERR_PTR(-ENODEV);
The patch is still returning an error and aborting initialization. Only the
warning is eliminated. But as per the commit description, the execution
should continue as normal after disabling stolen lmem support.
Regards,
Bala
> + }
> dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
> }
>
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-19 21:26 [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning Jonathan Cavitt
` (2 preceding siblings ...)
2024-04-22 6:29 ` [PATCH] " Vivekanandan, Balasubramani
@ 2024-04-22 8:28 ` Jani Nikula
2024-04-25 21:10 ` Cavitt, Jonathan
3 siblings, 1 reply; 7+ messages in thread
From: Jani Nikula @ 2024-04-22 8:28 UTC (permalink / raw
To: Jonathan Cavitt, intel-gfx
Cc: saurabhg.gupta, jonathan.cavitt, John.C.Harrison, chris.p.wilson,
andi.shyti, nirmoy.das
On Fri, 19 Apr 2024, Jonathan Cavitt <jonathan.cavitt@intel.com> wrote:
> In the case where lmem_size < dsm_base, hardware is reporting that
> stolen lmem is unusable. In this case, instead of throwing a warning,
> we can continue execution as normal by disabling stolen LMEM support.
> For example, this change will allow the following error report from
> ATS-M to no longer apply:
>
> <6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
> <6> [144.859900] pcieport 0000:4b:00.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> <6> [144.859917] pcieport 0000:4c:01.0: PCI bridge to [bus 4d-4e]
> <6> [144.859932] pcieport 0000:4c:01.0: bridge window [mem 0xb1000000-0xb11fffff]
> <6> [144.859945] pcieport 0000:4c:01.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> <6> [144.859984] i915 0000:4d:00.0: [drm] BAR2 resized to 256M
> <6> [144.860640] i915 0000:4d:00.0: [drm] Using a reduced BAR size of 256MiB. Consider enabling 'Resizable BAR' or similar, if available in the BIOS.
> <4> [144.860719] -----------[ cut here ]-----------
> <4> [144.860727] WARNING: CPU: 17 PID: 1815 at drivers/gpu/drm/i915/gem/i915_gem_stolen.c:939 i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> <4> [144.861430] Modules linked in: i915 snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm vgem drm_shmem_helper prime_numbers i2c_algo_bit ttm video drm_display_helper drm_buddy fuse x86_pkg_temp_thermal coretemp kvm_intel kvm ixgbe mdio irqbypass ptp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pps_core i2c_i801 mei_me i2c_smbus mei wmi acpi_power_meter [last unloaded: i915]
> <4> [144.861611] CPU: 17 PID: 1815 Comm: i915_module_loa Tainted: G U 6.8.0-rc5-drmtip_1515-g78f49af27723+ #1
> <4> [144.861624] Hardware name: Intel Corporation WHITLEY/WHITLEY, BIOS SE5C6200.86B.0020.P41.2109300305 09/30/2021
> <4> [144.861632] RIP: 0010:i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> <4> [144.862287] Code: ff 41 c1 e4 05 e9 ac fe ff ff 4d 63 e4 48 89 ef 48 85 ed 74 04 48 8b 7d 08 48 c7 c6 10 a3 7b a0 e8 e9 90 43 e1 e9 ee fd ff ff <0f> 0b 49 c7 c4 ed ff ff ff e9 e0 fd ff ff 0f b7 d2 48 c7 c6 00 d9
> <4> [144.862299] RSP: 0018:ffffc90005607980 EFLAGS: 00010207
> <4> [144.862315] RAX: fffffffffff00000 RBX: 0000000000000003 RCX: 0000000000000000
>
> Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
>
No blank lines between trailers please.
BR,
Jani.
> Suggested-by: Chris Wilson <chris.p.wilson@linux.intel.com>
> Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> ---
> drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> index ad6dd7f3259bc..efa632a9e61c6 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> @@ -936,8 +936,12 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
> } else {
> /* Use DSM base address instead for stolen memory */
> dsm_base = intel_uncore_read64(uncore, GEN6_DSMBASE) & GEN11_BDSM_MASK;
> - if (WARN_ON(lmem_size < dsm_base))
> + if (lmem_size < dsm_base) {
> + drm_dbg(&i915->drm,
> + "Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
> + lmem_size, dsm_base);
> return ERR_PTR(-ENODEV);
> + }
> dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
> }
--
Jani Nikula, Intel
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-22 6:29 ` [PATCH] " Vivekanandan, Balasubramani
@ 2024-04-25 21:09 ` Cavitt, Jonathan
0 siblings, 0 replies; 7+ messages in thread
From: Cavitt, Jonathan @ 2024-04-25 21:09 UTC (permalink / raw
To: Vivekanandan, Balasubramani, intel-gfx@lists.freedesktop.org
Cc: Cavitt, Jonathan
-----Original Message-----
From: Vivekanandan, Balasubramani <balasubramani.vivekanandan@intel.com>
Sent: Sunday, April 21, 2024 11:29 PM
To: Cavitt, Jonathan <jonathan.cavitt@intel.com>; intel-gfx@lists.freedesktop.org
Cc: Gupta, saurabhg <saurabhg.gupta@intel.com>; Harrison, John C <john.c.harrison@intel.com>; chris.p.wilson@linux.intel.com; andi.shyti@linux.intel.com; Das, Nirmoy <nirmoy.das@intel.com>
Subject: Re: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
>
> On 19.04.2024 14:26, Jonathan Cavitt wrote:
> > In the case where lmem_size < dsm_base, hardware is reporting that
> > stolen lmem is unusable. In this case, instead of throwing a warning,
> > we can continue execution as normal by disabling stolen LMEM support.
> > For example, this change will allow the following error report from
> > ATS-M to no longer apply:
> >
> > <6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
> > <6> [144.859900] pcieport 0000:4b:00.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> > <6> [144.859917] pcieport 0000:4c:01.0: PCI bridge to [bus 4d-4e]
> > <6> [144.859932] pcieport 0000:4c:01.0: bridge window [mem 0xb1000000-0xb11fffff]
> > <6> [144.859945] pcieport 0000:4c:01.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> > <6> [144.859984] i915 0000:4d:00.0: [drm] BAR2 resized to 256M
> > <6> [144.860640] i915 0000:4d:00.0: [drm] Using a reduced BAR size of 256MiB. Consider enabling 'Resizable BAR' or similar, if available in the BIOS.
> > <4> [144.860719] -----------[ cut here ]-----------
> > <4> [144.860727] WARNING: CPU: 17 PID: 1815 at drivers/gpu/drm/i915/gem/i915_gem_stolen.c:939 i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> > <4> [144.861430] Modules linked in: i915 snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm vgem drm_shmem_helper prime_numbers i2c_algo_bit ttm video drm_display_helper drm_buddy fuse x86_pkg_temp_thermal coretemp kvm_intel kvm ixgbe mdio irqbypass ptp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pps_core i2c_i801 mei_me i2c_smbus mei wmi acpi_power_meter [last unloaded: i915]
> > <4> [144.861611] CPU: 17 PID: 1815 Comm: i915_module_loa Tainted: G U 6.8.0-rc5-drmtip_1515-g78f49af27723+ #1
> > <4> [144.861624] Hardware name: Intel Corporation WHITLEY/WHITLEY, BIOS SE5C6200.86B.0020.P41.2109300305 09/30/2021
> > <4> [144.861632] RIP: 0010:i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> > <4> [144.862287] Code: ff 41 c1 e4 05 e9 ac fe ff ff 4d 63 e4 48 89 ef 48 85 ed 74 04 48 8b 7d 08 48 c7 c6 10 a3 7b a0 e8 e9 90 43 e1 e9 ee fd ff ff <0f> 0b 49 c7 c4 ed ff ff ff e9 e0 fd ff ff 0f b7 d2 48 c7 c6 00 d9
> > <4> [144.862299] RSP: 0018:ffffc90005607980 EFLAGS: 00010207
> > <4> [144.862315] RAX: fffffffffff00000 RBX: 0000000000000003 RCX: 0000000000000000
> >
> > Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
> >
> > Suggested-by: Chris Wilson <chris.p.wilson@linux.intel.com>
> > Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> > ---
> > drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 6 +++++-
> > 1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > index ad6dd7f3259bc..efa632a9e61c6 100644
> > --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > @@ -936,8 +936,12 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
> > } else {
> > /* Use DSM base address instead for stolen memory */
> > dsm_base = intel_uncore_read64(uncore, GEN6_DSMBASE) & GEN11_BDSM_MASK;
> > - if (WARN_ON(lmem_size < dsm_base))
> > + if (lmem_size < dsm_base) {
> > + drm_dbg(&i915->drm,
> > + "Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
> > + lmem_size, dsm_base);
> > return ERR_PTR(-ENODEV);
> The patch is still returning an error and aborting initialization. Only the
> warning is eliminated. But as per the commit description, the execution
> should continue as normal after disabling stolen lmem support.
>
Fixed in latest revision. Thank you for the revision notes.
-Jonathan Cavitt
> Regards,
> Bala
>
> > + }
> > dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
> > }
> >
> > --
> > 2.25.1
> >
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
2024-04-22 8:28 ` Jani Nikula
@ 2024-04-25 21:10 ` Cavitt, Jonathan
0 siblings, 0 replies; 7+ messages in thread
From: Cavitt, Jonathan @ 2024-04-25 21:10 UTC (permalink / raw
To: Jani Nikula, intel-gfx@lists.freedesktop.org; +Cc: Cavitt, Jonathan
-----Original Message-----
From: Jani Nikula <jani.nikula@linux.intel.com>
Sent: Monday, April 22, 2024 1:29 AM
To: Cavitt, Jonathan <jonathan.cavitt@intel.com>; intel-gfx@lists.freedesktop.org
Cc: Gupta, saurabhg <saurabhg.gupta@intel.com>; Cavitt, Jonathan <jonathan.cavitt@intel.com>; Harrison, John C <john.c.harrison@intel.com>; chris.p.wilson@linux.intel.com; andi.shyti@linux.intel.com; Das, Nirmoy <nirmoy.das@intel.com>
Subject: Re: [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning
>
> On Fri, 19 Apr 2024, Jonathan Cavitt <jonathan.cavitt@intel.com> wrote:
> > In the case where lmem_size < dsm_base, hardware is reporting that
> > stolen lmem is unusable. In this case, instead of throwing a warning,
> > we can continue execution as normal by disabling stolen LMEM support.
> > For example, this change will allow the following error report from
> > ATS-M to no longer apply:
> >
> > <6> [144.859887] pcieport 0000:4b:00.0: bridge window [mem 0xb1000000-0xb11fffff]
> > <6> [144.859900] pcieport 0000:4b:00.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> > <6> [144.859917] pcieport 0000:4c:01.0: PCI bridge to [bus 4d-4e]
> > <6> [144.859932] pcieport 0000:4c:01.0: bridge window [mem 0xb1000000-0xb11fffff]
> > <6> [144.859945] pcieport 0000:4c:01.0: bridge window [mem 0x3bbc00000000-0x3bbc17ffffff 64bit pref]
> > <6> [144.859984] i915 0000:4d:00.0: [drm] BAR2 resized to 256M
> > <6> [144.860640] i915 0000:4d:00.0: [drm] Using a reduced BAR size of 256MiB. Consider enabling 'Resizable BAR' or similar, if available in the BIOS.
> > <4> [144.860719] -----------[ cut here ]-----------
> > <4> [144.860727] WARNING: CPU: 17 PID: 1815 at drivers/gpu/drm/i915/gem/i915_gem_stolen.c:939 i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> > <4> [144.861430] Modules linked in: i915 snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm vgem drm_shmem_helper prime_numbers i2c_algo_bit ttm video drm_display_helper drm_buddy fuse x86_pkg_temp_thermal coretemp kvm_intel kvm ixgbe mdio irqbypass ptp crct10dif_pclmul crc32_pclmul ghash_clmulni_intel pps_core i2c_i801 mei_me i2c_smbus mei wmi acpi_power_meter [last unloaded: i915]
> > <4> [144.861611] CPU: 17 PID: 1815 Comm: i915_module_loa Tainted: G U 6.8.0-rc5-drmtip_1515-g78f49af27723+ #1
> > <4> [144.861624] Hardware name: Intel Corporation WHITLEY/WHITLEY, BIOS SE5C6200.86B.0020.P41.2109300305 09/30/2021
> > <4> [144.861632] RIP: 0010:i915_gem_stolen_lmem_setup+0x38c/0x430 [i915]
> > <4> [144.862287] Code: ff 41 c1 e4 05 e9 ac fe ff ff 4d 63 e4 48 89 ef 48 85 ed 74 04 48 8b 7d 08 48 c7 c6 10 a3 7b a0 e8 e9 90 43 e1 e9 ee fd ff ff <0f> 0b 49 c7 c4 ed ff ff ff e9 e0 fd ff ff 0f b7 d2 48 c7 c6 00 d9
> > <4> [144.862299] RSP: 0018:ffffc90005607980 EFLAGS: 00010207
> > <4> [144.862315] RAX: fffffffffff00000 RBX: 0000000000000003 RCX: 0000000000000000
> >
> > Fixes: https://gitlab.freedesktop.org/drm/intel/-/issues/10833
> >
>
> No blank lines between trailers please.
>
> BR,
> Jani.
Fixed in latest revision. Thank you for the revision notes.
-Jonathan Cavitt
>
> > Suggested-by: Chris Wilson <chris.p.wilson@linux.intel.com>
> > Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> > ---
> > drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 6 +++++-
> > 1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > index ad6dd7f3259bc..efa632a9e61c6 100644
> > --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c
> > @@ -936,8 +936,12 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
> > } else {
> > /* Use DSM base address instead for stolen memory */
> > dsm_base = intel_uncore_read64(uncore, GEN6_DSMBASE) & GEN11_BDSM_MASK;
> > - if (WARN_ON(lmem_size < dsm_base))
> > + if (lmem_size < dsm_base) {
> > + drm_dbg(&i915->drm,
> > + "Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
> > + lmem_size, dsm_base);
> > return ERR_PTR(-ENODEV);
> > + }
> > dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
> > }
>
> --
> Jani Nikula, Intel
>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-04-25 21:10 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-19 21:26 [PATCH] drm/i915/gem: Downgrade stolen lmem setup warning Jonathan Cavitt
2024-04-19 22:09 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2024-04-19 22:19 ` ✗ Fi.CI.BAT: failure " Patchwork
2024-04-22 6:29 ` [PATCH] " Vivekanandan, Balasubramani
2024-04-25 21:09 ` Cavitt, Jonathan
2024-04-22 8:28 ` Jani Nikula
2024-04-25 21:10 ` Cavitt, Jonathan
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.