linux-kernel - Re: [bp:tip-mm 11/13] drivers/video/fbdev/aty/atyfb_base.c:3461:2: error: implicit declaration of function 'ioremap

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 26 Jun 2015 11:29:05 +0200
From:	Borislav Petkov <bp@...e.de>
To:	kbuild test robot <fengguang.wu@...el.com>
Cc:	"Luis R. Rodriguez" <mcgrof@...e.com>, kbuild-all@...org,
	Chris Zankel <chris@...kel.net>,
	Max Filippov <jcmvbkbc@...il.com>,
	Will Deacon <will.deacon@....com>,
	Ingo Molnar <mingo@...nel.org>, Toshi Kani <toshi.kani@...com>,
	linux-xtensa@...ux-xtensa.org, linux-kernel@...r.kernel.org
Subject: Re: [bp:tip-mm 11/13] drivers/video/fbdev/aty/atyfb_base.c:3461:2:
 error: implicit declaration of function 'ioremap_uc'

On Fri, Jun 26, 2015 at 04:28:51PM +0800, kbuild test robot wrote:
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp tip-mm
> head:   7b0982330a1b4d4bdb99bb8382e2363165231a90
> commit: fd50bea9cedd8fdbd091dfccb4b526be0ef9839f [11/13] drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
> config: xtensa-allyesconfig (attached as .config)
> 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 fd50bea9cedd8fdbd091dfccb4b526be0ef9839f
>   # save the attached .config to linux build tree
>   make.cross ARCH=xtensa 

Cool script, btw. I'll use that locally too :-)

> All error/warnings (new ones prefixed by >>):
> 
>    drivers/video/fbdev/aty/atyfb_base.c: In function 'atyfb_setup_generic':
> >> drivers/video/fbdev/aty/atyfb_base.c:3461:2: error: implicit declaration of function 'ioremap_uc' [-Werror=implicit-function-declaration]
>      par->ati_regbase = ioremap_uc(info->fix.mmio_start, 0x1000);
>      ^
> >> drivers/video/fbdev/aty/atyfb_base.c:3461:19: warning: assignment makes pointer from integer without a cast
>      par->ati_regbase = ioremap_uc(info->fix.mmio_start, 0x1000);
>                       ^
>    cc1: some warnings being treated as errors
> 
> vim +/ioremap_uc +3461 drivers/video/fbdev/aty/atyfb_base.c
> 
>   3455	
>   3456		info->fix.mmio_start = raddr;
>   3457		/*
>   3458		 * By using strong UC we force the MTRR to never have an
>   3459		 * effect on the MMIO region on both non-PAT and PAT systems.
>   3460		 */
> > 3461		par->ati_regbase = ioremap_uc(info->fix.mmio_start, 0x1000);
>   3462		if (par->ati_regbase == NULL)
>   3463			return -ENOMEM;
>   3464	

Looks to me, xtensa needs something like that:

---
>From d8ac67e37a1a99e305c5ec57cff084f9f3513e61 Mon Sep 17 00:00:00 2001
From: Borislav Petkov <bp@...e.de>
Date: Fri, 26 Jun 2015 10:54:40 +0200
Subject: [PATCH] xtensa/asm: Add ioremap_uc

... and default to the uncacheable version.

Reported-by: kbuild test robot <fengguang.wu@...el.com>
Signed-off-by: Borislav Petkov <bp@...e.de>
Cc: "Luis R. Rodriguez" <mcgrof@...e.com>
Cc: Chris Zankel <chris@...kel.net>
Cc: Max Filippov <jcmvbkbc@...il.com>
Cc: Will Deacon <will.deacon@....com>
Cc: Ingo Molnar <mingo@...nel.org>
Cc: Toshi Kani <toshi.kani@...com>
Cc: linux-xtensa@...ux-xtensa.org
---
 arch/xtensa/include/asm/io.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/xtensa/include/asm/io.h b/arch/xtensa/include/asm/io.h
index c39bb6e61911..34f707ea619b 100644
--- a/arch/xtensa/include/asm/io.h
+++ b/arch/xtensa/include/asm/io.h
@@ -60,6 +60,7 @@ static inline void __iomem *ioremap_cache(unsigned long offset,
 
 #define ioremap_wc ioremap_nocache
 #define ioremap_wt ioremap_nocache
+#define ioremap_uc ioremap_nocache
 
 static inline void __iomem *ioremap(unsigned long offset, unsigned long size)
 {
-- 
2.3.5

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/