public inbox for linux-s390@vger.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Xunlei Pang <xlpang@redhat.com>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
	kexec@lists.infradead.org, akpm@linux-foundation.org,
	Eric Biederman <ebiederm@xmission.com>,
	Dave Young <dyoung@redhat.com>, Baoquan He <bhe@redhat.com>,
	Michael Holzheu <holzheu@linux.vnet.ibm.com>,
	linux-s390@vger.kernel.org, Dave Anderson <anderson@redhat.com>,
	Hari Bathini <hbathini@linux.vnet.ibm.com>,
	Gustavo Luiz Duarte <gustavold@linux.vnet.ibm.com>
Subject: Re: [PATCH] s390/crash: Fix KEXEC_NOTE_BYTES definition
Date: Mon, 12 Jun 2017 03:46:54 +0800	[thread overview]
Message-ID: <201706120335.KCTusZph%fengguang.wu@intel.com> (raw)
In-Reply-To: <1496974625-10891-1-git-send-email-xlpang@redhat.com>

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

Hi Xunlei,

[auto build test ERROR on next-20170605]
[also build test ERROR on v4.12-rc4]
[cannot apply to linus/master linux/master s390/features v4.9-rc8 v4.9-rc7 v4.9-rc6]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Xunlei-Pang/s390-crash-Fix-KEXEC_NOTE_BYTES-definition/20170610-173047
config: arm-sa1100 (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All error/warnings (new ones prefixed by >>):

   In file included from arch/arm/include/asm/memory.h:378:0,
                    from arch/arm/include/asm/page.h:165,
                    from arch/arm/include/asm/thread_info.h:17,
                    from include/linux/thread_info.h:37,
                    from include/asm-generic/preempt.h:4,
                    from ./arch/arm/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:80,
                    from include/linux/spinlock.h:50,
                    from include/linux/seqlock.h:35,
                    from include/linux/time.h:5,
                    from include/linux/efi.h:16,
                    from arch/arm/kernel/setup.c:10:
   arch/arm/include/asm/kexec.h: In function 'page_to_boot_pfn':
>> include/asm-generic/memory_model.h:63:14: error: implicit declaration of function 'page_to_section' [-Werror=implicit-function-declaration]
     int __sec = page_to_section(__pg);   \
                 ^
   include/asm-generic/memory_model.h:80:21: note: in expansion of macro '__page_to_pfn'
    #define page_to_pfn __page_to_pfn
                        ^~~~~~~~~~~~~
   arch/arm/include/asm/kexec.h:70:9: note: in expansion of macro 'page_to_pfn'
     return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
            ^~~~~~~~~~~
   In file included from include/linux/pid_namespace.h:6:0,
                    from include/linux/ptrace.h:9,
                    from include/uapi/linux/elfcore.h:7,
                    from include/linux/elfcore.h:9,
                    from include/linux/crash_core.h:5,
                    from include/linux/kexec.h:27,
                    from arch/arm/kernel/setup.c:24:
   include/linux/mm.h: At top level:
>> include/linux/mm.h:979:29: error: conflicting types for 'page_to_section'
    static inline unsigned long page_to_section(const struct page *page)
                                ^~~~~~~~~~~~~~~
   In file included from arch/arm/include/asm/memory.h:378:0,
                    from arch/arm/include/asm/page.h:165,
                    from arch/arm/include/asm/thread_info.h:17,
                    from include/linux/thread_info.h:37,
                    from include/asm-generic/preempt.h:4,
                    from ./arch/arm/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:80,
                    from include/linux/spinlock.h:50,
                    from include/linux/seqlock.h:35,
                    from include/linux/time.h:5,
                    from include/linux/efi.h:16,
                    from arch/arm/kernel/setup.c:10:
   include/asm-generic/memory_model.h:63:14: note: previous implicit declaration of 'page_to_section' was here
     int __sec = page_to_section(__pg);   \
                 ^
   include/asm-generic/memory_model.h:80:21: note: in expansion of macro '__page_to_pfn'
    #define page_to_pfn __page_to_pfn
                        ^~~~~~~~~~~~~
   arch/arm/include/asm/kexec.h:70:9: note: in expansion of macro 'page_to_pfn'
     return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
            ^~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   In file included from arch/arm/include/asm/memory.h:378:0,
                    from arch/arm/include/asm/page.h:165,
                    from arch/arm/include/asm/thread_info.h:17,
                    from include/linux/thread_info.h:37,
                    from include/asm-generic/preempt.h:4,
                    from ./arch/arm/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:80,
                    from include/linux/spinlock.h:50,
                    from arch/arm/kernel/traps.c:18:
   arch/arm/include/asm/kexec.h: In function 'page_to_boot_pfn':
>> include/asm-generic/memory_model.h:63:14: error: implicit declaration of function 'page_to_section' [-Werror=implicit-function-declaration]
     int __sec = page_to_section(__pg);   \
                 ^
   include/asm-generic/memory_model.h:80:21: note: in expansion of macro '__page_to_pfn'
    #define page_to_pfn __page_to_pfn
                        ^~~~~~~~~~~~~
   arch/arm/include/asm/kexec.h:70:9: note: in expansion of macro 'page_to_pfn'
     return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
            ^~~~~~~~~~~
   include/asm-generic/memory_model.h:64:23: error: arithmetic on pointer to an incomplete type
     (unsigned long)(__pg - __section_mem_map_addr(__nr_to_section(__sec))); \
                          ^
   include/asm-generic/memory_model.h:80:21: note: in expansion of macro '__page_to_pfn'
    #define page_to_pfn __page_to_pfn
                        ^~~~~~~~~~~~~
   arch/arm/include/asm/kexec.h:70:9: note: in expansion of macro 'page_to_pfn'
     return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
            ^~~~~~~~~~~
   arch/arm/include/asm/kexec.h: In function 'boot_pfn_to_page':
>> include/asm-generic/memory_model.h:69:9: error: invalid use of undefined type 'struct page'
     struct mem_section *__sec = __pfn_to_section(__pfn); \
            ^
   include/asm-generic/memory_model.h:81:21: note: in expansion of macro '__pfn_to_page'
    #define pfn_to_page __pfn_to_page
                        ^~~~~~~~~~~~~
>> arch/arm/include/asm/kexec.h:76:9: note: in expansion of macro 'pfn_to_page'
     return pfn_to_page(boot_pfn - (arch_phys_to_idmap_offset >> PAGE_SHIFT));
            ^~~~~~~~~~~
   In file included from include/linux/pid_namespace.h:6:0,
                    from include/linux/ptrace.h:9,
                    from include/uapi/linux/elfcore.h:7,
                    from include/linux/elfcore.h:9,
                    from include/linux/crash_core.h:5,
                    from include/linux/kexec.h:27,
                    from arch/arm/kernel/traps.c:23:
   include/linux/mm.h: At top level:
>> include/linux/mm.h:979:29: error: conflicting types for 'page_to_section'
    static inline unsigned long page_to_section(const struct page *page)
                                ^~~~~~~~~~~~~~~
   In file included from arch/arm/include/asm/memory.h:378:0,
                    from arch/arm/include/asm/page.h:165,
                    from arch/arm/include/asm/thread_info.h:17,
                    from include/linux/thread_info.h:37,
                    from include/asm-generic/preempt.h:4,
                    from ./arch/arm/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:80,
                    from include/linux/spinlock.h:50,
                    from arch/arm/kernel/traps.c:18:
   include/asm-generic/memory_model.h:63:14: note: previous implicit declaration of 'page_to_section' was here
     int __sec = page_to_section(__pg);   \
                 ^
   include/asm-generic/memory_model.h:80:21: note: in expansion of macro '__page_to_pfn'
    #define page_to_pfn __page_to_pfn
                        ^~~~~~~~~~~~~
   arch/arm/include/asm/kexec.h:70:9: note: in expansion of macro 'page_to_pfn'
     return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
            ^~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/pfn_to_page +76 arch/arm/include/asm/kexec.h

0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  60  #define phys_to_boot_phys phys_to_boot_phys
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  61  
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  62  static inline phys_addr_t boot_phys_to_phys(unsigned long entry)
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  63  {
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  64  	return idmap_to_phys(entry);
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  65  }
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  66  #define boot_phys_to_phys boot_phys_to_phys
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  67  
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  68  static inline unsigned long page_to_boot_pfn(struct page *page)
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  69  {
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  70  	return page_to_pfn(page) + (arch_phys_to_idmap_offset >> PAGE_SHIFT);
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  71  }
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  72  #define page_to_boot_pfn page_to_boot_pfn
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  73  
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  74  static inline struct page *boot_pfn_to_page(unsigned long boot_pfn)
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  75  {
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02 @76  	return pfn_to_page(boot_pfn - (arch_phys_to_idmap_offset >> PAGE_SHIFT));
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  77  }
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  78  #define boot_pfn_to_page boot_pfn_to_page
0719392a arch/arm/include/asm/kexec.h Russell King   2016-08-02  79  
c587e4a6 include/asm-arm/kexec.h      Richard Purdie 2007-02-06  80  #endif /* __ASSEMBLY__ */
c587e4a6 include/asm-arm/kexec.h      Richard Purdie 2007-02-06  81  
c587e4a6 include/asm-arm/kexec.h      Richard Purdie 2007-02-06  82  #endif /* CONFIG_KEXEC */
c587e4a6 include/asm-arm/kexec.h      Richard Purdie 2007-02-06  83  
c587e4a6 include/asm-arm/kexec.h      Richard Purdie 2007-02-06  84  #endif /* _ARM_KEXEC_H */

:::::: The code at line 76 was first introduced by commit
:::::: 0719392a61a9dbc2c850bc7bd1a17efba953fcf5 ARM: kexec: fix kexec for Keystone 2

:::::: TO: Russell King <rmk+kernel@arm.linux.org.uk>
:::::: 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/gzip, Size: 23734 bytes --]

  parent reply	other threads:[~2017-06-11 19:46 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-09  2:17 [PATCH] s390/crash: Fix KEXEC_NOTE_BYTES definition Xunlei Pang
2017-06-09  2:29 ` Dave Young
2017-06-09  7:45   ` Dave Young
2017-06-11  8:43     ` Xunlei Pang
2017-06-09  4:12 ` Hari Bathini
2017-06-11 19:46 ` kbuild test robot [this message]
2017-06-21  9:00 ` Michael Holzheu
2017-06-21 17:44 ` Michael Holzheu
2017-06-22  9:12   ` Xunlei Pang

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=201706120335.KCTusZph%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=anderson@redhat.com \
    --cc=bhe@redhat.com \
    --cc=dyoung@redhat.com \
    --cc=ebiederm@xmission.com \
    --cc=gustavold@linux.vnet.ibm.com \
    --cc=hbathini@linux.vnet.ibm.com \
    --cc=holzheu@linux.vnet.ibm.com \
    --cc=kbuild-all@01.org \
    --cc=kexec@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=xlpang@redhat.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