All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master
@ 2018-05-14 10:19 Katarzyna Dec
  2018-05-14 10:44 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (11 more replies)
  0 siblings, 12 replies; 21+ messages in thread
From: Katarzyna Dec @ 2018-05-14 10:19 UTC (permalink / raw)
  To: igt-dev

Let's add additional information to igt_device_drop_master
in case we try to run tests with e.g. x11 enabled.

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_device.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/igt_device.c b/lib/igt_device.c
index 8a2dd0e3..96fde899 100644
--- a/lib/igt_device.c
+++ b/lib/igt_device.c
@@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
  */
 void igt_device_drop_master(int fd)
 {
-	igt_assert_eq(__igt_device_drop_master(fd), 0);
+	if (__igt_device_drop_master(fd)) {
+		igt_require_f(__igt_device_drop_master(fd) == 0,
+			      "Can't drop DRM master, "
+			      "please check if no other DRM client is running.\n");
+	}
 }
-- 
2.14.3

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
@ 2018-05-14 10:44 ` Patchwork
  2018-05-14 10:52 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-14 10:44 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4175 -> IGTPW_1352 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/1/mbox/

== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@debugfs_test@read_all_entries:
      fi-snb-2520m:       PASS -> INCOMPLETE (fdo#103713)

    igt@drv_module_reload@basic-reload:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106248)

    igt@kms_chamelium@dp-crc-fast:
      fi-kbl-7500u:       PASS -> DMESG-FAIL (fdo#103841)

    
    ==== Possible fixes ====

    igt@gem_exec_flush@basic-uc-set-default:
      fi-byt-j1900:       INCOMPLETE (fdo#102657) -> PASS

    
  fdo#102657 https://bugs.freedesktop.org/show_bug.cgi?id=102657
  fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
  fdo#103841 https://bugs.freedesktop.org/show_bug.cgi?id=103841
  fdo#106248 https://bugs.freedesktop.org/show_bug.cgi?id=106248


== Participating hosts (41 -> 37) ==

  Missing    (4): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-skl-6700hq 


== Build changes ==

    * IGT: IGT_4476 -> IGTPW_1352

  CI_DRM_4175: a57366b6029ac86436ad36bbf8b9a31549ef2905 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1352: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1352/
  IGT_4476: 03a62cf055481f66b4f58e6228bc45f8ca454216 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4476: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1352/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
  2018-05-14 10:44 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
@ 2018-05-14 10:52 ` Chris Wilson
  2018-05-14 11:37 ` [igt-dev] [PATCH i-g-t v2] " Katarzyna Dec
                   ` (9 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Chris Wilson @ 2018-05-14 10:52 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Katarzyna Dec (2018-05-14 11:19:30)
> Let's add additional information to igt_device_drop_master
> in case we try to run tests with e.g. x11 enabled.
> 
> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_device.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_device.c b/lib/igt_device.c
> index 8a2dd0e3..96fde899 100644
> --- a/lib/igt_device.c
> +++ b/lib/igt_device.c
> @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
>   */
>  void igt_device_drop_master(int fd)
>  {
> -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> +       if (__igt_device_drop_master(fd)) {
> +               igt_require_f(__igt_device_drop_master(fd) == 0,
> +                             "Can't drop DRM master, "
> +                             "please check if no other DRM client is running.\n");

How does one check? Could we not present that information to the user?  ;)

It should remain an assert here I think.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] [PATCH i-g-t v2] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
  2018-05-14 10:44 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
  2018-05-14 10:52 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
@ 2018-05-14 11:37 ` Katarzyna Dec
  2018-05-14 12:39   ` Chris Wilson
  2018-05-14 12:51   ` [igt-dev] [PATCH i-g-t v3] " Katarzyna Dec
  2018-05-14 14:19 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork
                   ` (8 subsequent siblings)
  11 siblings, 2 replies; 21+ messages in thread
From: Katarzyna Dec @ 2018-05-14 11:37 UTC (permalink / raw)
  To: igt-dev

Let's add additional information to igt_device_drop_master
in case we try to run tests with e.g. x11 enabled.

v2: Changed comment and added listing clients.

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_device.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/igt_device.c b/lib/igt_device.c
index 8a2dd0e3..b2a0a160 100644
--- a/lib/igt_device.c
+++ b/lib/igt_device.c
@@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
  */
 void igt_device_drop_master(int fd)
 {
-	igt_assert_eq(__igt_device_drop_master(fd), 0);
+	if (__igt_device_drop_master(fd)) {
+		igt_debugfs_dump(fd, "clients");
+		igt_require_f(__igt_device_drop_master(fd) == 0,
+			      "Failed to drop DRM master.\n");
+	}
 }
-- 
2.14.3

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v2] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 11:37 ` [igt-dev] [PATCH i-g-t v2] " Katarzyna Dec
@ 2018-05-14 12:39   ` Chris Wilson
  2018-05-14 12:44     ` Chris Wilson
  2018-05-14 12:51   ` [igt-dev] [PATCH i-g-t v3] " Katarzyna Dec
  1 sibling, 1 reply; 21+ messages in thread
From: Chris Wilson @ 2018-05-14 12:39 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Katarzyna Dec (2018-05-14 12:37:22)
> Let's add additional information to igt_device_drop_master
> in case we try to run tests with e.g. x11 enabled.
> 
> v2: Changed comment and added listing clients.
> 
> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_device.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_device.c b/lib/igt_device.c
> index 8a2dd0e3..b2a0a160 100644
> --- a/lib/igt_device.c
> +++ b/lib/igt_device.c
> @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
>   */
>  void igt_device_drop_master(int fd)
>  {
> -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> +       if (__igt_device_drop_master(fd)) {
> +               igt_debugfs_dump(fd, "clients");
> +               igt_require_f(__igt_device_drop_master(fd) == 0,
> +                             "Failed to drop DRM master.\n");
> +       }

I still stand by this should be an assert.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v2] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 12:39   ` Chris Wilson
@ 2018-05-14 12:44     ` Chris Wilson
  2018-05-14 12:51       ` Katarzyna Dec
  0 siblings, 1 reply; 21+ messages in thread
From: Chris Wilson @ 2018-05-14 12:44 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Chris Wilson (2018-05-14 13:39:19)
> Quoting Katarzyna Dec (2018-05-14 12:37:22)
> > Let's add additional information to igt_device_drop_master
> > in case we try to run tests with e.g. x11 enabled.
> > 
> > v2: Changed comment and added listing clients.
> > 
> > Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > ---
> >  lib/igt_device.c | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> > 
> > diff --git a/lib/igt_device.c b/lib/igt_device.c
> > index 8a2dd0e3..b2a0a160 100644
> > --- a/lib/igt_device.c
> > +++ b/lib/igt_device.c
> > @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
> >   */
> >  void igt_device_drop_master(int fd)
> >  {
> > -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> > +       if (__igt_device_drop_master(fd)) {
> > +               igt_debugfs_dump(fd, "clients");
> > +               igt_require_f(__igt_device_drop_master(fd) == 0,
> > +                             "Failed to drop DRM master.\n");
> > +       }
> 
> I still stand by this should be an assert.

The essence of the problem is that the debug isn't shown for a require;
and for dropping master we are trying to give away our privileges. If we
fail to do so, that's a big problem.

To avoid the current problem, you might want to do:

if (__igt_device_set_master(fd) == 0)
	igt_assert_eq(__igt_device_drop_master(fd), 0);

In that case, we do only drop master when we were master. And don't fail
if we didn't have master privileges to begin with. So what's more
robust? In retrospect, I'd go with the don't fail if we weren't already
master (as that fits in with the caller's intentions).
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v2] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 12:44     ` Chris Wilson
@ 2018-05-14 12:51       ` Katarzyna Dec
  0 siblings, 0 replies; 21+ messages in thread
From: Katarzyna Dec @ 2018-05-14 12:51 UTC (permalink / raw)
  To: Chris Wilson; +Cc: igt-dev

On Mon, May 14, 2018 at 01:44:00PM +0100, Chris Wilson wrote:
> Quoting Chris Wilson (2018-05-14 13:39:19)
> > Quoting Katarzyna Dec (2018-05-14 12:37:22)
> > > Let's add additional information to igt_device_drop_master
> > > in case we try to run tests with e.g. x11 enabled.
> > > 
> > > v2: Changed comment and added listing clients.
> > > 
> > > Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> > > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > > ---
> > >  lib/igt_device.c | 6 +++++-
> > >  1 file changed, 5 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/lib/igt_device.c b/lib/igt_device.c
> > > index 8a2dd0e3..b2a0a160 100644
> > > --- a/lib/igt_device.c
> > > +++ b/lib/igt_device.c
> > > @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
> > >   */
> > >  void igt_device_drop_master(int fd)
> > >  {
> > > -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> > > +       if (__igt_device_drop_master(fd)) {
> > > +               igt_debugfs_dump(fd, "clients");
> > > +               igt_require_f(__igt_device_drop_master(fd) == 0,
> > > +                             "Failed to drop DRM master.\n");
> > > +       }
> > 
> > I still stand by this should be an assert.
> 
> The essence of the problem is that the debug isn't shown for a require;
> and for dropping master we are trying to give away our privileges. If we
> fail to do so, that's a big problem.
> 
> To avoid the current problem, you might want to do:
> 
> if (__igt_device_set_master(fd) == 0)
> 	igt_assert_eq(__igt_device_drop_master(fd), 0);
> 
> In that case, we do only drop master when we were master. And don't fail
> if we didn't have master privileges to begin with. So what's more
> robust? In retrospect, I'd go with the don't fail if we weren't already
> master (as that fits in with the caller's intentions).
> -Chris

I do not know how I missed that part with assert, but I did.
I will fix that in v3.
Kasia
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] [PATCH i-g-t v3] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 11:37 ` [igt-dev] [PATCH i-g-t v2] " Katarzyna Dec
  2018-05-14 12:39   ` Chris Wilson
@ 2018-05-14 12:51   ` Katarzyna Dec
  2018-05-15  7:45     ` Chris Wilson
  2018-05-15  8:40     ` [igt-dev] [PATCH i-g-t v4] " Katarzyna Dec
  1 sibling, 2 replies; 21+ messages in thread
From: Katarzyna Dec @ 2018-05-14 12:51 UTC (permalink / raw)
  To: igt-dev

Let's add additional information to igt_device_drop_master
in case we try to run tests with e.g. x11 enabled.

v2: Changed comment and added listing clients.
v3: Changed igt_require to igt_assert

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_device.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/igt_device.c b/lib/igt_device.c
index 8a2dd0e3..7135fc20 100644
--- a/lib/igt_device.c
+++ b/lib/igt_device.c
@@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
  */
 void igt_device_drop_master(int fd)
 {
-	igt_assert_eq(__igt_device_drop_master(fd), 0);
+	if (__igt_device_drop_master(fd)) {
+		igt_debugfs_dump(fd, "clients");
+		igt_assert_f(__igt_device_drop_master(fd) == 0,
+			      "Failed to drop DRM master.\n");
+	}
 }
-- 
2.14.3

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [igt-dev] ✗ Fi.CI.IGT: failure for lib/igt_device: Add information why cannot drop drm master
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (2 preceding siblings ...)
  2018-05-14 11:37 ` [igt-dev] [PATCH i-g-t v2] " Katarzyna Dec
@ 2018-05-14 14:19 ` Patchwork
  2018-05-14 14:53 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
                   ` (7 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-14 14:19 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master
URL   : https://patchwork.freedesktop.org/series/43125/
State : failure

== Summary ==

= CI Bug Log - changes from IGT_4476_full -> IGTPW_1352_full =

== Summary - FAILURE ==

  Serious unknown changes coming with IGTPW_1352_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1352_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/1/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Possible regressions ====

    igt@kms_atomic_transition@1x-modeset-transitions-nonblocking-fencing:
      shard-glk:          PASS -> FAIL

    igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
      shard-glk:          SKIP -> FAIL

    igt@kms_cursor_legacy@cursorb-vs-flipb-toggle:
      shard-glk:          SKIP -> DMESG-WARN

    
    ==== Warnings ====

    igt@gem_exec_schedule@deep-bsd1:
      shard-kbl:          SKIP -> PASS +1

    igt@gem_exec_schedule@deep-vebox:
      shard-kbl:          PASS -> SKIP +2

    igt@kms_ccs@pipe-a-bad-aux-stride:
      shard-apl:          SKIP -> PASS +1

    igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:
      shard-glk:          SKIP -> PASS +145

    igt@kms_hdmi_inject@inject-audio:
      shard-glk:          PASS -> SKIP +2

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@gem_ppgtt@blt-vs-render-ctxn:
      shard-kbl:          PASS -> INCOMPLETE (fdo#106023, fdo#103665)

    igt@kms_ccs@pipe-a-bad-pixel-format:
      shard-kbl:          PASS -> DMESG-WARN (fdo#105602, fdo#103313, fdo#103558) +3

    igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
      shard-apl:          PASS -> FAIL (fdo#106510) +1

    igt@kms_flip@2x-absolute-wf_vblank-interruptible:
      shard-glk:          SKIP -> FAIL (fdo#106087)

    igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
      shard-glk:          SKIP -> FAIL (fdo#100368) +1

    igt@kms_flip@absolute-wf_vblank-interruptible:
      shard-snb:          PASS -> FAIL (fdo#106087)

    igt@kms_flip@flip-vs-expired-vblank:
      shard-glk:          PASS -> FAIL (fdo#105363, fdo#102887)

    igt@kms_flip@plain-flip-ts-check-interruptible:
      shard-glk:          PASS -> FAIL (fdo#100368)

    igt@kms_flip_tiling@flip-to-x-tiled:
      shard-glk:          PASS -> FAIL (fdo#103822, fdo#104724)

    igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc:
      shard-glk:          SKIP -> FAIL (fdo#103167, fdo#104724)

    igt@kms_rotation_crc@sprite-rotation-180:
      shard-snb:          PASS -> FAIL (fdo#103925, fdo#104724) +1

    igt@kms_setmode@basic:
      shard-kbl:          PASS -> FAIL (fdo#99912)

    igt@kms_sysfs_edid_timing:
      shard-apl:          PASS -> FAIL (fdo#100047)

    igt@kms_vblank@pipe-c-accuracy-idle:
      shard-hsw:          PASS -> FAIL (fdo#102583)

    igt@testdisplay:
      shard-glk:          PASS -> INCOMPLETE (fdo#103359, k.org#198133)

    
    ==== Possible fixes ====

    igt@kms_flip@flip-vs-wf_vblank-interruptible:
      shard-glk:          FAIL (fdo#100368) -> PASS +1

    igt@kms_flip@wf_vblank-ts-check:
      shard-hsw:          FAIL (fdo#103928) -> PASS

    igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
      shard-apl:          FAIL (fdo#104724, fdo#103166) -> PASS

    igt@kms_plane_lowres@pipe-b-tiling-y:
      shard-kbl:          DMESG-WARN (fdo#105602, fdo#103313, fdo#103558) -> PASS +7

    igt@kms_setmode@basic:
      shard-apl:          FAIL (fdo#99912) -> PASS

    igt@pm_rpm@dpms-mode-unset-non-lpsp:
      shard-kbl:          DMESG-WARN (fdo#105602, fdo#103558) -> PASS +2

    
  fdo#100047 https://bugs.freedesktop.org/show_bug.cgi?id=100047
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#102583 https://bugs.freedesktop.org/show_bug.cgi?id=102583
  fdo#102887 https://bugs.freedesktop.org/show_bug.cgi?id=102887
  fdo#103166 https://bugs.freedesktop.org/show_bug.cgi?id=103166
  fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
  fdo#103313 https://bugs.freedesktop.org/show_bug.cgi?id=103313
  fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
  fdo#103558 https://bugs.freedesktop.org/show_bug.cgi?id=103558
  fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
  fdo#103822 https://bugs.freedesktop.org/show_bug.cgi?id=103822
  fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
  fdo#103928 https://bugs.freedesktop.org/show_bug.cgi?id=103928
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
  fdo#105602 https://bugs.freedesktop.org/show_bug.cgi?id=105602
  fdo#106023 https://bugs.freedesktop.org/show_bug.cgi?id=106023
  fdo#106087 https://bugs.freedesktop.org/show_bug.cgi?id=106087
  fdo#106510 https://bugs.freedesktop.org/show_bug.cgi?id=106510
  fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912
  k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4476 -> IGTPW_1352
    * Linux: CI_DRM_4172 -> CI_DRM_4175

  CI_DRM_4172: b4c14c0809e8f6dba9a4fdf1a659cea5a98750d3 @ git://anongit.freedesktop.org/gfx-ci/linux
  CI_DRM_4175: a57366b6029ac86436ad36bbf8b9a31549ef2905 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1352: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1352/
  IGT_4476: 03a62cf055481f66b4f58e6228bc45f8ca454216 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4476: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1352/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev3)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (3 preceding siblings ...)
  2018-05-14 14:19 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork
@ 2018-05-14 14:53 ` Patchwork
  2018-05-14 22:32 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-14 14:53 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev3)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4178 -> IGTPW_1356 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/3/mbox/

== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@kms_frontbuffer_tracking@basic:
      fi-hsw-4200u:       PASS -> DMESG-FAIL (fdo#106103, fdo#102614)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
      fi-snb-2520m:       PASS -> INCOMPLETE (fdo#103713)

    igt@prime_vgem@basic-fence-flip:
      fi-ilk-650:         PASS -> FAIL (fdo#104008)

    
    ==== Possible fixes ====

    igt@drv_module_reload@basic-reload:
      fi-glk-j4005:       DMESG-WARN (fdo#106248) -> PASS

    igt@kms_busy@basic-flip-c:
      fi-glk-j4005:       DMESG-WARN (fdo#106097) -> PASS +2

    
  fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
  fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
  fdo#104008 https://bugs.freedesktop.org/show_bug.cgi?id=104008
  fdo#106097 https://bugs.freedesktop.org/show_bug.cgi?id=106097
  fdo#106103 https://bugs.freedesktop.org/show_bug.cgi?id=106103
  fdo#106248 https://bugs.freedesktop.org/show_bug.cgi?id=106248


== Participating hosts (41 -> 37) ==

  Missing    (4): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-skl-6700hq 


== Build changes ==

    * IGT: IGT_4478 -> IGTPW_1356

  CI_DRM_4178: 8213a085ddd82871fab4bf94d1a3555fcdf9e6ac @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1356: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1356/
  IGT_4478: b871b10f2a6250d6dbe31665b267820fee829c84 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4478: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1356/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✗ Fi.CI.IGT: failure for lib/igt_device: Add information why cannot drop drm master (rev3)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (4 preceding siblings ...)
  2018-05-14 14:53 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
@ 2018-05-14 22:32 ` Patchwork
  2018-05-15 10:12 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-14 22:32 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev3)
URL   : https://patchwork.freedesktop.org/series/43125/
State : failure

== Summary ==

= CI Bug Log - changes from IGT_4478_full -> IGTPW_1356_full =

== Summary - FAILURE ==

  Serious unknown changes coming with IGTPW_1356_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1356_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/3/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Possible regressions ====

    igt@drv_module_reload@basic-reload:
      shard-glk:          PASS -> DMESG-WARN +1

    igt@gem_eio@hibernate:
      shard-hsw:          PASS -> DMESG-WARN
      shard-glk:          NOTRUN -> DMESG-WARN

    igt@gem_eio@in-flight-contexts-10ms:
      shard-kbl:          PASS -> DMESG-WARN +4

    igt@gem_eio@unwedge-stress:
      shard-snb:          PASS -> DMESG-WARN
      shard-apl:          PASS -> DMESG-WARN +2

    igt@kms_atomic_transition@1x-modeset-transitions-nonblocking:
      shard-glk:          PASS -> FAIL

    
    ==== Warnings ====

    igt@gem_exec_schedule@deep-render:
      shard-kbl:          PASS -> SKIP

    igt@gem_exec_schedule@deep-vebox:
      shard-kbl:          SKIP -> PASS

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@kms_flip@2x-modeset-vs-vblank-race:
      shard-glk:          PASS -> FAIL (fdo#103060)

    igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
      shard-glk:          PASS -> FAIL (fdo#100368)

    igt@kms_flip@2x-plain-flip-ts-check-interruptible:
      shard-hsw:          PASS -> FAIL (fdo#100368)

    igt@kms_flip_tiling@flip-to-y-tiled:
      shard-glk:          PASS -> FAIL (fdo#104724)

    igt@kms_flip_tiling@flip-x-tiled:
      shard-glk:          PASS -> FAIL (fdo#104724, fdo#103822) +1

    igt@kms_properties@plane-properties-legacy:
      shard-kbl:          PASS -> DMESG-WARN (fdo#106247)

    
    ==== Possible fixes ====

    igt@gem_eio@in-flight-internal-10ms:
      shard-glk:          DMESG-WARN -> PASS

    igt@gem_eio@in-flight-suspend:
      shard-apl:          DMESG-WARN -> PASS +4

    igt@gem_eio@throttle:
      shard-kbl:          DMESG-WARN -> PASS +4

    igt@kms_flip@blocking-wf_vblank:
      shard-glk:          FAIL (fdo#100368) -> PASS

    igt@kms_flip@modeset-vs-vblank-race:
      shard-hsw:          FAIL (fdo#103060) -> PASS +1

    igt@kms_flip@plain-flip-fb-recreate:
      shard-hsw:          FAIL (fdo#100368) -> PASS

    igt@kms_flip_tiling@flip-y-tiled:
      shard-glk:          FAIL (fdo#104724) -> PASS

    igt@kms_rotation_crc@sprite-rotation-180:
      shard-snb:          FAIL (fdo#104724, fdo#103925) -> PASS

    igt@kms_setmode@basic:
      shard-apl:          FAIL (fdo#99912) -> PASS

    igt@kms_universal_plane@cursor-fb-leak-pipe-b:
      shard-kbl:          DMESG-WARN (fdo#106247) -> PASS

    
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103060 https://bugs.freedesktop.org/show_bug.cgi?id=103060
  fdo#103822 https://bugs.freedesktop.org/show_bug.cgi?id=103822
  fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#106247 https://bugs.freedesktop.org/show_bug.cgi?id=106247
  fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4478 -> IGTPW_1356
    * Linux: CI_DRM_4177 -> CI_DRM_4178

  CI_DRM_4177: 2a481dd2c319c85c0233d292c5d08a8dce0784cd @ git://anongit.freedesktop.org/gfx-ci/linux
  CI_DRM_4178: 8213a085ddd82871fab4bf94d1a3555fcdf9e6ac @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1356: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1356/
  IGT_4478: b871b10f2a6250d6dbe31665b267820fee829c84 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4478: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1356/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v3] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 12:51   ` [igt-dev] [PATCH i-g-t v3] " Katarzyna Dec
@ 2018-05-15  7:45     ` Chris Wilson
  2018-05-15  8:40     ` [igt-dev] [PATCH i-g-t v4] " Katarzyna Dec
  1 sibling, 0 replies; 21+ messages in thread
From: Chris Wilson @ 2018-05-15  7:45 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Katarzyna Dec (2018-05-14 13:51:55)
> Let's add additional information to igt_device_drop_master
> in case we try to run tests with e.g. x11 enabled.
> 
> v2: Changed comment and added listing clients.
> v3: Changed igt_require to igt_assert
> 
> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_device.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_device.c b/lib/igt_device.c
> index 8a2dd0e3..7135fc20 100644
> --- a/lib/igt_device.c
> +++ b/lib/igt_device.c
> @@ -74,5 +74,9 @@ int __igt_device_drop_master(int fd)
>   */
>  void igt_device_drop_master(int fd)
>  {

I think there's still room for a 

	/* Check if we are master before dropping */
	if (__igt_device_set_master(fd))
		return;

upfront so that we only drop master when master then the mystery fails
just never happen.

> -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> +       if (__igt_device_drop_master(fd)) {
> +               igt_debugfs_dump(fd, "clients");
> +               igt_assert_f(__igt_device_drop_master(fd) == 0,
> +                             "Failed to drop DRM master.\n");
> +       }
>  }

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] [PATCH i-g-t v4] lib/igt_device: Add information why cannot drop drm master
  2018-05-14 12:51   ` [igt-dev] [PATCH i-g-t v3] " Katarzyna Dec
  2018-05-15  7:45     ` Chris Wilson
@ 2018-05-15  8:40     ` Katarzyna Dec
  2018-05-15 10:15       ` Chris Wilson
  1 sibling, 1 reply; 21+ messages in thread
From: Katarzyna Dec @ 2018-05-15  8:40 UTC (permalink / raw)
  To: igt-dev

Let's add additional information to igt_device_drop_master
in case we try to run tests with e.g. x11 enabled.

v2: Changed comment and added listing clients.
v3: Changed igt_require to igt_assert.
v4: Added checking if we are master already.

Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_device.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/lib/igt_device.c b/lib/igt_device.c
index 8a2dd0e3..78db540d 100644
--- a/lib/igt_device.c
+++ b/lib/igt_device.c
@@ -70,9 +70,16 @@ int __igt_device_drop_master(int fd)
  * @fd: the device
  *
  * Tell the kernel we no longer want this device fd to be the DRM master;
- * asserting that we lose the privilege.
+ * asserting that we lose the privilege. Return if we are master already.
  */
 void igt_device_drop_master(int fd)
 {
-	igt_assert_eq(__igt_device_drop_master(fd), 0);
+	/* Check if we are master before dropping */
+	if (__igt_device_set_master(fd))
+		return;
+	if (__igt_device_drop_master(fd)) {
+		igt_debugfs_dump(fd, "clients");
+		igt_assert_f(__igt_device_drop_master(fd) == 0,
+			      "Failed to drop DRM master.\n");
+	}
 }
-- 
2.14.3

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev3)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (5 preceding siblings ...)
  2018-05-14 22:32 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
@ 2018-05-15 10:12 ` Patchwork
  2018-05-15 11:14 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
                   ` (4 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-15 10:12 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev3)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4182 -> IGTPW_1362 =

== Summary - WARNING ==

  Minor unknown changes coming with IGTPW_1362 need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1362, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/3/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Warnings ====

    igt@gem_exec_gttfill@basic:
      fi-pnv-d510:        SKIP -> PASS

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@kms_psr_sink_crc@basic:
      fi-hsw-4200u:       NOTRUN -> FAIL (fdo#106346)

    
    ==== Possible fixes ====

    igt@kms_chamelium@dp-crc-fast:
      fi-kbl-7500u:       DMESG-FAIL (fdo#103841) -> PASS

    igt@kms_flip@basic-flip-vs-wf_vblank:
      fi-cfl-s3:          FAIL (fdo#100368, fdo#103928) -> PASS

    
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103841 https://bugs.freedesktop.org/show_bug.cgi?id=103841
  fdo#103928 https://bugs.freedesktop.org/show_bug.cgi?id=103928
  fdo#106346 https://bugs.freedesktop.org/show_bug.cgi?id=106346


== Participating hosts (42 -> 42) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4479 -> IGTPW_1362
    * Piglit: piglit_4479 -> piglit_4480

  CI_DRM_4182: 74146e495c113f6cf8721bb55feec6b459199cd4 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1362: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1362/
  IGT_4479: 89ae332745e31a075747a63ac5acc5baccf75769 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4479: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4480: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1362/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v4] lib/igt_device: Add information why cannot drop drm master
  2018-05-15  8:40     ` [igt-dev] [PATCH i-g-t v4] " Katarzyna Dec
@ 2018-05-15 10:15       ` Chris Wilson
  2018-05-15 11:30         ` Chris Wilson
  0 siblings, 1 reply; 21+ messages in thread
From: Chris Wilson @ 2018-05-15 10:15 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Katarzyna Dec (2018-05-15 09:40:55)
> Let's add additional information to igt_device_drop_master
> in case we try to run tests with e.g. x11 enabled.
> 
> v2: Changed comment and added listing clients.
> v3: Changed igt_require to igt_assert.
> v4: Added checking if we are master already.
> 
> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

> ---
>  lib/igt_device.c | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/igt_device.c b/lib/igt_device.c
> index 8a2dd0e3..78db540d 100644
> --- a/lib/igt_device.c
> +++ b/lib/igt_device.c
> @@ -70,9 +70,16 @@ int __igt_device_drop_master(int fd)
>   * @fd: the device
>   *
>   * Tell the kernel we no longer want this device fd to be the DRM master;
> - * asserting that we lose the privilege.
> + * asserting that we lose the privilege. Return if we are master already.
>   */
>  void igt_device_drop_master(int fd)
>  {
> -       igt_assert_eq(__igt_device_drop_master(fd), 0);
> +       /* Check if we are master before dropping */
> +       if (__igt_device_set_master(fd))
> +               return;

Visually break up the logical phases but keep sequences clustered;
whitespace can be our friend.

> +       if (__igt_device_drop_master(fd)) {
> +               igt_debugfs_dump(fd, "clients");
> +               igt_assert_f(__igt_device_drop_master(fd) == 0,
> +                             "Failed to drop DRM master.\n");
> +       }
>  }
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev4)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (6 preceding siblings ...)
  2018-05-15 10:12 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
@ 2018-05-15 11:14 ` Patchwork
  2018-05-15 14:52 ` Patchwork
                   ` (3 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-15 11:14 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev4)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4183 -> IGTPW_1363 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/4/mbox/

== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@kms_frontbuffer_tracking@basic:
      fi-hsw-peppy:       NOTRUN -> DMESG-FAIL (fdo#102614, fdo#106103)
      fi-cnl-y3:          PASS -> FAIL (fdo#104724, fdo#103167)

    igt@kms_psr_sink_crc@basic:
      fi-hsw-4200u:       NOTRUN -> FAIL (fdo#106346)

    
    ==== Possible fixes ====

    igt@gem_exec_suspend@basic-s4-devices:
      fi-glk-j4005:       DMESG-WARN (fdo#106097) -> PASS

    
  fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
  fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#106097 https://bugs.freedesktop.org/show_bug.cgi?id=106097
  fdo#106103 https://bugs.freedesktop.org/show_bug.cgi?id=106103
  fdo#106346 https://bugs.freedesktop.org/show_bug.cgi?id=106346


== Participating hosts (38 -> 37) ==

  Additional (2): fi-hsw-peppy fi-hsw-4200u 
  Missing    (3): fi-ctg-p8600 fi-ilk-m540 fi-skl-6700hq 


== Build changes ==

    * IGT: IGT_4480 -> IGTPW_1363
    * Piglit: piglit_4480 -> piglit_4481

  CI_DRM_4183: 9d97c6e3fd3ac8dd73f107ca85e2a12c746a2ccc @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1363: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1363/
  IGT_4480: 18fa0c11012da6aa9a5511dc9b1736064a02b429 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4480: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4481: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1363/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [igt-dev] [PATCH i-g-t v4] lib/igt_device: Add information why cannot drop drm master
  2018-05-15 10:15       ` Chris Wilson
@ 2018-05-15 11:30         ` Chris Wilson
  0 siblings, 0 replies; 21+ messages in thread
From: Chris Wilson @ 2018-05-15 11:30 UTC (permalink / raw)
  To: Katarzyna Dec, igt-dev

Quoting Chris Wilson (2018-05-15 11:15:53)
> Quoting Katarzyna Dec (2018-05-15 09:40:55)
> > Let's add additional information to igt_device_drop_master
> > in case we try to run tests with e.g. x11 enabled.
> > 
> > v2: Changed comment and added listing clients.
> > v3: Changed igt_require to igt_assert.
> > v4: Added checking if we are master already.
> > 
> > Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

References: https://bugs.freedesktop.org/show_bug.cgi?id=105990
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev4)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (7 preceding siblings ...)
  2018-05-15 11:14 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
@ 2018-05-15 14:52 ` Patchwork
  2018-05-15 16:01 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-15 14:52 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev4)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4186 -> IGTPW_1368 =

== Summary - WARNING ==

  Minor unknown changes coming with IGTPW_1368 need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1368, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/4/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Warnings ====

    igt@gem_exec_gttfill@basic:
      fi-pnv-d510:        SKIP -> PASS

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@kms_frontbuffer_tracking@basic:
      fi-hsw-peppy:       PASS -> DMESG-FAIL (fdo#106103, fdo#102614)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
      fi-bxt-dsi:         NOTRUN -> INCOMPLETE (fdo#103927)

    igt@prime_vgem@basic-fence-flip:
      fi-ilk-650:         PASS -> FAIL (fdo#104008)

    
  fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
  fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
  fdo#104008 https://bugs.freedesktop.org/show_bug.cgi?id=104008
  fdo#106103 https://bugs.freedesktop.org/show_bug.cgi?id=106103


== Participating hosts (41 -> 37) ==

  Additional (1): fi-bxt-dsi 
  Missing    (5): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-skl-6700hq 


== Build changes ==

    * IGT: IGT_4481 -> IGTPW_1368
    * Piglit: piglit_4481 -> piglit_4482

  CI_DRM_4186: c03987223c762e4a61142f0a9be6027bb181cdfa @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1368: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1368/
  IGT_4481: 94df67655566f18f05e599bb53a4090b598057f2 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4481: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4482: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1368/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev3)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (8 preceding siblings ...)
  2018-05-15 14:52 ` Patchwork
@ 2018-05-15 16:01 ` Patchwork
  2018-05-15 18:09 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
  2018-05-16  0:40 ` Patchwork
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-15 16:01 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev3)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from IGT_4479_full -> IGTPW_1362_full =

== Summary - WARNING ==

  Minor unknown changes coming with IGTPW_1362_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1362_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/3/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Warnings ====

    igt@perf_pmu@rc6:
      shard-kbl:          SKIP -> PASS +1

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@gem_eio@in-flight-1us:
      shard-apl:          PASS -> DMESG-WARN (fdo#106523) +3

    igt@gem_eio@throttle:
      shard-kbl:          PASS -> DMESG-WARN (fdo#106523) +2
      shard-glk:          PASS -> DMESG-WARN (fdo#106523)

    igt@gem_tiled_blits@normal:
      shard-apl:          PASS -> INCOMPLETE (fdo#103927)

    igt@gem_workarounds@suspend-resume:
      shard-kbl:          PASS -> INCOMPLETE (fdo#103665)

    igt@kms_atomic_interruptible@legacy-cursor:
      shard-kbl:          PASS -> DMESG-WARN (fdo#105602, fdo#103558) +10

    igt@kms_atomic_transition@plane-all-modeset-transition-fencing:
      shard-hsw:          PASS -> DMESG-WARN (fdo#102614)

    igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
      shard-hsw:          PASS -> FAIL (fdo#102670)

    igt@kms_rotation_crc@sprite-rotation-180:
      shard-snb:          PASS -> FAIL (fdo#104724, fdo#103925)

    
    ==== Possible fixes ====

    igt@gem_eio@hibernate:
      shard-hsw:          DMESG-WARN (fdo#106523) -> PASS +6

    igt@gem_eio@in-flight-contexts-1us:
      shard-apl:          DMESG-WARN (fdo#106523) -> PASS +2

    igt@gem_eio@in-flight-internal-immediate:
      shard-glk:          DMESG-WARN (fdo#106523) -> PASS +8

    igt@gem_eio@suspend:
      shard-kbl:          DMESG-WARN (fdo#106523) -> PASS +4

    igt@gem_eio@unwedge-stress:
      shard-snb:          DMESG-WARN (fdo#106523) -> PASS +6

    igt@gem_ppgtt@blt-vs-render-ctxn:
      shard-kbl:          INCOMPLETE (fdo#103665, fdo#106023) -> PASS

    igt@kms_atomic_transition@1x-modeset-transitions-nonblocking:
      shard-glk:          FAIL (fdo#105703) -> PASS

    igt@kms_flip@plain-flip-fb-recreate-interruptible:
      shard-glk:          FAIL (fdo#100368) -> PASS +1

    igt@kms_vblank@pipe-b-ts-continuation-suspend:
      shard-kbl:          INCOMPLETE (fdo#103665) -> PASS

    
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614
  fdo#102670 https://bugs.freedesktop.org/show_bug.cgi?id=102670
  fdo#103558 https://bugs.freedesktop.org/show_bug.cgi?id=103558
  fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
  fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
  fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#105602 https://bugs.freedesktop.org/show_bug.cgi?id=105602
  fdo#105703 https://bugs.freedesktop.org/show_bug.cgi?id=105703
  fdo#106023 https://bugs.freedesktop.org/show_bug.cgi?id=106023
  fdo#106523 https://bugs.freedesktop.org/show_bug.cgi?id=106523


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4479 -> IGTPW_1362
    * Linux: CI_DRM_4178 -> CI_DRM_4182
    * Piglit: piglit_4479 -> piglit_4480

  CI_DRM_4178: 8213a085ddd82871fab4bf94d1a3555fcdf9e6ac @ git://anongit.freedesktop.org/gfx-ci/linux
  CI_DRM_4182: 74146e495c113f6cf8721bb55feec6b459199cd4 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1362: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1362/
  IGT_4479: 89ae332745e31a075747a63ac5acc5baccf75769 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4479: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4480: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1362/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev4)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (9 preceding siblings ...)
  2018-05-15 16:01 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
@ 2018-05-15 18:09 ` Patchwork
  2018-05-16  0:40 ` Patchwork
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-15 18:09 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev4)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from IGT_4480_full -> IGTPW_1363_full =

== Summary - WARNING ==

  Minor unknown changes coming with IGTPW_1363_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1363_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/4/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Warnings ====

    igt@gem_linear_blits@normal:
      shard-kbl:          PASS -> SKIP +1
      shard-apl:          PASS -> SKIP

    igt@gem_mocs_settings@mocs-rc6-blt:
      shard-kbl:          SKIP -> PASS +1

    igt@pm_rc6_residency@rc6-accuracy:
      shard-snb:          SKIP -> PASS

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@gem_eio@hibernate:
      shard-hsw:          PASS -> DMESG-WARN (fdo#106523) +4

    igt@gem_eio@in-flight-contexts-1us:
      shard-snb:          PASS -> DMESG-WARN (fdo#106523) +3
      shard-glk:          PASS -> DMESG-WARN (fdo#106523)

    igt@gem_eio@in-flight-suspend:
      shard-kbl:          PASS -> DMESG-WARN (fdo#106523) +3
      shard-apl:          PASS -> DMESG-WARN (fdo#106523) +3

    igt@gem_eio@unwedge-stress:
      shard-glk:          NOTRUN -> DMESG-WARN (fdo#106523) +3

    igt@kms_atomic_transition@1x-modeset-transitions-nonblocking-fencing:
      shard-glk:          PASS -> FAIL (fdo#105703)

    igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
      shard-glk:          NOTRUN -> FAIL (fdo#100368)

    igt@kms_flip@2x-plain-flip-ts-check:
      shard-glk:          PASS -> FAIL (fdo#100368)

    igt@kms_flip@flip-vs-modeset-vs-hang:
      shard-kbl:          PASS -> DMESG-WARN (fdo#103558, fdo#105602, fdo#103313) +2

    igt@kms_flip@wf_vblank-ts-check-interruptible:
      shard-kbl:          PASS -> DMESG-WARN (fdo#103558, fdo#105602) +17

    igt@kms_flip_tiling@flip-x-tiled:
      shard-glk:          NOTRUN -> FAIL (fdo#103822, fdo#104724) +1

    igt@kms_rotation_crc@sprite-rotation-90-pos-100-0:
      shard-apl:          PASS -> DMESG-WARN (fdo#105602) +9

    igt@kms_setmode@basic:
      shard-apl:          PASS -> FAIL (fdo#99912)
      shard-glk:          NOTRUN -> FAIL (fdo#99912)
      shard-kbl:          PASS -> FAIL (fdo#99912)

    
    ==== Possible fixes ====

    igt@gem_eio@in-flight-internal-10ms:
      shard-glk:          DMESG-WARN (fdo#106523) -> PASS +3
      shard-apl:          DMESG-WARN (fdo#106523) -> PASS +1

    igt@gem_eio@suspend:
      shard-kbl:          DMESG-WARN (fdo#106523) -> PASS +2

    igt@gem_eio@throttle:
      shard-snb:          DMESG-WARN (fdo#106523) -> PASS +4

    igt@gem_eio@wait-wedge-immediate:
      shard-hsw:          DMESG-WARN (fdo#106523) -> PASS +2

    igt@kms_cursor_crc@cursor-64x21-onscreen:
      shard-kbl:          DMESG-WARN (fdo#103558, fdo#105602, fdo#103313) -> PASS +9

    igt@kms_fbcon_fbt@fbc-suspend:
      shard-kbl:          INCOMPLETE (fdo#103665) -> PASS

    igt@kms_flip@flip-vs-expired-vblank-interruptible:
      shard-glk:          FAIL (fdo#105363) -> PASS

    igt@kms_flip@flip-vs-panning-vs-hang:
      shard-snb:          DMESG-WARN (fdo#103821) -> PASS

    igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
      shard-kbl:          DMESG-WARN (fdo#103841, fdo#103558, fdo#105602) -> PASS

    igt@kms_rotation_crc@sprite-rotation-180:
      shard-snb:          FAIL (fdo#103925, fdo#104724) -> PASS

    igt@kms_rotation_crc@sprite-rotation-270:
      shard-apl:          FAIL (fdo#103925, fdo#104724) -> PASS

    igt@kms_universal_plane@cursor-fb-leak-pipe-b:
      shard-kbl:          DMESG-WARN (fdo#103558, fdo#105602) -> PASS +17

    
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103313 https://bugs.freedesktop.org/show_bug.cgi?id=103313
  fdo#103558 https://bugs.freedesktop.org/show_bug.cgi?id=103558
  fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
  fdo#103821 https://bugs.freedesktop.org/show_bug.cgi?id=103821
  fdo#103822 https://bugs.freedesktop.org/show_bug.cgi?id=103822
  fdo#103841 https://bugs.freedesktop.org/show_bug.cgi?id=103841
  fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
  fdo#105602 https://bugs.freedesktop.org/show_bug.cgi?id=105602
  fdo#105703 https://bugs.freedesktop.org/show_bug.cgi?id=105703
  fdo#106523 https://bugs.freedesktop.org/show_bug.cgi?id=106523
  fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4480 -> IGTPW_1363
    * Linux: CI_DRM_4182 -> CI_DRM_4183
    * Piglit: piglit_4480 -> piglit_4481

  CI_DRM_4182: 74146e495c113f6cf8721bb55feec6b459199cd4 @ git://anongit.freedesktop.org/gfx-ci/linux
  CI_DRM_4183: 9d97c6e3fd3ac8dd73f107ca85e2a12c746a2ccc @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1363: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1363/
  IGT_4480: 18fa0c11012da6aa9a5511dc9b1736064a02b429 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4480: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4481: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1363/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev4)
  2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
                   ` (10 preceding siblings ...)
  2018-05-15 18:09 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
@ 2018-05-16  0:40 ` Patchwork
  11 siblings, 0 replies; 21+ messages in thread
From: Patchwork @ 2018-05-16  0:40 UTC (permalink / raw)
  To: Katarzyna Dec; +Cc: igt-dev

== Series Details ==

Series: lib/igt_device: Add information why cannot drop drm master (rev4)
URL   : https://patchwork.freedesktop.org/series/43125/
State : success

== Summary ==

= CI Bug Log - changes from IGT_4481_full -> IGTPW_1368_full =

== Summary - WARNING ==

  Minor unknown changes coming with IGTPW_1368_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1368_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/43125/revisions/4/mbox/

== Possible new issues ==

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

  === IGT changes ===

    ==== Warnings ====

    igt@gem_exec_schedule@deep-vebox:
      shard-kbl:          PASS -> SKIP +4

    igt@gem_mocs_settings@mocs-rc6-vebox:
      shard-kbl:          SKIP -> PASS

    
== Known issues ==

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

  === IGT changes ===

    ==== Issues hit ====

    igt@gem_ctx_isolation@vcs0-s3:
      shard-kbl:          PASS -> INCOMPLETE (fdo#103665)

    igt@gem_eio@hibernate:
      shard-kbl:          PASS -> DMESG-WARN (fdo#106523) +1

    igt@gem_eio@in-flight-contexts-immediate:
      shard-hsw:          PASS -> DMESG-WARN (fdo#106523) +4

    igt@gem_eio@in-flight-internal-10ms:
      shard-glk:          PASS -> DMESG-WARN (fdo#106523) +2

    igt@gem_eio@suspend:
      shard-apl:          PASS -> DMESG-WARN (fdo#106523) +2

    igt@gem_eio@throttle:
      shard-snb:          PASS -> DMESG-WARN (fdo#106523) +4

    igt@gem_exec_schedule@pi-ringfull-render:
      shard-apl:          NOTRUN -> FAIL (fdo#103158)

    igt@gem_ppgtt@blt-vs-render-ctxn:
      shard-kbl:          PASS -> INCOMPLETE (fdo#106023, fdo#103665)

    igt@kms_cursor_legacy@2x-nonblocking-modeset-vs-cursor-atomic:
      shard-glk:          PASS -> FAIL (fdo#105454)

    igt@kms_flip@dpms-vs-vblank-race:
      shard-glk:          PASS -> FAIL (fdo#103060)

    igt@kms_flip@plain-flip-ts-check-interruptible:
      shard-glk:          PASS -> FAIL (fdo#100368) +2

    igt@kms_flip_tiling@flip-x-tiled:
      shard-glk:          PASS -> FAIL (fdo#104724)

    igt@pm_rpm@fences:
      shard-kbl:          PASS -> DMESG-WARN (fdo#105602, fdo#103558) +9

    
    ==== Possible fixes ====

    igt@gem_eio@in-flight-1us:
      shard-snb:          DMESG-WARN (fdo#106523) -> PASS +4

    igt@gem_eio@in-flight-external:
      shard-hsw:          DMESG-WARN (fdo#106523) -> PASS +4

    igt@gem_eio@in-flight-immediate:
      shard-apl:          DMESG-WARN (fdo#106523) -> PASS +5

    igt@gem_eio@in-flight-internal-immediate:
      shard-glk:          DMESG-WARN (fdo#106523) -> PASS +4

    igt@gem_eio@in-flight-suspend:
      shard-kbl:          DMESG-WARN (fdo#106523) -> PASS +5

    igt@kms_atomic_transition@1x-modeset-transitions-nonblocking:
      shard-glk:          FAIL (fdo#105703) -> PASS

    igt@kms_flip@plain-flip-fb-recreate-interruptible:
      shard-glk:          FAIL (fdo#100368) -> PASS +1

    igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-blt:
      shard-apl:          INCOMPLETE (fdo#103927) -> PASS

    igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-blt:
      shard-glk:          FAIL (fdo#104724, fdo#103167) -> PASS

    igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
      shard-snb:          FAIL (fdo#103166, fdo#104724) -> PASS

    igt@kms_setmode@basic:
      shard-apl:          FAIL (fdo#99912) -> PASS

    igt@pm_rpm@system-suspend:
      shard-kbl:          INCOMPLETE (fdo#103665) -> PASS

    
  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103060 https://bugs.freedesktop.org/show_bug.cgi?id=103060
  fdo#103158 https://bugs.freedesktop.org/show_bug.cgi?id=103158
  fdo#103166 https://bugs.freedesktop.org/show_bug.cgi?id=103166
  fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
  fdo#103558 https://bugs.freedesktop.org/show_bug.cgi?id=103558
  fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
  fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
  fdo#104724 https://bugs.freedesktop.org/show_bug.cgi?id=104724
  fdo#105454 https://bugs.freedesktop.org/show_bug.cgi?id=105454
  fdo#105602 https://bugs.freedesktop.org/show_bug.cgi?id=105602
  fdo#105703 https://bugs.freedesktop.org/show_bug.cgi?id=105703
  fdo#106023 https://bugs.freedesktop.org/show_bug.cgi?id=106023
  fdo#106523 https://bugs.freedesktop.org/show_bug.cgi?id=106523
  fdo#99912 https://bugs.freedesktop.org/show_bug.cgi?id=99912


== Participating hosts (5 -> 5) ==

  No changes in participating hosts


== Build changes ==

    * IGT: IGT_4481 -> IGTPW_1368
    * Linux: CI_DRM_4183 -> CI_DRM_4186
    * Piglit: piglit_4481 -> piglit_4482

  CI_DRM_4183: 9d97c6e3fd3ac8dd73f107ca85e2a12c746a2ccc @ git://anongit.freedesktop.org/gfx-ci/linux
  CI_DRM_4186: c03987223c762e4a61142f0a9be6027bb181cdfa @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1368: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1368/
  IGT_4481: 94df67655566f18f05e599bb53a4090b598057f2 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  piglit_4481: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit
  piglit_4482: 3ba0657bff4216d1ec7179935590261855f1651e @ git://anongit.freedesktop.org/piglit

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1368/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2018-05-16  0:40 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-14 10:19 [igt-dev] [PATCH i-g-t] lib/igt_device: Add information why cannot drop drm master Katarzyna Dec
2018-05-14 10:44 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-05-14 10:52 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
2018-05-14 11:37 ` [igt-dev] [PATCH i-g-t v2] " Katarzyna Dec
2018-05-14 12:39   ` Chris Wilson
2018-05-14 12:44     ` Chris Wilson
2018-05-14 12:51       ` Katarzyna Dec
2018-05-14 12:51   ` [igt-dev] [PATCH i-g-t v3] " Katarzyna Dec
2018-05-15  7:45     ` Chris Wilson
2018-05-15  8:40     ` [igt-dev] [PATCH i-g-t v4] " Katarzyna Dec
2018-05-15 10:15       ` Chris Wilson
2018-05-15 11:30         ` Chris Wilson
2018-05-14 14:19 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork
2018-05-14 14:53 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
2018-05-14 22:32 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2018-05-15 10:12 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2018-05-15 11:14 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
2018-05-15 14:52 ` Patchwork
2018-05-15 16:01 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev3) Patchwork
2018-05-15 18:09 ` [igt-dev] ✓ Fi.CI.IGT: success for lib/igt_device: Add information why cannot drop drm master (rev4) Patchwork
2018-05-16  0:40 ` Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.