[Fwd: [parisc-linux] linux and gcc 3.x question?]

[Fwd: [parisc-linux] linux and gcc 3.x question?]

[Joel Soete]

-------- Original Message --------
Subject: [parisc-linux] linux and gcc 3.x question?
Date: Thu, 3 Jun 2004 17:17:01 +0200
From: Joel Soete <soete.joel@tiscali.be>
To: parisc-linux@parisc-linux.org

Hello all,

I just noticed this harmless detail when I pre-compile some ncr53c8xx.c with gcc-3.3.4 on
my parisc-linux box as follow:
"gcc -Wp,-MD,drivers/scsi/.zalon53c720.o.d -nostdinc -iwithprefix include
-D__KERNEL__ -Iinclude  -Wall -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing
-fno-common -pipe -mno-space-regs -mfast-indirect-calls -mdisable-fpregs
-ffunction-sections -O2
-DCONFIG_NCR53C8XX_PREFETCH -DSCSI_NCR_BIG_ENDIAN -DCONFIG_SCSI_NCR53C8XX_NO_WORD_TRANSFERS
   -DKBUILD_BASENAME=ncr53c8xx -DKBUILD_MODNAME=zalon7xx -E -o drivers/scsi/ncr53c8xx.o
drivers/scsi/ncr53c8xx.c"

I got (in drivers/scsi/ncr53c8xx.o):
[...]
static __inline__ __attribute__((always_inline)) __attribute__((always_inline))
int __attribute__((deprecated)) check_region(unsigned long s, unsigned long
n)
[...]

(2 time __attribute__((always_inline))?)
It comes from:

include/linux/compiler-gcc3.h
[...]

#if __GNUC_MINOR__ >= 1  && __GNUC_MINOR__ < 4
# define inline         __inline__ __attribute__((always_inline))
# define __inline__     __inline__ __attribute__((always_inline))
# define __inline       __inline__ __attribute__((always_inline))
#endif
[...]

because when I comment the second "define", I got better:
[...]
static __inline__ __attribute__((always_inline)) int __attribute__((deprecated))
check_region(unsigned long s, unsigned long n)
[...]

I very doubt that's the right thing to do so if somebody as any idea?


TIA,
     Joel

PS: please make me in cc (I am not in this list)