[mmotm:master 152/178] include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt'

[mmotm:master 152/178] include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt'

[kbuild test robot]

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

tree:   git://git.cmpxchg.org/linux-mmotm.git master
head:   2e0066ec9585a5074c8040d639c3c669eb4e905f
commit: 60c8a7d9e20b888121b304895074928bf9b69029 [152/178] kexec: allow architectures to override boot mapping
config: sh-sh7785lcr_32bit_defconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 60c8a7d9e20b888121b304895074928bf9b69029
        # save the attached .config to linux build tree
        make.cross ARCH=sh 

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

   In file included from arch/sh/kernel/reboot.c:2:0:
   include/linux/kexec.h: In function 'boot_phys_to_virt':
>> include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt' [-Werror=implicit-function-declaration]
     return phys_to_virt(boot_phys_to_phys(entry));
            ^
>> include/linux/kexec.h:356:9: error: return makes pointer from integer without a cast [-Werror=int-conversion]
   cc1: all warnings being treated as errors
--
   In file included from kernel/ksysfs.c:16:0:
   include/linux/kexec.h: In function 'boot_phys_to_virt':
>> include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt' [-Werror=implicit-function-declaration]
     return phys_to_virt(boot_phys_to_phys(entry));
            ^
>> include/linux/kexec.h:356:9: warning: return makes pointer from integer without a cast [-Wint-conversion]
   cc1: some warnings being treated as errors

vim +/phys_to_virt +356 include/linux/kexec.h

   350	{
   351		return phys_to_boot_phys(__pa((unsigned long)addr));
   352	}
   353	
   354	static inline void *boot_phys_to_virt(unsigned long entry)
   355	{
 > 356		return phys_to_virt(boot_phys_to_phys(entry));
   357	}
   358	
   359	#else /* !CONFIG_KEXEC_CORE */

---
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: 16556 bytes --]

Re: [mmotm:master 152/178] include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt'

[Russell King - ARM Linux]

On Sat, Jun 04, 2016 at 10:11:47AM +0800, kbuild test robot wrote:
> tree:   git://git.cmpxchg.org/linux-mmotm.git master
> head:   2e0066ec9585a5074c8040d639c3c669eb4e905f
> commit: 60c8a7d9e20b888121b304895074928bf9b69029 [152/178] kexec: allow architectures to override boot mapping
> config: sh-sh7785lcr_32bit_defconfig (attached as .config)
> compiler: sh4-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
> reproduce:
>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 60c8a7d9e20b888121b304895074928bf9b69029
>         # save the attached .config to linux build tree
>         make.cross ARCH=sh 
> 
> All error/warnings (new ones prefixed by >>):
> 
>    In file included from arch/sh/kernel/reboot.c:2:0:
>    include/linux/kexec.h: In function 'boot_phys_to_virt':
> >> include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt' [-Werror=implicit-function-declaration]
>      return phys_to_virt(boot_phys_to_phys(entry));
>             ^

Is there a reason SH doesn't provide phys_to_virt()?  Isn't that a basic
requirement for every architecture?

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [mmotm:master 152/178] include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt'

[Andrew Morton]

On Sat, 4 Jun 2016 11:42:43 +0100 Russell King - ARM Linux <linux@armlinux.org.uk> wrote:

> On Sat, Jun 04, 2016 at 10:11:47AM +0800, kbuild test robot wrote:
> > tree:   git://git.cmpxchg.org/linux-mmotm.git master
> > head:   2e0066ec9585a5074c8040d639c3c669eb4e905f
> > commit: 60c8a7d9e20b888121b304895074928bf9b69029 [152/178] kexec: allow architectures to override boot mapping
> > config: sh-sh7785lcr_32bit_defconfig (attached as .config)
> > compiler: sh4-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
> > reproduce:
> >         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         git checkout 60c8a7d9e20b888121b304895074928bf9b69029
> >         # save the attached .config to linux build tree
> >         make.cross ARCH=sh 
> > 
> > All error/warnings (new ones prefixed by >>):
> > 
> >    In file included from arch/sh/kernel/reboot.c:2:0:
> >    include/linux/kexec.h: In function 'boot_phys_to_virt':
> > >> include/linux/kexec.h:356:9: error: implicit declaration of function 'phys_to_virt' [-Werror=implicit-function-declaration]
> >      return phys_to_virt(boot_phys_to_phys(entry));
> >             ^
> 
> Is there a reason SH doesn't provide phys_to_virt()?  Isn't that a basic
> requirement for every architecture?

It's there, in arch/sh/include/asm/io.h. 
kexec-allow-architectures-to-override-boot-mapping-fix.patch fixes this
error.

From: Andrew Morton <akpm@linux-foundation.org>
Subject: kexec-allow-architectures-to-override-boot-mapping-fix

kexec.h needs asm/io.h for phys_to_virt()

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 include/linux/kexec.h |    2 ++
 1 file changed, 2 insertions(+)

diff -puN include/linux/kexec.h~kexec-allow-architectures-to-override-boot-mapping-fix include/linux/kexec.h
--- a/include/linux/kexec.h~kexec-allow-architectures-to-override-boot-mapping-fix
+++ a/include/linux/kexec.h
@@ -14,6 +14,8 @@
 
 #if !defined(__ASSEMBLY__)
 
+#include <asm/io.h>
+
 #include <uapi/linux/kexec.h>
 
 #ifdef CONFIG_KEXEC_CORE
_

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>