== 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
