Skip to contentSkip to conversation

add -Werror-implicit-function-declaration to CFLAGS

PATCH In review Adrian Bunk 8 participants 20 messages Nov 7, 2005 – Nov 16, 2005

Subsystems (6)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 add -Werror-implicit-function-declaration to CFLAGS

Currently, using an undeclared function gives a compile warning, but it can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

This patch also removes some unneeded spaces between two tabs in the following line.

---

This patch was already sent on:
- 30 Aug 2005
- 30 Jul 2005

Makefile renamed from Makefile.old +2 -1
@@ -351,7 +351,8 @@
 CPPFLAGS        := -D__KERNEL__ $(LINUXINCLUDE)
 
 CFLAGS 		:= -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \
-	  	   -fno-strict-aliasing -fno-common \
+		   -Werror-implicit-function-declaration \
+		   -fno-strict-aliasing -fno-common \
 		   -ffreestanding
 AFLAGS		:= -D__ASSEMBLY__
 
#1 lore raw
Quoted 5 lines · Adrian Bunk

Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

Quoted 7 lines

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

#2 lore raw
Quoted 9 lines · Andrew Morton
Quoted 8 lines · Adrian Bunk

Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

Quoted 7 lines
Quoted 4 lines

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Can you send me the list of compile errors so that I can work on fixing them?

TIA
Adrian

Signature · 4 lines
       "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
#3 lore raw
Quoted 16 lines · Adrian Bunk
Quoted 14 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk

Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

Quoted 6 lines
Quoted 2 lines

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Can you send me the list of compile errors so that I can work on fixing them?

No handily, sorry. Missing virt_to_bus() is the typical problem.

The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.

#4 lore raw
Quoted 20 lines · On Thu, 10 Nov 2005 18:24:43 -0800 Andrew Morton wrote:
Quoted 19 lines · Adrian Bunk
Quoted 16 lines · Andrew Morton
Quoted 5 lines · Adrian Bunk

Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

Quoted 7 lines

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Quoted 8 lines
Quoted 3 lines

Can you send me the list of compile errors so that I can work on fixing them?

No handily, sorry. Missing virt_to_bus() is the typical problem.

The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.

Ack that. Those are the ones I use.

---
~Randy

#5 lore raw
In reply to Andrew Morton
Quoted 17 lines · Andrew Morton
Quoted 16 lines · Adrian Bunk
Quoted 14 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk

Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.

With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.

There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.

Quoted 9 lines
Quoted 6 lines
Quoted 2 lines

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Can you send me the list of compile errors so that I can work on fixing them?

No handily, sorry. Missing virt_to_bus() is the typical problem.

But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.

If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned long virt_to_bus_not_defined_use_pci_map(volatile void *addr); #define virt_to_bus virt_to_bus_not_defined_use_pci_map
extern unsigned long bus_to_virt_not_defined_use_pci_map(volatile void *addr); #define bus_to_virt bus_to_virt_not_defined_use_pci_map

Would a patch to mark the ISA legacy functions as __deprecated be OK?

This might give some motivation for people to convert drivers and would avoid new code like the recently introduced kexec to use this obsolete API.

Quoted 2 lines

The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.

Thanks, I've tried it.

Other problems I found until I gave up on compiling:
- a problem in sk98lin indirectly corrected by my SkPciWriteCfgDWord() patch
- drivers/net/wireless/tiacx/: missing #include <linux/vmalloc.h>'s (see my patch) - this seems to be a real bug

cu
Adrian

Signature · 4 lines
       "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
#6 lore raw
Quoted 12 lines · Adrian Bunk

...
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.

If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned long virt_to_bus_not_defined_use_pci_map(volatile void *addr); #define virt_to_bus virt_to_bus_not_defined_use_pci_map
extern unsigned long bus_to_virt_not_defined_use_pci_map(volatile void *addr); #define bus_to_virt bus_to_virt_not_defined_use_pci_map

Quoted 2 lines

Would a patch to mark the ISA legacy functions as __deprecated be OK?
...

Sorry, this were two separate thoughts:

Would a patch to mark both virt_to_bus/bus_to_virt and the ISA legacy functions (that cause similar problems) as __deprecated be OK?

cu
Adrian

Signature · 4 lines
       "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
#7 lore raw
Quoted 13 lines · Adrian Bunk
Quoted 12 lines · Adrian Bunk

...
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.

If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned long virt_to_bus_not_defined_use_pci_map(volatile void *addr); #define virt_to_bus virt_to_bus_not_defined_use_pci_map
extern unsigned long bus_to_virt_not_defined_use_pci_map(volatile void *addr); #define bus_to_virt bus_to_virt_not_defined_use_pci_map

Quoted 7 lines
Quoted 2 lines

Would a patch to mark the ISA legacy functions as __deprecated be OK?
...

Sorry, this were two separate thoughts:

Would a patch to mark both virt_to_bus/bus_to_virt and the ISA legacy functions (that cause similar problems) as __deprecated be OK?

Rethinking about this:
The ISA legacy functions were declared obsolete five years ago. Since there are not many drivers using them, we could simply mark the drivers still using them as BROKEN.

cu
Adrian

Signature · 4 lines
       "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
#8 lore raw

RFC PATCH 1/1 remove ISA legacy functions

This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.

The 7 drivers that were still using this obsolete API are now marked as BROKEN.

---

 Documentation/DocBook/deviceiobook.tmpl |   19 -----
 arch/i386/Kconfig                       |    1 
 drivers/net/Kconfig                     |    6 -
 drivers/net/arcnet/Kconfig              |    4 -
 drivers/scsi/Kconfig                    |    2 
 include/asm-alpha/io.h                  |   83 ------------------------
 include/asm-arm/arch-aaec2000/io.h      |    1 
 include/asm-arm/arch-clps711x/io.h      |    1 
 include/asm-arm/arch-ebsa285/io.h       |    8 --
 include/asm-arm/arch-integrator/io.h    |    1 
 include/asm-arm/arch-iop3xx/io.h        |    1 
 include/asm-arm/arch-l7200/io.h         |    1 
 include/asm-arm/arch-lh7a40x/io.h       |    1 
 include/asm-arm/arch-omap/io.h          |    1 
 include/asm-arm/arch-pxa/io.h           |    1 
 include/asm-arm/arch-realview/io.h      |    1 
 include/asm-arm/arch-sa1100/io.h        |    1 
 include/asm-arm/arch-versatile/io.h     |    1 
 include/asm-arm/io.h                    |   36 ----------
 include/asm-i386/io.h                   |   11 ---
 include/asm-m68k/io.h                   |   21 ++----
 include/asm-mips/io.h                   |   23 ------
 include/asm-parisc/io.h                 |   18 -----
 include/asm-sh/io.h                     |   13 ---
 include/asm-x86_64/io.h                 |   11 ---
 25 files changed, 15 insertions(+), 252 deletions(-)
Documentation/DocBook/deviceiobook.tmpl renamed from Documentation/DocBook/deviceiobook.tmpl.old +0 -19
@@ -270,25 +270,6 @@
       </para>
     </sect1>
 
-    <sect1>
-      <title>ISA legacy functions</title>
-      <para>
-	On older kernels (2.2 and earlier) the ISA bus could be read or
-	written with these functions and without ioremap being used. This is
-	no longer true in Linux 2.4. A set of equivalent functions exist for
-	easy legacy driver porting. The functions available are prefixed
-	with 'isa_' and are <function>isa_readb</function>,
-	<function>isa_writeb</function>, <function>isa_readw</function>, 
-	<function>isa_writew</function>, <function>isa_readl</function>,
-	<function>isa_writel</function>, <function>isa_memcpy_fromio</function>
-	and <function>isa_memcpy_toio</function>
-      </para>
-      <para>
-	These functions should not be used in new drivers, and will
-	eventually be going away.
-      </para>
-    </sect1>
-
   </chapter>
 
   <chapter>
include/asm-alpha/io.h renamed from include/asm-alpha/io.h.old +0 -83

95 lines not on this page · raw message

include/asm-arm/io.h renamed from include/asm-arm/io.h.old +0 -36
@@ -215,42 +215,6 @@
 #endif	/* __mem_pci */
 
 /*
- * If this architecture has ISA IO, then define the isa_read/isa_write
- * macros.
- */
-#ifdef __mem_isa
-
-#define isa_readb(addr)			__raw_readb(__mem_isa(addr))
-#define isa_readw(addr)			__raw_readw(__mem_isa(addr))
-#define isa_readl(addr)			__raw_readl(__mem_isa(addr))
-#define isa_writeb(val,addr)		__raw_writeb(val,__mem_isa(addr))
-#define isa_writew(val,addr)		__raw_writew(val,__mem_isa(addr))
-#define isa_writel(val,addr)		__raw_writel(val,__mem_isa(addr))
-#define isa_memset_io(a,b,c)		_memset_io(__mem_isa(a),(b),(c))
-#define isa_memcpy_fromio(a,b,c)	_memcpy_fromio((a),__mem_isa(b),(c))
-#define isa_memcpy_toio(a,b,c)		_memcpy_toio(__mem_isa((a)),(b),(c))
-
-#define isa_eth_io_copy_and_sum(a,b,c,d) \
-				eth_copy_and_sum((a),__mem_isa(b),(c),(d))
-
-#else	/* __mem_isa */
-
-#define isa_readb(addr)			(__readwrite_bug("isa_readb"),0)
-#define isa_readw(addr)			(__readwrite_bug("isa_readw"),0)
-#define isa_readl(addr)			(__readwrite_bug("isa_readl"),0)
-#define isa_writeb(val,addr)		__readwrite_bug("isa_writeb")
-#define isa_writew(val,addr)		__readwrite_bug("isa_writew")
-#define isa_writel(val,addr)		__readwrite_bug("isa_writel")
-#define isa_memset_io(a,b,c)		__readwrite_bug("isa_memset_io")
-#define isa_memcpy_fromio(a,b,c)	__readwrite_bug("isa_memcpy_fromio")
-#define isa_memcpy_toio(a,b,c)		__readwrite_bug("isa_memcpy_toio")
-
-#define isa_eth_io_copy_and_sum(a,b,c,d) \
-				__readwrite_bug("isa_eth_io_copy_and_sum")
-
-#endif	/* __mem_isa */
-
-/*
  * ioremap and friends.
  *
  * ioremap takes a PCI memory address, as specified in
include/asm-i386/io.h renamed from include/asm-i386/io.h.old +0 -11
@@ -214,23 +214,12 @@
  */
 #define __ISA_IO_base ((char __iomem *)(PAGE_OFFSET))
 
-#define isa_readb(a) readb(__ISA_IO_base + (a))
-#define isa_readw(a) readw(__ISA_IO_base + (a))
-#define isa_readl(a) readl(__ISA_IO_base + (a))
-#define isa_writeb(b,a) writeb(b,__ISA_IO_base + (a))
-#define isa_writew(w,a) writew(w,__ISA_IO_base + (a))
-#define isa_writel(l,a) writel(l,__ISA_IO_base + (a))
-#define isa_memset_io(a,b,c)		memset_io(__ISA_IO_base + (a),(b),(c))
-#define isa_memcpy_fromio(a,b,c)	memcpy_fromio((a),__ISA_IO_base + (b),(c))
-#define isa_memcpy_toio(a,b,c)		memcpy_toio(__ISA_IO_base + (a),(b),(c))
-
 
 /*
  * Again, i386 does not require mem IO specific function.
  */
 
 #define eth_io_copy_and_sum(a,b,c,d)		eth_copy_and_sum((a),(void __force *)(b),(c),(d))
-#define isa_eth_io_copy_and_sum(a,b,c,d)	eth_copy_and_sum((a),(void __force *)(__ISA_IO_base + (b)),(c),(d))
 
 /**
  *	check_signature		-	find BIOS signatures
include/asm-mips/io.h renamed from include/asm-mips/io.h.old +0 -23
@@ -563,33 +563,10 @@
 extern void pci_iounmap(struct pci_dev *dev, void __iomem *);
 
 /*
- * ISA space is 'always mapped' on currently supported MIPS systems, no need
- * to explicitly ioremap() it. The fact that the ISA IO space is mapped
- * to PAGE_OFFSET is pure coincidence - it does not mean ISA values
- * are physical addresses. The following constant pointer can be
- * used as the IO-area pointer (it can be iounmapped as well, so the
- * analogy with PCI is quite large):
- */
-#define __ISA_IO_base ((char *)(isa_slot_offset))
-
-#define isa_readb(a)		readb(__ISA_IO_base + (a))
-#define isa_readw(a)		readw(__ISA_IO_base + (a))
-#define isa_readl(a)		readl(__ISA_IO_base + (a))
-#define isa_readq(a)		readq(__ISA_IO_base + (a))
-#define isa_writeb(b,a)		writeb(b,__ISA_IO_base + (a))
-#define isa_writew(w,a)		writew(w,__ISA_IO_base + (a))
-#define isa_writel(l,a)		writel(l,__ISA_IO_base + (a))
-#define isa_writeq(q,a)		writeq(q,__ISA_IO_base + (a))
-#define isa_memset_io(a,b,c)	memset_io(__ISA_IO_base + (a),(b),(c))
-#define isa_memcpy_fromio(a,b,c) memcpy_fromio((a),__ISA_IO_base + (b),(c))
-#define isa_memcpy_toio(a,b,c)	memcpy_toio(__ISA_IO_base + (a),(b),(c))
-
-/*
  * We don't have csum_partial_copy_fromio() yet, so we cheat here and
  * just copy it. The net code will then do the checksum later.
  */
 #define eth_io_copy_and_sum(skb,src,len,unused) memcpy_fromio((skb)->data,(src),(len))
-#define isa_eth_io_copy_and_sum(a,b,c,d) eth_copy_and_sum((a),(b),(c),(d))
 
 /*
  *     check_signature         -       find BIOS signatures
include/asm-parisc/io.h renamed from include/asm-parisc/io.h.old +0 -18
@@ -294,22 +294,6 @@
 void memcpy_fromio(void *dst, const volatile void __iomem *src, int count);
 void memcpy_toio(volatile void __iomem *dst, const void *src, int count);
 
-/* Support old drivers which don't ioremap.
- * NB this interface is scheduled to disappear in 2.5
- */
-
-#define __isa_addr(x) (void __iomem *)(F_EXTEND(0xfc000000) | (x))
-#define isa_readb(a) readb(__isa_addr(a))
-#define isa_readw(a) readw(__isa_addr(a))
-#define isa_readl(a) readl(__isa_addr(a))
-#define isa_writeb(b,a) writeb((b), __isa_addr(a))
-#define isa_writew(b,a) writew((b), __isa_addr(a))
-#define isa_writel(b,a) writel((b), __isa_addr(a))
-#define isa_memset_io(a,b,c) memset_io(__isa_addr(a), (b), (c))
-#define isa_memcpy_fromio(a,b,c) memcpy_fromio((a), __isa_addr(b), (c))
-#define isa_memcpy_toio(a,b,c) memcpy_toio(__isa_addr(a), (b), (c))
-
-
 /*
  * XXX - We don't have csum_partial_copy_fromio() yet, so we cheat here and 
  * just copy it. The net code will then do the checksum later. Presently 
@@ -318,8 +302,6 @@
 
 #define eth_io_copy_and_sum(skb,src,len,unused) \
   memcpy_fromio((skb)->data,(src),(len))
-#define isa_eth_io_copy_and_sum(skb,src,len,unused) \
-  isa_memcpy_fromio((skb)->data,(src),(len))
 
 /* Port-space IO */
 
include/asm-x86_64/io.h renamed from include/asm-x86_64/io.h.old +0 -11
@@ -264,23 +264,12 @@
  */
 #define __ISA_IO_base ((char __iomem *)(PAGE_OFFSET))
 
-#define isa_readb(a) readb(__ISA_IO_base + (a))
-#define isa_readw(a) readw(__ISA_IO_base + (a))
-#define isa_readl(a) readl(__ISA_IO_base + (a))
-#define isa_writeb(b,a) writeb(b,__ISA_IO_base + (a))
-#define isa_writew(w,a) writew(w,__ISA_IO_base + (a))
-#define isa_writel(l,a) writel(l,__ISA_IO_base + (a))
-#define isa_memset_io(a,b,c)		memset_io(__ISA_IO_base + (a),(b),(c))
-#define isa_memcpy_fromio(a,b,c)	memcpy_fromio((a),__ISA_IO_base + (b),(c))
-#define isa_memcpy_toio(a,b,c)		memcpy_toio(__ISA_IO_base + (a),(b),(c))
-
 
 /*
  * Again, x86-64 does not require mem IO specific function.
  */
 
 #define eth_io_copy_and_sum(a,b,c,d)		eth_copy_and_sum((a),(void *)(b),(c),(d))
-#define isa_eth_io_copy_and_sum(a,b,c,d)	eth_copy_and_sum((a),(void *)(__ISA_IO_base + (b)),(c),(d))
 
 /**
  *	check_signature		-	find BIOS signatures
include/asm-m68k/io.h renamed from include/asm-m68k/io.h.old +8 -13
@@ -191,15 +191,6 @@
 #define isa_outb(val,port) out_8(isa_itb(port),(val))
 #define isa_outw(val,port) (ISA_SEX ? out_be16(isa_itw(port),(val)) : out_le16(isa_itw(port),(val)))
 
-#define isa_readb(p)       in_8(isa_mtb((unsigned long)(p)))
-#define isa_readw(p)       \
-	(ISA_SEX ? in_be16(isa_mtw((unsigned long)(p)))	\
-		 : in_le16(isa_mtw((unsigned long)(p))))
-#define isa_writeb(val,p)  out_8(isa_mtb((unsigned long)(p)),(val))
-#define isa_writew(val,p)  \
-	(ISA_SEX ? out_be16(isa_mtw((unsigned long)(p)),(val))	\
-		 : out_le16(isa_mtw((unsigned long)(p)),(val)))
-
 static inline void isa_delay(void)
 {
   switch(ISA_TYPE)
@@ -254,10 +245,14 @@
 #define insw    isa_insw
 #define outsb   isa_outsb
 #define outsw   isa_outsw
-#define readb   isa_readb
-#define readw   isa_readw
-#define writeb  isa_writeb
-#define writew  isa_writew
+#define readb(p) in_8(isa_mtb((unsigned long)(p)))
+#define readw(p)       \
+	(ISA_SEX ? in_be16(isa_mtw((unsigned long)(p)))	\
+		 : in_le16(isa_mtw((unsigned long)(p))))
+#define writeb(val,p)  out_8(isa_mtb((unsigned long)(p)),(val))
+#define writew(val,p)  \
+	(ISA_SEX ? out_be16(isa_mtw((unsigned long)(p)),(val))	\
+		 : out_le16(isa_mtw((unsigned long)(p)),(val)))
 #endif /* CONFIG_ISA */
 
 #if defined(CONFIG_PCI)
include/asm-sh/io.h renamed from include/asm-sh/io.h.old +0 -13
@@ -159,19 +159,6 @@
 	generic_io_base = pbase;
 }
 
-#define isa_readb(a) readb(isa_port2addr(a))
-#define isa_readw(a) readw(isa_port2addr(a))
-#define isa_readl(a) readl(isa_port2addr(a))
-#define isa_writeb(b,a) writeb(b,isa_port2addr(a))
-#define isa_writew(w,a) writew(w,isa_port2addr(a))
-#define isa_writel(l,a) writel(l,isa_port2addr(a))
-#define isa_memset_io(a,b,c) \
-  memset((void *)(isa_port2addr((unsigned long)a)),(b),(c))
-#define isa_memcpy_fromio(a,b,c) \
-  memcpy((a),(void *)(isa_port2addr((unsigned long)(b))),(c))
-#define isa_memcpy_toio(a,b,c) \
-  memcpy((void *)(isa_port2addr((unsigned long)(a))),(b),(c))
-
 /* We really want to try and get these to memcpy etc */
 extern void memcpy_fromio(void *, unsigned long, unsigned long);
 extern void memcpy_toio(unsigned long, const void *, unsigned long);
arch/i386/Kconfig renamed from arch/i386/Kconfig.old +1 -0
@@ -307,6 +307,7 @@
 
 config TOSHIBA
 	tristate "Toshiba Laptop support"
+	depends on BROKEN
 	---help---
 	  This adds a driver to safely access the System Management Mode of
 	  the CPU on Toshiba portables with a genuine Toshiba BIOS. It does
drivers/net/Kconfig renamed from drivers/net/Kconfig.old +3 -3
@@ -726,7 +726,7 @@
 
 config LANCE
 	tristate "AMD LANCE and PCnet (AT1500 and NE2100) support"
-	depends on NET_ETHERNET && ISA && ISA_DMA_API
+	depends on NET_ETHERNET && ISA && ISA_DMA_API && BROKEN
 	help
 	  If you have a network (Ethernet) card of this type, say Y and read
 	  the Ethernet-HOWTO, available from
@@ -935,7 +935,7 @@
 
 config HP100
 	tristate "HP 10/100VG PCLAN (ISA, EISA, PCI) support"
-	depends on NET_ETHERNET && (ISA || EISA || PCI)
+	depends on NET_ETHERNET && (ISA || EISA || PCI) && BROKEN
 	help
 	  If you have a network (Ethernet) card of this type, say Y and read
 	  the Ethernet-HOWTO, available from
@@ -1021,7 +1021,7 @@
 
 config HPLAN_PLUS
 	tristate "HP PCLAN+ (27247B and 27252A) support"
-	depends on NET_ISA
+	depends on NET_ISA && BROKEN
 	select CRC32
 	help
 	  If you have a network (Ethernet) card of this type, say Y and read
drivers/net/arcnet/Kconfig renamed from drivers/net/arcnet/Kconfig.old +2 -2
@@ -80,7 +80,7 @@
 
 config ARCNET_COM90xx
 	tristate "ARCnet COM90xx (normal) chipset driver"
-	depends on ARCNET
+	depends on ARCNET && BROKEN
 	help
 	  This is the chipset driver for the standard COM90xx cards. If you
 	  have always used the old ARCnet driver without knowing what type of
@@ -105,7 +105,7 @@
 
 config ARCNET_RIM_I
 	tristate "ARCnet COM90xx (RIM I) chipset driver"
-	depends on ARCNET
+	depends on ARCNET && BROKEN
 	---help---
 	  This is yet another chipset driver for the COM90xx cards, but this
 	  time only using memory-mapped mode, and no IO ports at all. This
drivers/scsi/Kconfig renamed from drivers/scsi/Kconfig.old +1 -1
@@ -474,7 +474,7 @@
 
 config SCSI_IN2000
 	tristate "Always IN2000 SCSI support"
-	depends on ISA && SCSI
+	depends on ISA && SCSI && BROKEN
 	help
 	  This is support for an ISA bus SCSI host adapter.  You'll find more
 	  information in <file:Documentation/scsi/in2000.txt>. If it doesn't work
include/asm-arm/arch-aaec2000/io.h renamed from include/asm-arm/arch-aaec2000/io.h.old +0 -1
@@ -16,6 +16,5 @@
  */
 #define __io(a)			((void __iomem *)(a))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
include/asm-arm/arch-clps711x/io.h renamed from include/asm-arm/arch-clps711x/io.h.old +0 -1
@@ -26,7 +26,6 @@
 
 #define __io(a)			((void __iomem *)(a))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 /*
  * We don't support ins[lb]/outs[lb].  Make them fault.
include/asm-arm/arch-ebsa285/io.h renamed from include/asm-arm/arch-ebsa285/io.h.old +0 -8
@@ -24,7 +24,6 @@
 #define __io(a)			((void __iomem *)(PCIO_BASE + (a)))
 #if 1
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		((a) + PCIMEM_BASE)
 #else
 
 static inline void __iomem *___mem_pci(void __iomem *p)
@@ -34,14 +33,7 @@
 	return p;
 }
 
-static inline void __iomem *___mem_isa(void __iomem *p)
-{
-	unsigned long a = (unsigned long)p;
-	BUG_ON(a >= 16*1048576);
-	return p + PCIMEM_BASE;
-}
 #define __mem_pci(a)		___mem_pci(a)
-#define __mem_isa(a)		___mem_isa(a)
 #endif
 
 #endif
include/asm-arm/arch-integrator/io.h renamed from include/asm-arm/arch-integrator/io.h.old +0 -1
@@ -32,6 +32,5 @@
 
 #define __io(a)			((void __iomem *)(PCI_IO_VADDR + (a)))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		((a) + PCI_MEMORY_VADDR)
 
 #endif
include/asm-arm/arch-iop3xx/io.h renamed from include/asm-arm/arch-iop3xx/io.h.old +0 -1
@@ -17,6 +17,5 @@
 
 #define __io(p)			((void __iomem *)(p))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
include/asm-arm/arch-l7200/io.h renamed from include/asm-arm/arch-l7200/io.h.old +0 -1
@@ -19,7 +19,6 @@
  */
 #define __io_pci(a)		((void __iomem *)(PCIO_BASE + (a)))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #define __ioaddr(p)             __io_pci(p)
 
include/asm-arm/arch-lh7a40x/io.h renamed from include/asm-arm/arch-lh7a40x/io.h.old +0 -1
@@ -18,6 +18,5 @@
 /* No ISA or PCI bus on this machine. */
 #define __io(a)			((void __iomem *)(a))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif /* __ASM_ARCH_IO_H */
include/asm-arm/arch-omap/io.h renamed from include/asm-arm/arch-omap/io.h.old +0 -1
@@ -44,7 +44,6 @@
  */
 #define __io(a)			((void __iomem *)(PCIO_BASE + (a)))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 /*
  * ----------------------------------------------------------------------------
include/asm-arm/arch-pxa/io.h renamed from include/asm-arm/arch-pxa/io.h.old +0 -1
@@ -16,6 +16,5 @@
  */
 #define __io(a)			((void __iomem *)(a))
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
include/asm-arm/arch-sa1100/io.h renamed from include/asm-arm/arch-sa1100/io.h.old +0 -1
@@ -24,6 +24,5 @@
 }
 #define __io(a)			__io(a)
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
include/asm-arm/arch-versatile/io.h renamed from include/asm-arm/arch-versatile/io.h.old +0 -1
@@ -28,6 +28,5 @@
 }
 #define __io(a)	__io(a)
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
include/asm-arm/arch-realview/io.h renamed from include/asm-arm/arch-realview/io.h.old +0 -1
@@ -29,6 +29,5 @@
 
 #define __io(a)			__io(a)
 #define __mem_pci(a)		(a)
-#define __mem_isa(a)		(a)
 
 #endif
#9 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 7 lines · Adrian Bunk

This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.

The 7 drivers that were still using this obsolete API are now marked as BROKEN.

it's not valid to mark working drivers broken, IMO.

Mark them x86-only, perhaps.

Jeff

#10 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 3 lines · Jeff Garzik

it's not valid to mark working drivers broken, IMO.

Mark them x86-only, perhaps.

hp100 works fine on parisc.

#11 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 8 lines · Matthew Wilcox
Quoted 3 lines · Jeff Garzik

it's not valid to mark working drivers broken, IMO.

Mark them x86-only, perhaps.

hp100 works fine on parisc.

Certainly. The point was, don't mark them broken, limit them to the arches where they work.

Jeff

#12 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 5 lines · Jeff Garzik
Quoted 4 lines · Matthew Wilcox
Quoted 2 lines · Jeff Garzik

it's not valid to mark working drivers broken, IMO.

Quoted 7 lines
Quoted 4 lines
Quoted 1 line

Mark them x86-only, perhaps.

hp100 works fine on parisc.

Certainly. The point was, don't mark them broken, limit them to the arches where they work.

I think they work fine everywhere. Adrian wants to remove the API they use.

I think this is a bad idea. The drivers should be converted.

#13 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 4 lines · Matthew Wilcox

I think they work fine everywhere. Adrian wants to remove the API they use.

I think this is a bad idea. The drivers should be converted.

They are - I'll send patches later today...

#14 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

Quoted 7 lines · Al Viro
Quoted 4 lines · Matthew Wilcox

I think they work fine everywhere. Adrian wants to remove the API they use.

I think this is a bad idea. The drivers should be converted.

They are - I'll send patches later today...

NB: never say these words on a Friday night or you'll get a visit from Murphy.

Apologies for delay, patches sent.

#15 lore raw

Re: [RFC: 2.6 patch] remove ISA legacy functions

In reply to Adrian Bunk
Quoted 5 lines · Adrian Bunk

This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.

The 7 drivers that were still using this obsolete API are now marked as BROKEN.

NACK. Al has patches pending to convert them to proper APIs, please wait until after that patchseries has landed.

#16 lore raw

PATCH 1/1 mark virt_to_bus/bus_to_virt as __deprecated on i386

In reply to Adrian Bunk

virt_to_bus/bus_to_virt are long deprecated, mark them as __deprecated on i386.

include/asm-i386/io.h renamed from include/asm-i386/io.h.old +8 -2
@@ -144,8 +144,14 @@
  *
  * Allow them on x86 for legacy drivers, though.
  */
-#define virt_to_bus virt_to_phys
-#define bus_to_virt phys_to_virt
+static inline unsigned long __deprecated virt_to_bus(volatile void * address)
+{
+	return __pa(address);
+}
+static inline void * __deprecated bus_to_virt(unsigned long address)
+{
+	return __va(address);
+}
 
 /*
  * readX/writeX() are used to access memory mapped devices. On some
#17 lore raw
In reply to Adrian Bunk
Quoted 9 lines · Adrian Bunk
Quoted 9 lines
Quoted 8 lines
Quoted 4 lines

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Can you send me the list of compile errors so that I can work on fixing them?

Quoted 6 lines
Quoted 2 lines

No handily, sorry. Missing virt_to_bus() is the typical problem.

But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.

It is a quite inconvenient change if you want to get full coverage with `make allmodconfig'.

Maybe one can do `make -i' and then weed through the noise - I haven't tried.

Quoted 6 lines

If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned long virt_to_bus_not_defined_use_pci_map(volatile void *addr); #define virt_to_bus virt_to_bus_not_defined_use_pci_map
extern unsigned long bus_to_virt_not_defined_use_pci_map(volatile void *addr); #define bus_to_virt bus_to_virt_not_defined_use_pci_map

Maybe. There were some other failures.

#18 lore raw
Quoted 13 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk
Quoted 11 lines
Quoted 8 lines
Quoted 4 lines

Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.

Can you send me the list of compile errors so that I can work on fixing them?

No handily, sorry. Missing virt_to_bus() is the typical problem.

Quoted 6 lines
Quoted 3 lines

But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.

It is a quite inconvenient change if you want to get full coverage with `make allmodconfig'.

It could be a Kconfig item if enabled or not.
Then you could use the new mechanishm in kconfig to disable it for your allmodconfig builds.

cat allmod.config
CONFIG_CC_ERROR_IMPLICIT_FUNCTION_DECLARATION = 0

That should do the trick, but maybe too inconvinient??

Sam

#19 lore raw
In reply to Adrian Bunk
Quoted 5 lines · Adrian Bunk

...
This might give some motivation for people to convert drivers and would avoid new code like the recently introduced kexec to use this obsolete API.
...

/me sits too long in front of the computer

s/kexec//

cu
Adrian

Signature · 4 lines
       "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