== Series Details == Series: series starting with [1/3] drm/i915: Switch obj->mm.lock lockdep annotations on its head URL : https://patchwork.freedesktop.org/series/68956/ State : warning == Summary == $ dim checkpatch origin/drm-tip c495bd921770 drm/i915: Switch obj->mm.lock lockdep annotations on its head -:345: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 213 lines checked ffcadad25d96 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 <daniel.vetter@xxxxxxxx>' total: 0 errors, 2 warnings, 1 checks, 20 lines checked db49a54b7b5c drm/i915: use might_lock_nested in get_pages annotation -:80: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@xxxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 51 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
