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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



== Series Details ==

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

== Summary ==

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

total: 0 errors, 0 warnings, 1 checks, 361 lines checked
e2b8f4a825d1 drm/i915/perf: implement active wait for noa configurations
-:39: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#39: FILE: drivers/gpu/drm/i915/gt/intel_gpu_commands.h:237:
+#define MI_MATH(x) MI_INSTR(0x1a, (x)-1)
                                      ^

-:118: CHECK:LINE_SPACING: Please don't use multiple blank lines
#118: FILE: drivers/gpu/drm/i915/i915_debugfs.c:3692:
+
+

-:175: CHECK:LINE_SPACING: Please don't use multiple blank lines
#175: FILE: drivers/gpu/drm/i915/i915_perf.c:448:
 
+

-:200: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u32' over 'uint32_t'
#200: FILE: drivers/gpu/drm/i915/i915_perf.c:1665:
+	uint32_t d;

-:225: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#225: FILE: drivers/gpu/drm/i915/i915_perf.c:1690:
+		DIV64_U64_ROUND_UP(

-:250: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#250: FILE: drivers/gpu/drm/i915/i915_perf.c:1715:
+	batch = cs = i915_gem_object_pin_map(bo, I915_MAP_WB);

-:258: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#258: FILE: drivers/gpu/drm/i915/i915_perf.c:1723:
+		cs = save_restore_register(

-:262: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#262: FILE: drivers/gpu/drm/i915/i915_perf.c:1727:
+	cs = save_restore_register(

-:362: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#362: FILE: drivers/gpu/drm/i915/i915_perf.c:1827:
+		cs = save_restore_register(

-:366: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#366: FILE: drivers/gpu/drm/i915/i915_perf.c:1831:
+	cs = save_restore_register(

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

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

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

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

total: 1 errors, 2 warnings, 0 checks, 530 lines checked
1f832dfa8af0 drm/i915: add a new perf configuration execbuf parameter
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
v7: Hold drm.struct_mutex when serializing the request with OA config (Chris)

-:52: CHECK:LINE_SPACING: Please don't use multiple blank lines
#52: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1218:
 
+

-:76: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#76: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1242:
+		err = i915_perf_get_oa_config(

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

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

-:244: CHECK:LINE_SPACING: Please don't use multiple blank lines
#244: FILE: drivers/gpu/drm/i915/gt/intel_engine_cs.c:852:
 
+

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

total: 0 errors, 1 warnings, 0 checks, 123 lines checked
40c2a9f98613 drm/i915/perf: execute OA configuration from command stream
828b572dc0bf drm/i915: add support for perf configuration queries
-:195: WARNING:BRACES: braces {} are not necessary for single statement blocks
#195: FILE: drivers/gpu/drm/i915/i915_query.c:250:
+		if (__get_user(config_id, &user_query_config_ptr->config)) {
+			return -EFAULT;
+		}

total: 0 errors, 1 warnings, 0 checks, 422 lines checked

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




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux