[OpenBIOS] [commit] r1209 - trunk/openbios-devel/arch/sparc32

[Aurelien Jarno]

On Mon, Aug 19, 2013 at 09:40:13AM +0200, repository service wrote:
> Author: mcayland
> Date: Mon Aug 19 09:40:12 2013
> New Revision: 1209
> URL: http://tracker.coreboot.org/trac/openbios/changeset/1209
> 
> Log:
> SPARC32: fix romvec stdin/stdout field initialisation
> 
> Previously the romvec stdin/stdout ihandles were configured so that they were
> set to the current stdin and stdout paths at initialisation time. Unfortunately
> as stdin/stdout can be changed with the input and output words, they can end
> up pointing to invalid ihandles causing a crash when trying to output to the
> console.
> 
> Fix this by pointing the romvec structure to the address of the stdin/stdout
> variables so that they are always in sync. Similarly we also resolve the
> text path stdin/stdout variables at boot time to ensure that they will never
> be stale.
> 
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland at ilande.co.uk>
> 
> Modified:
>    trunk/openbios-devel/arch/sparc32/boot.c
>    trunk/openbios-devel/arch/sparc32/romvec.c

Since this patch has been applied, I am unable to boot a Debian SPARC32
guest using SILO as a bootloader. OpenBios stops at "Trying disk".

One can find such a guest there:
http://people.debian.org/~aurel32/qemu/sparc/debian_etch_sparc_small.qcow2

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien at aurel32.net                 http://www.aurel32.net