| Message ID | 20190424180223.253025-1-ndesaulniers@google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] sh: vsyscall: drop unnecessary cc-ldoption | expand |
On Wed, Apr 24, 2019 at 11:02 AM Nick Desaulniers <ndesaulniers@google.com> wrote: > > Towards the goal of removing cc-ldoption, it seems that --hash-style= > was added to binutils 2.17.50.0.2 in 2006. The minimal required version > of binutils for the kernel according to > Documentation/process/changes.rst is 2.20. > > Link: https://gcc.gnu.org/ml/gcc/2007-01/msg01141.html > Cc: clang-built-linux@googlegroups.com > Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Signed-off-by: Nick Desaulniers <ndesaulniers@google.com> > --- > Changes V1 -> V2: > * update commit subject and message as per Masahiro/Geert. > > To Geert's question about minimum binutils versions; no change needed to > binutils. > > > arch/sh/kernel/vsyscall/Makefile | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/sh/kernel/vsyscall/Makefile b/arch/sh/kernel/vsyscall/Makefile > index 5db6579bc44c..6e8664448048 100644 > --- a/arch/sh/kernel/vsyscall/Makefile > +++ b/arch/sh/kernel/vsyscall/Makefile > @@ -15,8 +15,7 @@ quiet_cmd_syscall = SYSCALL $@ > > export CPPFLAGS_vsyscall.lds += -P -C -Ush > > -vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 \ > - $(call cc-ldoption, -Wl$(comma)--hash-style=sysv) > +vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 -Wl,--hash-style=sysv > > SYSCFLAGS_vsyscall-trapa.so = $(vsyscall-flags) > > -- > 2.21.0.593.g511ec345e18-goog > bumping for review
bumping for review, as the merge window is now open. On Tue, Apr 30, 2019 at 1:26 PM Nick Desaulniers <ndesaulniers@google.com> wrote: > > On Wed, Apr 24, 2019 at 11:02 AM Nick Desaulniers > <ndesaulniers@google.com> wrote: > > > > Towards the goal of removing cc-ldoption, it seems that --hash-style= > > was added to binutils 2.17.50.0.2 in 2006. The minimal required version > > of binutils for the kernel according to > > Documentation/process/changes.rst is 2.20. > > > > Link: https://gcc.gnu.org/ml/gcc/2007-01/msg01141.html > > Cc: clang-built-linux@googlegroups.com > > Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > Signed-off-by: Nick Desaulniers <ndesaulniers@google.com> > > --- > > Changes V1 -> V2: > > * update commit subject and message as per Masahiro/Geert. > > > > To Geert's question about minimum binutils versions; no change needed to > > binutils. > > > > > > arch/sh/kernel/vsyscall/Makefile | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/arch/sh/kernel/vsyscall/Makefile b/arch/sh/kernel/vsyscall/Makefile > > index 5db6579bc44c..6e8664448048 100644 > > --- a/arch/sh/kernel/vsyscall/Makefile > > +++ b/arch/sh/kernel/vsyscall/Makefile > > @@ -15,8 +15,7 @@ quiet_cmd_syscall = SYSCALL $@ > > > > export CPPFLAGS_vsyscall.lds += -P -C -Ush > > > > -vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 \ > > - $(call cc-ldoption, -Wl$(comma)--hash-style=sysv) > > +vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 -Wl,--hash-style=sysv > > > > SYSCFLAGS_vsyscall-trapa.so = $(vsyscall-flags) > > > > -- > > 2.21.0.593.g511ec345e18-goog > > > > bumping for review > -- > Thanks, > ~Nick Desaulniers
On Thu, May 9, 2019 at 5:13 AM Nick Desaulniers <ndesaulniers@google.com> wrote: > > bumping for review, as the merge window is now open. sh is not very active these days. I applied this to my kbuild tree. I will send PR for this in the current MW. Thanks. > On Tue, Apr 30, 2019 at 1:26 PM Nick Desaulniers > <ndesaulniers@google.com> wrote: > > > > On Wed, Apr 24, 2019 at 11:02 AM Nick Desaulniers > > <ndesaulniers@google.com> wrote: > > > > > > Towards the goal of removing cc-ldoption, it seems that --hash-style= > > > was added to binutils 2.17.50.0.2 in 2006. The minimal required version > > > of binutils for the kernel according to > > > Documentation/process/changes.rst is 2.20. > > > > > > Link: https://gcc.gnu.org/ml/gcc/2007-01/msg01141.html > > > Cc: clang-built-linux@googlegroups.com > > > Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > > Signed-off-by: Nick Desaulniers <ndesaulniers@google.com> > > > --- > > > Changes V1 -> V2: > > > * update commit subject and message as per Masahiro/Geert. > > > > > > To Geert's question about minimum binutils versions; no change needed to > > > binutils. > > > > > > > > > arch/sh/kernel/vsyscall/Makefile | 3 +-- > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > diff --git a/arch/sh/kernel/vsyscall/Makefile b/arch/sh/kernel/vsyscall/Makefile > > > index 5db6579bc44c..6e8664448048 100644 > > > --- a/arch/sh/kernel/vsyscall/Makefile > > > +++ b/arch/sh/kernel/vsyscall/Makefile > > > @@ -15,8 +15,7 @@ quiet_cmd_syscall = SYSCALL $@ > > > > > > export CPPFLAGS_vsyscall.lds += -P -C -Ush > > > > > > -vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 \ > > > - $(call cc-ldoption, -Wl$(comma)--hash-style=sysv) > > > +vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 -Wl,--hash-style=sysv > > > > > > SYSCFLAGS_vsyscall-trapa.so = $(vsyscall-flags) > > > > > > -- > > > 2.21.0.593.g511ec345e18-goog > > > > > > > bumping for review > > -- > > Thanks, > > ~Nick Desaulniers > > > > -- > Thanks, > ~Nick Desaulniers
On 5/9/19 2:44 AM, Masahiro Yamada wrote: > On Thu, May 9, 2019 at 5:13 AM Nick Desaulniers <ndesaulniers@google.com> wrote: >> >> bumping for review, as the merge window is now open. > > > sh is not very active these days. We're still supporting SH with an up-to-date kernel and userland in Debian, so if your patch broke something, we will notice that soonish. This also applies to ia64 and many other architectures. So, if in doubt, there are folks in Debian which can test your kernel patch if you fear that it breaks something. Adrian
On Thu, May 9, 2019 at 3:11 PM John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote: > > On 5/9/19 2:44 AM, Masahiro Yamada wrote: > > On Thu, May 9, 2019 at 5:13 AM Nick Desaulniers <ndesaulniers@google.com> wrote: > >> > >> bumping for review, as the merge window is now open. > > > > > > sh is not very active these days. > > We're still supporting SH with an up-to-date kernel and userland in Debian, so > if your patch broke something, we will notice that soonish. This also applies > to ia64 and many other architectures. > > So, if in doubt, there are folks in Debian which can test your kernel patch > if you fear that it breaks something. Thanks for taking care of them in Debian. I am talking about the sh sub-system in the kernel here. SH patches occasionally fly around in LKML, but the SH maintainers are not picking up them these days. This one is trivial from the build system point of view. So, I offer to apply it to my tree. Thanks. > Adrian > > -- > .''`. John Paul Adrian Glaubitz > : :' : Debian Developer - glaubitz@debian.org > `. `' Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de > `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
Hi Masahiro! On 5/9/19 9:07 AM, Masahiro Yamada wrote: >> So, if in doubt, there are folks in Debian which can test your kernel patch >> if you fear that it breaks something. > > Thanks for taking care of them in Debian. > > I am talking about the sh sub-system in the kernel here. Yes, I'm aware of that :). > SH patches occasionally fly around in LKML, but > the SH maintainers are not picking up them these days. Rich was normally the one maintaining the SH tree and collecting the patches, not sure whether he is still active. It's been a while since he sent Linus a pull request. Sato-san was recently a patch himself, but I think he doesn't have a tree himself. @Sato: Could you have a look at the patch and ack it if it's okay? > This one is trivial from the build system point of view. > So, I offer to apply it to my tree. I'm fine with that. If you need to test build the kernel for SH, Debian has an easily installable cross-compiler for SH and every other architecture except ia64 (working on fixing that) available. In any case, thanks for taking care of SH in this case. Adrian
On Thu, 25 Apr 2019 03:02:21 +0900, Nick Desaulniers wrote: > > Towards the goal of removing cc-ldoption, it seems that --hash-style= > was added to binutils 2.17.50.0.2 in 2006. The minimal required version > of binutils for the kernel according to > Documentation/process/changes.rst is 2.20. > > Link: https://gcc.gnu.org/ml/gcc/2007-01/msg01141.html > Cc: clang-built-linux@googlegroups.com > Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Signed-off-by: Nick Desaulniers <ndesaulniers@google.com> Acked-by: Yoshinori Sato <ysato@users.sourceforge.jp> > --- > Changes V1 -> V2: > * update commit subject and message as per Masahiro/Geert. > > To Geert's question about minimum binutils versions; no change needed to > binutils. > > > arch/sh/kernel/vsyscall/Makefile | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/sh/kernel/vsyscall/Makefile b/arch/sh/kernel/vsyscall/Makefile > index 5db6579bc44c..6e8664448048 100644 > --- a/arch/sh/kernel/vsyscall/Makefile > +++ b/arch/sh/kernel/vsyscall/Makefile > @@ -15,8 +15,7 @@ quiet_cmd_syscall = SYSCALL $@ > > export CPPFLAGS_vsyscall.lds += -P -C -Ush > > -vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 \ > - $(call cc-ldoption, -Wl$(comma)--hash-style=sysv) > +vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 -Wl,--hash-style=sysv > > SYSCFLAGS_vsyscall-trapa.so = $(vsyscall-flags) > > -- > 2.21.0.593.g511ec345e18-goog >
On Thu, 09 May 2019 17:44:45 +0900, John Paul Adrian Glaubitz wrote: > > Hi Masahiro! > > On 5/9/19 9:07 AM, Masahiro Yamada wrote: > >> So, if in doubt, there are folks in Debian which can test your kernel patch > >> if you fear that it breaks something. > > > > Thanks for taking care of them in Debian. > > > > I am talking about the sh sub-system in the kernel here. > > Yes, I'm aware of that :). > > > SH patches occasionally fly around in LKML, but > > the SH maintainers are not picking up them these days. > > Rich was normally the one maintaining the SH tree and collecting the patches, > not sure whether he is still active. It's been a while since he sent Linus > a pull request. > > Sato-san was recently a patch himself, but I think he doesn't have a tree > himself. > > @Sato: Could you have a look at the patch and ack it if it's okay? I sent ack now. Sorry too late reply. > > This one is trivial from the build system point of view. > > So, I offer to apply it to my tree. > > I'm fine with that. If you need to test build the kernel for SH, Debian > has an easily installable cross-compiler for SH and every other architecture > except ia64 (working on fixing that) available. > > In any case, thanks for taking care of SH in this case. > > Adrian > > -- > .''`. John Paul Adrian Glaubitz > : :' : Debian Developer - glaubitz@debian.org > `. `' Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de > `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
diff --git a/arch/sh/kernel/vsyscall/Makefile b/arch/sh/kernel/vsyscall/Makefile index 5db6579bc44c..6e8664448048 100644 --- a/arch/sh/kernel/vsyscall/Makefile +++ b/arch/sh/kernel/vsyscall/Makefile @@ -15,8 +15,7 @@ quiet_cmd_syscall = SYSCALL $@ export CPPFLAGS_vsyscall.lds += -P -C -Ush -vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 \ - $(call cc-ldoption, -Wl$(comma)--hash-style=sysv) +vsyscall-flags = -shared -s -Wl,-soname=linux-gate.so.1 -Wl,--hash-style=sysv SYSCFLAGS_vsyscall-trapa.so = $(vsyscall-flags)
Towards the goal of removing cc-ldoption, it seems that --hash-style= was added to binutils 2.17.50.0.2 in 2006. The minimal required version of binutils for the kernel according to Documentation/process/changes.rst is 2.20. Link: https://gcc.gnu.org/ml/gcc/2007-01/msg01141.html Cc: clang-built-linux@googlegroups.com Suggested-by: Masahiro Yamada <yamada.masahiro@socionext.com> Signed-off-by: Nick Desaulniers <ndesaulniers@google.com> --- Changes V1 -> V2: * update commit subject and message as per Masahiro/Geert. To Geert's question about minimum binutils versions; no change needed to binutils. arch/sh/kernel/vsyscall/Makefile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)