Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[John Paul Adrian Glaubitz]

Hi Steven!

>> The Super-H architecure seems to be without any maintainer.
>> Maybe Steven cares to pick this patch up. Dunno...

> Yeah, I remember when the SH maintainer left.

> I can pick it up. How urgent is it? Does it need to go in before the
> next merge window? And does it need to be tagged for stable?

I am one of the Debian porters of the SH port. Currently, the kernel
package fails to build from source on our automatic build machines
due to this bug. Here is the full build log where you can see it
fail with gcc-4.8:

>
http://buildd.debian-ports.org/status/fetch.php?pkg=linux&arch=sh4&ver=3.16.7-ckt4-1&stamp=1421425783

Since the SH port of Debian was long time poorly maintained, this
issue hasn't got any attention for quite a while which means the
most current kernel version we have at the moment on SH is 3.2.0
something. I asked Michael to submit this change on my behalf since
he has submitted kernel patches in the past.

Thus, it would be nice if this fix could be merged ASAP upstream
and included in all stable releases as well such that we get a
working up-to-date kernel on SH soon.

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

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Steven Rostedt]

On Mon, 19 Jan 2015 09:18:28 +0100
John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote:

> Hi Steven!
> 
> >> The Super-H architecure seems to be without any maintainer.
> >> Maybe Steven cares to pick this patch up. Dunno...
> 
> > Yeah, I remember when the SH maintainer left.
> 
> > I can pick it up. How urgent is it? Does it need to go in before the
> > next merge window? And does it need to be tagged for stable?
> 
> I am one of the Debian porters of the SH port. Currently, the kernel
> package fails to build from source on our automatic build machines
> due to this bug. Here is the full build log where you can see it
> fail with gcc-4.8:

Unfortunately I don't have a 4.8 cross compiler for superH. It seems to
work fine with -m32 with 4.6 and earlier (which I do have).

> 
> >
> http://buildd.debian-ports.org/status/fetch.php?pkg=linux&arch=sh4&ver=3.16.7-ckt4-1&stamp=1421425783
> 
> Since the SH port of Debian was long time poorly maintained, this
> issue hasn't got any attention for quite a while which means the
> most current kernel version we have at the moment on SH is 3.2.0
> something. I asked Michael to submit this change on my behalf since
> he has submitted kernel patches in the past.
> 
> Thus, it would be nice if this fix could be merged ASAP upstream
> and included in all stable releases as well such that we get a
> working up-to-date kernel on SH soon.

I'll add this info as well to the change log.

-- Steve

[PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Michael Karcher]

Signed-Off-By: Michael Karcher <kernel@mkarcher.dialup.fu-berlin.de>
---
 scripts/recordmcount.pl | 1 -
 1 file changed, 1 deletion(-)

diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
index 56ea99a..537c38c 100755
--- a/scripts/recordmcount.pl
+++ b/scripts/recordmcount.pl
@@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
     # force flags for this arch
     $ld .= " -m shlelf_linux";
     $objcopy .= " -O elf32-sh-linux";
-    $cc .= " -m32";
 
 } elsif ($arch eq "powerpc") {
     $local_regex = "^[0-9a-fA-F]+\\s+t\\s+(\\.?\\S+)";
-- 
2.1.3

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Heiko Carstens]

On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> Signed-Off-By: Michael Karcher <kernel@mkarcher.dialup.fu-berlin.de>
> ---
>  scripts/recordmcount.pl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> index 56ea99a..537c38c 100755
> --- a/scripts/recordmcount.pl
> +++ b/scripts/recordmcount.pl
> @@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
>      # force flags for this arch
>      $ld .= " -m shlelf_linux";
>      $objcopy .= " -O elf32-sh-linux";
> -    $cc .= " -m32";
> 
>  } elsif ($arch eq "powerpc") {
>      $local_regex = "^[0-9a-fA-F]+\\s+t\\s+(\\.?\\S+)";
> -- 
> 2.1.3

The Super-H architecure seems to be without any maintainer.
Maybe Steven cares to pick this patch up. Dunno...

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Steven Rostedt]

On Sun, 18 Jan 2015 19:07:40 +0100
Heiko Carstens <heiko.carstens@de.ibm.com> wrote:

> On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> > Signed-Off-By: Michael Karcher <kernel@mkarcher.dialup.fu-berlin.de>
> > ---
> >  scripts/recordmcount.pl | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> > diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> > index 56ea99a..537c38c 100755
> > --- a/scripts/recordmcount.pl
> > +++ b/scripts/recordmcount.pl
> > @@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
> >      # force flags for this arch
> >      $ld .= " -m shlelf_linux";
> >      $objcopy .= " -O elf32-sh-linux";
> > -    $cc .= " -m32";
> > 
> >  } elsif ($arch eq "powerpc") {
> >      $local_regex = "^[0-9a-fA-F]+\\s+t\\s+(\\.?\\S+)";
> > -- 
> > 2.1.3
> 
> The Super-H architecure seems to be without any maintainer.
> Maybe Steven cares to pick this patch up. Dunno...

Yeah, I remember when the SH maintainer left.

I can pick it up. How urgent is it? Does it need to go in before the
next merge window? And does it need to be tagged for stable?

-- Steve

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Steven Rostedt]

Matt,

You initially added this code. Are you OK with this change?

-- Steve

On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> Signed-Off-By: Michael Karcher <kernel@mkarcher.dialup.fu-berlin.de>
> ---
>  scripts/recordmcount.pl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> index 56ea99a..537c38c 100755
> --- a/scripts/recordmcount.pl
> +++ b/scripts/recordmcount.pl
> @@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
>      # force flags for this arch
>      $ld .= " -m shlelf_linux";
>      $objcopy .= " -O elf32-sh-linux";
> -    $cc .= " -m32";
>  
>  } elsif ($arch eq "powerpc") {
>      $local_regex = "^[0-9a-fA-F]+\\s+t\\s+(\\.?\\S+)";
> -- 
> 2.1.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H

[Steven Rostedt]

Resending with Matt's current email address.

-- Steve

On Mon, Jan 19, 2015 at 12:51:25PM -0500, Steven Rostedt wrote:
> 
> Matt,
> 
> You initially added this code. Are you OK with this change?
> 
> -- Steve
> 
> On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> > Signed-Off-By: Michael Karcher <kernel@mkarcher.dialup.fu-berlin.de>
> > ---
> >  scripts/recordmcount.pl | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> > diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> > index 56ea99a..537c38c 100755
> > --- a/scripts/recordmcount.pl
> > +++ b/scripts/recordmcount.pl
> > @@ -255,7 +255,6 @@ if ($arch eq "x86_64") {
> >      # force flags for this arch
> >      $ld .= " -m shlelf_linux";
> >      $objcopy .= " -O elf32-sh-linux";
> > -    $cc .= " -m32";
> >  
> >  } elsif ($arch eq "powerpc") {
> >      $local_regex = "^[0-9a-fA-F]+\\s+t\\s+(\\.?\\S+)";
> > -- 
> > 2.1.3
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/