== Series Details == Series: series starting with [01/22] drm/i915/gem: Mark the buffer pool as active for the cmdparser URL : https://patchwork.freedesktop.org/series/77996/ State : warning == Summary == $ dim checkpatch origin/drm-tip 47e5bfbf30e3 drm/i915/gem: Mark the buffer pool as active for the cmdparser 7ec3c01e7a42 drm/i915: Trim set_timer_ms() intervals 00d105eacda7 drm/i915/gt: Set timeslicing priority from queue 7e77d0a206b9 drm/i915/gt: Always check to enable timeslicing if not submitting b26c3cc35bd3 Restore "drm/i915: drop engine_pin/unpin_breadcrumbs_irq" fb7bb70a7283 drm/i915/gt: Couple tasklet scheduling for all CS interrupts cba96e7cdecf drm/i915/gt: Support creation of 'internal' rings ce8cae5ec931 drm/i915/gt: Use client timeline address for seqno writes 710e6262e4f4 drm/i915/gt: Infrastructure for ring scheduling -:79: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #79: new file mode 100644 total: 0 errors, 1 warnings, 0 checks, 842 lines checked f5112dc2ae3b drm/i915/gt: Enable busy-stats for ring-scheduler -:13: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #13: new file mode 100644 -:200: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #200: FILE: drivers/gpu/drm/i915/gt/selftest_engine_pm.c:47: + udelay(100); -:230: CHECK:USLEEP_RANGE: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst #230: FILE: drivers/gpu/drm/i915/gt/selftest_engine_pm.c:77: + udelay(100); total: 0 errors, 1 warnings, 2 checks, 236 lines checked d8fa9292586c drm/i915/gt: Track if an engine requires forcewake w/a 2cdc09a8fc16 drm/i915/gt: Implement ring scheduler for gen6/7 -:68: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #68: FILE: drivers/gpu/drm/i915/gt/intel_ring_scheduler.c:324: + *cs++ = i915_mmio_reg_offset( -:70: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #70: FILE: drivers/gpu/drm/i915/gt/intel_ring_scheduler.c:326: + *cs++ = _MASKED_BIT_ENABLE( -:105: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #105: FILE: drivers/gpu/drm/i915/gt/intel_ring_scheduler.c:361: + *cs++ = _MASKED_BIT_DISABLE( total: 0 errors, 0 warnings, 3 checks, 506 lines checked 2ff1bb716df2 drm/i915/gt: Enable ring scheduling for gen6/7 cfed75109fb2 drm/i915/gem: Async GPU relocations only dd90a9050b61 drm/i915: Add list_for_each_entry_safe_continue_reverse -:20: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pos' - possible side-effects? #20: FILE: drivers/gpu/drm/i915/i915_utils.h:269: +#define list_for_each_entry_safe_continue_reverse(pos, n, head, member) \ + for (pos = list_prev_entry(pos, member), \ + n = list_prev_entry(pos, member); \ + &pos->member != (head); \ + pos = n, n = list_prev_entry(n, member)) -:20: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects? #20: FILE: drivers/gpu/drm/i915/i915_utils.h:269: +#define list_for_each_entry_safe_continue_reverse(pos, n, head, member) \ + for (pos = list_prev_entry(pos, member), \ + n = list_prev_entry(pos, member); \ + &pos->member != (head); \ + pos = n, n = list_prev_entry(n, member)) -:20: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'member' - possible side-effects? #20: FILE: drivers/gpu/drm/i915/i915_utils.h:269: +#define list_for_each_entry_safe_continue_reverse(pos, n, head, member) \ + for (pos = list_prev_entry(pos, member), \ + n = list_prev_entry(pos, member); \ + &pos->member != (head); \ + pos = n, n = list_prev_entry(n, member)) total: 0 errors, 0 warnings, 3 checks, 12 lines checked a5768b96efc5 drm/i915/gem: Separate reloc validation into an earlier step -:101: WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return #101: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1408: + return (int)offset; + } else { total: 0 errors, 1 warnings, 0 checks, 217 lines checked 37ab424926eb drm/i915/gem: Lift GPU relocation allocation 38ad7f8804a1 drm/i915/gem: Build the reloc request first 7cb671e36bbd drm/i915/gem: Add all GPU reloc awaits/signals en masse 1f32dc179ff3 dma-buf: Proxy fence, an unsignaled fence placeholder -:45: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #45: new file mode 100644 -:438: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #438: FILE: drivers/dma-buf/st-dma-fence-proxy.c:20: + spinlock_t lock; total: 0 errors, 1 warnings, 1 checks, 1158 lines checked e665d3e71c75 drm/i915: Unpeel awaits on a proxy fence f155632b03ef drm/i915/gem: Make relocations atomic within execbuf _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
