Re: [PATCH] reiserfs: remove workaround code for GCC 3.x
Masahiro Yamada writes: > Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") > bumped the minimum GCC version to 4.6 for all architectures. > > The workaround code in fs/reiserfs/Makefile is obsolete now. > > Signed-off-by: Masahiro Yamada > --- > > fs/reiserfs/Makefile | 9 - > 1 file changed, 9 deletions(-) > > diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile > index a39a562..bd29c58 100644 > --- a/fs/reiserfs/Makefile > +++ b/fs/reiserfs/Makefile > @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) > reiserfs-objs += xattr_acl.o > endif > > -# 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. > -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) > - > TAGS: > etags *.c Oh wow that's gross. Thanks for cleaning it up. Acked-by: Michael Ellerman cheers
Re: [PATCH] reiserfs: remove workaround code for GCC 3.x
Masahiro Yamada writes: > Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") > bumped the minimum GCC version to 4.6 for all architectures. > > The workaround code in fs/reiserfs/Makefile is obsolete now. > > Signed-off-by: Masahiro Yamada > --- > > fs/reiserfs/Makefile | 9 - > 1 file changed, 9 deletions(-) > > diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile > index a39a562..bd29c58 100644 > --- a/fs/reiserfs/Makefile > +++ b/fs/reiserfs/Makefile > @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) > reiserfs-objs += xattr_acl.o > endif > > -# 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. > -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) > - > TAGS: > etags *.c Oh wow that's gross. Thanks for cleaning it up. Acked-by: Michael Ellerman cheers
Re: [PATCH] reiserfs: remove workaround code for GCC 3.x
On 8/26/18 10:33 PM, Masahiro Yamada wrote: > Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") > bumped the minimum GCC version to 4.6 for all architectures. > > The workaround code in fs/reiserfs/Makefile is obsolete now. > > Signed-off-by: Masahiro Yamada Acked-by: Jeff Mahoney Thanks, -Jeff > --- > > fs/reiserfs/Makefile | 9 - > 1 file changed, 9 deletions(-) > > diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile > index a39a562..bd29c58 100644 > --- a/fs/reiserfs/Makefile > +++ b/fs/reiserfs/Makefile > @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) > reiserfs-objs += xattr_acl.o > endif > > -# 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. > -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) > - > TAGS: > etags *.c > - > -- Jeff Mahoney SUSE Labs signature.asc Description: OpenPGP digital signature
Re: [PATCH] reiserfs: remove workaround code for GCC 3.x
On 8/26/18 10:33 PM, Masahiro Yamada wrote: > Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") > bumped the minimum GCC version to 4.6 for all architectures. > > The workaround code in fs/reiserfs/Makefile is obsolete now. > > Signed-off-by: Masahiro Yamada Acked-by: Jeff Mahoney Thanks, -Jeff > --- > > fs/reiserfs/Makefile | 9 - > 1 file changed, 9 deletions(-) > > diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile > index a39a562..bd29c58 100644 > --- a/fs/reiserfs/Makefile > +++ b/fs/reiserfs/Makefile > @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) > reiserfs-objs += xattr_acl.o > endif > > -# 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. > -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) > - > TAGS: > etags *.c > - > -- Jeff Mahoney SUSE Labs signature.asc Description: OpenPGP digital signature
[PATCH] reiserfs: remove workaround code for GCC 3.x
Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") bumped the minimum GCC version to 4.6 for all architectures. The workaround code in fs/reiserfs/Makefile is obsolete now. Signed-off-by: Masahiro Yamada --- fs/reiserfs/Makefile | 9 - 1 file changed, 9 deletions(-) diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile index a39a562..bd29c58 100644 --- a/fs/reiserfs/Makefile +++ b/fs/reiserfs/Makefile @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) reiserfs-objs += xattr_acl.o endif -# 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. -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) - TAGS: etags *.c - -- 2.7.4
[PATCH] reiserfs: remove workaround code for GCC 3.x
Commit cafa0010cd51 ("Raise the minimum required gcc version to 4.6") bumped the minimum GCC version to 4.6 for all architectures. The workaround code in fs/reiserfs/Makefile is obsolete now. Signed-off-by: Masahiro Yamada --- fs/reiserfs/Makefile | 9 - 1 file changed, 9 deletions(-) diff --git a/fs/reiserfs/Makefile b/fs/reiserfs/Makefile index a39a562..bd29c58 100644 --- a/fs/reiserfs/Makefile +++ b/fs/reiserfs/Makefile @@ -26,14 +26,5 @@ ifeq ($(CONFIG_REISERFS_FS_POSIX_ACL),y) reiserfs-objs += xattr_acl.o endif -# 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. -ccflags-$(CONFIG_PPC32) := $(call cc-ifversion, -lt, 0400, -O1) - TAGS: etags *.c - -- 2.7.4