Pci driver can not handle multiple root complex system

[Jilong Kuang - SISA]

Hi Alex,

In my opinion, a simple fix is to remove the two lines in _find_next() function in pci_session_component.h

If (!bus_valid(bus))
  Continue;

Another way is to scan all buses/devices/functions at the first call of bus_valid(). A simple three level loop will do.

Best,
Jilong

-- 
Jilong Kuang Ph.D. C.Sci. | Samsung Research America - Silicon Valley (SRA-SV)

Senior Research Engineer, Computer Science Lab
t: 408-544-2705   c: 951-318-2995
jilong.kuang at ...60...

Please consider the environment before printing this email.

This message is intended only for the named recipient(s) above and may contain confidential or privileged information or protected attorney work product.  If you are not the intended recipient, any review, dissemination, distribution or copying is strictly prohibited.  If you have received this message in error, please immediately notify the sender and delete this message and its attachments from your computer and dispose of all other copies or printouts.  Thank you.

-----Original Message-----
From: Alexander Boettcher [mailto:alexander.boettcher at ...1...] 
Sent: Wednesday, June 12, 2013 5:11 AM
To: Genode OS Framework Mailing List
Subject: Re: Pci driver can not handle multiple root complex system

Hello,

On 08.06.2013 01:13, Jilong Kuang - SISA wrote:
> I just found out current Genode pci driver does not support a system 
> with multiple root complexes.

yes, this is true.
Until now we had neither the demand nor the hardware to develop for this feature.

I suggest to open at Genode at ...197... a feature request. If you have any insights or pointers to some public low level documentation about multiple PCI root complexes in regards to how to use, to handle them, it would be nice to share these information at the github feature request.

Regards,

Alex.
> 
>  
> 
> More specifically, in the bus_valid() function, it only scans all the 
> devices under the first root complex (starting from bus number 0).  As 
> a result, other devices under different root complexes are considered 
> INVALID, thus will be skipped by pci driver code.
> 
>  
> 
> Maybe a simple patch will fix the problem.
> 
>  
> 
> Best regards,
> 
> Jilong
> 
>  
> 
>  
> 
> --
> *Jilong Kuang Ph.D. C.Sci. | Samsung Research America - Silicon Valley
> (SRA-SV)*
> 
> Senior Research Engineer, Computer Science Lab
> t: 408-544-2705   c: 951-318-2995
> jilong.kuang at ...60... <mailto:jilong.kuang at ...60...>
> 
> *P*lease consider the environment before printing this email.
> 
> This message is intended only for the named recipient(s) above and may 
> contain confidential or privileged information or protected attorney 
> work product.  If you are not the intended recipient, any review, 
> dissemination, distribution or copying is strictly prohibited.  If you 
> have received this message in error, please immediately notify the 
> sender and delete this message and its attachments from your computer 
> and dispose of all other copies or printouts.  Thank you.
> 
>  
> 
> 
> 
> ----------------------------------------------------------------------
> -------- How ServiceNow helps IT people transform IT departments:
> 1. A cloud service to automate IT design, transition and operations 2. 
> Dashboards that offer high-level views of enterprise services 3. A 
> single system of record for all IT processes 
> http://p.sf.net/sfu/servicenow-d2d-j
> 
> 
> 
> _______________________________________________
> Genode-main mailing list
> Genode-main at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/genode-main
> 


------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Genode-main mailing list
Genode-main at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/genode-main