linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: js1304@gmail.com
Cc: kbuild-all@01.org, Andrew Morton <akpm@linux-foundation.org>,
	Hugh Dickins <hughd@google.com>,
	Johannes Berg <johannes@sipsolutions.net>,
	"David S. Miller" <davem@davemloft.net>,
	Sunil Goutham <sgoutham@cavium.com>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	Joonsoo Kim <iamjoonsoo.kim@lge.com>
Subject: Re: [PATCH 2/2] mm: rename _count, field of the struct page, to _refcount
Date: Mon, 28 Mar 2016 14:07:52 +0800	[thread overview]
Message-ID: <201603281456.e4kLKip8%fengguang.wu@intel.com> (raw)
In-Reply-To: <1459144748-13664-2-git-send-email-iamjoonsoo.kim@lge.com>

[-- Attachment #1: Type: text/plain, Size: 5924 bytes --]

Hi Joonsoo,

[auto build test ERROR on net/master]
[also build test ERROR on v4.6-rc1 next-20160327]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/js1304-gmail-com/mm-page_ref-use-page_ref-helper-instead-of-direct-modification-of-_count/20160328-140113
config: i386-tinyconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   In file included from include/linux/mm.h:25:0,
                    from include/linux/suspend.h:8,
                    from arch/x86/kernel/asm-offsets.c:12:
   include/linux/page_ref.h: In function 'page_ref_count':
>> include/linux/page_ref.h:66:26: error: 'struct page' has no member named '_count'
     return atomic_read(&page->_count);
                             ^
   include/linux/page_ref.h: In function 'page_count':
   include/linux/page_ref.h:71:41: error: 'struct page' has no member named '_count'
     return atomic_read(&compound_head(page)->_count);
                                            ^
   include/linux/page_ref.h: In function 'set_page_count':
   include/linux/page_ref.h:76:18: error: 'struct page' has no member named '_count'
     atomic_set(&page->_count, v);
                     ^
   include/linux/page_ref.h: In function 'page_ref_add':
   include/linux/page_ref.h:92:22: error: 'struct page' has no member named '_count'
     atomic_add(nr, &page->_count);
                         ^
   include/linux/page_ref.h: In function 'page_ref_sub':
   include/linux/page_ref.h:99:22: error: 'struct page' has no member named '_count'
     atomic_sub(nr, &page->_count);
                         ^
   include/linux/page_ref.h: In function 'page_ref_inc':
   include/linux/page_ref.h:106:18: error: 'struct page' has no member named '_count'
     atomic_inc(&page->_count);
                     ^
   include/linux/page_ref.h: In function 'page_ref_dec':
   include/linux/page_ref.h:113:18: error: 'struct page' has no member named '_count'
     atomic_dec(&page->_count);
                     ^
   include/linux/page_ref.h: In function 'page_ref_sub_and_test':
   include/linux/page_ref.h:120:41: error: 'struct page' has no member named '_count'
     int ret = atomic_sub_and_test(nr, &page->_count);
                                            ^
   include/linux/page_ref.h: In function 'page_ref_dec_and_test':
   include/linux/page_ref.h:129:37: error: 'struct page' has no member named '_count'
     int ret = atomic_dec_and_test(&page->_count);
                                        ^
   In file included from include/linux/atomic.h:4:0,
                    from include/linux/crypto.h:20,
                    from arch/x86/kernel/asm-offsets.c:8:
   include/linux/page_ref.h: In function 'page_ref_dec_return':
   include/linux/page_ref.h:138:35: error: 'struct page' has no member named '_count'
     int ret = atomic_dec_return(&page->_count);
                                      ^
   arch/x86/include/asm/atomic.h:172:53: note: in definition of macro 'atomic_dec_return'
    #define atomic_dec_return(v)  (atomic_sub_return(1, v))
                                                        ^
   In file included from include/linux/mm.h:25:0,
                    from include/linux/suspend.h:8,
                    from arch/x86/kernel/asm-offsets.c:12:
   include/linux/page_ref.h: In function 'page_ref_add_unless':
   include/linux/page_ref.h:147:35: error: 'struct page' has no member named '_count'
     int ret = atomic_add_unless(&page->_count, nr, u);
                                      ^
   In file included from arch/x86/include/asm/atomic.h:4:0,
                    from include/linux/atomic.h:4,
                    from include/linux/crypto.h:20,
                    from arch/x86/kernel/asm-offsets.c:8:
   include/linux/page_ref.h: In function 'page_ref_freeze':
   include/linux/page_ref.h:156:39: error: 'struct page' has no member named '_count'
     int ret = likely(atomic_cmpxchg(&page->_count, count, 0) == count);
                                          ^
   include/linux/compiler.h:169:40: note: in definition of macro 'likely'
    # define likely(x) __builtin_expect(!!(x), 1)
                                           ^
   In file included from include/linux/mm.h:25:0,
                    from include/linux/suspend.h:8,
                    from arch/x86/kernel/asm-offsets.c:12:
   include/linux/page_ref.h: In function 'page_ref_unfreeze':
   include/linux/page_ref.h:168:18: error: 'struct page' has no member named '_count'
     atomic_set(&page->_count, count);
                     ^
   make[2]: *** [arch/x86/kernel/asm-offsets.s] Error 1
   make[2]: Target '__build' not remade because of errors.
   make[1]: *** [prepare0] Error 2
   make[1]: Target 'prepare' not remade because of errors.
   make: *** [sub-make] Error 2

vim +66 include/linux/page_ref.h

95813b8f Joonsoo Kim 2016-03-17  60  }
95813b8f Joonsoo Kim 2016-03-17  61  
95813b8f Joonsoo Kim 2016-03-17  62  #endif
fe896d18 Joonsoo Kim 2016-03-17  63  
fe896d18 Joonsoo Kim 2016-03-17  64  static inline int page_ref_count(struct page *page)
fe896d18 Joonsoo Kim 2016-03-17  65  {
fe896d18 Joonsoo Kim 2016-03-17 @66  	return atomic_read(&page->_count);
fe896d18 Joonsoo Kim 2016-03-17  67  }
fe896d18 Joonsoo Kim 2016-03-17  68  
fe896d18 Joonsoo Kim 2016-03-17  69  static inline int page_count(struct page *page)

:::::: The code at line 66 was first introduced by commit
:::::: fe896d1878949ea92ba547587bc3075cc688fb8f mm: introduce page reference manipulation functions

:::::: TO: Joonsoo Kim <iamjoonsoo.kim@lge.com>
:::::: CC: Linus Torvalds <torvalds@linux-foundation.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 6258 bytes --]

  parent reply	other threads:[~2016-03-28  6:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-28  5:59 [PATCH 1/2] mm/page_ref: use page_ref helper instead of direct modification of _count js1304
2016-03-28  5:59 ` [PATCH 2/2] mm: rename _count, field of the struct page, to _refcount js1304
2016-03-28  6:05   ` Joonsoo Kim
2016-03-28  6:07   ` kbuild test robot [this message]
2016-03-28  6:19     ` Joonsoo Kim
2016-03-28  6:11   ` kbuild test robot
2016-03-28  6:14   ` kbuild test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201603281456.e4kLKip8%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=hughd@google.com \
    --cc=iamjoonsoo.kim@lge.com \
    --cc=johannes@sipsolutions.net \
    --cc=js1304@gmail.com \
    --cc=kbuild-all@01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=sgoutham@cavium.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox