[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/ttm: don't set page->mapping (rev2)

Patchwork patchwork at emeril.freedesktop.org
Wed Nov 25 17:09:24 UTC 2020


== Series Details ==

Series: drm/ttm: don't set page->mapping (rev2)
URL   : https://patchwork.freedesktop.org/series/84098/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
ca1f0b04859a mm: Track mmu notifiers in fs_reclaim_acquire/release
-:12: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 23b68395c7c7 ("mm/mmu_notifiers: add a lockdep map for invalidate_range_start/end")'
#12: 
recursions we do have lockdep annotations since 23b68395c7c7

-:41: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 66204f1d2d1b ("mm/mmu_notifiers: prime lockdep")'
#41: 
With this we can also remove the lockdep priming added in 66204f1d2d1b

-:58: WARNING:BAD_SIGN_OFF: email address 'Thomas Hellström (Intel) <thomas_os at shipmail.org>' might be better as '"Thomas Hellström"(Intel) <thomas_os at shipmail.org>'
#58: 
Cc: Thomas Hellström (Intel) <thomas_os at shipmail.org>

-:138: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#138: FILE: mm/page_alloc.c:4307:
+
+	}

-:154: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: Daniel Vetter <daniel.vetter at ffwll.ch>' != 'Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>'

total: 2 errors, 2 warnings, 1 checks, 74 lines checked
56ce3e4d0329 mm: Extract might_alloc() debug check
-:126: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: Daniel Vetter <daniel.vetter at ffwll.ch>' != 'Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>'

total: 0 errors, 1 warnings, 0 checks, 51 lines checked
2e6343b1c471 locking/selftests: Add testcases for fs_reclaim
-:18: WARNING:BAD_SIGN_OFF: email address 'Thomas Hellström (Intel) <thomas_os at shipmail.org>' might be better as '"Thomas Hellström"(Intel) <thomas_os at shipmail.org>'
#18: 
Cc: Thomas Hellström (Intel) <thomas_os at shipmail.org>

-:75: WARNING:PRINTK_WITHOUT_KERN_LEVEL: printk() should include KERN_<LEVEL> facility level
#75: FILE: lib/locking-selftest.c:2388:
+	printk("  --------------------\n");

-:76: WARNING:PRINTK_WITHOUT_KERN_LEVEL: printk() should include KERN_<LEVEL> facility level
#76: FILE: lib/locking-selftest.c:2389:
+	printk("  | fs_reclaim tests |\n");

-:77: WARNING:PRINTK_WITHOUT_KERN_LEVEL: printk() should include KERN_<LEVEL> facility level
#77: FILE: lib/locking-selftest.c:2390:
+	printk("  --------------------\n");

-:81: WARNING:LOGGING_CONTINUATION: Avoid logging continuation uses where feasible
#81: FILE: lib/locking-selftest.c:2394:
+	pr_cont("\n");

-:85: WARNING:LOGGING_CONTINUATION: Avoid logging continuation uses where feasible
#85: FILE: lib/locking-selftest.c:2398:
+	pr_cont("\n");

-:89: WARNING:LOGGING_CONTINUATION: Avoid logging continuation uses where feasible
#89: FILE: lib/locking-selftest.c:2402:
+	pr_cont("\n");

-:103: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: Daniel Vetter <daniel.vetter at ffwll.ch>' != 'Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>'

total: 0 errors, 8 warnings, 0 checks, 65 lines checked




More information about the Intel-gfx mailing list