[Intel-gfx] [PATCH] drm/i915/gt: Validation rotated vma bounds are within the object

[Intel-gfx] [PATCH] drm/i915/gt: Validation rotated vma bounds are within the object

[Chris Wilson]

Quite understandably, we bug out when asked to find a page that doesn't
belong to the object. However, we should report the error back to the
user long before we attempt the out-of-bound access! In this case, it is
insufficient validation on the rotated vma, with the simplest/cheapest
point for us to insert a bound check when we are computing the rotated
page lookups.

Similarly, it might be wise to see if we can validate the user input
upon creating the rotated framebuffer.

Closes: https://gitlab.freedesktop.org/drm/intel/issues/951
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Matthew Auld <matthew.auld@intel.com
---
 drivers/gpu/drm/i915/gt/intel_ggtt.c | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
index 99189cdba8a9..59a60968a6da 100644
--- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
@@ -1283,6 +1283,7 @@ static noinline struct sg_table *
 intel_rotate_pages(struct intel_rotation_info *rot_info,
 		   struct drm_i915_gem_object *obj)
 {
+	const unsigned long npages = obj->base.size >> PAGE_SHIFT;
 	unsigned int size = intel_rotation_info_size(rot_info);
 	struct sg_table *st;
 	struct scatterlist *sg;
@@ -1302,9 +1303,23 @@ intel_rotate_pages(struct intel_rotation_info *rot_info,
 	sg = st->sgl;
 
 	for (i = 0 ; i < ARRAY_SIZE(rot_info->plane); i++) {
-		sg = rotate_pages(obj, rot_info->plane[i].offset,
-				  rot_info->plane[i].width, rot_info->plane[i].height,
-				  rot_info->plane[i].stride, st, sg);
+		const struct intel_remapped_plane_info *plane =
+			&rot_info->plane[i];
+		unsigned long last;
+
+		last = plane->offset;
+		last += (plane->height - 1) * plane->stride;
+		last += plane->width - 1;
+		if (last >= npages) {
+			ret = -EINVAL;
+			goto err_sg_alloc;
+		}
+
+		sg = rotate_pages(obj,
+				  plane->offset,
+				  plane->width, plane->height,
+				  plane->stride,
+				  st, sg);
 	}
 
 	return st;
-- 
2.25.0.rc2

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

Re: [Intel-gfx] [PATCH] drm/i915/gt: Validation rotated vma bounds are within the object

[Ville Syrjälä]

On Thu, Jan 09, 2020 at 02:11:52PM +0000, Chris Wilson wrote:
> Quite understandably, we bug out when asked to find a page that doesn't
> belong to the object. However, we should report the error back to the
> user long before we attempt the out-of-bound access! In this case, it is
> insufficient validation on the rotated vma, with the simplest/cheapest
> point for us to insert a bound check when we are computing the rotated
> page lookups.
> 
> Similarly, it might be wise to see if we can validate the user input
> upon creating the rotated framebuffer.

We do. Did someone break it?

> 
> Closes: https://gitlab.freedesktop.org/drm/intel/issues/951
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Matthew Auld <matthew.auld@intel.com
> ---
>  drivers/gpu/drm/i915/gt/intel_ggtt.c | 21 ++++++++++++++++++---
>  1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
> index 99189cdba8a9..59a60968a6da 100644
> --- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
> +++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
> @@ -1283,6 +1283,7 @@ static noinline struct sg_table *
>  intel_rotate_pages(struct intel_rotation_info *rot_info,
>  		   struct drm_i915_gem_object *obj)
>  {
> +	const unsigned long npages = obj->base.size >> PAGE_SHIFT;
>  	unsigned int size = intel_rotation_info_size(rot_info);
>  	struct sg_table *st;
>  	struct scatterlist *sg;
> @@ -1302,9 +1303,23 @@ intel_rotate_pages(struct intel_rotation_info *rot_info,
>  	sg = st->sgl;
>  
>  	for (i = 0 ; i < ARRAY_SIZE(rot_info->plane); i++) {
> -		sg = rotate_pages(obj, rot_info->plane[i].offset,
> -				  rot_info->plane[i].width, rot_info->plane[i].height,
> -				  rot_info->plane[i].stride, st, sg);
> +		const struct intel_remapped_plane_info *plane =
> +			&rot_info->plane[i];
> +		unsigned long last;
> +
> +		last = plane->offset;
> +		last += (plane->height - 1) * plane->stride;
> +		last += plane->width - 1;
> +		if (last >= npages) {
> +			ret = -EINVAL;
> +			goto err_sg_alloc;
> +		}
> +
> +		sg = rotate_pages(obj,
> +				  plane->offset,
> +				  plane->width, plane->height,
> +				  plane->stride,
> +				  st, sg);
>  	}
>  
>  	return st;
> -- 
> 2.25.0.rc2

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

Re: [Intel-gfx] [PATCH] drm/i915/gt: Validation rotated vma bounds are within the object

[Ville Syrjälä]

On Thu, Jan 09, 2020 at 04:52:41PM +0200, Ville Syrjälä wrote:
> On Thu, Jan 09, 2020 at 02:11:52PM +0000, Chris Wilson wrote:
> > Quite understandably, we bug out when asked to find a page that doesn't
> > belong to the object. However, we should report the error back to the
> > user long before we attempt the out-of-bound access! In this case, it is
> > insufficient validation on the rotated vma, with the simplest/cheapest
> > point for us to insert a bound check when we are computing the rotated
> > page lookups.
> > 
> > Similarly, it might be wise to see if we can validate the user input
> > upon creating the rotated framebuffer.
> 
> We do. Did someone break it?

One theory on how this could happens is that we are using a stale gtt
view here. But AFAICS the only way that could happen is that we take
a shortcut out from the plane check somewhere before populating
plane_state->gtt_view afresh, after using a rotated fb previously so
that plane_state->gtt_view has been populated with a rotated view.

The first such path I see is:
intel_plane_atomic_check_with_state()
{
...
	if (!new_plane_state->hw.crtc && !old_plane_state->hw.crtc)
		return 0;

but that should also imply new_plane_state->hw.fb==NULL and so we
should not end up pinning the fb.

The second path is:
intel_plane_compute_gtt()
{
	const struct intel_framebuffer *fb =
	        to_intel_framebuffer(plane_state->hw.fb);

	if (!fb)
		return 0;

and so we won't have a new fb there either and so shouldn't try
to pin it.

So can't see how that could happen from these normal paths. Which
leads me to wonder if this might have something to do with nv12
slave planes...

> 
> > 
> > Closes: https://gitlab.freedesktop.org/drm/intel/issues/951
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Cc: Matthew Auld <matthew.auld@intel.com
> > ---
> >  drivers/gpu/drm/i915/gt/intel_ggtt.c | 21 ++++++++++++++++++---
> >  1 file changed, 18 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
> > index 99189cdba8a9..59a60968a6da 100644
> > --- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
> > +++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
> > @@ -1283,6 +1283,7 @@ static noinline struct sg_table *
> >  intel_rotate_pages(struct intel_rotation_info *rot_info,
> >  		   struct drm_i915_gem_object *obj)
> >  {
> > +	const unsigned long npages = obj->base.size >> PAGE_SHIFT;
> >  	unsigned int size = intel_rotation_info_size(rot_info);
> >  	struct sg_table *st;
> >  	struct scatterlist *sg;
> > @@ -1302,9 +1303,23 @@ intel_rotate_pages(struct intel_rotation_info *rot_info,
> >  	sg = st->sgl;
> >  
> >  	for (i = 0 ; i < ARRAY_SIZE(rot_info->plane); i++) {
> > -		sg = rotate_pages(obj, rot_info->plane[i].offset,
> > -				  rot_info->plane[i].width, rot_info->plane[i].height,
> > -				  rot_info->plane[i].stride, st, sg);
> > +		const struct intel_remapped_plane_info *plane =
> > +			&rot_info->plane[i];
> > +		unsigned long last;
> > +
> > +		last = plane->offset;
> > +		last += (plane->height - 1) * plane->stride;
> > +		last += plane->width - 1;
> > +		if (last >= npages) {
> > +			ret = -EINVAL;
> > +			goto err_sg_alloc;
> > +		}
> > +
> > +		sg = rotate_pages(obj,
> > +				  plane->offset,
> > +				  plane->width, plane->height,
> > +				  plane->stride,
> > +				  st, sg);
> >  	}
> >  
> >  	return st;
> > -- 
> > 2.25.0.rc2
> 
> -- 
> Ville Syrjälä
> Intel

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

Re: [Intel-gfx] [PATCH] drm/i915/gt: Validation rotated vma bounds are within the object

[Ville Syrjälä]

On Thu, Jan 09, 2020 at 08:37:09PM +0200, Ville Syrjälä wrote:
> On Thu, Jan 09, 2020 at 04:52:41PM +0200, Ville Syrjälä wrote:
> > On Thu, Jan 09, 2020 at 02:11:52PM +0000, Chris Wilson wrote:
> > > Quite understandably, we bug out when asked to find a page that doesn't
> > > belong to the object. However, we should report the error back to the
> > > user long before we attempt the out-of-bound access! In this case, it is
> > > insufficient validation on the rotated vma, with the simplest/cheapest
> > > point for us to insert a bound check when we are computing the rotated
> > > page lookups.
> > > 
> > > Similarly, it might be wise to see if we can validate the user input
> > > upon creating the rotated framebuffer.
> > 
> > We do. Did someone break it?
> 
> One theory on how this could happens is that we are using a stale gtt
> view here. But AFAICS the only way that could happen is that we take
> a shortcut out from the plane check somewhere before populating
> plane_state->gtt_view afresh, after using a rotated fb previously so
> that plane_state->gtt_view has been populated with a rotated view.
> 
> The first such path I see is:
> intel_plane_atomic_check_with_state()
> {
> ...
> 	if (!new_plane_state->hw.crtc && !old_plane_state->hw.crtc)
> 		return 0;
> 
> but that should also imply new_plane_state->hw.fb==NULL and so we
> should not end up pinning the fb.
> 
> The second path is:
> intel_plane_compute_gtt()
> {
> 	const struct intel_framebuffer *fb =
> 	        to_intel_framebuffer(plane_state->hw.fb);
> 
> 	if (!fb)
> 		return 0;
> 
> and so we won't have a new fb there either and so shouldn't try
> to pin it.
> 
> So can't see how that could happen from these normal paths. Which
> leads me to wonder if this might have something to do with nv12
> slave planes...

That may well be it. Looks like we may not end up calling
intel_plane_copy_uapi_to_hw_state() for old slave planes at all,
thus leaving a stale plane_state->hw.fb pointer behind.

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

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Validation rotated vma bounds are within the object

[Patchwork]

== Series Details ==

Series: drm/i915/gt: Validation rotated vma bounds are within the object
URL   : https://patchwork.freedesktop.org/series/71827/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
cd35fb1ef5a1 drm/i915/gt: Validation rotated vma bounds are within the object
-:23: ERROR:BAD_SIGN_OFF: Unrecognized email address: 'Matthew Auld <matthew.auld@intel.com'
#23: 
Cc: Matthew Auld <matthew.auld@intel.com

total: 1 errors, 0 warnings, 0 checks, 33 lines checked

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

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gt: Validation rotated vma bounds are within the object

[Patchwork]

== Series Details ==

Series: drm/i915/gt: Validation rotated vma bounds are within the object
URL   : https://patchwork.freedesktop.org/series/71827/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_7714 -> Patchwork_16042
====================================================

Summary
-------

  **FAILURE**

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

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live_vma:
    - fi-gdg-551:         NOTRUN -> [DMESG-WARN][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-gdg-551/igt@i915_selftest@live_vma.html
    - fi-bsw-nick:        [PASS][2] -> [DMESG-WARN][3]
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-bsw-nick/igt@i915_selftest@live_vma.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-bsw-nick/igt@i915_selftest@live_vma.html
    - fi-skl-lmem:        [PASS][4] -> [DMESG-WARN][5]
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-skl-lmem/igt@i915_selftest@live_vma.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-lmem/igt@i915_selftest@live_vma.html
    - fi-bdw-5557u:       [PASS][6] -> [DMESG-WARN][7]
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-bdw-5557u/igt@i915_selftest@live_vma.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-bdw-5557u/igt@i915_selftest@live_vma.html
    - fi-snb-2600:        [PASS][8] -> [DMESG-WARN][9]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-snb-2600/igt@i915_selftest@live_vma.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-snb-2600/igt@i915_selftest@live_vma.html
    - fi-skl-6770hq:      NOTRUN -> [DMESG-WARN][10]
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-6770hq/igt@i915_selftest@live_vma.html
    - fi-icl-u3:          [PASS][11] -> [DMESG-WARN][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-u3/igt@i915_selftest@live_vma.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-u3/igt@i915_selftest@live_vma.html
    - fi-kbl-soraka:      [PASS][13] -> [DMESG-WARN][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-kbl-soraka/igt@i915_selftest@live_vma.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-kbl-soraka/igt@i915_selftest@live_vma.html
    - fi-skl-6700k2:      [PASS][15] -> [DMESG-WARN][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-skl-6700k2/igt@i915_selftest@live_vma.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-6700k2/igt@i915_selftest@live_vma.html
    - fi-hsw-4770:        [PASS][17] -> [DMESG-WARN][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-hsw-4770/igt@i915_selftest@live_vma.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-hsw-4770/igt@i915_selftest@live_vma.html
    - fi-byt-j1900:       [PASS][19] -> [DMESG-WARN][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-byt-j1900/igt@i915_selftest@live_vma.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-byt-j1900/igt@i915_selftest@live_vma.html
    - fi-blb-e6850:       [PASS][21] -> [DMESG-WARN][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-blb-e6850/igt@i915_selftest@live_vma.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-blb-e6850/igt@i915_selftest@live_vma.html
    - fi-elk-e7500:       [PASS][23] -> [DMESG-WARN][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-elk-e7500/igt@i915_selftest@live_vma.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-elk-e7500/igt@i915_selftest@live_vma.html
    - fi-ilk-650:         [PASS][25] -> [DMESG-WARN][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-ilk-650/igt@i915_selftest@live_vma.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-ilk-650/igt@i915_selftest@live_vma.html
    - fi-cml-u2:          [PASS][27] -> [DMESG-WARN][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-cml-u2/igt@i915_selftest@live_vma.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-cml-u2/igt@i915_selftest@live_vma.html
    - fi-icl-y:           [PASS][29] -> [DMESG-WARN][30]
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-y/igt@i915_selftest@live_vma.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-y/igt@i915_selftest@live_vma.html
    - fi-ivb-3770:        [PASS][31] -> [DMESG-WARN][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-ivb-3770/igt@i915_selftest@live_vma.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-ivb-3770/igt@i915_selftest@live_vma.html
    - fi-bsw-n3050:       NOTRUN -> [DMESG-WARN][33]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-bsw-n3050/igt@i915_selftest@live_vma.html
    - fi-cfl-guc:         [PASS][34] -> [DMESG-WARN][35]
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-cfl-guc/igt@i915_selftest@live_vma.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-cfl-guc/igt@i915_selftest@live_vma.html
    - fi-skl-guc:         [PASS][36] -> [DMESG-WARN][37]
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-skl-guc/igt@i915_selftest@live_vma.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-guc/igt@i915_selftest@live_vma.html
    - fi-pnv-d510:        [PASS][38] -> [DMESG-WARN][39]
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-pnv-d510/igt@i915_selftest@live_vma.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-pnv-d510/igt@i915_selftest@live_vma.html
    - fi-bsw-kefka:       NOTRUN -> [DMESG-WARN][40]
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-bsw-kefka/igt@i915_selftest@live_vma.html
    - fi-kbl-x1275:       [PASS][41] -> [DMESG-WARN][42]
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-kbl-x1275/igt@i915_selftest@live_vma.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-kbl-x1275/igt@i915_selftest@live_vma.html
    - fi-icl-dsi:         [PASS][43] -> [DMESG-WARN][44]
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-dsi/igt@i915_selftest@live_vma.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-dsi/igt@i915_selftest@live_vma.html
    - fi-apl-guc:         [PASS][45] -> [DMESG-WARN][46]
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-apl-guc/igt@i915_selftest@live_vma.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-apl-guc/igt@i915_selftest@live_vma.html
    - fi-glk-dsi:         [PASS][47] -> [DMESG-WARN][48]
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-glk-dsi/igt@i915_selftest@live_vma.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-glk-dsi/igt@i915_selftest@live_vma.html
    - fi-icl-u2:          [PASS][49] -> [DMESG-WARN][50]
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-u2/igt@i915_selftest@live_vma.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-u2/igt@i915_selftest@live_vma.html
    - fi-cfl-8700k:       [PASS][51] -> [DMESG-WARN][52]
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-cfl-8700k/igt@i915_selftest@live_vma.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-cfl-8700k/igt@i915_selftest@live_vma.html
    - fi-kbl-r:           [PASS][53] -> [DMESG-WARN][54]
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-kbl-r/igt@i915_selftest@live_vma.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-kbl-r/igt@i915_selftest@live_vma.html
    - fi-byt-n2820:       [PASS][55] -> [DMESG-WARN][56]
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-byt-n2820/igt@i915_selftest@live_vma.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-byt-n2820/igt@i915_selftest@live_vma.html
    - fi-kbl-guc:         [PASS][57] -> [DMESG-WARN][58]
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-kbl-guc/igt@i915_selftest@live_vma.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-kbl-guc/igt@i915_selftest@live_vma.html
    - fi-icl-guc:         [PASS][59] -> [DMESG-WARN][60]
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-guc/igt@i915_selftest@live_vma.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-guc/igt@i915_selftest@live_vma.html

  
#### Suppressed ####

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

  * igt@i915_selftest@live_vma:
    - {fi-tgl-u}:         [PASS][61] -> [DMESG-WARN][62]
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-tgl-u/igt@i915_selftest@live_vma.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-tgl-u/igt@i915_selftest@live_vma.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-bxt-dsi:         [PASS][63] -> [INCOMPLETE][64] ([fdo#103927])
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6700k2:      [PASS][65] -> [DMESG-WARN][66] ([i915#889])
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html

  * igt@i915_selftest@live_blt:
    - fi-ivb-3770:        [PASS][67] -> [DMESG-FAIL][68] ([i915#770])
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-ivb-3770/igt@i915_selftest@live_blt.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-ivb-3770/igt@i915_selftest@live_blt.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s3:
    - fi-icl-u2:          [FAIL][69] ([fdo#103375]) -> [PASS][70]
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-u2/igt@gem_exec_suspend@basic-s3.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-u2/igt@gem_exec_suspend@basic-s3.html

  * igt@gem_exec_suspend@basic-s4-devices:
    - fi-icl-u2:          [FAIL][71] ([fdo#111550]) -> [PASS][72]
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-u2/igt@gem_exec_suspend@basic-s4-devices.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-u2/igt@gem_exec_suspend@basic-s4-devices.html

  * igt@i915_selftest@live_blt:
    - fi-hsw-4770:        [DMESG-FAIL][73] ([i915#725]) -> [PASS][74]
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-hsw-4770/igt@i915_selftest@live_blt.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-hsw-4770/igt@i915_selftest@live_blt.html

  * igt@i915_selftest@live_gem_contexts:
    - fi-skl-lmem:        [INCOMPLETE][75] ([i915#424]) -> [PASS][76]
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-skl-lmem/igt@i915_selftest@live_gem_contexts.html
    - fi-cfl-8700k:       [DMESG-FAIL][77] ([i915#623]) -> [PASS][78]
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html
    - fi-byt-n2820:       [DMESG-FAIL][79] ([i915#722]) -> [PASS][80]
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-byt-n2820/igt@i915_selftest@live_gem_contexts.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-byt-n2820/igt@i915_selftest@live_gem_contexts.html

  
#### Warnings ####

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-icl-u2:          [FAIL][81] ([fdo#103375]) -> [DMESG-WARN][82] ([IGT#4] / [i915#263])
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7714/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16042/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html

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

  [IGT#4]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/4
  [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#111550]: https://bugs.freedesktop.org/show_bug.cgi?id=111550
  [i915#263]: https://gitlab.freedesktop.org/drm/intel/issues/263
  [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424
  [i915#623]: https://gitlab.freedesktop.org/drm/intel/issues/623
  [i915#722]: https://gitlab.freedesktop.org/drm/intel/issues/722
  [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
  [i915#770]: https://gitlab.freedesktop.org/drm/intel/issues/770
  [i915#889]: https://gitlab.freedesktop.org/drm/intel/issues/889


Participating hosts (46 -> 39)
------------------------------

  Additional (5): fi-bsw-n3050 fi-skl-6770hq fi-gdg-551 fi-bsw-kefka fi-skl-6600u 
  Missing    (12): fi-hsw-4770r fi-ehl-1 fi-hsw-4200u fi-hsw-peppy fi-byt-squawks fi-bwr-2160 fi-snb-2520m fi-kbl-7500u fi-ctg-p8600 fi-whl-u fi-tgl-y fi-byt-clapper 


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

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_7714 -> Patchwork_16042

  CI-20190529: 20190529
  CI_DRM_7714: b633f28f2de80cdb861d6c1c3b4df6fd2d53239f @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5358: c6fc013f414b806175dc4143c58ab445e5235ea5 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_16042: cd35fb1ef5a1de344a9a544883c5f16e3b7630fd @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

cd35fb1ef5a1 drm/i915/gt: Validation rotated vma bounds are within the object

== Logs ==

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

[Intel-gfx] [PATCH v2] drm/i915/gt: Validate rotated vma bounds are within the object

[Chris Wilson]

Quite understandably, we bug out when asked to find a page that doesn't
belong to the object. However, we should report the error back to the
user long before we attempt the out-of-bound access! In this case, it is
insufficient validation on the rotated vma, with the simplest/cheapest
point for us to insert a bound check when we are computing the rotated
page lookups.

Similarly, it might be wise to see if we can validate the user input
upon creating the rotated framebuffer.

v2: Skip empty planes

Closes: https://gitlab.freedesktop.org/drm/intel/issues/951
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Matthew Auld <matthew.auld@intel.com
---
 drivers/gpu/drm/i915/gt/intel_ggtt.c      | 25 +++++++++++++++++++----
 drivers/gpu/drm/i915/selftests/i915_vma.c |  1 +
 2 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
index 79096722ce16..4a646d74327a 100644
--- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
@@ -1266,6 +1266,7 @@ static noinline struct sg_table *
 intel_rotate_pages(struct intel_rotation_info *rot_info,
 		   struct drm_i915_gem_object *obj)
 {
+	const unsigned long npages = obj->base.size >> PAGE_SHIFT;
 	unsigned int size = intel_rotation_info_size(rot_info);
 	struct sg_table *st;
 	struct scatterlist *sg;
@@ -1285,9 +1286,26 @@ intel_rotate_pages(struct intel_rotation_info *rot_info,
 	sg = st->sgl;
 
 	for (i = 0 ; i < ARRAY_SIZE(rot_info->plane); i++) {
-		sg = rotate_pages(obj, rot_info->plane[i].offset,
-				  rot_info->plane[i].width, rot_info->plane[i].height,
-				  rot_info->plane[i].stride, st, sg);
+		const struct intel_remapped_plane_info *plane =
+			&rot_info->plane[i];
+		unsigned long last;
+
+		if (!plane->height || !plane->width)
+			continue;
+
+		last = plane->offset;
+		last += (plane->height - 1) * plane->stride;
+		last += plane->width - 1;
+		if (last >= npages) {
+			ret = -EINVAL;
+			goto err_sg_alloc;
+		}
+
+		sg = rotate_pages(obj,
+				  plane->offset,
+				  plane->width, plane->height,
+				  plane->stride,
+				  st, sg);
 	}
 
 	return st;
@@ -1295,7 +1313,6 @@ intel_rotate_pages(struct intel_rotation_info *rot_info,
 err_sg_alloc:
 	kfree(st);
 err_st_alloc:
-
 	DRM_DEBUG_DRIVER("Failed to create rotated mapping for object size %zu! (%ux%u tiles, %u pages)\n",
 			 obj->base.size, rot_info->plane[0].width, rot_info->plane[0].height, size);
 
diff --git a/drivers/gpu/drm/i915/selftests/i915_vma.c b/drivers/gpu/drm/i915/selftests/i915_vma.c
index 58b5f40a07dd..24e7806867ef 100644
--- a/drivers/gpu/drm/i915/selftests/i915_vma.c
+++ b/drivers/gpu/drm/i915/selftests/i915_vma.c
@@ -962,6 +962,7 @@ static int igt_vma_remapped_gtt(void *arg)
 						       *t == I915_GGTT_VIEW_ROTATED ? "Rotated" : "Remapped",
 						       val, exp);
 						i915_vma_unpin_iomap(vma);
+						err = -EINVAL;
 						goto out;
 					}
 				}
-- 
2.25.0.rc2

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

Re: [Intel-gfx] [PATCH v2] drm/i915/gt: Validate rotated vma bounds are within the object

[Chris Wilson]

Quoting Chris Wilson (2020-01-10 15:53:34)
> Quite understandably, we bug out when asked to find a page that doesn't
> belong to the object. However, we should report the error back to the
> user long before we attempt the out-of-bound access! In this case, it is
> insufficient validation on the rotated vma, with the simplest/cheapest
> point for us to insert a bound check when we are computing the rotated
> page lookups.
> 
> Similarly, it might be wise to see if we can validate the user input
> upon creating the rotated framebuffer.
Reminder to self, scratch this comment. They are, it's just a bug.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gt: Validation rotated vma bounds are within the object (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/gt: Validation rotated vma bounds are within the object (rev2)
URL   : https://patchwork.freedesktop.org/series/71827/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
933176d41e65 drm/i915/gt: Validate rotated vma bounds are within the object
-:25: ERROR:BAD_SIGN_OFF: Unrecognized email address: 'Matthew Auld <matthew.auld@intel.com'
#25: 
Cc: Matthew Auld <matthew.auld@intel.com

total: 1 errors, 0 warnings, 0 checks, 50 lines checked

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

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Validation rotated vma bounds are within the object (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/gt: Validation rotated vma bounds are within the object (rev2)
URL   : https://patchwork.freedesktop.org/series/71827/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_7720 -> Patchwork_16056
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/index.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_close_race@basic-threads:
    - fi-byt-j1900:       [PASS][1] -> [TIMEOUT][2] ([i915#816])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-byt-j1900/igt@gem_close_race@basic-threads.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-byt-j1900/igt@gem_close_race@basic-threads.html

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-skl-6770hq:      [PASS][3] -> [DMESG-WARN][4] ([i915#889])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html

  * igt@i915_selftest@live_blt:
    - fi-hsw-4770:        [PASS][5] -> [DMESG-FAIL][6] ([i915#725])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-hsw-4770/igt@i915_selftest@live_blt.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-hsw-4770/igt@i915_selftest@live_blt.html

  * igt@i915_selftest@live_gem_contexts:
    - fi-cfl-guc:         [PASS][7] -> [INCOMPLETE][8] ([fdo#106070] / [i915#424])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html

  * igt@i915_selftest@live_hangcheck:
    - fi-skl-guc:         [PASS][9] -> [INCOMPLETE][10] ([fdo#108744])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-skl-guc/igt@i915_selftest@live_hangcheck.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-skl-guc/igt@i915_selftest@live_hangcheck.html

  * igt@kms_chamelium@dp-edid-read:
    - fi-icl-u2:          [PASS][11] -> [FAIL][12] ([fdo#109635] / [i915#217])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       [PASS][13] -> [FAIL][14] ([fdo#111096] / [i915#323])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html

  
#### Possible fixes ####

  * igt@gem_exec_parallel@basic:
    - {fi-ehl-1}:         [INCOMPLETE][15] ([i915#937]) -> [PASS][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-ehl-1/igt@gem_exec_parallel@basic.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-ehl-1/igt@gem_exec_parallel@basic.html

  * igt@i915_selftest@live_blt:
    - fi-hsw-4770r:       [DMESG-FAIL][17] ([i915#770]) -> [PASS][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-hsw-4770r/igt@i915_selftest@live_blt.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-hsw-4770r/igt@i915_selftest@live_blt.html

  
#### Warnings ####

  * igt@i915_pm_rpm@module-reload:
    - fi-skl-6770hq:      [FAIL][19] ([i915#178]) -> [INCOMPLETE][20] ([i915#151])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html

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

  [fdo#106070]: https://bugs.freedesktop.org/show_bug.cgi?id=106070
  [fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744
  [fdo#109635]: https://bugs.freedesktop.org/show_bug.cgi?id=109635
  [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
  [i915#151]: https://gitlab.freedesktop.org/drm/intel/issues/151
  [i915#178]: https://gitlab.freedesktop.org/drm/intel/issues/178
  [i915#217]: https://gitlab.freedesktop.org/drm/intel/issues/217
  [i915#323]: https://gitlab.freedesktop.org/drm/intel/issues/323
  [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424
  [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
  [i915#770]: https://gitlab.freedesktop.org/drm/intel/issues/770
  [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816
  [i915#889]: https://gitlab.freedesktop.org/drm/intel/issues/889
  [i915#937]: https://gitlab.freedesktop.org/drm/intel/issues/937


Participating hosts (51 -> 40)
------------------------------

  Additional (1): fi-tgl-u 
  Missing    (12): fi-hsw-4200u fi-hsw-peppy fi-glk-dsi fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-ivb-3770 fi-bsw-kefka fi-skl-lmem fi-blb-e6850 fi-byt-clapper 


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

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_7720 -> Patchwork_16056

  CI-20190529: 20190529
  CI_DRM_7720: 3770105165843883b1422bb3bc5bc8601dfc8051 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5364: b7cb6ffdb65cbd233f5ddee2f2dabf97b34fa640 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_16056: 933176d41e65348c1a9ce7eeaadb6cc8c97dae32 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

933176d41e65 drm/i915/gt: Validate rotated vma bounds are within the object

== Logs ==

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

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Validation rotated vma bounds are within the object (rev2)

[Patchwork]

== Series Details ==

Series: drm/i915/gt: Validation rotated vma bounds are within the object (rev2)
URL   : https://patchwork.freedesktop.org/series/71827/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_7720_full -> Patchwork_16056_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_isolation@rcs0-s3:
    - shard-kbl:          [PASS][1] -> [DMESG-WARN][2] ([i915#180]) +4 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-kbl2/igt@gem_ctx_isolation@rcs0-s3.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-kbl6/igt@gem_ctx_isolation@rcs0-s3.html

  * igt@gem_ctx_persistence@processes:
    - shard-tglb:         [PASS][3] -> [FAIL][4] ([i915#570])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb5/igt@gem_ctx_persistence@processes.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb3/igt@gem_ctx_persistence@processes.html

  * igt@gem_ctx_persistence@vcs1-mixed-process:
    - shard-iclb:         [PASS][5] -> [SKIP][6] ([fdo#109276] / [fdo#112080]) +3 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@gem_ctx_persistence@vcs1-mixed-process.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb6/igt@gem_ctx_persistence@vcs1-mixed-process.html

  * igt@gem_ctx_shared@q-smoketest-all:
    - shard-tglb:         [PASS][7] -> [INCOMPLETE][8] ([fdo#111735]) +1 similar issue
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb7/igt@gem_ctx_shared@q-smoketest-all.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb8/igt@gem_ctx_shared@q-smoketest-all.html

  * igt@gem_eio@in-flight-contexts-1us:
    - shard-snb:          [PASS][9] -> [FAIL][10] ([i915#490])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-snb4/igt@gem_eio@in-flight-contexts-1us.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-snb4/igt@gem_eio@in-flight-contexts-1us.html

  * igt@gem_exec_balancer@nop:
    - shard-tglb:         [PASS][11] -> [INCOMPLETE][12] ([fdo#111736] / [i915#472])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb3/igt@gem_exec_balancer@nop.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb1/igt@gem_exec_balancer@nop.html

  * igt@gem_exec_gttfill@basic:
    - shard-tglb:         [PASS][13] -> [INCOMPLETE][14] ([fdo#111593] / [i915#472])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb5/igt@gem_exec_gttfill@basic.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb3/igt@gem_exec_gttfill@basic.html

  * igt@gem_exec_parallel@vcs1-fds:
    - shard-iclb:         [PASS][15] -> [SKIP][16] ([fdo#112080]) +10 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb4/igt@gem_exec_parallel@vcs1-fds.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb3/igt@gem_exec_parallel@vcs1-fds.html

  * igt@gem_exec_schedule@pi-shared-iova-bsd:
    - shard-iclb:         [PASS][17] -> [SKIP][18] ([i915#677]) +1 similar issue
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb7/igt@gem_exec_schedule@pi-shared-iova-bsd.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb2/igt@gem_exec_schedule@pi-shared-iova-bsd.html

  * igt@gem_exec_schedule@pi-shared-iova-vebox:
    - shard-glk:          [PASS][19] -> [TIMEOUT][20] ([fdo#112271])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk4/igt@gem_exec_schedule@pi-shared-iova-vebox.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk9/igt@gem_exec_schedule@pi-shared-iova-vebox.html

  * igt@gem_exec_schedule@preempt-queue-blt:
    - shard-tglb:         [PASS][21] -> [INCOMPLETE][22] ([fdo#111677] / [i915#472]) +1 similar issue
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb1/igt@gem_exec_schedule@preempt-queue-blt.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb8/igt@gem_exec_schedule@preempt-queue-blt.html

  * igt@gem_exec_schedule@preempt-queue-contexts-bsd1:
    - shard-tglb:         [PASS][23] -> [INCOMPLETE][24] ([fdo#111606] / [fdo#111677] / [i915#472])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb3/igt@gem_exec_schedule@preempt-queue-contexts-bsd1.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb8/igt@gem_exec_schedule@preempt-queue-contexts-bsd1.html

  * igt@gem_exec_schedule@reorder-wide-bsd:
    - shard-iclb:         [PASS][25] -> [SKIP][26] ([fdo#112146]) +4 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb6/igt@gem_exec_schedule@reorder-wide-bsd.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb4/igt@gem_exec_schedule@reorder-wide-bsd.html

  * igt@gem_persistent_relocs@forked-faulting-reloc-thrashing:
    - shard-glk:          [PASS][27] -> [INCOMPLETE][28] ([CI#80] / [i915#58] / [k.org#198133])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk2/igt@gem_persistent_relocs@forked-faulting-reloc-thrashing.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk8/igt@gem_persistent_relocs@forked-faulting-reloc-thrashing.html

  * igt@gem_ppgtt@blt-vs-render-ctxn:
    - shard-tglb:         [PASS][29] -> [INCOMPLETE][30] ([i915#470] / [i915#475])
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb1/igt@gem_ppgtt@blt-vs-render-ctxn.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb3/igt@gem_ppgtt@blt-vs-render-ctxn.html

  * igt@gem_softpin@noreloc-s3:
    - shard-skl:          [PASS][31] -> [INCOMPLETE][32] ([i915#69]) +1 similar issue
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl10/igt@gem_softpin@noreloc-s3.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl8/igt@gem_softpin@noreloc-s3.html

  * igt@gem_sync@basic-many-each:
    - shard-tglb:         [PASS][33] -> [INCOMPLETE][34] ([i915#472] / [i915#707])
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb1/igt@gem_sync@basic-many-each.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb3/igt@gem_sync@basic-many-each.html

  * igt@i915_pm_rps@waitboost:
    - shard-iclb:         [PASS][35] -> [FAIL][36] ([i915#413])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb8/igt@i915_pm_rps@waitboost.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb8/igt@i915_pm_rps@waitboost.html

  * igt@kms_color@pipe-b-ctm-0-75:
    - shard-skl:          [PASS][37] -> [DMESG-WARN][38] ([i915#109]) +1 similar issue
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl1/igt@kms_color@pipe-b-ctm-0-75.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl8/igt@kms_color@pipe-b-ctm-0-75.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
    - shard-glk:          [PASS][39] -> [FAIL][40] ([i915#79])
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk4/igt@kms_flip@flip-vs-expired-vblank-interruptible.html

  * igt@kms_flip@flip-vs-suspend:
    - shard-apl:          [PASS][41] -> [DMESG-WARN][42] ([i915#180])
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-apl2/igt@kms_flip@flip-vs-suspend.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-apl1/igt@kms_flip@flip-vs-suspend.html
    - shard-iclb:         [PASS][43] -> [DMESG-WARN][44] ([fdo#111764])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@kms_flip@flip-vs-suspend.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb2/igt@kms_flip@flip-vs-suspend.html

  * igt@kms_frontbuffer_tracking@fbc-stridechange:
    - shard-tglb:         [PASS][45] -> [FAIL][46] ([i915#49]) +1 similar issue
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb2/igt@kms_frontbuffer_tracking@fbc-stridechange.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb7/igt@kms_frontbuffer_tracking@fbc-stridechange.html

  * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
    - shard-skl:          [PASS][47] -> [FAIL][48] ([fdo#108145] / [i915#265])
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl8/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl8/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html

  * igt@kms_psr@psr2_sprite_mmap_gtt:
    - shard-iclb:         [PASS][49] -> [SKIP][50] ([fdo#109441])
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb5/igt@kms_psr@psr2_sprite_mmap_gtt.html

  * igt@perf_pmu@enable-race-vcs1:
    - shard-tglb:         [PASS][51] -> [INCOMPLETE][52] ([i915#472]) +2 similar issues
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb1/igt@perf_pmu@enable-race-vcs1.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb8/igt@perf_pmu@enable-race-vcs1.html

  * igt@prime_vgem@fence-wait-bsd2:
    - shard-iclb:         [PASS][53] -> [SKIP][54] ([fdo#109276]) +20 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb4/igt@prime_vgem@fence-wait-bsd2.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb3/igt@prime_vgem@fence-wait-bsd2.html

  
#### Possible fixes ####

  * igt@gem_ctx_isolation@rcs0-s3:
    - shard-skl:          [INCOMPLETE][55] ([i915#69]) -> [PASS][56]
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl8/igt@gem_ctx_isolation@rcs0-s3.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl10/igt@gem_ctx_isolation@rcs0-s3.html

  * igt@gem_ctx_isolation@vcs1-dirty-create:
    - shard-iclb:         [SKIP][57] ([fdo#109276] / [fdo#112080]) -> [PASS][58] +2 similar issues
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb7/igt@gem_ctx_isolation@vcs1-dirty-create.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb2/igt@gem_ctx_isolation@vcs1-dirty-create.html

  * igt@gem_ctx_shared@exec-single-timeline-bsd:
    - shard-iclb:         [SKIP][59] ([fdo#110841]) -> [PASS][60]
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@gem_ctx_shared@exec-single-timeline-bsd.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb8/igt@gem_ctx_shared@exec-single-timeline-bsd.html

  * igt@gem_ctx_shared@q-smoketest-bsd1:
    - shard-tglb:         [INCOMPLETE][61] ([fdo#111735]) -> [PASS][62]
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb3/igt@gem_ctx_shared@q-smoketest-bsd1.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb2/igt@gem_ctx_shared@q-smoketest-bsd1.html

  * igt@gem_eio@unwedge-stress:
    - shard-snb:          [FAIL][63] ([i915#232]) -> [PASS][64]
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-snb6/igt@gem_eio@unwedge-stress.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-snb6/igt@gem_eio@unwedge-stress.html

  * igt@gem_exec_create@basic:
    - shard-tglb:         [INCOMPLETE][65] ([fdo#111736] / [i915#472]) -> [PASS][66]
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb4/igt@gem_exec_create@basic.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb2/igt@gem_exec_create@basic.html

  * igt@gem_exec_parallel@basic:
    - shard-tglb:         [INCOMPLETE][67] ([i915#472] / [i915#476]) -> [PASS][68]
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb3/igt@gem_exec_parallel@basic.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb5/igt@gem_exec_parallel@basic.html

  * igt@gem_exec_schedule@in-order-bsd:
    - shard-iclb:         [SKIP][69] ([fdo#112146]) -> [PASS][70] +7 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@gem_exec_schedule@in-order-bsd.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb7/igt@gem_exec_schedule@in-order-bsd.html

  * igt@gem_exec_schedule@independent-bsd2:
    - shard-iclb:         [SKIP][71] ([fdo#109276]) -> [PASS][72] +25 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb7/igt@gem_exec_schedule@independent-bsd2.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb4/igt@gem_exec_schedule@independent-bsd2.html

  * igt@gem_exec_schedule@pi-common-bsd:
    - shard-iclb:         [SKIP][73] ([i915#677]) -> [PASS][74]
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb2/igt@gem_exec_schedule@pi-common-bsd.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb6/igt@gem_exec_schedule@pi-common-bsd.html

  * igt@gem_exec_schedule@smoketest-bsd1:
    - shard-tglb:         [INCOMPLETE][75] ([i915#463] / [i915#472]) -> [PASS][76]
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb9/igt@gem_exec_schedule@smoketest-bsd1.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb1/igt@gem_exec_schedule@smoketest-bsd1.html

  * igt@gem_exec_schedule@smoketest-bsd2:
    - shard-tglb:         [INCOMPLETE][77] ([i915#472] / [i915#707]) -> [PASS][78]
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb3/igt@gem_exec_schedule@smoketest-bsd2.html
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb3/igt@gem_exec_schedule@smoketest-bsd2.html

  * igt@gem_persistent_relocs@forked-interruptible:
    - shard-kbl:          [FAIL][79] ([i915#520]) -> [PASS][80]
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-kbl6/igt@gem_persistent_relocs@forked-interruptible.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-kbl7/igt@gem_persistent_relocs@forked-interruptible.html

  * igt@gem_softpin@noreloc-s3:
    - shard-apl:          [DMESG-WARN][81] ([i915#180]) -> [PASS][82] +1 similar issue
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-apl4/igt@gem_softpin@noreloc-s3.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-apl3/igt@gem_softpin@noreloc-s3.html

  * igt@gem_sync@basic-store-each:
    - shard-tglb:         [INCOMPLETE][83] ([i915#472]) -> [PASS][84]
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb5/igt@gem_sync@basic-store-each.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb7/igt@gem_sync@basic-store-each.html

  * igt@kms_flip@plain-flip-fb-recreate:
    - shard-skl:          [FAIL][85] ([i915#34]) -> [PASS][86]
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl3/igt@kms_flip@plain-flip-fb-recreate.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl10/igt@kms_flip@plain-flip-fb-recreate.html

  * igt@kms_flip@plain-flip-ts-check:
    - shard-glk:          [FAIL][87] ([i915#34]) -> [PASS][88]
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk3/igt@kms_flip@plain-flip-ts-check.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk1/igt@kms_flip@plain-flip-ts-check.html

  * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite:
    - shard-tglb:         [FAIL][89] ([i915#49]) -> [PASS][90] +1 similar issue
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb8/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb6/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html

  * igt@kms_plane@pixel-format-pipe-a-planes:
    - shard-tglb:         [INCOMPLETE][91] -> [PASS][92]
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb8/igt@kms_plane@pixel-format-pipe-a-planes.html
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb9/igt@kms_plane@pixel-format-pipe-a-planes.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
    - shard-kbl:          [DMESG-WARN][93] ([i915#180]) -> [PASS][94] +8 similar issues
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-kbl3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-kbl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html

  * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
    - shard-skl:          [FAIL][95] ([fdo#108145]) -> [PASS][96]
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl6/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl3/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html

  * igt@kms_psr@psr2_primary_mmap_gtt:
    - shard-iclb:         [SKIP][97] ([fdo#109441]) -> [PASS][98] +2 similar issues
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb1/igt@kms_psr@psr2_primary_mmap_gtt.html
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb2/igt@kms_psr@psr2_primary_mmap_gtt.html

  * igt@kms_setmode@basic:
    - shard-apl:          [FAIL][99] ([i915#31]) -> [PASS][100]
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-apl1/igt@kms_setmode@basic.html
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-apl6/igt@kms_setmode@basic.html
    - shard-skl:          [FAIL][101] ([i915#31]) -> [PASS][102]
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl1/igt@kms_setmode@basic.html
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl9/igt@kms_setmode@basic.html

  * igt@perf@disabled-read-error:
    - shard-iclb:         [DMESG-WARN][103] ([i915#645]) -> [PASS][104]
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb6/igt@perf@disabled-read-error.html
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb3/igt@perf@disabled-read-error.html

  * igt@perf_pmu@idle-vcs1:
    - shard-iclb:         [SKIP][105] ([fdo#112080]) -> [PASS][106] +9 similar issues
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb5/igt@perf_pmu@idle-vcs1.html
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb1/igt@perf_pmu@idle-vcs1.html

  
#### Warnings ####

  * igt@gem_ctx_isolation@vcs1-nonpriv:
    - shard-iclb:         [SKIP][107] ([fdo#109276] / [fdo#112080]) -> [FAIL][108] ([IGT#28])
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-iclb5/igt@gem_ctx_isolation@vcs1-nonpriv.html
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-iclb4/igt@gem_ctx_isolation@vcs1-nonpriv.html

  * igt@gem_ctx_isolation@vcs2-none:
    - shard-tglb:         [SKIP][109] ([fdo#112080]) -> [SKIP][110] ([fdo#111912] / [fdo#112080])
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb9/igt@gem_ctx_isolation@vcs2-none.html
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb1/igt@gem_ctx_isolation@vcs2-none.html

  * igt@gem_pwrite@self:
    - shard-snb:          [DMESG-WARN][111] ([i915#975]) -> [DMESG-WARN][112] ([i915#478])
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-snb2/igt@gem_pwrite@self.html
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-snb1/igt@gem_pwrite@self.html

  * igt@gem_tiled_blits@interruptible:
    - shard-hsw:          [FAIL][113] ([i915#694]) -> [FAIL][114] ([i915#818])
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-hsw7/igt@gem_tiled_blits@interruptible.html
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-hsw1/igt@gem_tiled_blits@interruptible.html

  * igt@gem_tiled_blits@normal:
    - shard-hsw:          [FAIL][115] ([i915#818]) -> [FAIL][116] ([i915#694])
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-hsw5/igt@gem_tiled_blits@normal.html
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-hsw7/igt@gem_tiled_blits@normal.html

  * igt@i915_pm_dc@dc6-dpms:
    - shard-tglb:         [SKIP][117] ([i915#468]) -> [FAIL][118] ([i915#454])
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-tglb8/igt@i915_pm_dc@dc6-dpms.html
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-tglb9/igt@i915_pm_dc@dc6-dpms.html

  * igt@i915_selftest@mock_timelines:
    - shard-glk:          [INCOMPLETE][119] ([i915#58] / [k.org#198133]) -> [INCOMPLETE][120] ([i915#58] / [i915#974] / [k.org#198133])
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk4/igt@i915_selftest@mock_timelines.html
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk2/igt@i915_selftest@mock_timelines.html
    - shard-kbl:          [INCOMPLETE][121] ([fdo#103665]) -> [INCOMPLETE][122] ([fdo#103665] / [i915#974])
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-kbl7/igt@i915_selftest@mock_timelines.html
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-kbl2/igt@i915_selftest@mock_timelines.html
    - shard-apl:          [INCOMPLETE][123] ([fdo#103927]) -> [INCOMPLETE][124] ([fdo#103927] / [i915#974])
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-apl2/igt@i915_selftest@mock_timelines.html
   [124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-apl8/igt@i915_selftest@mock_timelines.html

  * igt@runner@aborted:
    - shard-apl:          [FAIL][125] ([fdo#103927]) -> [FAIL][126] ([fdo#103927] / [i915#974])
   [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-apl2/igt@runner@aborted.html
   [126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-apl8/igt@runner@aborted.html
    - shard-glk:          [FAIL][127] ([k.org#202321]) -> ([FAIL][128], [FAIL][129]) ([i915#974] / [k.org#202321])
   [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-glk4/igt@runner@aborted.html
   [128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk9/igt@runner@aborted.html
   [129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-glk2/igt@runner@aborted.html
    - shard-skl:          [FAIL][130] ([i915#69]) -> [FAIL][131] ([i915#69] / [i915#974])
   [130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7720/shard-skl8/igt@runner@aborted.html
   [131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16056/shard-skl3/igt@runner@aborted.html

  
  [CI#80]: https://gitlab.freedesktop.org/gfx-ci/i915-infra/issues/80
  [IGT#28]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/28
  [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841
  [fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
  [fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
  [fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
  [fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735
  [fdo#111736]: https://bugs.freedesktop.org/show_bug.cgi?id=111736
  [fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764
  [fdo#111912]: https://bugs.freedesktop.org/show_bug.cgi?id=111912
  [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
  [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
  [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271
  [i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109
  [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
  [i915#232]: https://gitlab.freedesktop.org/drm/intel/issues/232
  [i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
  [i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31
  [i915#34]: https://gitlab.freedesktop.org/drm/intel/issues/34
  [i915#413]: https://gitlab.freedesktop.org/drm/intel/issues/413
  [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
  [i915#463]: https://gitlab.freedesktop.org/drm/intel/issues/463
  [i915#468]: https://gitlab.freedesktop.org/drm/intel/issues/468
  [i915#470]: https://gitlab.freedesktop.org/drm/intel/issues/470
  [i915#472]: https://gitlab.freedesktop.org/drm/intel/issues/472
  [i915#475]: https://gitlab.freedesktop.org/drm/intel/issues/475
  [i915#476]: https://gitlab.freedesktop.org/drm/intel/issues/476
  [i915#478]: https://gitlab.freedesktop.org/drm/intel/issues/478
  [i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
  [i915#490]: https://gitlab.freedesktop.org/drm/intel/issues/490
  [i915#520]: https://gitlab.freedesktop.org/drm/intel/issues/520
  [i915#570]: https://gitlab.freedesktop.org/drm/intel/issues/570
  [i915#58]: https://gitlab.freedesktop.org/drm/intel/issues/58
  [i915#645]: https://gitlab.freedesktop.org/drm/intel/issues/645
  [i915#677]: https://gitlab.freedesktop.org/drm/intel/issues/677
  [i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69
  [i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694
  [i915#707]: https://gitlab.freedesktop.org/drm/intel/issues/707
  [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
  [i915#818]: https://gitlab.freedesktop.org/drm/intel/issues/818
  [i915#974]: https://gitlab.freedesktop.org/drm/intel/issues/974
  [i915#975]: https://gitlab.freedesktop.org/drm/intel/issues/975
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
  [k.org#202321]: https://bugzilla.kernel.org/show_bug.cgi?id=202321


Participating hosts (10 -> 11)
------------------------------

  Additional (1): pig-hsw-4770r 


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

  * CI: CI-20190529 -> None
  * Linux: CI_DRM_7720 -> Patchwork_16056

  CI-20190529: 20190529
  CI_DRM_7720: 3770105165843883b1422bb3bc5bc8601dfc8051 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_5364: b7cb6ffdb65cbd233f5ddee2f2dabf97b34fa640 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_16056: 933176d41e65348c1a9ce7eeaadb6cc8c97dae32 @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

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