== Series Details == Series: series starting with [CI,1/9] drm/i915/gt: Mark up racy read of intel_ring.head URL : https://patchwork.freedesktop.org/series/75667/ State : warning == Summary == $ dim checkpatch origin/drm-tip 78a5ccdb8c47 drm/i915/gem: Apply more mb() around clflush relocation paths -:24: WARNING:MEMORY_BARRIER: memory barrier without comment #24: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1041: + mb(); -:41: WARNING:MEMORY_BARRIER: memory barrier without comment #41: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1063: + mb(); -:52: WARNING:MEMORY_BARRIER: memory barrier without comment #52: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1170: + mb(); total: 0 errors, 3 warnings, 0 checks, 36 lines checked 5383336232fb drm/i915/gem: Include PIN_GLOBAL prior to using I915_DISPATCH_SECURE 390e4f1dba9b drm/i915/gem: Assign context id for async work 50f81cbd53ef drm/i915: Export a preallocate variant of i915_active_acquire() 2538bbe3b0dd drm/i915/gem: Separate the ww_mutex walker into its own list -:92: WARNING:LONG_LINE: line over 100 characters #92: FILE: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c:1642: + list_for_each_entry_safe_continue_reverse(unlock, en, &eb->lock, lock_link) { -:140: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pos' - possible side-effects? #140: 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)) -:140: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'n' - possible side-effects? #140: 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)) -:140: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'member' - possible side-effects? #140: 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, 1 warnings, 3 checks, 120 lines checked 27d347f131a9 drm/i915/gem: Asynchronous GTT unbinding c498c6b039e8 drm/i915/gem: Bind the fence async for execbuf _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx