device: Add support for PCIe Resizable BARs
Section 7.8.6 of the PCIe spec (rev 4) indicates that some devices can
indicates support for "Resizable BARs" via a PCIe extended capability.
When support this capability is indicated by the device, the size of
each BAR is determined in a different way than the normal "moving
bits" method. Instead, a pair of capability and control registers is
allocated in config space for each BAR, which can be used to both
indicate the different sizes the device is capable of supporting for
the BAR (powers-of-2 number of bits from 20 [1 MiB] to 63 [8 EiB]), and
to also inform the device of the size that the allocator actually
reserved for the MMIO range.
This patch adds a Kconfig for a mainboard to select if it knows that it
will have a device that requires this support during PCI enumeration.
If so, there is a corresponding Kconfig to indicate the maximum number
of bits of address space to hand out to devices this way (again, limited
by what devices can support and each individual system may want to
support, but just like above, this number can range from 20 to 63) If
the device can support more bits than this Kconfig, the resource request
is truncated to the number indicated by this Kconfig.
BUG=b:214443809
TEST=compile (device with this capability not available yet),
also verify that no changes are seen in resource allocation for
google/brya0 before and after this change.
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Change-Id: I14fcbe0ef09fdc7f6061bcf7439d1160d3bc4abf
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61215
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 5938f0d..2acc517 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -19,6 +19,7 @@
#include <device/pci_ids.h>
#include <device/pcix.h>
#include <device/pciexp.h>
+#include <lib.h>
#include <pc80/i8259.h>
#include <security/vboot/vbnv.h>
#include <timestamp.h>
@@ -309,6 +310,127 @@
return (struct msix_entry *)((uintptr_t)res->base + offset);
}
+static unsigned int get_rebar_offset(const struct device *dev, unsigned long index)
+{
+ uint32_t offset = pciexp_find_extended_cap(dev, PCIE_EXT_CAP_RESIZABLE_BAR);
+ if (!offset)
+ return 0;
+
+ /* Convert PCI_BASE_ADDRESS_0, ..._1, ..._2 into 0, 1, 2... */
+ const unsigned int find_bar_idx = (index - PCI_BASE_ADDRESS_0) /
+ sizeof(uint32_t);
+
+ /* Although all of the Resizable BAR Control Registers contain an
+ "NBARs" field, it is only valid in the Control Register for BAR 0 */
+ const uint32_t rebar_ctrl0 = pci_read_config32(dev, offset + PCI_REBAR_CTRL_OFFSET);
+ const unsigned int nbars = (rebar_ctrl0 & PCI_REBAR_CTRL_NBARS_MASK) >>
+ PCI_REBAR_CTRL_NBARS_SHIFT;
+
+ for (unsigned int i = 0; i < nbars; i++, offset += sizeof(uint64_t)) {
+ const uint32_t rebar_ctrl = pci_read_config32(
+ dev, offset + PCI_REBAR_CTRL_OFFSET);
+ const uint32_t bar_idx = rebar_ctrl & PCI_REBAR_CTRL_IDX_MASK;
+ if (bar_idx == find_bar_idx)
+ return offset;
+ }
+
+ return 0;
+}
+
+/* Bit 20 = 1 MiB, bit 21 = 2 MiB, bit 22 = 4 MiB, ... bit 63 = 8 EiB */
+static uint64_t get_rebar_sizes_mask(const struct device *dev,
+ unsigned long index)
+{
+ uint64_t size_mask = 0ULL;
+ const uint32_t offset = get_rebar_offset(dev, index);
+ if (!offset)
+ return 0;
+
+ /* Get 1 MB - 128 TB support from CAP register */
+ const uint32_t cap = pci_read_config32(dev, offset + PCI_REBAR_CAP_OFFSET);
+ /* Shift the bits from 4-31 to 0-27 (i.e., down by 4 bits) */
+ size_mask |= ((cap & PCI_REBAR_CAP_SIZE_MASK) >> 4);
+
+ /* Get 256 TB - 8 EB support from CTRL register and store it in bits 28-43 */
+ const uint64_t ctrl = pci_read_config32(dev, offset + PCI_REBAR_CTRL_OFFSET);
+ /* Shift ctrl mask from bit 16 to bit 28, so that the two
+ masks (fom cap and ctrl) form a contiguous bitmask when
+ concatenated (i.e., up by 12 bits). */
+ size_mask |= ((ctrl & PCI_REBAR_CTRL_SIZE_MASK) << 12);
+
+ /* Now that the mask occupies bits 0-43, shift it up to 20-63, so they
+ represent the actual powers of 2. */
+ return size_mask << 20;
+}
+
+static void pci_store_rebar_size(const struct device *dev,
+ const struct resource *resource)
+{
+ const unsigned int num_bits = __fls64(resource->size);
+ const uint32_t offset = get_rebar_offset(dev, resource->index);
+ if (!offset)
+ return;
+
+ pci_update_config32(dev, offset + PCI_REBAR_CTRL_OFFSET,
+ ~PCI_REBAR_CTRL_SIZE_MASK,
+ num_bits << PCI_REBAR_CTRL_SIZE_SHIFT);
+}
+
+static void configure_adjustable_base(const struct device *dev,
+ unsigned long index,
+ struct resource *res)
+{
+ /*
+ * Excerpt from an implementation note from the PCIe spec:
+ *
+ * System software uses this capability in place of the above mentioned
+ * method of determining the resource size[0], and prior to assigning
+ * the base address to the BAR. Potential usable resource sizes are
+ * reported by the Function via its Resizable BAR Capability and Control
+ * registers. It is intended that the software allocate the largest of
+ * the reported sizes that it can, since allocating less address space
+ * than the largest reported size can result in lower
+ * performance. Software then writes the size to the Resizable BAR
+ * Control register for the appropriate BAR for the Function. Following
+ * this, the base address is written to the BAR.
+ *
+ * [0] Referring to using the moving bits in the BAR to determine the
+ * requested size of the MMIO region
+ */
+ const uint64_t size_mask = get_rebar_sizes_mask(dev, index);
+ if (!size_mask)
+ return;
+
+ int max_requested_bits = __fls64(size_mask);
+ if (max_requested_bits > CONFIG_PCIEXP_DEFAULT_MAX_RESIZABLE_BAR_BITS) {
+ printk(BIOS_WARNING, "WARNING: Device %s requests a BAR with"
+ "%u bits of address space, which coreboot is not"
+ "configured to hand out, truncating to %u bits\n",
+ dev_path(dev), max_requested_bits,
+ CONFIG_PCIEXP_DEFAULT_MAX_RESIZABLE_BAR_BITS);
+ max_requested_bits = CONFIG_PCIEXP_DEFAULT_MAX_RESIZABLE_BAR_BITS;
+ }
+
+ if (!(res->flags & IORESOURCE_PCI64) && max_requested_bits > 32) {
+ printk(BIOS_ERR, "ERROR: Resizable BAR requested"
+ "above 32 bits, but PCI function reported a"
+ "32-bit BAR.");
+ return;
+ }
+
+ /* Configure the resource parameters for the adjustable BAR */
+ res->size = 1ULL << max_requested_bits;
+ res->align = max_requested_bits;
+ res->gran = max_requested_bits;
+ res->limit = (res->flags & IORESOURCE_PCI64) ? UINT64_MAX : UINT32_MAX;
+ res->flags |= IORESOURCE_PCIE_RESIZABLE_BAR;
+
+ printk(BIOS_INFO, "%s: Adjusting resource index %lu: base: %llx size: %llx "
+ "align: %d gran: %d limit: %llx\n",
+ dev_path(dev), res->index, res->base, res->size,
+ res->align, res->gran, res->limit);
+}
+
/**
* Read the base address registers for a given device.
*
@@ -323,6 +445,11 @@
(index < PCI_BASE_ADDRESS_0 + (howmany << 2));) {
struct resource *resource;
resource = pci_get_resource(dev, index);
+
+ const bool is_pcie = pci_find_capability(dev, PCI_CAP_ID_PCIE) != 0;
+ if (CONFIG(PCIEXP_SUPPORT_RESIZABLE_BARS) && is_pcie)
+ configure_adjustable_base(dev, index, resource);
+
index += (resource->flags & IORESOURCE_PCI64) ? 8 : 4;
}
@@ -547,11 +674,17 @@
/* Now store the resource. */
resource->flags |= IORESOURCE_STORED;
- if (resource->flags & IORESOURCE_PCI_BRIDGE)
- pci_store_bridge_resource(dev, resource);
- else
+ if (!(resource->flags & IORESOURCE_PCI_BRIDGE)) {
+ if (CONFIG(PCIEXP_SUPPORT_RESIZABLE_BARS) &&
+ (resource->flags & IORESOURCE_PCIE_RESIZABLE_BAR))
+ pci_store_rebar_size(dev, resource);
+
pci_store_resource(dev, resource);
+ } else {
+ pci_store_bridge_resource(dev, resource);
+ }
+
report_resource_stored(dev, resource, "");
}