2016-03-31 01:29:53

by Joonsoo Kim

[permalink] [raw]
Subject: Re: + mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix.patch added to -mm tree

On Wed, Mar 30, 2016 at 02:50:29PM -0700, [email protected] wrote:
>
> The patch titled
> Subject: mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix
> has been added to the -mm tree. Its filename is
> mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix.patch
>
> This patch should soon appear at
> http://ozlabs.org/~akpm/mmots/broken-out/mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix.patch
> and later at
> http://ozlabs.org/~akpm/mmotm/broken-out/mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix.patch
>
> Before you just go and hit "reply", please:
> a) Consider who else should be cc'ed
> b) Prefer to cc a suitable mailing list as well
> c) Ideally: find the original patch on the mailing list and do a
> reply-to-all to that, adding suitable additional cc's
>
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
>
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
>
> ------------------------------------------------------
> From: Andrew Morton <[email protected]>
> Subject: mm-rename-_count-field-of-the-struct-page-to-_refcount-fix-fix
>
> Documentation/vm/transhuge.txt too

Hello, Andrew.

There is one more site to fix.

Thanks.

--------->8---------------
>From 046a1e1934e1fa490cc4e36bc8d556b28a8707ea Mon Sep 17 00:00:00 2001
From: Joonsoo Kim <[email protected]>
Date: Thu, 31 Mar 2016 09:42:54 +0900
Subject: [PATCH] mm: rename _count filed of the struct page to _refcount fix
fix fix

String in mm/page_alloc.c too

Signed-off-by: Joonsoo Kim <[email protected]>
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index b563403..8120ce3 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -794,7 +794,7 @@ static inline int free_pages_check(struct page *page)
if (unlikely(page->mapping != NULL))
bad_reason = "non-NULL mapping";
if (unlikely(page_ref_count(page) != 0))
- bad_reason = "nonzero _count";
+ bad_reason = "nonzero _refcount";
if (unlikely(page->flags & PAGE_FLAGS_CHECK_AT_FREE)) {
bad_reason = "PAGE_FLAGS_CHECK_AT_FREE flag(s) set";
bad_flags = PAGE_FLAGS_CHECK_AT_FREE;
--
1.9.1