* Re: sparc next boot failure
2008-12-01 2:01 sparc next boot failure Robert Reif
@ 2008-12-01 5:50 ` Sam Ravnborg
2008-12-01 8:55 ` Sam Ravnborg
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Sam Ravnborg @ 2008-12-01 5:50 UTC (permalink / raw)
To: sparclinux
On Sun, Nov 30, 2008 at 09:01:52PM -0500, Robert Reif wrote:
> Current sparc next fails to boot (hangs at Booting Linux...).
>
> I bisected it down to this:
>
> 21cb8a43216e0595b466e903e0f2a7ae615d9e11 is first bad commit
> commit 21cb8a43216e0595b466e903e0f2a7ae615d9e11
> Author: Sam Ravnborg <sam@ravnborg.org>
> Date: Sat Nov 29 21:52:12 2008 -0800
>
> sparc: refactor Makefile
>
> The btfixup step needs knowledge of all the .o files,
> but there is no need to pass them in independent variables.
> Simplify it to use only two variables.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Signed-off-by: David S. Miller <davem@davemloft.net>
>
> :040000 040000 d5815835afa54122531d5bc8060f3831b145a1a4
> c8aad1a3c22a8243c5dd0ab2c528198d7d23af3c M arch
Hi Robert.
Thanks for testing and bisecting!
Can I ask you to send me the output of make V=1 for
your kernel before and after the above patch is applied.
It's only the last few lines that are of interest (from btfixup
and onwards).
I hope from that to say what is causing the boot regression.
Sam
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: sparc next boot failure
2008-12-01 2:01 sparc next boot failure Robert Reif
2008-12-01 5:50 ` Sam Ravnborg
@ 2008-12-01 8:55 ` Sam Ravnborg
2008-12-01 12:41 ` Robert Reif
2008-12-01 13:15 ` Sam Ravnborg
3 siblings, 0 replies; 5+ messages in thread
From: Sam Ravnborg @ 2008-12-01 8:55 UTC (permalink / raw)
To: sparclinux
On Sun, Nov 30, 2008 at 09:01:52PM -0500, Robert Reif wrote:
> Current sparc next fails to boot (hangs at Booting Linux...).
>
> I bisected it down to this:
>
> 21cb8a43216e0595b466e903e0f2a7ae615d9e11 is first bad commit
> commit 21cb8a43216e0595b466e903e0f2a7ae615d9e11
> Author: Sam Ravnborg <sam@ravnborg.org>
> Date: Sat Nov 29 21:52:12 2008 -0800
>
> sparc: refactor Makefile
>
> The btfixup step needs knowledge of all the .o files,
> but there is no need to pass them in independent variables.
> Simplify it to use only two variables.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Signed-off-by: David S. Miller <davem@davemloft.net>
>
> :040000 040000 d5815835afa54122531d5bc8060f3831b145a1a4
> c8aad1a3c22a8243c5dd0ab2c528198d7d23af3c M arch
Hi Robert.
Can you try to add the following patch on top of
the " sparc: refactor Makefile" patch.
This was the only difference I could see in my
limited testing here.
Sam
diff --git a/arch/sparc/Makefile b/arch/sparc/Makefile
index 04ea968..da82ba6 100644
--- a/arch/sparc/Makefile
+++ b/arch/sparc/Makefile
@@ -44,6 +44,8 @@ ifdef CONFIG_KALLSYMS
kallsyms.o := .tmp_kallsyms2.o
endif
+export kallsyms.o
+
# Default target
all: zImage
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: sparc next boot failure
2008-12-01 2:01 sparc next boot failure Robert Reif
2008-12-01 5:50 ` Sam Ravnborg
2008-12-01 8:55 ` Sam Ravnborg
@ 2008-12-01 12:41 ` Robert Reif
2008-12-01 13:15 ` Sam Ravnborg
3 siblings, 0 replies; 5+ messages in thread
From: Robert Reif @ 2008-12-01 12:41 UTC (permalink / raw)
To: sparclinux
Sam Ravnborg wrote:
> On Sun, Nov 30, 2008 at 09:01:52PM -0500, Robert Reif wrote:
>
>> Current sparc next fails to boot (hangs at Booting Linux...).
>>
>> I bisected it down to this:
>>
>> 21cb8a43216e0595b466e903e0f2a7ae615d9e11 is first bad commit
>> commit 21cb8a43216e0595b466e903e0f2a7ae615d9e11
>> Author: Sam Ravnborg <sam@ravnborg.org>
>> Date: Sat Nov 29 21:52:12 2008 -0800
>>
>> sparc: refactor Makefile
>>
>> The btfixup step needs knowledge of all the .o files,
>> but there is no need to pass them in independent variables.
>> Simplify it to use only two variables.
>>
>> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
>> Signed-off-by: David S. Miller <davem@davemloft.net>
>>
>> :040000 040000 d5815835afa54122531d5bc8060f3831b145a1a4
>> c8aad1a3c22a8243c5dd0ab2c528198d7d23af3c M arch
>>
>
> Hi Robert.
>
> Can you try to add the following patch on top of
> the " sparc: refactor Makefile" patch.
>
> This was the only difference I could see in my
> limited testing here.
>
> Sam
>
> diff --git a/arch/sparc/Makefile b/arch/sparc/Makefile
> index 04ea968..da82ba6 100644
> --- a/arch/sparc/Makefile
> +++ b/arch/sparc/Makefile
> @@ -44,6 +44,8 @@ ifdef CONFIG_KALLSYMS
> kallsyms.o := .tmp_kallsyms2.o
> endif
>
> +export kallsyms.o
> +
> # Default target
> all: zImage
>
>
>
I hand applied it to yesterdays sparc-next and it seems to have fixed it.
I'll do some more testing tonight when I have the time.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: sparc next boot failure
2008-12-01 2:01 sparc next boot failure Robert Reif
` (2 preceding siblings ...)
2008-12-01 12:41 ` Robert Reif
@ 2008-12-01 13:15 ` Sam Ravnborg
3 siblings, 0 replies; 5+ messages in thread
From: Sam Ravnborg @ 2008-12-01 13:15 UTC (permalink / raw)
To: sparclinux
On Mon, Dec 01, 2008 at 07:41:07AM -0500, Robert Reif wrote:
> Sam Ravnborg wrote:
> >On Sun, Nov 30, 2008 at 09:01:52PM -0500, Robert Reif wrote:
> >
> >>Current sparc next fails to boot (hangs at Booting Linux...).
> >>
> >>I bisected it down to this:
> >>
> >>21cb8a43216e0595b466e903e0f2a7ae615d9e11 is first bad commit
> >>commit 21cb8a43216e0595b466e903e0f2a7ae615d9e11
> >>Author: Sam Ravnborg <sam@ravnborg.org>
> >>Date: Sat Nov 29 21:52:12 2008 -0800
> >>
> >> sparc: refactor Makefile
> >>
> >> The btfixup step needs knowledge of all the .o files,
> >> but there is no need to pass them in independent variables.
> >> Simplify it to use only two variables.
> >>
> >> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> >> Signed-off-by: David S. Miller <davem@davemloft.net>
> >>
> >>:040000 040000 d5815835afa54122531d5bc8060f3831b145a1a4
> >>c8aad1a3c22a8243c5dd0ab2c528198d7d23af3c M arch
> >>
> >
> >Hi Robert.
> >
> >Can you try to add the following patch on top of
> >the " sparc: refactor Makefile" patch.
> >
> >This was the only difference I could see in my
> >limited testing here.
> >
> > Sam
> >
> >diff --git a/arch/sparc/Makefile b/arch/sparc/Makefile
> >index 04ea968..da82ba6 100644
> >--- a/arch/sparc/Makefile
> >+++ b/arch/sparc/Makefile
> >@@ -44,6 +44,8 @@ ifdef CONFIG_KALLSYMS
> > kallsyms.o := .tmp_kallsyms2.o
> > endif
> >
> >+export kallsyms.o
> >+
> > # Default target
> > all: zImage
> >
> >
> >
> I hand applied it to yesterdays sparc-next and it seems to have fixed it.
>
> I'll do some more testing tonight when I have the time.
Thanks. I do not see why it cured the boot but I know how to fix it.
I will try to find time tonight to look at it.
Sam
^ permalink raw reply [flat|nested] 5+ messages in thread