✗ Fi.CI.CHECKPATCH: warning for series starting with [01/17] drm/i915: Apply batch location restrictions before pinning

[Patchwork <patchwork@emeril.freedesktop.org>]

== Series Details ==

Series: series starting with [01/17] drm/i915: Apply batch location restrictions before pinning
URL   : https://patchwork.freedesktop.org/series/44539/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
17cecb85f303 drm/i915: Apply batch location restrictions before pinning
-:30: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#30: FILE: drivers/gpu/drm/i915/i915_gem_execbuffer.c:493:
+	   unsigned int i, unsigned batch_idx,

total: 0 errors, 1 warnings, 0 checks, 86 lines checked
a01d03491a31 drm/i915/ringbuffer: Brute force context restore
47f79a76e877 drm/i915/ringbuffer: Fix context restore upon reset
82fd75bde6dd drm/i915: Wrap around the tail offset before setting ring->tail
eb6f6414086e drm/i915/gtt: Invalidate GGTT caches after writing the gen6 page directories
99a2bbbf20d1 drm/i915/ringbuffer: Serialize load of PD_DIR
483cfe1c9f3a drm/i915/gtt: Subclass gen6_hw_ppgtt
-:290: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'base' - possible side-effects?
#290: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:417:
+#define __to_gen6_ppgtt(base) container_of(base, struct gen6_hw_ppgtt, base)

total: 0 errors, 0 warnings, 1 checks, 294 lines checked
2e7a6a454a92 drm/i915/gtt: Onionify error handling for gen6_ppgtt_create
314ece1f1246 drm/i915/gtt: Make gen6 page directories evictable
ac8215c1f544 drm/i915/gtt: Only keep gen6 page directories pinned while active
651e6ba71483 drm/i915/gtt: Lazily allocate page directories for gen7
5e6d68c4ceb4 drm/i915/gtt: Free unused page tables on unbind the context
e699cee5e36d drm/i915/gtt: Skip initializing PT with scratch if full
-:9: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#9: 
References: 14826673247e ("drm/i915: Only initialize partially filled pagetables")

-:9: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 14826673247e ("drm/i915: Only initialize partially filled pagetables")'
#9: 
References: 14826673247e ("drm/i915: Only initialize partially filled pagetables")

total: 1 errors, 1 warnings, 0 checks, 9 lines checked
13b6bde56af2 drm/i915/gtt: Cache the PTE encoding of the scratch page
1648f5cc38eb drm/i915/gtt: Reduce a pair of runtime asserts
-:39: CHECK:SPACING: spaces preferred around that '|' (ctx:VxV)
#39: FILE: drivers/gpu/drm/i915/i915_gem_gtt.h:468:
+	GEM_BUG_ON(offset_in_page(addr|length));
 	                              ^

total: 0 errors, 0 warnings, 1 checks, 18 lines checked
3f18536c8f2a drm/i915/gtt: Skip clearing the GGTT under gen6+ full-ppgtt
77568d77af36 drm/i915/gtt: Enable full-ppgtt by default everywhere

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx