]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Add pci_early function to detect Intel stolen memory.
authorkib <kib@FreeBSD.org>
Wed, 31 Oct 2018 23:17:00 +0000 (23:17 +0000)
committerkib <kib@FreeBSD.org>
Wed, 31 Oct 2018 23:17:00 +0000 (23:17 +0000)
commitb00247de3f3c568d8630c04b651f5d87894ce65d
tree796bce4d51a3cc4bdc92da995eec251bbe34834e
parent371f3d1fe1a0f0bc9659b23848c432e1b2f22481
Add pci_early function to detect Intel stolen memory.

On some Intel devices BIOS does not properly reserve memory (called
"stolen memory") for the GPU.  If the stolen memory is claimed by the
OS, functions that depend on stolen memory (like frame buffer
compression) can't be used.

A function called pci_early_quirks that is called before the virtual
memory system is started was added. In Linux, this PCI early quirks
function iterates through all PCI slots to check for any device that
require quirks.  While this more generic solution is preferable I only
ported the Intel graphics specific parts because I think my
implementation would be too similar to Linux GPL'd solution after
looking at the Linux code too much.

The code regarding Intel graphics stolen memory was ported from
Linux. In the case of Intel graphics stolen memory this
pci_early_quirks will read the stolen memory base and size from north
bridge registers.  The values are stored in global variables that is
later read by linuxkpi_gplv2. Linuxkpi stores these values in a
Linux-specific structure that is read by the drm driver.

Relevant linuxkpi code is here:
https://github.com/FreeBSDDesktop/kms-drm/blob/drm-v4.16/linuxkpi/gplv2/src/linux_compat.c#L37

For now, only amd64 arch is suppor ted since that is the only arch
supported by the new drm drivers. I was told that Intel GPUs are
always located on 0:2:0 so these values are hard coded for now.

Note that the structure and early execution of the detection code is
not required in its current form, but we expect that the code will be
added shortly which fixes the potential BIOS bugs by reserving the
stolen range in phys_avail[].  This must be done as early as possible
to avoid conflicts with the potential usage of the memory in kernel.

Submitted by: Johannes Lundberg <johalun0@gmail.com>
Reviewed by: bwidawsk, imp
MFC after: 1 week
Differential revision: https://reviews.freebsd.org/D16719
Differential revision: https://reviews.freebsd.org/D17775
sys/amd64/amd64/machdep.c
sys/amd64/include/md_var.h
sys/conf/files.amd64
sys/x86/pci/pci_early_quirks.c [new file with mode: 0644]
sys/x86/pci/pci_early_quirks.h [new file with mode: 0644]