== Series Details == Series: series starting with [01/17] drm/i915/execlists: Track begin/end of execlists submission sequences URL : https://patchwork.freedesktop.org/series/40927/ State : warning == Summary == $ dim checkpatch origin/drm-tip d71000599823 drm/i915/execlists: Track begin/end of execlists submission sequences 934e303733f6 drm/i915/execlists: Set queue priority from secondary port -:25: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #25: References: f6322eddaff7 ("drm/i915/preemption: Allow preemption between submission ports") -:25: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit f6322eddaff7 ("drm/i915/preemption: Allow preemption between submission ports")' #25: References: f6322eddaff7 ("drm/i915/preemption: Allow preemption between submission ports") total: 1 errors, 1 warnings, 0 checks, 9 lines checked 06aa4edeb023 drm/i915/execlists: Refactor out complete_preempt_context() 7150789d6e56 drm/i915: Move engine reset prepare/finish to backends 25b87d182879 drm/i915: Split execlists/guc reset prepartions a46f451f9827 drm/i915/execlists: Flush pending preemption events during reset -:69: WARNING:LONG_LINE: line over 100 characters #69: FILE: drivers/gpu/drm/i915/intel_lrc.c:907: + (i915->regs + i915_mmio_reg_offset(RING_CONTEXT_STATUS_BUF_LO(engine, 0))); -:87: WARNING:LONG_LINE: line over 100 characters #87: FILE: drivers/gpu/drm/i915/intel_lrc.c:921: + head = readl(i915->regs + i915_mmio_reg_offset(RING_CONTEXT_STATUS_PTR(engine))); -:104: WARNING:LONG_LINE: line over 100 characters #104: FILE: drivers/gpu/drm/i915/intel_lrc.c:935: + head, GEN8_CSB_READ_PTR(readl(i915->regs + i915_mmio_reg_offset(RING_CONTEXT_STATUS_PTR(engine)))), fw ? "" : "?", -:105: WARNING:LONG_LINE: line over 100 characters #105: FILE: drivers/gpu/drm/i915/intel_lrc.c:936: + tail, GEN8_CSB_WRITE_PTR(readl(i915->regs + i915_mmio_reg_offset(RING_CONTEXT_STATUS_PTR(engine)))), fw ? "" : "?"); total: 0 errors, 4 warnings, 0 checks, 192 lines checked 5548848bbbb2 drm/i915/selftests: Add basic sanitychecks for execlists -:43: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #43: new file mode 100644 -:119: CHECK:SPACING: spaces preferred around that '*' (ctx:VxV) #119: FILE: drivers/gpu/drm/i915/selftests/intel_lrc.c:72: + return hws->node.start + offset_in_page(sizeof(u32)*rq->fence.context); ^ -:247: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'W' - possible side-effects? #247: FILE: drivers/gpu/drm/i915/selftests/intel_lrc.c:200: +#define wedge_on_timeout(W, DEV, TIMEOUT) \ + for (__init_wedge((W), (DEV), (TIMEOUT), __builtin_return_address(0)); \ + (W)->i915; \ + __fini_wedge((W))) total: 0 errors, 1 warnings, 2 checks, 511 lines checked 165feb53f995 drm/i915/breadcrumbs: Keep the fake irq armed across reset 13708e4e6f94 drm/i915: Combine tasklet_kill and tasklet_disable -:39: WARNING:MEMORY_BARRIER: memory barrier without comment #39: FILE: drivers/gpu/drm/i915/intel_lrc.c:1755: + smp_mb(); total: 0 errors, 1 warnings, 0 checks, 26 lines checked 435c7f3e4181 drm/i915: Stop parking the signaler around reset -:13: WARNING:TYPO_SPELLING: 'choosen' may be misspelled - perhaps 'chosen'? #13: advancing so that the GPU doesn't emit the breadcrumb for our choosen total: 0 errors, 1 warnings, 0 checks, 85 lines checked b7b11d47f57b drm/i915: Be irqsafe inside reset 28278ff2c649 drm/i915: Allow init_breadcrumbs to be used from irq context b9c1f591cc6b drm/i915/execlists: Force preemption via reset on timeout -:228: ERROR:SPACING: spaces required around that '=' (ctx:VxW) #228: FILE: drivers/gpu/drm/i915/selftests/intel_lrc.c:508: + ctx= kernel_context(i915); ^ total: 1 errors, 0 warnings, 0 checks, 241 lines checked 1df46bdfe4a3 drm/i915/execlists: Try preempt-reset from softirq context -:115: ERROR:SPACING: spaces required around that '=' (ctx:VxW) #115: FILE: drivers/gpu/drm/i915/selftests/intel_lrc.c:593: + ctx= kernel_context(i915); ^ total: 1 errors, 0 warnings, 0 checks, 176 lines checked 57ea892580e5 drm/i915/preemption: Select timeout when scheduling c9542a8112fd drm/i915: Use a preemption timeout to enforce interactivity b742993888d6 drm/i915: Allow user control over preempt timeout on their important context _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
