From 894878f70476deaca209750445d5c6d370180a43 Mon Sep 17 00:00:00 2001 From: Andrew Morton Date: Wed, 14 May 2014 10:02:12 +1000 Subject: [PATCH] mm-use-the-light-version-__mod_zone_page_state-in-mlocked_vma_newpage-checkpatch-fixes 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 Signed-off-by: Andrew Morton --- mm/internal.h | 6 +++--- mm/rmap.c | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/mm/internal.h b/mm/internal.h index 32b064fe678d46..e067984bafa07c 100644 --- 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 30f4fa65ec68b6..91b4d7869198b2 100644 --- 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,