Hello build bot (Jenkins), Nico Huber, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/42459
to look at the new patch set (#7).
Change subject: [RFC] device: Add method for configuring bus master based on config option ......................................................................
[RFC] device: Add method for configuring bus master based on config option
The bus master bit is set at many places in coreboot's code, but the reason for that is not quite clear. We examined not setting the bus master bit whereever possible and tried booting without it, which worked fine for internal PCI devices but not for PCIe. As a PCIe device we used a Samsung M.2 NVMe SSD.
For security reasons, we would like to disable bus mastering where possible. But depending on the device, bus mastering might get enabled by the operating system (e.g. for iGPU) and it might be required for some devices to work properly. However, the idea is to leave it disabled and configure IOMMU first before enabling it.
To have some sort of "backwards compatibility", add a method which configures the bus master bit based on an additional config option and enable it by default to keep the current behaviour.
Tested with Siemens/Chili, a Coffee Lake based platform.
Change-Id: I876c48ea3fb4f9cf7b6a5c2dcaeda07ea36cbed3 Signed-off-by: Felix Singer felix.singer@secunet.com --- M src/device/Kconfig M src/device/pci_device.c M src/include/device/pci.h 3 files changed, 11 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/42459/7