* [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak
@ 2019-03-13 12:06 Petri Latvala
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines Petri Latvala
` (5 more replies)
0 siblings, 6 replies; 12+ messages in thread
From: Petri Latvala @ 2019-03-13 12:06 UTC (permalink / raw)
To: igt-dev; +Cc: Petri Latvala
by freeing batches in fillgtt()
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
---
tests/i915/gem_exec_gttfill.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/i915/gem_exec_gttfill.c b/tests/i915/gem_exec_gttfill.c
index efd612bb..c5627c80 100644
--- a/tests/i915/gem_exec_gttfill.c
+++ b/tests/i915/gem_exec_gttfill.c
@@ -196,6 +196,7 @@ static void fillgtt(int fd, unsigned ring, int timeout)
munmap(batches[i].ptr, BATCH_SIZE);
gem_close(fd, batches[i].handle);
}
+ free(batches);
shared[nengine] = 0;
for (unsigned i = 0; i < nengine; i++)
--
2.19.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
@ 2019-03-13 12:06 ` Petri Latvala
2019-03-13 12:33 ` Chris Wilson
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero Petri Latvala
` (4 subsequent siblings)
5 siblings, 1 reply; 12+ messages in thread
From: Petri Latvala @ 2019-03-13 12:06 UTC (permalink / raw)
To: igt-dev; +Cc: Petri Latvala
Continue instead of breaking on the first engine.
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
tests/i915/gem_exec_fence.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/i915/gem_exec_fence.c b/tests/i915/gem_exec_fence.c
index ba46595d..c6661626 100644
--- a/tests/i915/gem_exec_fence.c
+++ b/tests/i915/gem_exec_fence.c
@@ -274,7 +274,7 @@ static void test_fence_busy_all(int fd, unsigned flags)
if (all < 0) {
all = fence;
- break;
+ continue;
}
new = sync_fence_merge(all, fence);
--
2.19.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines Petri Latvala
@ 2019-03-13 12:06 ` Petri Latvala
2019-03-13 12:33 ` Chris Wilson
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c Petri Latvala
` (3 subsequent siblings)
5 siblings, 1 reply; 12+ messages in thread
From: Petri Latvala @ 2019-03-13 12:06 UTC (permalink / raw)
To: igt-dev; +Cc: Petri Latvala
It's just added to in a loop later without any initialization or
direct assignment.
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
---
tests/i915/gem_exec_latency.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
index 6dd191ec..1b022df5 100644
--- a/tests/i915/gem_exec_latency.c
+++ b/tests/i915/gem_exec_latency.c
@@ -597,7 +597,7 @@ rthog_latency_on_ring(int fd, unsigned int engine, const char *name, unsigned in
for (unsigned int child = 0; child < nengine; child++) {
struct rt_pkt normal = results[NPASS * child + 1];
igt_stats_t stats;
- double variance;
+ double variance = 0.0;
igt_stats_init_with_size(&stats, NPASS);
--
2.19.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines Petri Latvala
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero Petri Latvala
@ 2019-03-13 12:06 ` Petri Latvala
2019-03-13 12:12 ` Chris Wilson
2019-03-13 12:36 ` Chris Wilson
2019-03-13 12:32 ` [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Chris Wilson
` (2 subsequent siblings)
5 siblings, 2 replies; 12+ messages in thread
From: Petri Latvala @ 2019-03-13 12:06 UTC (permalink / raw)
To: igt-dev; +Cc: Petri Latvala
The code that uses c deceivingly looks like it works with any initial
value of c, it doesn't work the same for negative values. In addition
initializing it to 0 makes the results deterministic.
Signed-off-by: Petri Latvala <petri.latvala@intel.com>
---
tests/i915/gem_exec_lut_handle.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/i915/gem_exec_lut_handle.c b/tests/i915/gem_exec_lut_handle.c
index 98e6ae5a..d252d75d 100644
--- a/tests/i915/gem_exec_lut_handle.c
+++ b/tests/i915/gem_exec_lut_handle.c
@@ -85,7 +85,7 @@ igt_simple_main
{
uint32_t batch[2] = {MI_BATCH_BUFFER_END};
uint32_t cycle[16];
- int fd, n, m, count, c;
+ int fd, n, m, count, c = 0;
const struct {
const char *name;
unsigned int flags;
--
2.19.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c Petri Latvala
@ 2019-03-13 12:12 ` Chris Wilson
2019-03-13 12:36 ` Chris Wilson
1 sibling, 0 replies; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:12 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Petri Latvala (2019-03-13 12:06:41)
> The code that uses c deceivingly looks like it works with any initial
> value of c, it doesn't work the same for negative values. In addition
> initializing it to 0 makes the results deterministic.
Meaning that there was non-determinism given all handles starting in
identical state. That would be a nasty bug, so care to write a test
proving that statement?
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
` (2 preceding siblings ...)
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c Petri Latvala
@ 2019-03-13 12:32 ` Chris Wilson
2019-03-13 13:47 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/4] " Patchwork
2019-03-13 17:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:32 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Petri Latvala (2019-03-13 12:06:38)
> by freeing batches in fillgtt()
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> ---
> tests/i915/gem_exec_gttfill.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/i915/gem_exec_gttfill.c b/tests/i915/gem_exec_gttfill.c
> index efd612bb..c5627c80 100644
> --- a/tests/i915/gem_exec_gttfill.c
> +++ b/tests/i915/gem_exec_gttfill.c
> @@ -196,6 +196,7 @@ static void fillgtt(int fd, unsigned ring, int timeout)
> munmap(batches[i].ptr, BATCH_SIZE);
> gem_close(fd, batches[i].handle);
> }
> + free(batches);
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] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines Petri Latvala
@ 2019-03-13 12:33 ` Chris Wilson
0 siblings, 0 replies; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:33 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Petri Latvala (2019-03-13 12:06:39)
> Continue instead of breaking on the first engine.
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero Petri Latvala
@ 2019-03-13 12:33 ` Chris Wilson
2019-03-13 12:34 ` Chris Wilson
0 siblings, 1 reply; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:33 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Petri Latvala (2019-03-13 12:06:40)
> It's just added to in a loop later without any initialization or
> direct assignment.
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> ---
> tests/i915/gem_exec_latency.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
> index 6dd191ec..1b022df5 100644
> --- a/tests/i915/gem_exec_latency.c
> +++ b/tests/i915/gem_exec_latency.c
> @@ -597,7 +597,7 @@ rthog_latency_on_ring(int fd, unsigned int engine, const char *name, unsigned in
> for (unsigned int child = 0; child < nengine; child++) {
> struct rt_pkt normal = results[NPASS * child + 1];
> igt_stats_t stats;
> - double variance;
> + double variance = 0.0;
Just = 0; would do.
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] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero
2019-03-13 12:33 ` Chris Wilson
@ 2019-03-13 12:34 ` Chris Wilson
0 siblings, 0 replies; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:34 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Chris Wilson (2019-03-13 12:33:48)
> Quoting Petri Latvala (2019-03-13 12:06:40)
> > It's just added to in a loop later without any initialization or
> > direct assignment.
> >
> > Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> > ---
> > tests/i915/gem_exec_latency.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
> > index 6dd191ec..1b022df5 100644
> > --- a/tests/i915/gem_exec_latency.c
> > +++ b/tests/i915/gem_exec_latency.c
> > @@ -597,7 +597,7 @@ rthog_latency_on_ring(int fd, unsigned int engine, const char *name, unsigned in
> > for (unsigned int child = 0; child < nengine; child++) {
> > struct rt_pkt normal = results[NPASS * child + 1];
> > igt_stats_t stats;
> > - double variance;
> > + double variance = 0.0;
>
> Just = 0; would do.
I would push it down to just before we compute variance though.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c Petri Latvala
2019-03-13 12:12 ` Chris Wilson
@ 2019-03-13 12:36 ` Chris Wilson
1 sibling, 0 replies; 12+ messages in thread
From: Chris Wilson @ 2019-03-13 12:36 UTC (permalink / raw)
To: Petri Latvala, igt-dev; +Cc: Petri Latvala
Quoting Petri Latvala (2019-03-13 12:06:41)
> The code that uses c deceivingly looks like it works with any initial
> value of c, it doesn't work the same for negative values. In addition
> initializing it to 0 makes the results deterministic.
Since c is reused lots, you will want to set c=0 before we use it for
CYCLE_BATCH.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/4] gem_exec_gttfill: Plug memory leak
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
` (3 preceding siblings ...)
2019-03-13 12:32 ` [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Chris Wilson
@ 2019-03-13 13:47 ` Patchwork
2019-03-13 17:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2019-03-13 13:47 UTC (permalink / raw)
To: Petri Latvala; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/4] gem_exec_gttfill: Plug memory leak
URL : https://patchwork.freedesktop.org/series/57930/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5737 -> IGTPW_2602
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/57930/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2602 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_cs_nop@fork-gfx0:
- fi-icl-u2: NOTRUN -> SKIP [fdo#109315] +17
* igt@gem_ctx_create@basic-files:
- fi-gdg-551: NOTRUN -> SKIP [fdo#109271] +106
* igt@gem_exec_basic@gtt-bsd2:
- fi-byt-clapper: NOTRUN -> SKIP [fdo#109271] +57
* igt@gem_exec_basic@readonly-bsd1:
- fi-snb-2520m: NOTRUN -> SKIP [fdo#109271] +57
- fi-icl-u2: NOTRUN -> SKIP [fdo#109276] +7
* igt@gem_exec_parse@basic-allowed:
- fi-icl-u2: NOTRUN -> SKIP [fdo#109289] +1
* igt@i915_module_load@reload:
- fi-blb-e6850: NOTRUN -> INCOMPLETE [fdo#107718]
* igt@i915_pm_rpm@basic-pci-d3-state:
- fi-bsw-kefka: PASS -> SKIP [fdo#109271]
* igt@i915_pm_rpm@basic-rte:
- fi-bsw-kefka: PASS -> FAIL [fdo#108800]
* igt@i915_selftest@live_contexts:
- fi-icl-u2: NOTRUN -> DMESG-FAIL [fdo#108569]
* igt@i915_selftest@live_execlists:
- fi-apl-guc: PASS -> INCOMPLETE [fdo#103927] / [fdo#109720]
* igt@kms_busy@basic-flip-c:
- fi-blb-e6850: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
- fi-byt-clapper: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
- fi-gdg-551: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
- fi-snb-2520m: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
* igt@kms_chamelium@dp-edid-read:
- fi-icl-u2: NOTRUN -> SKIP [fdo#109316] +2
* igt@kms_chamelium@hdmi-edid-read:
- fi-hsw-peppy: NOTRUN -> SKIP [fdo#109271] +46
* igt@kms_chamelium@vga-hpd-fast:
- fi-icl-u2: NOTRUN -> SKIP [fdo#109309] +1
* igt@kms_force_connector_basic@prune-stale-modes:
- fi-icl-u2: NOTRUN -> SKIP [fdo#109285] +3
* igt@kms_frontbuffer_tracking@basic:
- fi-icl-u3: PASS -> FAIL [fdo#103167]
- fi-hsw-peppy: NOTRUN -> DMESG-FAIL [fdo#102614] / [fdo#107814]
- fi-icl-u2: NOTRUN -> FAIL [fdo#103167]
- fi-byt-clapper: NOTRUN -> FAIL [fdo#103167]
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-c:
- fi-blb-e6850: NOTRUN -> SKIP [fdo#109271] +29
* igt@runner@aborted:
- fi-apl-guc: NOTRUN -> FAIL [fdo#108622] / [fdo#109720]
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3:
- fi-blb-e6850: INCOMPLETE [fdo#107718] -> PASS
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#107814]: https://bugs.freedesktop.org/show_bug.cgi?id=107814
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#108622]: https://bugs.freedesktop.org/show_bug.cgi?id=108622
[fdo#108800]: https://bugs.freedesktop.org/show_bug.cgi?id=108800
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
[fdo#109294]: https://bugs.freedesktop.org/show_bug.cgi?id=109294
[fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309
[fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
[fdo#109316]: https://bugs.freedesktop.org/show_bug.cgi?id=109316
[fdo#109720]: https://bugs.freedesktop.org/show_bug.cgi?id=109720
[fdo#110028]: https://bugs.freedesktop.org/show_bug.cgi?id=110028
Participating hosts (41 -> 42)
------------------------------
Additional (6): fi-hsw-peppy fi-icl-u2 fi-snb-2520m fi-gdg-551 fi-icl-y fi-byt-clapper
Missing (5): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-ctg-p8600 fi-bdw-samus
Build changes
-------------
* IGT: IGT_4883 -> IGTPW_2602
CI_DRM_5737: d5bb7d77aa77996702426496078a597f30bead58 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2602: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2602/
IGT_4883: b25e06d6ddf2e42044cd9c93b613cbc7339a8c33 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2602/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
* [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [i-g-t,1/4] gem_exec_gttfill: Plug memory leak
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
` (4 preceding siblings ...)
2019-03-13 13:47 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/4] " Patchwork
@ 2019-03-13 17:26 ` Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2019-03-13 17:26 UTC (permalink / raw)
To: Petri Latvala; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/4] gem_exec_gttfill: Plug memory leak
URL : https://patchwork.freedesktop.org/series/57930/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_5737_full -> IGTPW_2602_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_2602_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_2602_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/57930/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_2602_full:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_fence@basic-busy-all:
- shard-apl: PASS -> FAIL +1
- shard-kbl: PASS -> FAIL +1
- shard-glk: PASS -> FAIL
* igt@gem_exec_fence@basic-wait-all:
- shard-snb: PASS -> FAIL
- shard-hsw: PASS -> FAIL
#### Warnings ####
* igt@kms_plane_scaling@pipe-c-scaler-with-rotation:
- shard-glk: SKIP [fdo#109271] / [fdo#109278] -> FAIL
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* {igt@kms_plane@pixel-format-pipe-b-planes}:
- shard-apl: PASS -> FAIL
Known issues
------------
Here are the changes found in IGTPW_2602_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_param@invalid-param-set:
- shard-snb: NOTRUN -> FAIL [fdo#109674]
* igt@gem_eio@reset-stress:
- shard-snb: PASS -> INCOMPLETE [fdo#105411]
* igt@gem_exec_schedule@preempt-other-chain-blt:
- shard-snb: NOTRUN -> SKIP [fdo#109271] +102
* igt@kms_atomic_transition@3x-modeset-transitions:
- shard-glk: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
- shard-kbl: PASS -> DMESG-WARN [fdo#107956] +1
- shard-hsw: PASS -> DMESG-WARN [fdo#107956]
* igt@kms_color@pipe-a-degamma:
- shard-glk: NOTRUN -> FAIL [fdo#104782] / [fdo#108145]
* igt@kms_color@pipe-a-legacy-gamma:
- shard-glk: PASS -> FAIL [fdo#104782] / [fdo#108145]
* igt@kms_color@pipe-c-ctm-max:
- shard-apl: PASS -> FAIL [fdo#108147]
* igt@kms_cursor_crc@cursor-128x128-suspend:
- shard-glk: NOTRUN -> FAIL [fdo#103232] +1
* igt@kms_cursor_crc@cursor-64x64-dpms:
- shard-apl: PASS -> FAIL [fdo#103232]
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-kbl: PASS -> INCOMPLETE [fdo#103665]
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-move:
- shard-glk: NOTRUN -> FAIL [fdo#103167]
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff:
- shard-glk: PASS -> FAIL [fdo#103167] +3
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render:
- shard-glk: NOTRUN -> SKIP [fdo#109271] +10
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-f:
- shard-kbl: NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +1
* igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
- shard-kbl: NOTRUN -> FAIL [fdo#108145] / [fdo#108590]
* igt@kms_psr@basic:
- shard-apl: NOTRUN -> SKIP [fdo#109271] +2
* igt@kms_universal_plane@cursor-fb-leak-pipe-d:
- shard-snb: NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +9
* igt@perf_pmu@rc6-runtime-pm:
- shard-glk: PASS -> FAIL [fdo#105010]
* igt@prime_nv_pcopy@test_semaphore:
- shard-kbl: NOTRUN -> SKIP [fdo#109271] +27
* igt@runner@aborted:
- shard-apl: NOTRUN -> ( 10 FAIL ) [fdo#109373]
#### Possible fixes ####
* igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-hsw: DMESG-WARN [fdo#107956] -> PASS +1
- shard-snb: DMESG-WARN [fdo#107956] -> PASS
* igt@kms_color@pipe-c-legacy-gamma:
- shard-apl: FAIL [fdo#104782] -> PASS
* igt@kms_cursor_crc@cursor-128x42-random:
- shard-apl: FAIL [fdo#103232] -> PASS +2
- shard-kbl: FAIL [fdo#103232] -> PASS
* igt@kms_cursor_crc@cursor-64x64-suspend:
- shard-apl: FAIL [fdo#103191] / [fdo#103232] -> PASS
- shard-kbl: FAIL [fdo#103191] / [fdo#103232] -> PASS
* igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
- shard-hsw: FAIL [fdo#105767] -> PASS
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-glk: FAIL [fdo#102887] / [fdo#105363] -> PASS
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-cpu:
- shard-apl: FAIL [fdo#103167] -> PASS +1
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen:
- shard-glk: FAIL [fdo#103167] -> PASS +2
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- shard-apl: DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +1
* {igt@kms_plane@plane-position-covered-pipe-a-planes}:
- shard-glk: FAIL [fdo#110038] -> PASS
- shard-apl: FAIL [fdo#110038] -> PASS
* igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
- shard-apl: FAIL [fdo#108145] -> PASS
- shard-glk: FAIL [fdo#108145] -> PASS
- shard-kbl: FAIL [fdo#108145] -> PASS
* {igt@kms_plane_multiple@atomic-pipe-a-tiling-none}:
- shard-apl: FAIL [fdo#110037] -> PASS
* {igt@kms_plane_multiple@atomic-pipe-a-tiling-y}:
- shard-glk: FAIL [fdo#110037] -> PASS
* igt@kms_rotation_crc@multiplane-rotation-cropping-top:
- shard-kbl: FAIL [fdo#109016] -> PASS
* igt@kms_setmode@basic:
- shard-hsw: FAIL [fdo#99912] -> PASS
* igt@kms_universal_plane@universal-plane-pipe-a-functional:
- shard-apl: FAIL [fdo#103166] -> PASS
* igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend:
- shard-apl: FAIL [fdo#104894] -> PASS
* igt@kms_vblank@pipe-c-ts-continuation-dpms-suspend:
- shard-kbl: INCOMPLETE [fdo#103665] -> PASS +1
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- shard-kbl: FAIL [fdo#104894] -> PASS
* igt@perf@rc6-disable:
- shard-kbl: FAIL [fdo#103179] -> PASS
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[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#103179]: https://bugs.freedesktop.org/show_bug.cgi?id=103179
[fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
[fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
[fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
[fdo#104894]: https://bugs.freedesktop.org/show_bug.cgi?id=104894
[fdo#105010]: https://bugs.freedesktop.org/show_bug.cgi?id=105010
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
[fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
[fdo#105767]: https://bugs.freedesktop.org/show_bug.cgi?id=105767
[fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
[fdo#108590]: https://bugs.freedesktop.org/show_bug.cgi?id=108590
[fdo#109016]: https://bugs.freedesktop.org/show_bug.cgi?id=109016
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109373]: https://bugs.freedesktop.org/show_bug.cgi?id=109373
[fdo#109674]: https://bugs.freedesktop.org/show_bug.cgi?id=109674
[fdo#110037]: https://bugs.freedesktop.org/show_bug.cgi?id=110037
[fdo#110038]: https://bugs.freedesktop.org/show_bug.cgi?id=110038
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
Participating hosts (10 -> 5)
------------------------------
Missing (5): shard-skl pig-hsw-4770r pig-glk-j5005 shard-iclb pig-skl-6260u
Build changes
-------------
* IGT: IGT_4883 -> IGTPW_2602
* Piglit: piglit_4509 -> None
CI_DRM_5737: d5bb7d77aa77996702426496078a597f30bead58 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2602: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2602/
IGT_4883: b25e06d6ddf2e42044cd9c93b613cbc7339a8c33 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2602/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-03-13 17:26 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-13 12:06 [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Petri Latvala
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 2/4] gem_exec_fence: Really loop through all engines Petri Latvala
2019-03-13 12:33 ` Chris Wilson
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 3/4] gem_exec_latency: Initialize variance to zero Petri Latvala
2019-03-13 12:33 ` Chris Wilson
2019-03-13 12:34 ` Chris Wilson
2019-03-13 12:06 ` [igt-dev] [PATCH i-g-t 4/4] gem_exec_lut_handle: Initialize c Petri Latvala
2019-03-13 12:12 ` Chris Wilson
2019-03-13 12:36 ` Chris Wilson
2019-03-13 12:32 ` [igt-dev] [PATCH i-g-t 1/4] gem_exec_gttfill: Plug memory leak Chris Wilson
2019-03-13 13:47 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/4] " Patchwork
2019-03-13 17:26 ` [igt-dev] ✗ Fi.CI.IGT: failure " 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.