On Tue, Apr 9, 2013 at 9:27 PM, Mark Cave-Ayland mark.cave-ayland@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 the 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 ok 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 initialisation.
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?
-- Regards, Artyom Tarasenko
linux/sparc and solaris/sparc under qemu blog: http://tyom.blogspot.com/search/label/qemu