Messages in this thread | | | Date | Fri, 7 Jul 2006 09:49:49 +0200 | From | Adrian Bunk <> | Subject | Re: [2.6 patch] add -Werror-implicit-function-declaration to CFLAGS |
| |
On Fri, Jul 07, 2006 at 08:42:18AM +0200, Sam Ravnborg wrote: >... > make -C /home/sam/kernel/kbuild.git O=/home/sam/kernel/osparc > Using /home/sam/kernel/kbuild.git as source for kernel > GEN /home/sam/kernel/osparc/Makefile > CHK include/linux/version.h > CHK include/linux/utsrelease.h > CHK include/linux/compile.h > CC arch/sparc/mm/init.o > In file included from include2/asm/tlb.h:22, > from /home/sam/kernel/kbuild.git/arch/sparc/mm/init.c:32: > /home/sam/kernel/kbuild.git/include/asm-generic/tlb.h: In function `tlb_flush_mmu': > /home/sam/kernel/kbuild.git/include/asm-generic/tlb.h:76: error: implicit declaration of function `release_pages' > /home/sam/kernel/kbuild.git/include/asm-generic/tlb.h: In function `tlb_remove_page': > /home/sam/kernel/kbuild.git/include/asm-generic/tlb.h:105: error: implicit declaration of function `page_cache_release' > make[3]: *** [arch/sparc/mm/init.o] Error 1 > make[2]: *** [arch/sparc/mm] Error 2 > make[1]: *** [_all] Error 2 > make: *** [all] Error 2 >...
OK, I tried starting with this one.
The problem is that in the CONFIG_SWAP=n case, linux/swap.h uses these functions.
These implicit declarations are bugs that should be fixed.
I tried adding an #include <linux/pagemap.h> to linux/swap.h, but this broke things faster than I could fix them.
Does anyone know our header mess good enough for being able to help me with this issue?
cu Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |