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

2015-01-19 Thread 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 
> > ---
> >  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 majord...@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 majord...@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 majord...@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

2015-01-19 Thread 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 
> ---
>  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 majord...@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 majord...@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

2015-01-19 Thread Steven Rostedt
On Mon, 19 Jan 2015 09:18:28 +0100
John Paul Adrian Glaubitz  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=sh4=3.16.7-ckt4-1=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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@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

2015-01-19 Thread 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=sh4=3.16.7-ckt4-1=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 - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@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

2015-01-19 Thread 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=linuxarch=sh4ver=3.16.7-ckt4-1stamp=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 - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@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

2015-01-19 Thread Steven Rostedt
On Mon, 19 Jan 2015 09:18:28 +0100
John Paul Adrian Glaubitz glaub...@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=linuxarch=sh4ver=3.16.7-ckt4-1stamp=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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@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

2015-01-19 Thread 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 ker...@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 majord...@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 majord...@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

2015-01-19 Thread 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 ker...@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 majord...@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 majord...@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 majord...@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

2015-01-18 Thread Steven Rostedt
On Sun, 18 Jan 2015 19:07:40 +0100
Heiko Carstens  wrote:

> On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> > Signed-Off-By: Michael Karcher 
> > ---
> >  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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@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

2015-01-18 Thread Heiko Carstens
On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
> Signed-Off-By: Michael Karcher 
> ---
>  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...

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@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

2015-01-18 Thread Steven Rostedt
On Sun, 18 Jan 2015 19:07:40 +0100
Heiko Carstens heiko.carst...@de.ibm.com wrote:

 On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
  Signed-Off-By: Michael Karcher ker...@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
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@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

2015-01-18 Thread Heiko Carstens
On Sun, Jan 18, 2015 at 12:36:15AM +0100, Michael Karcher wrote:
 Signed-Off-By: Michael Karcher ker...@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...

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/