[PATCH] make gcc -O1 in fs/reiserfs optional
olh at suse.de
Wed Oct 12 05:01:33 EST 2005
On Wed, Apr 27, Hans Reiser wrote:
> Olaf Hering wrote:
> >you added this EXTRA_CFLAGS= during 2.4 development, I think the broken
> >compiler was gcc 3.2 on SLES8. Can we turn this -O1 into a .config
> Sounds reasonable.
only compile with -O1 if the (very old) compiler is broken
We use reiserfs alot in SLES9 on ppc64, and it was never seen
Signed-off-by: Olaf Hering <olh at suse.de>
@@ -21,13 +21,7 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y)
reiserfs-objs += xattr_acl.o
-# gcc -O2 (the kernel default) is overaggressive on ppc32 when many inline
-# functions are used. This causes the compiler to advance the stack
-# pointer out of the available stack space, corrupting kernel space,
-# and causing a panic. Since this behavior only affects ppc32, this ifeq
-# will work around it. If any other architecture displays this behavior,
-# add it here.
EXTRA_CFLAGS := -O1
@@ -186,6 +186,18 @@ config REISERFS_FS
If you like it, you can pay us to add new features to it that you
need, buy a support contract, or pay us to port it to another OS.
+ bool "Reduce CC optimization level to workaround compiler bugs"
+ depends on PPC32
+ default n
+ gcc -O2 (the kernel default) is overaggressive on ppc32 when many inline
+ functions are used. This causes the compiler to advance the stack
+ pointer out of the available stack space, corrupting kernel space,
+ and causing a panic. Since this behavior only affects ppc32, this ifeq
+ will work around it. If any other architecture displays this behavior,
+ add it here.
bool "Enable reiserfs debug mode"
depends on REISERFS_FS
short story of a lazy sysadmin:
More information about the Linuxppc-dev