mm-use-the-light-version-__mod_zone_page_state-in-mlocked_vma_newpage-checkpatch...

WARNING: line over 80 characters
#39: FILE: mm/internal.h:207:
+		 * pte lock is held(spinlock), which implies preemption disabled.

WARNING: line over 80 characters
#55: FILE: mm/rmap.c:988:
+		 * pte lock(a spinlock) is held, which implies preemption disabled.

total: 0 errors, 2 warnings, 44 lines checked

./patches/mm-use-the-light-version-__mod_zone_page_state-in-mlocked_vma_newpage.patch has style problems, please review.

If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Jianyu Zhan <nasa4836@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/mm/internal.h b/mm/internal.h
index 32b064fe678d463932abab40368de40636173284..e067984bafa07c4ab34ef32419406c92e95a4902 100644 (file)
--- a/mm/internal.h
+++ b/mm/internal.h
@@ -202,9 +202,9 @@ static inline int mlocked_vma_newpage(struct vm_area_struct *vma,
 
        if (!TestSetPageMlocked(page)) {
                /*
-                * We use the irq-unsafe __mod_zone_page_stat because
-                * this counter is not modified from interrupt context, and the
-                * pte lock is held(spinlock), which implies preemption disabled.
+                * We use the irq-unsafe __mod_zone_page_stat because this
+                * counter is not modified from interrupt context, and the pte
+                * lock is held(spinlock), which implies preemption disabled.
                 */
                __mod_zone_page_state(page_zone(page), NR_MLOCK,
                                    hpage_nr_pages(page));

diff --git a/mm/rmap.c b/mm/rmap.c
index 30f4fa65ec68b63ebdf2fefe2f18466c74830e70..91b4d7869198b20d788e3abfac9864c39fd7aca8 100644 (file)
--- a/mm/rmap.c
+++ b/mm/rmap.c
@@ -985,7 +985,8 @@ void do_page_add_anon_rmap(struct page *page,
                /*
                 * We use the irq-unsafe __{inc|mod}_zone_page_stat because
                 * these counters are not modified from interrupt context, and
-                * pte lock(a spinlock) is held, which implies preemption disabled.
+                * pte lock(a spinlock) is held, which implies preemption
+                * disabled.
                 */
                if (PageTransHuge(page))
                        __inc_zone_page_state(page,