ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-06-13 04:38:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

pci: fix irq re-routing discovery

Browse Source 
The commit handles the case that a host bridge device is not at 0:0.0 located.

Issue #1547
This commit is contained in:
Alexander Boettcher
2015-06-10 11:28:00 +02:00
committed by Christian Helmuth
parent f78aaf2234
commit c72ea84bcd
1 changed files with 27 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
repos/os/src/drivers/platform/x86/pci_session_component.h
View File

@ -661,6 +661,8 @@ namespace Platform {
{ {
using namespace Genode; using namespace Genode;
Config_access config_access;
try { try {
Xml_node xml_acpi(acpi_rom); Xml_node xml_acpi(acpi_rom);
if (!xml_acpi.has_type("acpi")) if (!xml_acpi.has_type("acpi"))
@ -711,7 +713,31 @@ namespace Platform {
node.attribute("device").value(&device); node.attribute("device").value(&device);
node.attribute("device_pin").value(&device_pin); node.attribute("device_pin").value(&device_pin);
Irq_routing::list()->insert(new (env()->heap()) Irq_routing(gsi, bridge_bdf, device, device_pin)); /* check that bridge bdf is actually a valid device */
Device_config config((bridge_bdf >> 8 & 0xff),
(bridge_bdf >> 3) & 0x1f,
bridge_bdf & 0x7,
&config_access);
if (config.valid()) {
if (!config.is_pci_bridge() && bridge_bdf != 0)
/**
* If the bridge bdf has not a type header
* of a bridge in the pci config space,
* then it should be the host bridge
* device. The host bridge device need not
* to be necessarily at 0:0.0, it may be
* on another location. The irq routing
* information for the host bridge however
* contain entries for the bridge bdf to be
* 0:0.0 - therefore we override it here
* for the irq rerouting information of
* host bridge devices.
*/
bridge_bdf = 0;
Irq_routing::list()->insert(new (env()->heap()) Irq_routing(gsi, bridge_bdf, device, device_pin));
}
} }
} }
} catch (...) { } catch (...) {