]> git.proxmox.com Git - mirror_edk2.git/commit - OvmfPkg/OvmfPkg.dec
OvmfPkg: introduce PciCapPciSegmentLib
authorLaszlo Ersek <lersek@redhat.com>
Sat, 28 Apr 2018 21:22:59 +0000 (23:22 +0200)
committerLaszlo Ersek <lersek@redhat.com>
Thu, 24 May 2018 19:13:11 +0000 (21:13 +0200)
commit6a744d40d0c707b43202ae9b6a1d6ae89f6f864b
tree075eb52a8a5a273db61483188ec0078b2569268d
parent392a31467f4164423ae459bf28b39f282bb48e98
OvmfPkg: introduce PciCapPciSegmentLib

Add a library class, and a BASE lib instance, that are layered on top of
PciCapLib, and allow clients to plug a PciSegmentLib backend into
PciCapLib, for config space access.

(Side note:

The "MaxDomain" parameter is provided because, in practice, platforms
exist where a PCI Express device may show up on a root bridge such that
the root bridge doesn't support access to extended config space. Earlier
the same issue was handled for MdeModulePkg/PciHostBridgeDxe in commit
014b472053ae3. However, that solution does not apply to the PciSegmentLib
class, because:

(1) The config space accessor functions of the PciSegmentLib class, such
    as PciSegmentReadBuffer(), have no way of informing the caller whether
    access to extended config space actually succeeds.

    (For example, in the UefiPciSegmentLibPciRootBridgeIo instace, which
    could in theory benefit from commit 014b472053ae3, the
    EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.Pci.Read() status code is explicitly
    ignored, because there's no way for the lib instance to propagate it
    to the PciSegmentLib caller. If the
    EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL.Pci.Read() call fails, then
    DxePciSegmentLibPciRootBridgeIoReadWorker() returns Data with
    indeterminate value.)

(2) There is no *general* way for any firmware platform to provide, or
    use, a PciSegmentLib instance in which access to extended config space
    always succeeds.

In brief, on a platform where config space may be limited to 256 bytes,
access to extended config space through PciSegmentLib may invoke undefined
behavior; therefore PciCapPciSegmentLib must give platforms a way to
prevent such access.)

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
OvmfPkg/Include/Library/PciCapPciSegmentLib.h [new file with mode: 0644]
OvmfPkg/Library/BasePciCapPciSegmentLib/BasePciCapPciSegmentLib.c [new file with mode: 0644]
OvmfPkg/Library/BasePciCapPciSegmentLib/BasePciCapPciSegmentLib.h [new file with mode: 0644]
OvmfPkg/Library/BasePciCapPciSegmentLib/BasePciCapPciSegmentLib.inf [new file with mode: 0644]
OvmfPkg/OvmfPkg.dec