GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

pciexp_device: Propagate above-4G flag to all hotplug devices 

The `IORESOURCE_ABOVE_4G` flag was only explicitly set for our dummy
device that reserves resources behind a hotplug port. The current re-
source allocator implicitly extends this to all devices below the port,
including real ones. Let's make that explicit, so future changes to the
allocator can't break this rule.

Change-Id: Id4c90b60682cf5c8949cde25362d286625b3e953
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66719
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
This commit is contained in:
Nico Huber 2022-08-15 00:08:58 +02:00 committed by Martin L Roth
parent 526c64249a
commit 577c6b9225
4 changed files with 20 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/device/device_util.c
View File

@ -5,6 +5,7 @@
#include <device/path.h>
#include <device/pci_def.h>
#include <device/resource.h>
#include <stdbool.h>
#include <stdlib.h>
#include <string.h>
@ -939,6 +940,15 @@ const char *dev_path_name(enum device_path_type type)
return type_name;
}
bool dev_path_hotplug(const struct device *dev)
{
for (dev = dev->bus->dev; dev != dev->bus->dev; dev = dev->bus->dev) {
if (dev->hotplug_port)
return true;
}
return false;
}
void log_resource(const char *type, const struct device *dev, const struct resource *res,
const char *srcfile, const int line)
{

6
src/device/pci_device.c
View File

@ -164,8 +164,12 @@ struct resource *pci_get_resource(struct device *dev, unsigned long index)
/* A Memory mapped base address. */
attr &= PCI_BASE_ADDRESS_MEM_ATTR_MASK;
resource->flags |= IORESOURCE_MEM;
if (attr & PCI_BASE_ADDRESS_MEM_PREFETCH)
if (attr & PCI_BASE_ADDRESS_MEM_PREFETCH) {
resource->flags |= IORESOURCE_PREFETCH;
if (CONFIG(PCIEXP_HOTPLUG_PREFETCH_MEM_ABOVE_4G)
&& dev_path_hotplug(dev))
resource->flags |= IORESOURCE_ABOVE_4G;
}
attr &= PCI_BASE_ADDRESS_MEM_LIMIT_MASK;
if (attr == PCI_BASE_ADDRESS_MEM_LIMIT_32) {
/* 32bit limit. */

1
src/device/pciexp_device.c
View File

@ -646,6 +646,7 @@ static struct device_operations pciexp_hotplug_dummy_ops = {
void pciexp_hotplug_scan_bridge(struct device *dev)
{
dev->hotplug_port = 1;
dev->hotplug_buses = CONFIG_PCIEXP_HOTPLUG_BUSES;
/* Normal PCIe Scan */

4
src/include/device/device.h
View File

@ -128,6 +128,7 @@ struct device {
unsigned int hidden : 1;
/* set if this device is used even in minimum PCI cases */
unsigned int mandatory : 1;
unsigned int hotplug_port : 1;
u8 command;
uint16_t hotplug_buses; /* Number of hotplug buses to allocate */
@ -207,6 +208,9 @@ void add_more_links(struct device *dev, unsigned int total_links);
bool is_dev_enabled(const struct device *const dev);
bool is_devfn_enabled(unsigned int devfn);
/* Returns whether there is a hotplug port on the path to the given device. */
extern bool dev_path_hotplug(const struct device *);
/* Option ROM helper functions */
void run_bios(struct device *dev, unsigned long addr);