* [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE
@ 2019-12-23 20:44 Chris Wilson
2019-12-23 20:44 ` [Intel-gfx] [PATCH 2/2] drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co Chris Wilson
` (4 more replies)
0 siblings, 5 replies; 8+ messages in thread
From: Chris Wilson @ 2019-12-23 20:44 UTC (permalink / raw)
To: intel-gfx
Add a space between the prefixed format and the users format so that the
join are not mistakenly combined into one long word.
Fixes: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
---
drivers/gpu/drm/i915/gt/intel_context.h | 2 +-
drivers/gpu/drm/i915/i915_request.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_context.h b/drivers/gpu/drm/i915/gt/intel_context.h
index 1d4a1b1357cf..0f5ae4ff3b10 100644
--- a/drivers/gpu/drm/i915/gt/intel_context.h
+++ b/drivers/gpu/drm/i915/gt/intel_context.h
@@ -19,7 +19,7 @@
#define CE_TRACE(ce, fmt, ...) do { \
const struct intel_context *ce__ = (ce); \
- ENGINE_TRACE(ce__->engine, "context:%llx" fmt, \
+ ENGINE_TRACE(ce__->engine, "context:%llx " fmt, \
ce__->timeline->fence_context, \
##__VA_ARGS__); \
} while (0)
diff --git a/drivers/gpu/drm/i915/i915_request.h b/drivers/gpu/drm/i915/i915_request.h
index 565322640378..9784421a3b4d 100644
--- a/drivers/gpu/drm/i915/i915_request.h
+++ b/drivers/gpu/drm/i915/i915_request.h
@@ -51,7 +51,7 @@ struct i915_capture_list {
#define RQ_TRACE(rq, fmt, ...) do { \
const struct i915_request *rq__ = (rq); \
- ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d" fmt, \
+ ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d " fmt, \
rq__->fence.context, rq__->fence.seqno, \
hwsp_seqno(rq__), ##__VA_ARGS__); \
} while (0)
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Intel-gfx] [PATCH 2/2] drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
@ 2019-12-23 20:44 ` Chris Wilson
2019-12-23 20:53 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE Patchwork
` (3 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Chris Wilson @ 2019-12-23 20:44 UTC (permalink / raw)
To: intel-gfx
Convert the few remaining GEM_TRACE() used for debugging over to the
appropriate GT_TRACE or RQ_TRACE.
References: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
---
drivers/gpu/drm/i915/gt/intel_reset.c | 21 ++++++++-------------
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
index 1c51296646e0..5cade72c7701 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -147,11 +147,7 @@ static void mark_innocent(struct i915_request *rq)
void __i915_request_reset(struct i915_request *rq, bool guilty)
{
- GEM_TRACE("%s rq=%llx:%lld, guilty? %s\n",
- rq->engine->name,
- rq->fence.context,
- rq->fence.seqno,
- yesno(guilty));
+ RQ_TRACE(rq, "guilty? %s\n", yesno(guilty));
GEM_BUG_ON(i915_request_completed(rq));
@@ -625,7 +621,7 @@ int __intel_gt_reset(struct intel_gt *gt, intel_engine_mask_t engine_mask)
*/
intel_uncore_forcewake_get(gt->uncore, FORCEWAKE_ALL);
for (retry = 0; ret == -ETIMEDOUT && retry < retries; retry++) {
- GEM_TRACE("engine_mask=%x\n", engine_mask);
+ GT_TRACE(gt, "engine_mask=%x\n", engine_mask);
preempt_disable();
ret = reset(gt, engine_mask, retry);
preempt_enable();
@@ -785,8 +781,7 @@ static void nop_submit_request(struct i915_request *request)
struct intel_engine_cs *engine = request->engine;
unsigned long flags;
- GEM_TRACE("%s fence %llx:%lld -> -EIO\n",
- engine->name, request->fence.context, request->fence.seqno);
+ RQ_TRACE(request, "-EIO\n");
dma_fence_set_error(&request->fence, -EIO);
spin_lock_irqsave(&engine->active.lock, flags);
@@ -813,7 +808,7 @@ static void __intel_gt_set_wedged(struct intel_gt *gt)
intel_engine_dump(engine, &p, "%s\n", engine->name);
}
- GEM_TRACE("start\n");
+ GT_TRACE(gt, "start\n");
/*
* First, stop submission to hw, but do not yet complete requests by
@@ -844,7 +839,7 @@ static void __intel_gt_set_wedged(struct intel_gt *gt)
reset_finish(gt, awake);
- GEM_TRACE("end\n");
+ GT_TRACE(gt, "end\n");
}
void intel_gt_set_wedged(struct intel_gt *gt)
@@ -870,7 +865,7 @@ static bool __intel_gt_unset_wedged(struct intel_gt *gt)
if (test_bit(I915_WEDGED_ON_INIT, >->reset.flags))
return false;
- GEM_TRACE("start\n");
+ GT_TRACE(gt, "start\n");
/*
* Before unwedging, make sure that all pending operations
@@ -932,7 +927,7 @@ static bool __intel_gt_unset_wedged(struct intel_gt *gt)
*/
intel_engines_reset_default_submission(gt);
- GEM_TRACE("end\n");
+ GT_TRACE(gt, "end\n");
smp_mb__before_atomic(); /* complete takeover before enabling execbuf */
clear_bit(I915_WEDGED, >->reset.flags);
@@ -1007,7 +1002,7 @@ void intel_gt_reset(struct intel_gt *gt,
intel_engine_mask_t awake;
int ret;
- GEM_TRACE("flags=%lx\n", gt->reset.flags);
+ GT_TRACE(gt, "flags=%lx\n", gt->reset.flags);
might_sleep();
GEM_BUG_ON(!test_bit(I915_RESET_BACKOFF, >->reset.flags));
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
2019-12-23 20:44 ` [Intel-gfx] [PATCH 2/2] drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co Chris Wilson
@ 2019-12-23 20:53 ` Patchwork
2019-12-23 21:22 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
` (2 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-12-23 20:53 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
URL : https://patchwork.freedesktop.org/series/71331/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
a793707d6e12 drm/i915: Add spaces before compound GEM_TRACE
0e8083f32b63 drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co
-:9: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 639f2f24895f ("drm/i915: Introduce new macros for tracing")'
#9:
References: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
total: 1 errors, 0 warnings, 0 checks, 69 lines checked
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
2019-12-23 20:44 ` [Intel-gfx] [PATCH 2/2] drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co Chris Wilson
2019-12-23 20:53 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE Patchwork
@ 2019-12-23 21:22 ` Patchwork
2019-12-23 21:23 ` [Intel-gfx] [PATCH 1/2] " Lucas De Marchi
2019-12-26 8:54 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] " Patchwork
4 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-12-23 21:22 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
URL : https://patchwork.freedesktop.org/series/71331/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7630 -> Patchwork_15903
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/index.html
Known issues
------------
Here are the changes found in Patchwork_15903 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_gttfill@basic:
- fi-tgl-y: [PASS][1] -> [INCOMPLETE][2] ([fdo#111593])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-tgl-y/igt@gem_exec_gttfill@basic.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-tgl-y/igt@gem_exec_gttfill@basic.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-skl-6700k2: [PASS][3] -> [INCOMPLETE][4] ([i915#671])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [PASS][5] -> [FAIL][6] ([fdo#111407])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_frontbuffer_tracking@basic:
- fi-hsw-peppy: [PASS][7] -> [DMESG-WARN][8] ([i915#44])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
#### Possible fixes ####
* igt@gem_exec_create@basic:
- {fi-tgl-u}: [INCOMPLETE][9] ([fdo#111736]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-tgl-u/igt@gem_exec_create@basic.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-tgl-u/igt@gem_exec_create@basic.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-cfl-8700k: [INCOMPLETE][11] ([i915#505]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_selftest@live_execlists:
- fi-kbl-soraka: [DMESG-FAIL][13] ([i915#656]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-kbl-soraka/igt@i915_selftest@live_execlists.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-kbl-soraka/igt@i915_selftest@live_execlists.html
#### Warnings ####
* igt@i915_selftest@live_blt:
- fi-hsw-4770: [DMESG-FAIL][15] ([i915#725]) -> [DMESG-FAIL][16] ([i915#563])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-hsw-4770/igt@i915_selftest@live_blt.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-hsw-4770/igt@i915_selftest@live_blt.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][17] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][18] ([i915#62] / [i915#92]) +5 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][19] ([i915#62] / [i915#92]) -> [DMESG-WARN][20] ([i915#62] / [i915#92] / [i915#95]) +4 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
[fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
[fdo#111736]: https://bugs.freedesktop.org/show_bug.cgi?id=111736
[i915#435]: https://gitlab.freedesktop.org/drm/intel/issues/435
[i915#44]: https://gitlab.freedesktop.org/drm/intel/issues/44
[i915#505]: https://gitlab.freedesktop.org/drm/intel/issues/505
[i915#563]: https://gitlab.freedesktop.org/drm/intel/issues/563
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#656]: https://gitlab.freedesktop.org/drm/intel/issues/656
[i915#671]: https://gitlab.freedesktop.org/drm/intel/issues/671
[i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
[i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (43 -> 46)
------------------------------
Additional (10): fi-hsw-4770r fi-byt-j1900 fi-skl-6770hq fi-glk-dsi fi-whl-u fi-gdg-551 fi-ivb-3770 fi-kbl-7560u fi-bsw-nick fi-snb-2600
Missing (7): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-bwr-2160 fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7630 -> Patchwork_15903
CI-20190529: 20190529
CI_DRM_7630: 28a2aa0ebf1520ea8a0dd89299f7ceea80dfd96f @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5352: 0586d205f651674e575351c2d5a7d0760716c9f1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15903: 0e8083f32b6366df29e9383d31722eea390f9a39 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
0e8083f32b63 drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co
a793707d6e12 drm/i915: Add spaces before compound GEM_TRACE
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
` (2 preceding siblings ...)
2019-12-23 21:22 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2019-12-23 21:23 ` Lucas De Marchi
2019-12-23 21:54 ` Chris Wilson
2019-12-26 8:54 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] " Patchwork
4 siblings, 1 reply; 8+ messages in thread
From: Lucas De Marchi @ 2019-12-23 21:23 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
On Mon, Dec 23, 2019 at 08:44:10PM +0000, Chris Wilson wrote:
>Add a space between the prefixed format and the users format so that the
>join are not mistakenly combined into one long word.
>
>Fixes: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
>Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
>---
> drivers/gpu/drm/i915/gt/intel_context.h | 2 +-
> drivers/gpu/drm/i915/i915_request.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/i915/gt/intel_context.h b/drivers/gpu/drm/i915/gt/intel_context.h
>index 1d4a1b1357cf..0f5ae4ff3b10 100644
>--- a/drivers/gpu/drm/i915/gt/intel_context.h
>+++ b/drivers/gpu/drm/i915/gt/intel_context.h
>@@ -19,7 +19,7 @@
>
> #define CE_TRACE(ce, fmt, ...) do { \
> const struct intel_context *ce__ = (ce); \
>- ENGINE_TRACE(ce__->engine, "context:%llx" fmt, \
>+ ENGINE_TRACE(ce__->engine, "context:%llx " fmt, \
> ce__->timeline->fence_context, \
> ##__VA_ARGS__); \
> } while (0)
>diff --git a/drivers/gpu/drm/i915/i915_request.h b/drivers/gpu/drm/i915/i915_request.h
>index 565322640378..9784421a3b4d 100644
>--- a/drivers/gpu/drm/i915/i915_request.h
>+++ b/drivers/gpu/drm/i915/i915_request.h
>@@ -51,7 +51,7 @@ struct i915_capture_list {
>
> #define RQ_TRACE(rq, fmt, ...) do { \
> const struct i915_request *rq__ = (rq); \
>- ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d" fmt, \
>+ ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d " fmt, \
do we care about the trailing space if fmt is "\n"?
Lucas De Marchi
> rq__->fence.context, rq__->fence.seqno, \
> hwsp_seqno(rq__), ##__VA_ARGS__); \
> } while (0)
>--
>2.24.1
>
>_______________________________________________
>Intel-gfx mailing list
>Intel-gfx@lists.freedesktop.org
>https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 21:23 ` [Intel-gfx] [PATCH 1/2] " Lucas De Marchi
@ 2019-12-23 21:54 ` Chris Wilson
2019-12-23 22:00 ` Lucas De Marchi
0 siblings, 1 reply; 8+ messages in thread
From: Chris Wilson @ 2019-12-23 21:54 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
Quoting Lucas De Marchi (2019-12-23 21:23:36)
> On Mon, Dec 23, 2019 at 08:44:10PM +0000, Chris Wilson wrote:
> >Add a space between the prefixed format and the users format so that the
> >join are not mistakenly combined into one long word.
> >
> >Fixes: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
> >Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> >Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
> >---
> > drivers/gpu/drm/i915/gt/intel_context.h | 2 +-
> > drivers/gpu/drm/i915/i915_request.h | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/i915/gt/intel_context.h b/drivers/gpu/drm/i915/gt/intel_context.h
> >index 1d4a1b1357cf..0f5ae4ff3b10 100644
> >--- a/drivers/gpu/drm/i915/gt/intel_context.h
> >+++ b/drivers/gpu/drm/i915/gt/intel_context.h
> >@@ -19,7 +19,7 @@
> >
> > #define CE_TRACE(ce, fmt, ...) do { \
> > const struct intel_context *ce__ = (ce); \
> >- ENGINE_TRACE(ce__->engine, "context:%llx" fmt, \
> >+ ENGINE_TRACE(ce__->engine, "context:%llx " fmt, \
> > ce__->timeline->fence_context, \
> > ##__VA_ARGS__); \
> > } while (0)
> >diff --git a/drivers/gpu/drm/i915/i915_request.h b/drivers/gpu/drm/i915/i915_request.h
> >index 565322640378..9784421a3b4d 100644
> >--- a/drivers/gpu/drm/i915/i915_request.h
> >+++ b/drivers/gpu/drm/i915/i915_request.h
> >@@ -51,7 +51,7 @@ struct i915_capture_list {
> >
> > #define RQ_TRACE(rq, fmt, ...) do { \
> > const struct i915_request *rq__ = (rq); \
> >- ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d" fmt, \
> >+ ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d " fmt, \
>
> do we care about the trailing space if fmt is "\n"?
No. An extra space in a debug log, which you only see when something
blows up and only compiled in for CI, is the last of your worries.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 21:54 ` Chris Wilson
@ 2019-12-23 22:00 ` Lucas De Marchi
0 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-12-23 22:00 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
On Mon, Dec 23, 2019 at 09:54:55PM +0000, Chris Wilson wrote:
>Quoting Lucas De Marchi (2019-12-23 21:23:36)
>> On Mon, Dec 23, 2019 at 08:44:10PM +0000, Chris Wilson wrote:
>> >Add a space between the prefixed format and the users format so that the
>> >join are not mistakenly combined into one long word.
>> >
>> >Fixes: 639f2f24895f ("drm/i915: Introduce new macros for tracing")
>> >Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
>> >Cc: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
>> >---
>> > drivers/gpu/drm/i915/gt/intel_context.h | 2 +-
>> > drivers/gpu/drm/i915/i915_request.h | 2 +-
>> > 2 files changed, 2 insertions(+), 2 deletions(-)
>> >
>> >diff --git a/drivers/gpu/drm/i915/gt/intel_context.h b/drivers/gpu/drm/i915/gt/intel_context.h
>> >index 1d4a1b1357cf..0f5ae4ff3b10 100644
>> >--- a/drivers/gpu/drm/i915/gt/intel_context.h
>> >+++ b/drivers/gpu/drm/i915/gt/intel_context.h
>> >@@ -19,7 +19,7 @@
>> >
>> > #define CE_TRACE(ce, fmt, ...) do { \
>> > const struct intel_context *ce__ = (ce); \
>> >- ENGINE_TRACE(ce__->engine, "context:%llx" fmt, \
>> >+ ENGINE_TRACE(ce__->engine, "context:%llx " fmt, \
>> > ce__->timeline->fence_context, \
>> > ##__VA_ARGS__); \
>> > } while (0)
>> >diff --git a/drivers/gpu/drm/i915/i915_request.h b/drivers/gpu/drm/i915/i915_request.h
>> >index 565322640378..9784421a3b4d 100644
>> >--- a/drivers/gpu/drm/i915/i915_request.h
>> >+++ b/drivers/gpu/drm/i915/i915_request.h
>> >@@ -51,7 +51,7 @@ struct i915_capture_list {
>> >
>> > #define RQ_TRACE(rq, fmt, ...) do { \
>> > const struct i915_request *rq__ = (rq); \
>> >- ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d" fmt, \
>> >+ ENGINE_TRACE(rq__->engine, "fence %llx:%lld, current %d " fmt, \
>>
>> do we care about the trailing space if fmt is "\n"?
>
>No. An extra space in a debug log, which you only see when something
>blows up and only compiled in for CI, is the last of your worries.
oh, ok. This in the end uses trace_printk().
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Lucas De Marchi
>-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
` (3 preceding siblings ...)
2019-12-23 21:23 ` [Intel-gfx] [PATCH 1/2] " Lucas De Marchi
@ 2019-12-26 8:54 ` Patchwork
4 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-12-26 8:54 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE
URL : https://patchwork.freedesktop.org/series/71331/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7630_full -> Patchwork_15903_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15903_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15903_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15903_full:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_workarounds:
- shard-apl: [PASS][1] -> [DMESG-FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-apl3/igt@i915_selftest@live_workarounds.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-apl4/igt@i915_selftest@live_workarounds.html
New tests
---------
New tests have been introduced between CI_DRM_7630_full and Patchwork_15903_full:
### New Piglit tests (9) ###
* spec@arb_vertex_attrib_64bit@execution@vs_in@vs-input-double_dmat2-double_dvec3-position:
- Statuses : 1 fail(s)
- Exec time: [0.13] s
* spec@ext_framebuffer_object@fbo-stencil-gl_stencil_index4-copypixels:
- Statuses : 1 fail(s)
- Exec time: [0.08] s
* spec@glsl-4.20@execution@vs_in@vs-input-double_dmat2_array5-float_float-position:
- Statuses : 1 fail(s)
- Exec time: [0.14] s
* spec@glsl-4.20@execution@vs_in@vs-input-float_mat4x2-double_dmat4x2_array2-position:
- Statuses : 1 fail(s)
- Exec time: [0.13] s
* spec@glsl-4.20@execution@vs_in@vs-input-float_vec3-position-double_dmat3x2_array2:
- Statuses : 1 fail(s)
- Exec time: [0.13] s
* spec@glsl-4.20@execution@vs_in@vs-input-int_ivec3-position-double_dmat3x4_array2:
- Statuses : 1 fail(s)
- Exec time: [0.15] s
* spec@glsl-4.20@execution@vs_in@vs-input-int_ivec3_array3-position-double_dvec3:
- Statuses : 1 fail(s)
- Exec time: [0.16] s
* spec@glsl-4.20@execution@vs_in@vs-input-position-double_dmat3x4-float_mat2x3_array3:
- Statuses : 1 fail(s)
- Exec time: [0.12] s
* spec@glsl-4.20@execution@vs_in@vs-input-ushort_uint-position-double_dmat4:
- Statuses : 1 fail(s)
- Exec time: [0.17] s
Known issues
------------
Here are the changes found in Patchwork_15903_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_isolation@vcs1-s3:
- shard-tglb: [PASS][3] -> [INCOMPLETE][4] ([i915#456])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb6/igt@gem_ctx_isolation@vcs1-s3.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb4/igt@gem_ctx_isolation@vcs1-s3.html
* igt@gem_eio@in-flight-suspend:
- shard-tglb: [PASS][5] -> [INCOMPLETE][6] ([i915#456] / [i915#460] / [i915#534])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb3/igt@gem_eio@in-flight-suspend.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb7/igt@gem_eio@in-flight-suspend.html
* igt@gem_exec_create@madvise:
- shard-tglb: [PASS][7] -> [INCOMPLETE][8] ([i915#435])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb7/igt@gem_exec_create@madvise.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb4/igt@gem_exec_create@madvise.html
* igt@gem_exec_schedule@preempt-queue-contexts-bsd1:
- shard-tglb: [PASS][9] -> [INCOMPLETE][10] ([fdo#111606] / [fdo#111677])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb8/igt@gem_exec_schedule@preempt-queue-contexts-bsd1.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb6/igt@gem_exec_schedule@preempt-queue-contexts-bsd1.html
* igt@gem_exec_schedule@smoketest-all:
- shard-tglb: [PASS][11] -> [INCOMPLETE][12] ([i915#463])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb5/igt@gem_exec_schedule@smoketest-all.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb7/igt@gem_exec_schedule@smoketest-all.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-glk: [PASS][13] -> [FAIL][14] ([i915#644])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-glk6/igt@gem_ppgtt@flink-and-close-vma-leak.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-glk8/igt@gem_ppgtt@flink-and-close-vma-leak.html
- shard-apl: [PASS][15] -> [FAIL][16] ([i915#644])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-apl2/igt@gem_ppgtt@flink-and-close-vma-leak.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-apl4/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@i915_suspend@debugfs-reader:
- shard-tglb: [PASS][17] -> [INCOMPLETE][18] ([i915#456] / [i915#460])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb7/igt@i915_suspend@debugfs-reader.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb8/igt@i915_suspend@debugfs-reader.html
* igt@kms_cursor_crc@pipe-a-cursor-dpms:
- shard-hsw: [PASS][19] -> [DMESG-WARN][20] ([IGT#6])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-hsw6/igt@kms_cursor_crc@pipe-a-cursor-dpms.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-hsw6/igt@kms_cursor_crc@pipe-a-cursor-dpms.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-kbl: [PASS][21] -> [DMESG-WARN][22] ([i915#180]) +5 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_crc@pipe-d-cursor-suspend:
- shard-tglb: [PASS][23] -> [INCOMPLETE][24] ([i915#460]) +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-suspend.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb8/igt@kms_cursor_crc@pipe-d-cursor-suspend.html
* igt@kms_cursor_legacy@all-pipes-torture-move:
- shard-glk: [PASS][25] -> [DMESG-WARN][26] ([i915#128])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-glk2/igt@kms_cursor_legacy@all-pipes-torture-move.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-glk4/igt@kms_cursor_legacy@all-pipes-torture-move.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl: [PASS][27] -> [FAIL][28] ([i915#79])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl2/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-apl: [PASS][29] -> [DMESG-WARN][30] ([i915#180]) +2 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
- shard-tglb: [PASS][31] -> [FAIL][32] ([i915#49]) +1 similar issue
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb4/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb3/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-render:
- shard-skl: [PASS][33] -> [FAIL][34] ([i915#49])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl2/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-render.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl4/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-render.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
- shard-skl: [PASS][35] -> [FAIL][36] ([i915#53])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl2/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl4/igt@kms_pipe_crc_basic@hang-read-crc-pipe-a.html
* igt@kms_plane@pixel-format-pipe-b-planes-source-clamping:
- shard-skl: [PASS][37] -> [DMESG-WARN][38] ([i915#109]) +1 similar issue
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl9/igt@kms_plane@pixel-format-pipe-b-planes-source-clamping.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl9/igt@kms_plane@pixel-format-pipe-b-planes-source-clamping.html
* igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
- shard-skl: [PASS][39] -> [FAIL][40] ([fdo#108145])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl2/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl4/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: [PASS][41] -> [FAIL][42] ([fdo#108145] / [i915#265])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl9/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html
* igt@kms_setmode@basic:
- shard-apl: [PASS][43] -> [FAIL][44] ([i915#31])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-apl1/igt@kms_setmode@basic.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-apl3/igt@kms_setmode@basic.html
#### Possible fixes ####
* igt@gem_ctx_isolation@vcs1-s3:
- shard-kbl: [DMESG-WARN][45] ([i915#180]) -> [PASS][46] +1 similar issue
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-kbl2/igt@gem_ctx_isolation@vcs1-s3.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-kbl6/igt@gem_ctx_isolation@vcs1-s3.html
* igt@gem_ctx_persistence@bcs0-mixed-process:
- shard-skl: [FAIL][47] ([i915#679]) -> [PASS][48]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl7/igt@gem_ctx_persistence@bcs0-mixed-process.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl4/igt@gem_ctx_persistence@bcs0-mixed-process.html
* igt@gem_eio@unwedge-stress:
- shard-tglb: [INCOMPLETE][49] ([i915#469]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb6/igt@gem_eio@unwedge-stress.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb8/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_gttfill@basic:
- shard-tglb: [INCOMPLETE][51] ([fdo#111593]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb5/igt@gem_exec_gttfill@basic.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb4/igt@gem_exec_gttfill@basic.html
* igt@gem_exec_parallel@contexts:
- shard-tglb: [INCOMPLETE][53] ([i915#470]) -> [PASS][54]
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb9/igt@gem_exec_parallel@contexts.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb2/igt@gem_exec_parallel@contexts.html
* igt@gem_exec_reuse@single:
- shard-tglb: [INCOMPLETE][55] ([i915#435]) -> [PASS][56]
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb7/igt@gem_exec_reuse@single.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb9/igt@gem_exec_reuse@single.html
* igt@kms_color@pipe-a-ctm-0-5:
- shard-skl: [DMESG-WARN][57] ([i915#109]) -> [PASS][58]
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl4/igt@kms_color@pipe-a-ctm-0-5.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl8/igt@kms_color@pipe-a-ctm-0-5.html
* igt@kms_cursor_crc@pipe-c-cursor-256x256-sliding:
- shard-skl: [FAIL][59] ([i915#54]) -> [PASS][60]
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl6/igt@kms_cursor_crc@pipe-c-cursor-256x256-sliding.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl1/igt@kms_cursor_crc@pipe-c-cursor-256x256-sliding.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-tglb: [INCOMPLETE][61] ([i915#435] / [i915#456] / [i915#460]) -> [PASS][62]
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb2/igt@kms_fbcon_fbt@fbc-suspend.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb1/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-skl: [FAIL][63] ([i915#79]) -> [PASS][64]
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl4/igt@kms_flip@flip-vs-expired-vblank.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl5/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-glk: [FAIL][65] ([i915#79]) -> [PASS][66]
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-glk5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-glk4/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite:
- shard-tglb: [FAIL][67] ([i915#49]) -> [PASS][68] +1 similar issue
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb4/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb3/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
* igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min:
- shard-skl: [FAIL][69] ([fdo#108145]) -> [PASS][70] +2 similar issues
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-skl9/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-tglb: [INCOMPLETE][71] ([i915#456] / [i915#460]) -> [PASS][72]
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb1/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb9/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
* igt@kms_vblank@pipe-b-ts-continuation-suspend:
- shard-apl: [DMESG-WARN][73] ([i915#180]) -> [PASS][74] +1 similar issue
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-apl1/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-apl3/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
#### Warnings ####
* igt@gem_ctx_isolation@vcs2-dirty-create:
- shard-tglb: [SKIP][75] ([fdo#111912] / [fdo#112080]) -> [SKIP][76] ([fdo#112080]) +1 similar issue
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb1/igt@gem_ctx_isolation@vcs2-dirty-create.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb9/igt@gem_ctx_isolation@vcs2-dirty-create.html
* igt@i915_pm_dc@dc6-psr:
- shard-tglb: [FAIL][77] ([i915#454]) -> [SKIP][78] ([i915#468])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7630/shard-tglb3/igt@i915_pm_dc@dc6-psr.html
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/shard-tglb6/igt@i915_pm_dc@dc6-psr.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[IGT#6]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/6
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
[fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
[fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
[fdo#111912]: https://bugs.freedesktop.org/show_bug.cgi?id=111912
[fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
[i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109
[i915#128]: https://gitlab.freedesktop.org/drm/intel/issues/128
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31
[i915#435]: https://gitlab.freedesktop.org/drm/intel/issues/435
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#456]: https://gitlab.freedesktop.org/drm/intel/issues/456
[i915#460]: https://gitlab.freedesktop.org/drm/intel/issues/460
[i915#463]: https://gitlab.freedesktop.org/drm/intel/issues/463
[i915#468]: https://gitlab.freedesktop.org/drm/intel/issues/468
[i915#469]: https://gitlab.freedesktop.org/drm/intel/issues/469
[i915#470]: https://gitlab.freedesktop.org/drm/intel/issues/470
[i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
[i915#53]: https://gitlab.freedesktop.org/drm/intel/issues/53
[i915#534]: https://gitlab.freedesktop.org/drm/intel/issues/534
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644
[i915#679]: https://gitlab.freedesktop.org/drm/intel/issues/679
[i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
Participating hosts (11 -> 11)
------------------------------
No changes in participating hosts
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7630 -> Patchwork_15903
CI-20190529: 20190529
CI_DRM_7630: 28a2aa0ebf1520ea8a0dd89299f7ceea80dfd96f @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5352: 0586d205f651674e575351c2d5a7d0760716c9f1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15903: 0e8083f32b6366df29e9383d31722eea390f9a39 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15903/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-12-26 8:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-23 20:44 [Intel-gfx] [PATCH 1/2] drm/i915: Add spaces before compound GEM_TRACE Chris Wilson
2019-12-23 20:44 ` [Intel-gfx] [PATCH 2/2] drm/i915/gt: Convert the final GEM_TRACE to GT_TRACE and co Chris Wilson
2019-12-23 20:53 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Add spaces before compound GEM_TRACE Patchwork
2019-12-23 21:22 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2019-12-23 21:23 ` [Intel-gfx] [PATCH 1/2] " Lucas De Marchi
2019-12-23 21:54 ` Chris Wilson
2019-12-23 22:00 ` Lucas De Marchi
2019-12-26 8:54 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/2] " 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.