Re: [OpenBIOS] [commit] r1078 - trunk/openbios-devel/arch/ppc/qemu

[Olivier DANET]

> De : Mark Cave-Ayland
>  Envoyés : 10.01.13 20:54
>  Objet : Re: [OpenBIOS] [commit] r1078 - trunk/openbios-devel/arch/ppc/qemu
>
> On 10/01/13 18:25, Programmingkid wrote:
> > On Jan 9, 2013, at 7:27 PM, Mark Cave-Ayland wrote:
> > > (this is what I get with the patch I just posted applied):
> > >
> > > 0> here 10 0 fill ok
> > > 0> here 4 44332211 filll ok
> > > 0> here 8 dump
> > > fff503c0 44 33 22 11 00 00 00 00 00 00 00 00 00 00 00 00 D3".............
> >
> > Same as Open Firmware.
> > > 0> here 5 44332211 filll ok
> > > 0> here 8 dump
> > > fff503c0 44 33 22 11 44 00 00 00 00 00 00 00 00 00 00 00 D3".D...........
> >
> > This output in incorrect. This is what Open Firmware outputs:
> > ff94d75c: 44 33 22 11 44 33 22 11
> > > 0> here 6 44332211 filll ok
> > > 0> here 8 dump
> > > fff503c0 44 33 22 11 44 33 00 00 00 00 00 00 00 00 00 00 D3".D3..........
> >
> > This output is also incorrect. This is what Open Firmware outputs:
> > ff94d75c: 44 33 22 11 44 33 22 11
> > > 0> here 7 44332211 filll ok
> > > 0> here 8 dump
> > > fff503c0 44 33 22 11 44 33 22 00 00 00 00 00 00 00 00 00 D3".D3".........
> >
> > This output is incorrect. This is what Open Firmware outputs:
> > ff94d75c: 44 33 22 11 44 33 22 11
> > > 0> here 8 44332211 filll ok
> > > 0> here 8 dump
> > > fff503c0 44 33 22 11 44 33 22 11 00 00 00 00 00 00 00 00 D3".D3"........
>
> Hmmm well that looks as if it just rounds up to the next 4-byte word, 
> which is like the last patch but without the extra byte handling on the 
> end. Segher - can you confirm this on your hardware?
>
> ATB,
>
> Mark.

Out of curiosity, I have tried these sequences on a SparcStation5 and a PowerPC G3 iBook
Both get the same result.

here 4 44332211 fill
> 11 11 11 11

here 5 44332211 fill
> 11 11 11 11 11

here 8 44332211 fill
> 11 11 11 11 11 11 11 11

The lowest significant byte is repeated and the lenght is the byte count.
Hope this help ...

Olivier

[commit] r1078 - trunk/openbios-devel/arch/ppc/qemu

[Programmingkid]

On Jan 10, 2013, at 5:24 PM, Olivier DANET wrote:
> > De : Mark Cave-Ayland
> >  Envoyés : 10.01.13 20:54
> >  Objet : Re: [OpenBIOS] [commit] r1078 - trunk/openbios-devel/arch/ppc/qemu
> >
> > On 10/01/13 18:25, Programmingkid wrote:
> > > On Jan 9, 2013, at 7:27 PM, Mark Cave-Ayland wrote:
> > > > (this is what I get with the patch I just posted applied):
> > > >
> > > > 0> here 10 0 fill ok
> > > > 0> here 4 44332211 filll ok
> > > > 0> here 8 dump
> > > > fff503c0 44 33 22 11 00 00 00 00 00 00 00 00 00 00 00 00 D3".............
> > >
> > > Same as Open Firmware.
> > > > 0> here 5 44332211 filll ok
> > > > 0> here 8 dump
> > > > fff503c0 44 33 22 11 44 00 00 00 00 00 00 00 00 00 00 00 D3".D...........
> > >
> > > This output in incorrect. This is what Open Firmware outputs:
> > > ff94d75c: 44 33 22 11 44 33 22 11
> > > > 0> here 6 44332211 filll ok
> > > > 0> here 8 dump
> > > > fff503c0 44 33 22 11 44 33 00 00 00 00 00 00 00 00 00 00 D3".D3..........
> > >
> > > This output is also incorrect. This is what Open Firmware outputs:
> > > ff94d75c: 44 33 22 11 44 33 22 11
> > > > 0> here 7 44332211 filll ok
> > > > 0> here 8 dump
> > > > fff503c0 44 33 22 11 44 33 22 00 00 00 00 00 00 00 00 00 D3".D3".........
> > >
> > > This output is incorrect. This is what Open Firmware outputs:
> > > ff94d75c: 44 33 22 11 44 33 22 11
> > > > 0> here 8 44332211 filll ok
> > > > 0> here 8 dump
> > > > fff503c0 44 33 22 11 44 33 22 11 00 00 00 00 00 00 00 00 D3".D3"........
> >
> > Hmmm well that looks as if it just rounds up to the next 4-byte word, 
> > which is like the last patch but without the extra byte handling on the 
> > end. Segher - can you confirm this on your hardware?
> >
> > ATB,
> >
> > Mark.
>
> Out of curiosity, I have tried these sequences on a SparcStation5 and a PowerPC G3 iBook
> Both get the same result.
>
> here 4 44332211 fill
> > 11 11 11 11
>
> here 5 44332211 fill
> > 11 11 11 11 11
>
> here 8 44332211 fill
> > 11 11 11 11 11 11 11 11
>
> The lowest significant byte is repeated and the lenght is the byte count.
> Hope this help ...
>
> Olivier

Thank you for trying, but we are testing the filll (3 L's) word.