[OpenBIOS] [PATCH 2/2] Don't map the page 0 to make NULL-dereferencing more obvious
atar4qemu at gmail.com
Tue Apr 9 21:49:25 CEST 2013
On Tue, Apr 9, 2013 at 9:27 PM, Mark Cave-Ayland
<mark.cave-ayland at ilande.co.uk> wrote:
> On 08/04/13 21:25, Artyom Tarasenko wrote:
>>> You're probably calling ihandle>phandle with a phandle. I would expect
>>> return from active-package to be a phandle, not an ihandle.
>> You are absolutely right. Removing ihandle>phandle helped.
>> Thanks a lot!
> Actually the fix is much easier than this:
> Configuration device id QEMU version 1 machine id 32
> CPUs: 1 x FMI,MB86904
> UUID: 00000000-0000-0000-0000-000000000000
> Welcome to OpenBIOS v1.0 built on Apr 9 2013 18:40
> Type 'help' for detailed information
> 0 > cd /chosen ok
> 0 > .properties
> name "chosen"
> stdin ffce26f4
> stdout ffce278c
> memory ffce29c4
> mmu 0
> screen ffce2610
> 0 >
> According to the sample prtconf output, SPARC32 doesn't have an "mmu" chosen
> handle so we should just remove it. Now currently the property lives in
> forth/device/tree.fs so it exists across all architectures, however SPARC64
> and PPC are fine because they create the property on demand during
> I've just committed a patch to fix this (on a second attempt as I
> accidentally caught the NULL dereference patch in there too) so please feel
> free to test.
But we still don't need ihandle>phandle in (.property-by-name), do we?
linux/sparc and solaris/sparc under qemu blog:
More information about the OpenBIOS