[OpenBIOS] b?branch
Jd Lyons
lyons_dj at yahoo.com
Tue Jan 2 17:39:54 CET 2018
> On Jan 1, 2018, at 7:06 AM, Jd Lyons <lyons_dj at yahoo.com> wrote:
>
>
>
>> On Dec 31, 2017, at 1:09 PM, Mark Cave-Ayland <mark.cave-ayland at ilande.co.uk> wrote:
>>
>> On 31/12/17 12:59, Jd Lyons wrote:
>>
>>>> On Dec 30, 2017, at 6:04 AM, Segher Boessenkool <segher at kernel.crashing.org <mailto:segher at kernel.crashing.org>> wrote:
>>>>
>>>> On Sat, Dec 30, 2017 at 05:44:10AM -0500, Jd Lyons wrote:
>>>>>> On Dec 30, 2017, at 4:21 AM, Segher Boessenkool <segher at kernel.crashing.org <mailto:segher at kernel.crashing.org>> wrote:
>>>>>> On Fri, Dec 29, 2017 at 08:27:54PM -0500, Tarl Neustaedter wrote:
>>>>>>> [re-send, copying the list. For whatever reason, it seems messages
>>>>>>> aren't getting the reply-to: header.]
>>>>>>>
>>>>>>> On 2017-Dec-29 03:58 , Jd Lyons wrote:
>>>>>>>> 0 > " /pci/@e" open-dev to my-self ok
>>>>>>>> 0 > my-self . 5fc5ac34 ok
>>>>>>>> 0 > my-parent . 5fc5abfc ok
>>>>>>>> 0 > my-space . 0 ok <<---Seems my-space isn't returning a correct value?
>>>>>>>> 0 >
>>>>>>>>
>>>>>>> That's the problem. It appears that simply open-dev and assigning
>>>>>>> my-self isn't enough. my-space (and my-address and my-unit) aren't
>>>>>>> getting set up, so all config-space accesses are going to do the wrong
>>>>>>> thing (they'll go to device 0, which may or may not be the root).
>>>>>>>
>>>>>>> In the Sun/Oracle version, select would properly set things up, it
>>>>>>> appears no equivalent is available under openbios.
>>>>>>>
>>>>>>> I think you'll have to further debug this by getting the FCode to be
>>>>>>> pulled in at startup in place of the built-in vga fcode, rather than
>>>>>>> trying to fiddle things this way.
>>>>>>
>>>>>> Or set my-space to return 7000 and keep on fumbling :-)
>>>>>
>>>>> How would I set my-space to 7000?
>>>>>
>>>>> Is that specific to pci/@e?
>>>>>
>>>>> I noticed in SLOF that my-space . returned 1800, however the card was pci/@3.
>>>>
>>>> And that is correct :-)
>>>>
>>>> It is @dev,fn or if fn is 0, it is written as @dev . In the encoded
>>>> representation, it is 800*dev + 100*fn (dev is 5 bits, fn is 3 bits).
>>>>
>>>> In openbios, it looks like my-space gets its data from >dn.probe-addr in
>>>> the device node... And it is set via set-args... And then I got lost,
>>>> not sure how that is supposed to be called.
>>>>
>>> Looks like we need to change the way openbios handles my-space.
>>> SLOF deals with it in the nodes.fs
>>> : (my-phandle) ( -- phandle )
>>> my-self ?dup IF
>>> ihandle>phandle
>>> ELSE
>>> get-node dup 0= ABORT" no active node"
>>> THEN
>>> ;
>>> : my-space ( -- phys.hi )
>>> (my-phandle) >space
>>> ;
>>> I think we also need the >space word, the phandle word, and the ihandle word, I’ll have to track that down too.
>>> John, do you want to take a crack at fixing the >dn.probe-addr, or replacing it with something that returns a correct my-space .?
>>
>> After a bit of poking, it appears that the issue is simply that set-args isn't being called for PCI devices.
>>
>> The attached patch appears to fix the issue for me - can you confirm that it works for you?
>>
>> 0 > " /pci/QEMU,VGA" open-dev to my-self ok
>> 0 > my-space u. 800 ok
>>
>>
>
> Thanks Mark, that seems to work. my-space . now returns 7000 for /pci/@e.
>
> That seems correct to me?
>
>
> I’m still catching an exception at the same place, but now, at least, I think, we have the correct instance.
>
> Any insight on what I could/should try next Tarl?
>
> <> ( 0x03d )
> 25619: b?branch ( 0x014 ) 0x0026 ( =dec 38)
> 25620: (unnamed-fcode) [0x9bd]
> 25621: b(lit) ( 0x010 ) 0xff
> 25622: and ( 0x023 )
> 25623: my-space ( 0x103 )
> 25624: + ( 0x01e )
> 25625: (unnamed-fcode) [0xa08]
> 25626: b(lit) ( 0x010 ) 0x6
> 25627: and ( 0x023 )
> 25628: b(lit) ( 0x010 ) 0x4
> 25629: = ( 0x03c )
> 25630: b?branch ( 0x014 ) 0x0009 ()
> 25631: b(') ( 0x011 ) (unnamed-fcode) [0x9c1]
> 25632: b(to) ( 0x0c3 ) (unnamed-fcode) [0x9c0]
> 25633: b(>resolve) ( 0x0b2 )
> 25634: b(>resolve) ( 0x0b2 ) <<<————Still catching the exception here
> 25635: (unnamed-fcode) [0xddf]
>
>
>
>> ATB,
>>
>> Mark.
>> <openbios-pci-set-args.patch>
Seems we’re still not getting the correct instance.
https://lists.ozlabs.org/pipermail/slof/2018-January/002023.html
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/openbios/attachments/20180102/4e46e33a/attachment-0001.html>
More information about the OpenBIOS
mailing list