[OpenBIOS] sparc64's get_physical_address_code should not fault on TTE's with V=0

Jakub Jermar jakub at jermar.eu
Wed Jan 14 21:57:54 CET 2009


Blue Swirl wrote:
>>  I.e. it searches property 'screen' in /aliases and doesn't
>>  find any. If it is not there already, could you please add
>>  it (together with keyboard) to openbios?
> 
> Recent OpenBIOS (including the version in Qemu) should have both
> "screen" and "keyboard" aliases. In /chosen there is "display" (?) and
> "keyboard".

That's true, but somehow the following cycle used to detect number
of node's properties returns 1 even if there are more properties:

    name[0] = '\0';
    while (ofw_next_property(current, name, name) == 1)
        current_node->properties++;

The code above can be found in HelenOS's boot loader and so far
has worked on other machines. The ofw_next_property is a wrapper
for "nextprop" cif method.  I am not that good in Forth, so
I can't verify the implementation of "nextprop". Can you please
have a look if there is anything which could be causing my
wrapper to return incorrect number of properties?

Thanks,
Jakub



More information about the OpenBIOS mailing list