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

From: Patchwork <patchwork@emeril.freedesktop.org>
To: "Daniel Vetter" <daniel.vetter@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/ttm: don't set page->mapping
Date: Fri, 20 Nov 2020 10:14:57 -0000	[thread overview]
Message-ID: <160586729782.16285.1961156146337100921@emeril.freedesktop.org> (raw)
In-Reply-To: <20201120095445.1195585-1-daniel.vetter@ffwll.ch>

== Series Details ==

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

== Summary ==

$ dim checkpatch origin/drm-tip
cd6a3ea2fee2 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

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

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

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

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

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

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

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

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

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

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

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

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

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


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