Series without cover letter

Submitted by Chris Wilson on April 26, 2019, 4:33 p.m.

Details

Reviewer None
Submitted April 26, 2019, 4:33 p.m.
Last Updated April 26, 2019, 5:17 p.m.
Revision 1

Revisions

Patches download mbox

Tests

CI Bug Log - changes from CI_DRM_6006 -> Patchwork_12885
====================================================

Summary
-------

  **FAILURE**

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

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@gem_exec_create@basic:
    - fi-blb-e6850:       [PASS][1] -> [INCOMPLETE][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6006/fi-blb-e6850/igt@gem_exec_create@basic.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12885/fi-blb-e6850/igt@gem_exec_create@basic.html

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

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

### IGT changes ###

#### Possible fixes ####

  * igt@gem_basic@create-fd-close:
    - fi-skl-6700k2:      [INCOMPLETE][3] -> [PASS][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6006/fi-skl-6700k2/igt@gem_basic@create-fd-close.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12885/fi-skl-6700k2/igt@gem_basic@create-fd-close.html

  


Participating hosts (25 -> 23)
------------------------------

  ERROR: It appears as if the changes made in Patchwork_12885 prevented too many machines from booting.

  Additional (11): fi-skl-gvtdvm fi-byt-j1900 fi-skl-6260u fi-snb-2520m fi-kbl-7500u fi-bxt-j4205 fi-ivb-3770 fi-icl-u3 fi-cfl-8109u fi-skl-iommu fi-icl-dsi 
  Missing    (13): fi-kbl-soraka fi-hsw-4770r fi-ilk-m540 fi-skl-guc fi-bdw-gvtdvm fi-byt-squawks fi-apl-guc fi-byt-clapper fi-kbl-x1275 fi-icl-y fi-byt-n2820 fi-bsw-kefka fi-snb-2600 


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

  * Linux: CI_DRM_6006 -> Patchwork_12885

  CI_DRM_6006: f2b80d9616c3ced3d5e04bba276c4e57265faabb @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4968: caed251990f35bfe45368f803980071a73e36315 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_12885: 68ad0fba195d3e0399a2ae9aeee8c772323a34a5 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

68ad0fba195d drm/i915: Move i915_request_alloc into selftests/
3ad89dd1bd8f drm/i915: Remove intel_context.active_link
db367156d990 drm/i915: Switch back to an array of logical per-engine HW contexts
5a139a452c59 drm/i915: Split engine setup/init into two phases
6cd19bc51af2 drm/i915: Pass intel_context to intel_context_pin_lock()
e18447869df3 drm/i915/selftests: Pass around intel_context for sseu
059c3b36d6aa drm/i915/selftests: Use the real kernel context for sseu isolation tests
e21e4dc54c65 drm/i915: Export intel_context_instance()
df18fb0cb42a drm/i915/gvt: Pin the per-engine GVT shadow contexts
$ dim checkpatch origin/drm-tip
df18fb0cb42a drm/i915/gvt: Pin the per-engine GVT shadow contexts
e21e4dc54c65 drm/i915: Export intel_context_instance()
059c3b36d6aa drm/i915/selftests: Use the real kernel context for sseu isolation tests
e18447869df3 drm/i915/selftests: Pass around intel_context for sseu
6cd19bc51af2 drm/i915: Pass intel_context to intel_context_pin_lock()
5a139a452c59 drm/i915: Split engine setup/init into two phases
db367156d990 drm/i915: Switch back to an array of logical per-engine HW contexts
-:588: WARNING:LINE_SPACING: Missing a blank line after declarations
#588: FILE: drivers/gpu/drm/i915/i915_gem_context.h:214:
+		struct i915_gem_engines *e = rcu_dereference(ctx->engines);
+		if (likely(idx < e->num_engines && e->engines[idx]))

-:607: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'it' - possible side-effects?
#607: FILE: drivers/gpu/drm/i915/i915_gem_context.h:233:
+#define for_each_gem_engine(ce, engines, it) \
+	for (i915_gem_engines_iter_init(&(it), (engines)); \
+	     ((ce) = i915_gem_engines_iter_next(&(it)));)

total: 0 errors, 1 warnings, 1 checks, 962 lines checked
3ad89dd1bd8f drm/i915: Remove intel_context.active_link
68ad0fba195d drm/i915: Move i915_request_alloc into selftests/
-:421: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#421: 
new file mode 100644

-:426: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#426: FILE: drivers/gpu/drm/i915/selftests/igt_gem_utils.c:1:
+/*

-:427: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#427: FILE: drivers/gpu/drm/i915/selftests/igt_gem_utils.c:2:
+ * SPDX-License-Identifier: MIT

-:466: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#466: FILE: drivers/gpu/drm/i915/selftests/igt_gem_utils.h:1:
+/*

-:467: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead
#467: FILE: drivers/gpu/drm/i915/selftests/igt_gem_utils.h:2:
+ * SPDX-License-Identifier: MIT

total: 0 errors, 5 warnings, 0 checks, 408 lines checked
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915/gvt: Pin the per-engine GVT shadow contexts
Okay!

Commit: drm/i915: Export intel_context_instance()
-O:drivers/gpu/drm/i915/gt/intel_context.c:131:22: warning: context imbalance in 'intel_context_pin_lock' - wrong count at exit
+drivers/gpu/drm/i915/gt/intel_context.c:131:22: warning: context imbalance in 'intel_context_pin_lock' - wrong count at exit
+drivers/gpu/drm/i915/gt/intel_context.c:150:6: warning: context imbalance in 'intel_context_pin_unlock' - wrong count at exit

Commit: drm/i915/selftests: Use the real kernel context for sseu isolation tests
Okay!

Commit: drm/i915/selftests: Pass around intel_context for sseu
Okay!

Commit: drm/i915: Pass intel_context to intel_context_pin_lock()
-O:drivers/gpu/drm/i915/gt/intel_context.c:131:22: warning: context imbalance in 'intel_context_pin_lock' - wrong count at exit
-O:drivers/gpu/drm/i915/gt/intel_context.c:150:6: warning: context imbalance in 'intel_context_pin_unlock' - wrong count at exit

Commit: drm/i915: Split engine setup/init into two phases
Okay!

Commit: drm/i915: Switch back to an array of logical per-engine HW contexts
+./include/linux/overflow.h:285:13: error: incorrect type in conditional
+./include/linux/overflow.h:285:13: error: undefined identifier '__builtin_mul_overflow'
+./include/linux/overflow.h:285:13:    got void
+./include/linux/overflow.h:285:13: warning: call with no type!
+./include/linux/overflow.h:287:13: error: incorrect type in conditional
+./include/linux/overflow.h:287:13: error: undefined identifier '__builtin_add_overflow'
+./include/linux/overflow.h:287:13:    got void
+./include/linux/overflow.h:287:13: warning: call with no type!

Commit: drm/i915: Remove intel_context.active_link
Okay!

Commit: drm/i915: Move i915_request_alloc into selftests/
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)