Project

General

Profile

Actions

Bug #8949

open

support pcix/pciex extended iospace

Added by Toomas Soome over 5 years ago. Updated about 4 years ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
kernel
Start date:
2018-01-05
Due date:
% Done:

90%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:
External Bug:

Description

A bit cryptic message on boot:

WARNING: illegal PCI request: offset = 100, size = 4

The early PCI setup is using inb/inw/inl based IO access to PCI config space (as opposed to memory mapped reads) and the access is validated against pci_iocfg_max_offset global variable, which is set to value 0xff. Unfortunately the PCI-X 2.0 and PCIE extended config space is starting from offset 0x100 and the space is 4k, and accessing the extended config space will result with validation failure and the warning message above.

The quick fix is to set (for mech1) pci_iocfg_max_offset = 0xfff; So the extended config space access will be allowed. Note, the similar workaround is already in place for AMD.

The better approach would be to determine the cfg space size for each case and use that value in validation, but I would implement this quick fix for now, so we would get access to devices and address the proper fix later (especially considering the pci code is shared with sparc px driver and there are probably more complications).

Tested with qemu 2.10.1 + UEFI, this fix does enable nvme devices to attach.


Related issues

Related to illumos gate - Feature #11050: Startech USB 3.0 PCI Host 4 Port Adapter supportClosed2019-05-19

Actions
Actions #1

Updated by Michal Nowak about 4 years ago

  • Related to Feature #11050: Startech USB 3.0 PCI Host 4 Port Adapter support added
Actions #2

Updated by Michal Nowak about 4 years ago

OpenIndiana user at #11050 got the very same warning when they attached PCIe card, an xHCI adapter. The card seems to be misconfigured. Might be worth reiterating the proposed fix as we have a reproducer.

Actions #3

Updated by Toomas Soome about 4 years ago

Michal Nowak wrote:

OpenIndiana user at #11050 got the very same warning when they attached PCIe card, an xHCI adapter. The card seems to be misconfigured. Might be worth reiterating the proposed fix as we have a reproducer.

It is not forgotten, just postponed - the very first implementation is of course too crude, but proper fix needs time...

Actions

Also available in: Atom PDF