[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [01/18] drm/i915/selftests: Ensure we don't clamp a random offset to 32b

Patchwork patchwork at emeril.freedesktop.org
Thu Jul 11 09:05:37 UTC 2019


== Series Details ==

Series: series starting with [01/18] drm/i915/selftests: Ensure we don't clamp a random offset to 32b
URL   : https://patchwork.freedesktop.org/series/63533/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
912dcfca3fc2 drm/i915/selftests: Ensure we don't clamp a random offset to 32b
b15f72d4a124 drm/i915/selftests: Hold the vma manager lock while modifying mmap_offset
a7c1d3b64fac drm/i915: Lock the engine while dumping the active request
f890aa2256c6 drm/i915: Rely on spinlock protection for GPU error capture
8c30f33da09c drm/i915/oa: Reconfigure contexts on the fly
aecf3ec7aac8 drm/i915: Add to timeline requires the timeline mutex
6b518a7d8705 drm/i915: Teach execbuffer to take the engine wakeref not GT
382d7993f6df drm/i915/gt: Track timeline activeness in enter/exit
28ee984d9e7d drm/i915/gt: Convert timeline tracking to spinlock
f05a2009711d drm/i915/gt: Guard timeline pinning with its own mutex
0f981d4ad129 drm/i915: Protect request retirement with timeline->mutex
35f7d0cd1411 drm/i915: Replace struct_mutex for batch pool serialisation
-:305: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#305: 
new file mode 100644

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

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

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

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

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

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

-:557: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment
#557: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool_types.h:18:
+	spinlock_t lock;

total: 0 errors, 7 warnings, 1 checks, 613 lines checked
1e04df10d91b drm/i915/gt: Mark context->active_count as protected by timeline->mutex
b9d33aecc1c3 drm/i915: Forgo last_fence active request tracking
ba014d8c95c1 drm/i915/overlay: Switch to using i915_active tracking
88e901cf9678 drm/i915: Extract intel_frontbuffer active tracking
dbe8e637b2ac drm/i915: Markup expected timeline locks for i915_active
-:281: CHECK:UNCOMMENTED_DEFINITION: struct mutex definition without comment
#281: FILE: drivers/gpu/drm/i915/i915_active_types.h:28:
+	struct mutex *lock;

total: 0 errors, 0 warnings, 1 checks, 233 lines checked
fcc08729cca8 drm/i915/gt: Use intel_gt as the primary object for handling resets
-:27: WARNING:MEMORY_BARRIER: memory barrier without comment
#27: FILE: drivers/gpu/drm/i915/display/intel_display.c:4279:
+	smp_mb__after_atomic();

-:1826: WARNING:MEMORY_BARRIER: memory barrier without comment
#1826: FILE: drivers/gpu/drm/i915/gt/intel_reset.c:1269:
+	smp_mb__after_atomic();

-:2051: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'W' - possible side-effects?
#2051: FILE: drivers/gpu/drm/i915/gt/intel_reset.h:64:
+#define intel_wedge_on_timeout(W, GT, TIMEOUT)				\
+	for (__intel_init_wedge((W), (GT), (TIMEOUT), __func__);	\
+	     (W)->gt;							\
+	     __intel_fini_wedge((W)))

-:2066: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#2066: 
new file mode 100644

-:4479: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'T' - possible side-effects?
#4479: FILE: drivers/gpu/drm/i915/i915_selftest.h:94:
+#define intel_gt_live_subtests(T, data) ({ \
+	typecheck(struct intel_gt *, data); \
+	__i915_subtests(__func__, \
+			__intel_gt_live_setup, __intel_gt_live_teardown, \
+			T, ARRAY_SIZE(T), data); \
+})

total: 0 errors, 3 warnings, 2 checks, 4298 lines checked



More information about the Intel-gfx mailing list