✗ Fi.CI.CHECKPATCH: warning for drm/i915: Vulkan performance query support (rev4)

[Patchwork <patchwork@emeril.freedesktop.org>]

== Series Details ==

Series: drm/i915: Vulkan performance query support (rev4)
URL   : https://patchwork.freedesktop.org/series/60916/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
abfe9e305ec9 drm/i915/perf: introduce a versioning of the i915-perf uapi
32203e3c3c4c drm/i915/perf: allow for CS OA configs to be created lazily
-:124: CHECK:SPACING: No space is necessary after a cast
#124: FILE: drivers/gpu/drm/i915/i915_perf.c:395:
+					(u32) MI_LOAD_REGISTER_IMM_MAX_REGS);

total: 0 errors, 0 warnings, 1 checks, 311 lines checked
3aef0b772b8a drm/i915: introduce a mechanism to extend execbuf2
-:126: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#126: FILE: include/uapi/drm/i915_drm.h:1170:
+#define __I915_EXEC_UNKNOWN_FLAGS (-(I915_EXEC_EXT<<1))
                                                   ^

total: 0 errors, 0 warnings, 1 checks, 103 lines checked
bfa3bc4cccc0 drm/i915: add syncobj timeline support
-:341: WARNING:TYPO_SPELLING: 'transfered' may be misspelled - perhaps 'transferred'?
#341: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:2488:
+			 * The chain's ownership is transfered to the

-:365: ERROR:CODE_INDENT: code indent should use tabs where possible
#365: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:2512:
+        [DRM_I915_GEM_EXECBUFFER_EXT_TIMELINE_FENCES] = parse_timeline_fences,$

-:365: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#365: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:2512:
+        [DRM_I915_GEM_EXECBUFFER_EXT_TIMELINE_FENCES] = parse_timeline_fences,$

total: 1 errors, 2 warnings, 0 checks, 511 lines checked
151ed539e4ba drm/i915: add a new perf configuration execbuf parameter
-:44: CHECK:LINE_SPACING: Please don't use multiple blank lines
#44: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1206:
 
+

-:170: ERROR:CODE_INDENT: code indent should use tabs where possible
#170: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:2620:
+        [DRM_I915_GEM_EXECBUFFER_EXT_PERF] = parse_perf_config,$

-:170: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#170: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:2620:
+        [DRM_I915_GEM_EXECBUFFER_EXT_PERF] = parse_perf_config,$

total: 1 errors, 1 warnings, 1 checks, 416 lines checked
ccd1cbb63c8e drm/i915/perf: allow holding preemption on filtered ctx
-:140: WARNING:BRACES: braces {} are not necessary for single statement blocks
#140: FILE: drivers/gpu/drm/i915/i915_perf.c:2727:
+	if (IS_HASWELL(dev_priv) && specific_ctx && !props->hold_preemption) {
 		privileged_op = false;
+	}

total: 0 errors, 1 warnings, 0 checks, 165 lines checked
ce2af5b1a9ca drm/i915: add support for perf configuration queries

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx