== Series Details == Series: series starting with [01/23] drm/i915: Cache the error string URL : https://patchwork.freedesktop.org/series/48362/ State : warning == Summary == $ dim checkpatch origin/drm-tip 31a8de99a658 drm/i915: Cache the error string 07069ce0121b drm/i915/execlists: Avoid kicking priority on the current context -:49: ERROR:SPACING: space prohibited after that open parenthesis '(' #49: FILE: drivers/gpu/drm/i915/intel_guc_submission.c:767: + if ( __guc_dequeue(engine)) total: 1 errors, 0 warnings, 0 checks, 103 lines checked 59a7f61abead drm/i915: Missed interrupt simulation is no more, tell the world aa77959dacd7 drm/i915/selftests: Basic stress test for rapid context switching -:120: WARNING:ALLOC_ARRAY_ARGS: kcalloc uses number as first arg, sizeof is generally wrong #120: FILE: drivers/gpu/drm/i915/selftests/i915_gem_context.c:127: + ctx = kcalloc(sizeof(*ctx), nctx, GFP_KERNEL); total: 0 errors, 1 warnings, 0 checks, 203 lines checked 1994f89c4bee drm/i915/execlists: Delay updating ring register state after resume -:68: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV) #68: FILE: drivers/gpu/drm/i915/intel_lrc.c:2827: + ce->lrc_reg_state[CTX_RING_HEAD+1] = 0; ^ -:69: CHECK:SPACING: spaces preferred around that '+' (ctx:VxV) #69: FILE: drivers/gpu/drm/i915/intel_lrc.c:2828: + ce->lrc_reg_state[CTX_RING_TAIL+1] = 0; ^ total: 0 errors, 0 warnings, 2 checks, 52 lines checked e3c5fe224683 drm/i915/execlists: Use coherent writes into the context image -:56: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #56: FILE: drivers/gpu/drm/i915/i915_perf.c:1853: + regs = i915_gem_object_pin_map(ce->state->obj, + i915_coherent_map_type(dev_priv)); total: 0 errors, 0 warnings, 1 checks, 71 lines checked 961fcf08085e drm/i915/execlists: Onion unwind for logical_ring_init() failure b80a0049bd90 drm/i915: Report the number of closed vma held by each context in debugfs -:43: WARNING:LONG_LINE: line over 100 characters #43: FILE: drivers/gpu/drm/i915/i915_debugfs.c:350: + seq_printf(m, "%s: %lu objects, %llu bytes (%llu active, %llu inactive, %llu global, %llu shared, %llu unbound, %llu closed)\n", \ total: 0 errors, 1 warnings, 0 checks, 169 lines checked 9a5b5d34b0e7 drm/i915: Remove debugfs/i915_ppgtt_info 0b779c883955 drm/i915/execlists: Assert the queue is non-empty on unsubmitting -:9: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #9: <0>[ 531.960431] drv_self-4806 7.... 527402570us : intel_gpu_reset: engine_mask=1, ret=0, retry=0 total: 0 errors, 1 warnings, 0 checks, 7 lines checked 0471fde14d4f drm/i915/execlists: Reset queue_priority on cancellation 3e9a52f60a87 drm/i915/execlists: Include reset depth in traces 41987cac178b drm/i915/execlists: Use direct submission to restart HW after reset f701f7a446fc drm/i915: Attach the pci match data to the device upon creation -:83: WARNING:AVOID_BUG: Avoid crashing the kernel - try using WARN_ON & recovery code rather than BUG() or BUG_ON() #83: FILE: drivers/gpu/drm/i915/i915_drv.c:1351: + BUG_ON(device_info->gen > sizeof(device_info->gen_mask) * BITS_PER_BYTE); total: 0 errors, 1 warnings, 0 checks, 107 lines checked 06c35ac73fff drm/i915: Track all held rpm wakerefs -:121: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #121: FILE: drivers/gpu/drm/i915/i915_drv.h:1293: + spinlock_t debug_lock; total: 0 errors, 0 warnings, 1 checks, 586 lines checked 53d707353038 drm/i915: Markup paired operations on wakerefs -:692: WARNING:NEW_TYPEDEFS: do not add new typedefs #692: FILE: drivers/gpu/drm/i915/i915_drv.h:130: +typedef depot_stack_handle_t intel_wakeref_t; total: 0 errors, 1 warnings, 0 checks, 1868 lines checked cc5070256ace drm/i915: Syntatic sugar for using intel_runtime_pm -:491: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #491: FILE: drivers/gpu/drm/i915/intel_drv.h:2055: +#define with_intel_runtime_pm(i915, wf) \ + for (wf = intel_runtime_pm_get(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:491: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #491: FILE: drivers/gpu/drm/i915/intel_drv.h:2055: +#define with_intel_runtime_pm(i915, wf) \ + for (wf = intel_runtime_pm_get(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:495: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #495: FILE: drivers/gpu/drm/i915/intel_drv.h:2059: +#define with_intel_runtime_pm_if_in_use(i915, wf) \ + for (wf = intel_runtime_pm_get_if_in_use(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:495: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #495: FILE: drivers/gpu/drm/i915/intel_drv.h:2059: +#define with_intel_runtime_pm_if_in_use(i915, wf) \ + for (wf = intel_runtime_pm_get_if_in_use(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) total: 0 errors, 0 warnings, 4 checks, 569 lines checked 09683b7d42ed drm/i915: Markup paired operations on display power domains -:31: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #31: FILE: drivers/gpu/drm/i915/i915_debugfs.c:634: + wakeref = intel_display_power_get_if_enabled(dev_priv, + power_domain); total: 0 errors, 0 warnings, 1 checks, 1403 lines checked 28d161441228 drm/i915: Track the wakeref used to initialise display power domains -:198: WARNING:LINE_SPACING: Missing a blank line after declarations #198: FILE: drivers/gpu/drm/i915/intel_runtime_pm.c:4104: + struct i915_power_domains *power_domains = &i915->power_domains; + intel_wakeref_t wakeref __maybe_unused = -:213: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "i915->csr.dmc_payload" #213: FILE: drivers/gpu/drm/i915/intel_runtime_pm.c:4117: + i915->csr.dmc_payload != NULL) total: 0 errors, 1 warnings, 1 checks, 294 lines checked 7628024c7252 drm/i915/dp: Markup pps lock power well -:50: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dp' - possible side-effects? #50: FILE: drivers/gpu/drm/i915/intel_dp.c:688: +#define with_pps_lock(dp, wf) \ + for (wf = pps_lock(dp); wf; wf = pps_unlock(dp, wf)) -:50: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #50: FILE: drivers/gpu/drm/i915/intel_dp.c:688: +#define with_pps_lock(dp, wf) \ + for (wf = pps_lock(dp); wf; wf = pps_unlock(dp, wf)) total: 0 errors, 0 warnings, 2 checks, 428 lines checked 5803b4691f5e drm/i915: Complain if hsw_get_pipe_config acquires the same power well twice 39971cb567f5 drm/i915: Mark up Ironlake ips with rpm wakerefs -:210: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #210: FILE: drivers/gpu/drm/i915/intel_pm.c:7970: + chipset_val = graphics_val = 0; total: 0 errors, 0 warnings, 1 checks, 318 lines checked 147bda82fa16 drm/i915: Enable runtime-pm debugging by default for CI _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
