[PATCH v2 1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

[PATCH v2 1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

[Lyude Paul]

Currently, we don't call dma_set_max_seg_size() for i915 because we
intentionally do not limit the segment length that the device supports.
However, this results in a warning being emitted if we try to map
anything larger than SZ_64K on a kernel with CONFIG_DMA_API_DEBUG_SG
enabled:

[    7.751926] DMA-API: i915 0000:00:02.0: mapping sg segment longer
than device claims to support [len=98304] [max=65536]
[    7.751934] WARNING: CPU: 5 PID: 474 at kernel/dma/debug.c:1220
debug_dma_map_sg+0x20f/0x340

This was originally brought up on
https://bugs.freedesktop.org/show_bug.cgi?id=108517 , and the consensus
there was it wasn't really useful to set a limit (and that dma-debug
isn't really all that useful for i915 in the first place). Unfortunately
though, CONFIG_DMA_API_DEBUG_SG is enabled in the debug configs for
various distro kernels. Since a WARN_ON() will disable automatic problem
reporting (and cause any CI with said option enabled to start
complaining), we really should just fix the problem.

Note that as me and Chris Wilson discussed, the other solution for this
would be to make DMA-API not make such assumptions when a driver hasn't
explicitly set a maximum segment size. But, taking a look at the commit
which originally introduced this behavior, commit 78c47830a5cb
("dma-debug: check scatterlist segments"), there is an explicit mention
of this assumption and how it applies to devices with no segment size:

	Conversely, devices which are less limited than the rather
	conservative defaults, or indeed have no limitations at all
	(e.g. GPUs with their own internal MMU), should be encouraged to
	set appropriate dma_parms, as they may get more efficient DMA
	mapping performance out of it.

So unless there's any concerns (I'm open to discussion!), let's just
follow suite and call dma_set_max_seg_size() with UINT_MAX as our limit
to silence any warnings.

Signed-off-by: Lyude Paul <lyude@redhat.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: <stable@vger.kernel.org> # v4.18+
---
 drivers/gpu/drm/i915/i915_gem_gtt.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
index 0b81e0b64393..a1475039d182 100644
--- a/drivers/gpu/drm/i915/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
@@ -3152,6 +3152,11 @@ static int ggtt_probe_hw(struct i915_ggtt *ggtt, struct intel_gt *gt)
 	if (ret)
 		return ret;
 
+	/* We don't have a max segment size, so set it to the max so sg's
+	 * debugging layer doesn't complain
+	 */
+	dma_set_max_seg_size(ggtt->vm.dma, UINT_MAX);
+
 	if ((ggtt->vm.total - 1) >> 32) {
 		DRM_ERROR("We never expected a Global GTT with more than 32bits"
 			  " of address space! Found %lldM!\n",
-- 
2.21.0

[PATCH v2 2/2] drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci

[Lyude Paul]

Now that we've fixed i915 so that it sets a max SG segment length and
gotten rid of the relevant warnings, let's enable
CONFIG_DMA_API_DEBUG_SG for intel-ci so that we can catch issues like
this in the future as well.

Signed-off-by: Lyude Paul <lyude@redhat.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/Kconfig.debug | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug
index 00786a142ff0..ad8d3cd63c9f 100644
--- a/drivers/gpu/drm/i915/Kconfig.debug
+++ b/drivers/gpu/drm/i915/Kconfig.debug
@@ -32,6 +32,7 @@ config DRM_I915_DEBUG
 	select DRM_DEBUG_SELFTEST
 	select DMABUF_SELFTESTS
 	select SW_SYNC # signaling validation framework (igt/syncobj*)
+        select DMA_API_DEBUG_SG
 	select DRM_I915_SW_FENCE_DEBUG_OBJECTS
 	select DRM_I915_SELFTEST
 	select DRM_I915_DEBUG_RUNTIME_PM
-- 
2.21.0

✗ Fi.CI.SPARSE: warning for series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

[Patchwork]

== Series Details ==

Series: series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()
URL   : https://patchwork.freedesktop.org/series/65643/
State : warning

== Summary ==

$ dim sparse origin/drm-tip
Sparse version: v0.6.0
Commit: drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()
Okay!

Commit: drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci
+
+
+  Depends on [n]: DMA_API_DEBUG [=n]
+  Depends on [n]: DMA_API_DEBUG [=n]
+  Depends on [n]: DMA_API_DEBUG [=n]
+  - DRM_I915_DEBUG [=y] && HAS_IOMEM [=y] && EXPERT [=y] && DRM_I915 [=m]
+  - DRM_I915_DEBUG [=y] && HAS_IOMEM [=y] && EXPERT [=y] && DRM_I915 [=m]
+  - DRM_I915_DEBUG [=y] && HAS_IOMEM [=y] && EXPERT [=y] && DRM_I915 [=m]
+  Selected by [m]:
+  Selected by [m]:
+  Selected by [m]:
+WARNING: unmet direct dependencies detected for DMA_API_DEBUG_SG
+WARNING: unmet direct dependencies detected for DMA_API_DEBUG_SG
+WARNING: unmet direct dependencies detected for DMA_API_DEBUG_SG

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✓ Fi.CI.BAT: success for series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

[Patchwork]

== Series Details ==

Series: series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()
URL   : https://patchwork.freedesktop.org/series/65643/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_6768 -> Patchwork_14148
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/

Known issues
------------

  Here are the changes found in Patchwork_14148 that come from known issues:

### IGT changes ###

#### Possible fixes ####

  * igt@i915_selftest@live_execlists:
    - fi-skl-gvtdvm:      [DMESG-FAIL][1] ([fdo#111108]) -> [PASS][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6768/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html

  * igt@i915_selftest@live_reset:
    - fi-icl-u2:          [INCOMPLETE][3] ([fdo#107713]) -> [PASS][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6768/fi-icl-u2/igt@i915_selftest@live_reset.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/fi-icl-u2/igt@i915_selftest@live_reset.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-icl-u2:          [FAIL][5] ([fdo#103167]) -> [PASS][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6768/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html

  
#### Warnings ####

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-icl-u2:          [FAIL][7] ([fdo#109483]) -> [FAIL][8] ([fdo#111407])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6768/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html

  
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#109483]: https://bugs.freedesktop.org/show_bug.cgi?id=109483
  [fdo#111108]: https://bugs.freedesktop.org/show_bug.cgi?id=111108
  [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407


Participating hosts (55 -> 48)
------------------------------

  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_6768 -> Patchwork_14148

  CI-20190529: 20190529
  CI_DRM_6768: 37f6677cd1b8dbbb8fce0d2776439ff0645baafe @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5148: 50390dd7adaccae21cafa85b866c17606cec94c3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_14148: 973f8851dd0494d8702ee30f5c2df31de5968032 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

973f8851dd04 drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci
ac93099b78a2 drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14148/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH v2 2/2] drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci

[Chris Wilson]

Quoting Lyude Paul (2019-08-22 21:31:27)
> Now that we've fixed i915 so that it sets a max SG segment length and
> gotten rid of the relevant warnings, let's enable
> CONFIG_DMA_API_DEBUG_SG for intel-ci so that we can catch issues like
> this in the future as well.
> 
> Signed-off-by: Lyude Paul <lyude@redhat.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  drivers/gpu/drm/i915/Kconfig.debug | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug
> index 00786a142ff0..ad8d3cd63c9f 100644
> --- a/drivers/gpu/drm/i915/Kconfig.debug
> +++ b/drivers/gpu/drm/i915/Kconfig.debug
> @@ -32,6 +32,7 @@ config DRM_I915_DEBUG
>         select DRM_DEBUG_SELFTEST
>         select DMABUF_SELFTESTS
>         select SW_SYNC # signaling validation framework (igt/syncobj*)

	select DMA_API_DEBUG
as well for it to be enabled, no recursive dependency solver in Kconfig.

> +        select DMA_API_DEBUG_SG
>         select DRM_I915_SW_FENCE_DEBUG_OBJECTS
>         select DRM_I915_SELFTEST
>         select DRM_I915_DEBUG_RUNTIME_PM
> -- 
> 2.21.0
> 

[PATCH v3] drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci

[Lyude Paul]

Now that we've fixed i915 so that it sets a max SG segment length and
gotten rid of the relevant warnings, let's enable
CONFIG_DMA_API_DEBUG_SG for intel-ci so that we can catch issues like
this in the future as well.

Changes since v2:
* Also select DMA_API_DEBUG - Chris Wilson

Signed-off-by: Lyude Paul <lyude@redhat.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/Kconfig.debug | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug
index 00786a142ff0..cb929d0218e8 100644
--- a/drivers/gpu/drm/i915/Kconfig.debug
+++ b/drivers/gpu/drm/i915/Kconfig.debug
@@ -32,6 +32,8 @@ config DRM_I915_DEBUG
 	select DRM_DEBUG_SELFTEST
 	select DMABUF_SELFTESTS
 	select SW_SYNC # signaling validation framework (igt/syncobj*)
+        select DMA_API_DEBUG
+        select DMA_API_DEBUG_SG
 	select DRM_I915_SW_FENCE_DEBUG_OBJECTS
 	select DRM_I915_SELFTEST
 	select DRM_I915_DEBUG_RUNTIME_PM
-- 
2.21.0

[PATCH v3] drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci

[Lyude Paul]

Now that we've fixed i915 so that it sets a max SG segment length and
gotten rid of the relevant warnings, let's enable
CONFIG_DMA_API_DEBUG_SG for intel-ci so that we can catch issues like
this in the future as well.

Changes since v2:
* Also select DMA_API_DEBUG - Chris Wilson

Signed-off-by: Lyude Paul <lyude@redhat.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/Kconfig.debug | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/i915/Kconfig.debug b/drivers/gpu/drm/i915/Kconfig.debug
index 00786a142ff0..cb929d0218e8 100644
--- a/drivers/gpu/drm/i915/Kconfig.debug
+++ b/drivers/gpu/drm/i915/Kconfig.debug
@@ -32,6 +32,8 @@ config DRM_I915_DEBUG
 	select DRM_DEBUG_SELFTEST
 	select DMABUF_SELFTESTS
 	select SW_SYNC # signaling validation framework (igt/syncobj*)
+        select DMA_API_DEBUG
+        select DMA_API_DEBUG_SG
 	select DRM_I915_SW_FENCE_DEBUG_OBJECTS
 	select DRM_I915_SELFTEST
 	select DRM_I915_DEBUG_RUNTIME_PM
-- 
2.21.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

✗ Fi.CI.BAT: failure for series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw() (rev2)

[Patchwork]

== Series Details ==

Series: series starting with [v2,1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw() (rev2)
URL   : https://patchwork.freedesktop.org/series/65643/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_6769 -> Patchwork_14149
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_14149 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_14149, 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_14149/

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_14149:

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live_objects:
    - fi-whl-u:           [PASS][1] -> [DMESG-WARN][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-whl-u/igt@i915_selftest@live_objects.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-whl-u/igt@i915_selftest@live_objects.html
    - fi-cml-u2:          [PASS][3] -> [DMESG-WARN][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-cml-u2/igt@i915_selftest@live_objects.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cml-u2/igt@i915_selftest@live_objects.html
    - fi-bxt-dsi:         [PASS][5] -> [DMESG-WARN][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-bxt-dsi/igt@i915_selftest@live_objects.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bxt-dsi/igt@i915_selftest@live_objects.html
    - fi-kbl-8809g:       [PASS][7] -> [DMESG-WARN][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-8809g/igt@i915_selftest@live_objects.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-8809g/igt@i915_selftest@live_objects.html
    - fi-kbl-7500u:       [PASS][9] -> [DMESG-WARN][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-7500u/igt@i915_selftest@live_objects.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-7500u/igt@i915_selftest@live_objects.html
    - fi-bdw-5557u:       [PASS][11] -> [DMESG-WARN][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-bdw-5557u/igt@i915_selftest@live_objects.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bdw-5557u/igt@i915_selftest@live_objects.html
    - fi-cfl-8700k:       [PASS][13] -> [DMESG-WARN][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-cfl-8700k/igt@i915_selftest@live_objects.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cfl-8700k/igt@i915_selftest@live_objects.html
    - fi-kbl-x1275:       [PASS][15] -> [DMESG-WARN][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-x1275/igt@i915_selftest@live_objects.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-x1275/igt@i915_selftest@live_objects.html
    - fi-hsw-peppy:       [PASS][17] -> [DMESG-WARN][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-hsw-peppy/igt@i915_selftest@live_objects.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-hsw-peppy/igt@i915_selftest@live_objects.html
    - fi-skl-6600u:       [PASS][19] -> [DMESG-WARN][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-6600u/igt@i915_selftest@live_objects.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-6600u/igt@i915_selftest@live_objects.html
    - fi-pnv-d510:        [PASS][21] -> [DMESG-WARN][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-pnv-d510/igt@i915_selftest@live_objects.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-pnv-d510/igt@i915_selftest@live_objects.html
    - fi-glk-dsi:         [PASS][23] -> [DMESG-WARN][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-glk-dsi/igt@i915_selftest@live_objects.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-glk-dsi/igt@i915_selftest@live_objects.html
    - fi-hsw-4770r:       [PASS][25] -> [DMESG-WARN][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-hsw-4770r/igt@i915_selftest@live_objects.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-hsw-4770r/igt@i915_selftest@live_objects.html
    - fi-apl-guc:         [PASS][27] -> [DMESG-WARN][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-apl-guc/igt@i915_selftest@live_objects.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-apl-guc/igt@i915_selftest@live_objects.html
    - fi-skl-guc:         [PASS][29] -> [DMESG-WARN][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-guc/igt@i915_selftest@live_objects.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-guc/igt@i915_selftest@live_objects.html
    - fi-cfl-guc:         [PASS][31] -> [DMESG-WARN][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-cfl-guc/igt@i915_selftest@live_objects.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cfl-guc/igt@i915_selftest@live_objects.html
    - fi-blb-e6850:       [PASS][33] -> [DMESG-WARN][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-blb-e6850/igt@i915_selftest@live_objects.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-blb-e6850/igt@i915_selftest@live_objects.html
    - fi-skl-6700k2:      [PASS][35] -> [DMESG-WARN][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-6700k2/igt@i915_selftest@live_objects.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-6700k2/igt@i915_selftest@live_objects.html
    - fi-elk-e7500:       [PASS][37] -> [DMESG-WARN][38]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-elk-e7500/igt@i915_selftest@live_objects.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-elk-e7500/igt@i915_selftest@live_objects.html
    - fi-snb-2600:        [PASS][39] -> [DMESG-WARN][40]
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-snb-2600/igt@i915_selftest@live_objects.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-snb-2600/igt@i915_selftest@live_objects.html
    - fi-skl-6260u:       [PASS][41] -> [DMESG-WARN][42]
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-6260u/igt@i915_selftest@live_objects.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-6260u/igt@i915_selftest@live_objects.html
    - fi-byt-j1900:       [PASS][43] -> [DMESG-WARN][44]
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-byt-j1900/igt@i915_selftest@live_objects.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-byt-j1900/igt@i915_selftest@live_objects.html
    - fi-skl-lmem:        [PASS][45] -> [DMESG-WARN][46]
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-lmem/igt@i915_selftest@live_objects.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-lmem/igt@i915_selftest@live_objects.html
    - fi-icl-u3:          [PASS][47] -> [DMESG-WARN][48]
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u3/igt@i915_selftest@live_objects.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u3/igt@i915_selftest@live_objects.html
    - fi-cfl-8109u:       [PASS][49] -> [DMESG-WARN][50]
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-cfl-8109u/igt@i915_selftest@live_objects.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cfl-8109u/igt@i915_selftest@live_objects.html
    - fi-byt-n2820:       [PASS][51] -> [DMESG-WARN][52]
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-byt-n2820/igt@i915_selftest@live_objects.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-byt-n2820/igt@i915_selftest@live_objects.html
    - fi-skl-6770hq:      [PASS][53] -> [DMESG-WARN][54]
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-6770hq/igt@i915_selftest@live_objects.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-6770hq/igt@i915_selftest@live_objects.html
    - fi-kbl-r:           [PASS][55] -> [DMESG-WARN][56]
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-r/igt@i915_selftest@live_objects.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-r/igt@i915_selftest@live_objects.html
    - fi-bsw-kefka:       [PASS][57] -> [DMESG-WARN][58]
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-bsw-kefka/igt@i915_selftest@live_objects.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bsw-kefka/igt@i915_selftest@live_objects.html
    - fi-kbl-guc:         [PASS][59] -> [DMESG-WARN][60]
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-guc/igt@i915_selftest@live_objects.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-guc/igt@i915_selftest@live_objects.html
    - fi-icl-u2:          [PASS][61] -> [DMESG-WARN][62]
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u2/igt@i915_selftest@live_objects.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u2/igt@i915_selftest@live_objects.html
    - fi-snb-2520m:       [PASS][63] -> [DMESG-WARN][64]
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-snb-2520m/igt@i915_selftest@live_objects.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-snb-2520m/igt@i915_selftest@live_objects.html
    - fi-ivb-3770:        [PASS][65] -> [DMESG-WARN][66]
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-ivb-3770/igt@i915_selftest@live_objects.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-ivb-3770/igt@i915_selftest@live_objects.html
    - fi-kbl-7567u:       [PASS][67] -> [DMESG-WARN][68]
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-kbl-7567u/igt@i915_selftest@live_objects.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-7567u/igt@i915_selftest@live_objects.html
    - fi-skl-iommu:       [PASS][69] -> [DMESG-WARN][70]
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-skl-iommu/igt@i915_selftest@live_objects.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-skl-iommu/igt@i915_selftest@live_objects.html
    - fi-hsw-4770:        [PASS][71] -> [DMESG-WARN][72]
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-hsw-4770/igt@i915_selftest@live_objects.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-hsw-4770/igt@i915_selftest@live_objects.html
    - fi-ilk-650:         [PASS][73] -> [DMESG-WARN][74]
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-ilk-650/igt@i915_selftest@live_objects.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-ilk-650/igt@i915_selftest@live_objects.html
    - fi-bsw-n3050:       NOTRUN -> [DMESG-WARN][75]
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bsw-n3050/igt@i915_selftest@live_objects.html

  * igt@runner@aborted:
    - fi-ilk-650:         NOTRUN -> [FAIL][76]
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-ilk-650/igt@runner@aborted.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][77]
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-pnv-d510/igt@runner@aborted.html
    - fi-bdw-gvtdvm:      NOTRUN -> [FAIL][78]
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bdw-gvtdvm/igt@runner@aborted.html
    - fi-cfl-8109u:       NOTRUN -> [FAIL][79]
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cfl-8109u/igt@runner@aborted.html
    - fi-gdg-551:         NOTRUN -> [FAIL][80]
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-gdg-551/igt@runner@aborted.html
    - fi-kbl-7500u:       NOTRUN -> [FAIL][81]
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-7500u/igt@runner@aborted.html
    - fi-whl-u:           NOTRUN -> [FAIL][82]
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-whl-u/igt@runner@aborted.html
    - fi-cml-u2:          NOTRUN -> [FAIL][83]
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cml-u2/igt@runner@aborted.html
    - fi-bxt-dsi:         NOTRUN -> [FAIL][84]
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bxt-dsi/igt@runner@aborted.html
    - fi-kbl-7567u:       NOTRUN -> [FAIL][85]
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-7567u/igt@runner@aborted.html
    - fi-blb-e6850:       NOTRUN -> [FAIL][86]
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-blb-e6850/igt@runner@aborted.html
    - fi-kbl-x1275:       NOTRUN -> [FAIL][87]
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-x1275/igt@runner@aborted.html
    - fi-cfl-8700k:       NOTRUN -> [FAIL][88]
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-cfl-8700k/igt@runner@aborted.html
    - fi-kbl-8809g:       NOTRUN -> [FAIL][89]
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-8809g/igt@runner@aborted.html
    - fi-kbl-r:           NOTRUN -> [FAIL][90]
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-kbl-r/igt@runner@aborted.html
    - fi-bdw-5557u:       NOTRUN -> [FAIL][91]
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-bdw-5557u/igt@runner@aborted.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_selftest@live_objects:
    - {fi-icl-guc}:       [PASS][92] -> [DMESG-WARN][93]
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-guc/igt@i915_selftest@live_objects.html
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-guc/igt@i915_selftest@live_objects.html
    - {fi-icl-u4}:        [PASS][94] -> [DMESG-WARN][95]
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u4/igt@i915_selftest@live_objects.html
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u4/igt@i915_selftest@live_objects.html

  
Known issues
------------

  Here are the changes found in Patchwork_14149 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_cpu_reloc@basic:
    - fi-icl-u3:          [PASS][96] -> [DMESG-WARN][97] ([fdo#107724]) +1 similar issue
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u3/igt@gem_cpu_reloc@basic.html
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u3/igt@gem_cpu_reloc@basic.html

  
#### Possible fixes ####

  * igt@kms_frontbuffer_tracking@basic:
    - {fi-icl-u4}:        [FAIL][98] ([fdo#103167]) -> [PASS][99]
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u4/igt@kms_frontbuffer_tracking@basic.html
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u4/igt@kms_frontbuffer_tracking@basic.html
    - fi-icl-u2:          [FAIL][100] ([fdo#103167]) -> [PASS][101]
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6769/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100


Participating hosts (55 -> 48)
------------------------------

  Additional (1): fi-bsw-n3050 
  Missing    (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_6769 -> Patchwork_14149

  CI-20190529: 20190529
  CI_DRM_6769: 66b20c063386eb45b5617fce6c2dca606d56a09f @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5148: 50390dd7adaccae21cafa85b866c17606cec94c3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_14149: 6827154be39005013373f72b48e34de6e5df0e4c @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

6827154be390 drm/i915: Enable CONFIG_DMA_API_DEBUG_SG for intel-ci
5052874ed87e drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14149/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [PATCH v2 1/2] drm/i915: Call dma_set_max_seg_size() in i915_ggtt_probe_hw()

[Chris Wilson]

Quoting Lyude Paul (2019-08-22 21:31:26)
> Currently, we don't call dma_set_max_seg_size() for i915 because we
> intentionally do not limit the segment length that the device supports.
> However, this results in a warning being emitted if we try to map
> anything larger than SZ_64K on a kernel with CONFIG_DMA_API_DEBUG_SG
> enabled:
> 
> [    7.751926] DMA-API: i915 0000:00:02.0: mapping sg segment longer
> than device claims to support [len=98304] [max=65536]
> [    7.751934] WARNING: CPU: 5 PID: 474 at kernel/dma/debug.c:1220
> debug_dma_map_sg+0x20f/0x340
> 
> This was originally brought up on
> https://bugs.freedesktop.org/show_bug.cgi?id=108517 , and the consensus
> there was it wasn't really useful to set a limit (and that dma-debug
> isn't really all that useful for i915 in the first place). Unfortunately
> though, CONFIG_DMA_API_DEBUG_SG is enabled in the debug configs for
> various distro kernels. Since a WARN_ON() will disable automatic problem
> reporting (and cause any CI with said option enabled to start
> complaining), we really should just fix the problem.
> 
> Note that as me and Chris Wilson discussed, the other solution for this
> would be to make DMA-API not make such assumptions when a driver hasn't
> explicitly set a maximum segment size. But, taking a look at the commit
> which originally introduced this behavior, commit 78c47830a5cb
> ("dma-debug: check scatterlist segments"), there is an explicit mention
> of this assumption and how it applies to devices with no segment size:
> 
>         Conversely, devices which are less limited than the rather
>         conservative defaults, or indeed have no limitations at all
>         (e.g. GPUs with their own internal MMU), should be encouraged to
>         set appropriate dma_parms, as they may get more efficient DMA
>         mapping performance out of it.
> 
> So unless there's any concerns (I'm open to discussion!), let's just
> follow suite and call dma_set_max_seg_size() with UINT_MAX as our limit
> to silence any warnings.
> 
> Signed-off-by: Lyude Paul <lyude@redhat.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: <stable@vger.kernel.org> # v4.18+
> ---
>  drivers/gpu/drm/i915/i915_gem_gtt.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c b/drivers/gpu/drm/i915/i915_gem_gtt.c
> index 0b81e0b64393..a1475039d182 100644
> --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> @@ -3152,6 +3152,11 @@ static int ggtt_probe_hw(struct i915_ggtt *ggtt, struct intel_gt *gt)
>         if (ret)
>                 return ret;
>  
> +       /* We don't have a max segment size, so set it to the max so sg's
> +        * debugging layer doesn't complain
> +        */
> +       dma_set_max_seg_size(ggtt->vm.dma, UINT_MAX);

The rest of the dma setup is in i915_driver_hw_probe, so I would put it
there just after dma_set_coherent_mask() (maybe one day even being brave
enough to pull out those to their own function).

I think I've made my point about the futility of dma-debug and you've
made yours about the simplicity of the patch to shut it up, so move it
across and
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris