All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
@ 2015-01-17 23:36 Michael Karcher
  2015-01-18 18:07 ` Heiko Carstens
  2015-01-19 17:51 ` Steven Rostedt
  0 siblings, 2 replies; 7+ messages in thread
From: Michael Karcher @ 2015-01-17 23:36 UTC (permalink / raw)
  To: Heiko Carstens, Martin Schwidefsky, Will Deacon, AKASHI Takahiro,
	linux-kernel

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


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
  2015-01-17 23:36 [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H Michael Karcher
@ 2015-01-18 18:07 ` Heiko Carstens
  2015-01-18 20:01   ` Steven Rostedt
  2015-01-19 17:51 ` Steven Rostedt
  1 sibling, 1 reply; 7+ messages in thread
From: Heiko Carstens @ 2015-01-18 18:07 UTC (permalink / raw)
  To: Michael Karcher, Steven Rostedt
  Cc: Martin Schwidefsky, Will Deacon, AKASHI Takahiro, linux-kernel

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...


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
  2015-01-18 18:07 ` Heiko Carstens
@ 2015-01-18 20:01   ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2015-01-18 20:01 UTC (permalink / raw)
  To: Heiko Carstens
  Cc: Michael Karcher, Martin Schwidefsky, Will Deacon,
	AKASHI Takahiro, linux-kernel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
  2015-01-17 23:36 [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H Michael Karcher
  2015-01-18 18:07 ` Heiko Carstens
@ 2015-01-19 17:51 ` Steven Rostedt
  2015-01-19 18:10   ` Steven Rostedt
  1 sibling, 1 reply; 7+ messages in thread
From: Steven Rostedt @ 2015-01-19 17:51 UTC (permalink / raw)
  To: Matt Fleming
  Cc: Heiko Carstens, Michael Karcher, Martin Schwidefsky, Will Deacon,
	AKASHI Takahiro, linux-kernel


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/

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
  2015-01-19 17:51 ` Steven Rostedt
@ 2015-01-19 18:10   ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2015-01-19 18:10 UTC (permalink / raw)
  To: Matt Fleming
  Cc: Heiko Carstens, Michael Karcher, Martin Schwidefsky, Will Deacon,
	AKASHI Takahiro, linux-kernel

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/

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
  2015-01-19  8:18 John Paul Adrian Glaubitz
@ 2015-01-19 17:10 ` Steven Rostedt
  0 siblings, 0 replies; 7+ messages in thread
From: Steven Rostedt @ 2015-01-19 17:10 UTC (permalink / raw)
  To: John Paul Adrian Glaubitz; +Cc: Heiko Carstens, Michael Karcher, linux-kernel

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H
@ 2015-01-19  8:18 John Paul Adrian Glaubitz
  2015-01-19 17:10 ` Steven Rostedt
  0 siblings, 1 reply; 7+ messages in thread
From: John Paul Adrian Glaubitz @ 2015-01-19  8:18 UTC (permalink / raw)
  To: Steven Rostedt; +Cc: Heiko Carstens, Michael Karcher, linux-kernel

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-01-19 18:11 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-17 23:36 [PATCH 1/1] scripts/recordmcount.pl: There is no -m32 option on Super-H Michael Karcher
2015-01-18 18:07 ` Heiko Carstens
2015-01-18 20:01   ` Steven Rostedt
2015-01-19 17:51 ` Steven Rostedt
2015-01-19 18:10   ` Steven Rostedt
2015-01-19  8:18 John Paul Adrian Glaubitz
2015-01-19 17:10 ` Steven Rostedt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.