* Please revert "use todc time functions for PPC_PREP" changeset
@ 2004-02-28 8:23 Leigh Brown
2004-02-28 8:32 ` Andrew Morton
0 siblings, 1 reply; 7+ messages in thread
From: Leigh Brown @ 2004-02-28 8:23 UTC (permalink / raw)
To: Tom Rini; +Cc: akpm, linuxppc-dev
Hi Tom,
Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina, and
probably quite a few other RS/6000 based PRePs. I was a bit surprised
to see it 2.6.3.
http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
Can we revert this until there is a working version, please? I don't
really know enough about this stuff to fix it myself, unfortunately,
although I'm working through the code to see how it works.
Cheers,
Leigh.
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 8:32 ` Andrew Morton
@ 2004-02-28 8:28 ` Benjamin Herrenschmidt
2004-02-28 8:45 ` Leigh Brown
2004-02-28 8:43 ` Leigh Brown
1 sibling, 1 reply; 7+ messages in thread
From: Benjamin Herrenschmidt @ 2004-02-28 8:28 UTC (permalink / raw)
To: Andrew Morton; +Cc: Leigh Brown, Tom Rini, linuxppc-dev list
On Sat, 2004-02-28 at 19:32, Andrew Morton wrote:
> "Leigh Brown" <leigh@solinno.co.uk> wrote:
> >
> > Hi Tom,
> >
> > Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina, and
> > probably quite a few other RS/6000 based PRePs. I was a bit surprised
> > to see it 2.6.3.
> >
> > http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
> >
> > Can we revert this until there is a working version, please?
>
> It wouild help if you could define "breaks".
Yup, Leigh, we have at least a week I suppose before 2.6.4 goes final, so
we can probably fix the problem rather than reverting the cleanup.
Can you tell us more about what's going on ?
Ben.
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 8:23 Please revert "use todc time functions for PPC_PREP" changeset Leigh Brown
@ 2004-02-28 8:32 ` Andrew Morton
2004-02-28 8:28 ` Benjamin Herrenschmidt
2004-02-28 8:43 ` Leigh Brown
0 siblings, 2 replies; 7+ messages in thread
From: Andrew Morton @ 2004-02-28 8:32 UTC (permalink / raw)
To: Leigh Brown; +Cc: trini, linuxppc-dev
"Leigh Brown" <leigh@solinno.co.uk> wrote:
>
> Hi Tom,
>
> Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina, and
> probably quite a few other RS/6000 based PRePs. I was a bit surprised
> to see it 2.6.3.
>
> http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
>
> Can we revert this until there is a working version, please?
It wouild help if you could define "breaks".
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 8:32 ` Andrew Morton
2004-02-28 8:28 ` Benjamin Herrenschmidt
@ 2004-02-28 8:43 ` Leigh Brown
2004-02-28 15:33 ` Tom Rini
1 sibling, 1 reply; 7+ messages in thread
From: Leigh Brown @ 2004-02-28 8:43 UTC (permalink / raw)
To: Andrew Morton; +Cc: trini, linuxppc-dev
Andrew Morton said:
>
> "Leigh Brown" <leigh@solinno.co.uk> wrote:
>>
>> Hi Tom,
>>
>> Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina, and
>> probably quite a few other RS/6000 based PRePs. I was a bit surprised
>> to see it 2.6.3.
>>
>> http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
>>
>> Can we revert this until there is a working version, please?
>
> It wouild help if you could define "breaks".
Sorry! It hangs during boot, before any output on the serial console. I
got as far as working out that it hangs somewhere in
arch/ppc/kernel/time.c:time_init() (or one of the called functions).
I can do some more narrowing down if it helps.
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 8:28 ` Benjamin Herrenschmidt
@ 2004-02-28 8:45 ` Leigh Brown
0 siblings, 0 replies; 7+ messages in thread
From: Leigh Brown @ 2004-02-28 8:45 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: Andrew Morton, Tom Rini, linuxppc-dev list
Benjamin Herrenschmidt said:
> On Sat, 2004-02-28 at 19:32, Andrew Morton wrote:
>> "Leigh Brown" <leigh@solinno.co.uk> wrote:
>> >
>> > Hi Tom,
>> >
>> > Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina,
>> and
>> > probably quite a few other RS/6000 based PRePs. I was a bit
>> surprised
>> > to see it 2.6.3.
>> >
>> > http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
>> >
>> > Can we revert this until there is a working version, please?
>>
>> It wouild help if you could define "breaks".
>
> Yup, Leigh, we have at least a week I suppose before 2.6.4 goes final, so
> we can probably fix the problem rather than reverting the cleanup.
>
> Can you tell us more about what's going on ?
Okay, great. I have to go out now but I'll narrow down the exact function
that's hanging when I get back.
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 8:43 ` Leigh Brown
@ 2004-02-28 15:33 ` Tom Rini
2004-03-05 8:19 ` Leigh Brown
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2004-02-28 15:33 UTC (permalink / raw)
To: Leigh Brown; +Cc: Andrew Morton, linuxppc-dev
On Sat, Feb 28, 2004 at 08:43:06AM -0000, Leigh Brown wrote:
> Andrew Morton said:
> >
> > "Leigh Brown" <leigh@solinno.co.uk> wrote:
> >>
> >> Hi Tom,
> >>
> >> Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina, and
> >> probably quite a few other RS/6000 based PRePs. I was a bit surprised
> >> to see it 2.6.3.
> >>
> >> http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
> >>
> >> Can we revert this until there is a working version, please?
> >
> > It wouild help if you could define "breaks".
>
> Sorry! It hangs during boot, before any output on the serial console. I
> got as far as working out that it hangs somewhere in
> arch/ppc/kernel/time.c:time_init() (or one of the called functions).
Er.. Didn't we talk about this awhile ago, and get a different version
of this patch working back in 2.4? The problem was that we couldn't use
todc_calibrate_decr we had to still grab the value from the residual
data. And I thought I had kept that logic still..
--
Tom Rini
http://gate.crashing.org/~trini/
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Please revert "use todc time functions for PPC_PREP" changeset
2004-02-28 15:33 ` Tom Rini
@ 2004-03-05 8:19 ` Leigh Brown
0 siblings, 0 replies; 7+ messages in thread
From: Leigh Brown @ 2004-03-05 8:19 UTC (permalink / raw)
To: Andrew Morton; +Cc: linuxppc-dev, Tom Rini
Tom Rini said:
> On Sat, Feb 28, 2004 at 08:43:06AM -0000, Leigh Brown wrote:
>> Andrew Morton said:
>> >
>> > "Leigh Brown" <leigh@solinno.co.uk> wrote:
>> >> Your patch (ChangeSet@1.1557.41.25) breaks the 43p140 and Carolina,
>> and
>> >> probably quite a few other RS/6000 based PRePs. I was a bit
>> surprised
>> >> to see it 2.6.3.
>> >>
>> >> http://linux.bkbits.net:8080/linux-2.5/patch@1.1557.41.25?nav=index.html|ChangeSet@-3w|cset@1.1557.41.25
>> >>
>> >> Can we revert this until there is a working version, please?
>> >
>> > It wouild help if you could define "breaks".
>>
>> Sorry! It hangs during boot, before any output on the serial console.
>> I
>> got as far as working out that it hangs somewhere in
>> arch/ppc/kernel/time.c:time_init() (or one of the called functions).
>
> Er.. Didn't we talk about this awhile ago, and get a different version
> of this patch working back in 2.4? The problem was that we couldn't use
> todc_calibrate_decr we had to still grab the value from the residual
> data. And I thought I had kept that logic still..
Just a quick update on this. Tom has a patch that I've tested that
makes it work again. I very much hope it will be submitted shortly.
Thanks all for your help.
Leigh.
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2004-03-05 8:19 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-02-28 8:23 Please revert "use todc time functions for PPC_PREP" changeset Leigh Brown
2004-02-28 8:32 ` Andrew Morton
2004-02-28 8:28 ` Benjamin Herrenschmidt
2004-02-28 8:45 ` Leigh Brown
2004-02-28 8:43 ` Leigh Brown
2004-02-28 15:33 ` Tom Rini
2004-03-05 8:19 ` Leigh Brown
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.