== Series Details ==
Series: series starting with drm/i915: Switch obj->mm.lock lockdep annotations
on its head (rev2)
URL : https://patchwork.freedesktop.org/series/68956/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
c1f4ce7bcff4 drm/i915: Switch obj->mm.lock lockdep annotations on its head
-:74: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+
chars of sha1> ("<title line>")' - ie: 'commit 232a6ebae419 ("drm/i915:
introduce intel_memory_region")'
#74:
commit 232a6ebae419193f5b8da4fa869ae5089ab105c2
-:347: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch
author 'Daniel Vetter <[email protected]>'
total: 1 errors, 1 warnings, 0 checks, 199 lines checked
71463ddc096d lockdep: add might_lock_nested()
-:24: WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#24: FILE: include/linux/lockdep.h:631:
+# define might_lock_nested(lock, subclass) ^I^I^I^I\$
-:24: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'lock' - possible
side-effects?
#24: FILE: include/linux/lockdep.h:631:
+# define might_lock_nested(lock, subclass) \
+do { \
+ typecheck(struct lockdep_map *, &(lock)->dep_map); \
+ lock_acquire(&(lock)->dep_map, subclass, 0, 1, 1, NULL, \
+ _THIS_IP_); \
+ lock_release(&(lock)->dep_map, 0, _THIS_IP_); \
+} while (0)
-:41: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch
author 'Daniel Vetter <[email protected]>'
total: 0 errors, 2 warnings, 1 checks, 20 lines checked
57feff531c8e drm/i915: use might_lock_nested in get_pages annotation
-:81: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch
author 'Daniel Vetter <[email protected]>'
total: 0 errors, 1 warnings, 0 checks, 51 lines checked
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
