d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

bhyve: copy OpRegion into guest memory

Browse Source 
This makes the OpRegion accessible by the guest. However, the guest
doesn't know the address of the OpRegion. This will be fixed by an
upcoming commit.

The range of the OpRegion is added to the e820 table. This allows the
guest firmware to easily pick up this range and to reserve it properly.

Reviewed by:		markj
MFC after:		1 week
Sponsored by:		Beckhoff Automation GmbH & Co. KG
Differential Revision:	https://reviews.freebsd.org/D40041
This commit is contained in:
Corvin Köhne 2023-05-10 13:39:56 +02:00
parent 6952b9d25e
commit 1115cdcf7a
No known key found for this signature in database
GPG Key ID: D854DA56315E026A
1 changed files with 43 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
usr.sbin/bhyve/pci_gvt-d.c
View File

@ -222,8 +222,40 @@ gvt_d_setup_opregion(struct pci_devinst *const pi)
opregion->len = header->size * KB;
munmap(header, sizeof(header));
opregion->hva = mmap(NULL, opregion->len * KB, PROT_READ, MAP_SHARED,
memfd, opregion->hpa);
if (opregion->hva == MAP_FAILED) {
warn("%s: Unable to map host OpRegion", __func__);
close(memfd);
return (-1);
}
close(memfd);
opregion->gpa = gvt_d_alloc_mmio_memory(opregion->hpa, opregion->len,
E820_ALIGNMENT_NONE, E820_TYPE_NVS);
if (opregion->gpa == 0) {
warnx(
"%s: Unable to add OpRegion to E820 table (hpa 0x%lx len 0x%lx)",
__func__, opregion->hpa, opregion->len);
e820_dump_table();
return (-1);
}
opregion->gva = vm_map_gpa(pi->pi_vmctx, opregion->gpa, opregion->len);
if (opregion->gva == NULL) {
warnx("%s: Unable to map guest OpRegion", __func__);
return (-1);
}
if (opregion->gpa != opregion->hpa) {
/*
* A 1:1 host to guest mapping is not required but this could
* change in the future.
*/
warnx(
"Warning: Unable to reuse host address of OpRegion. GPU passthrough might not work properly.");
}
memcpy(opregion->gva, opregion->hva, opregion->len);
return (0);
}
@ -247,8 +279,18 @@ gvt_d_init(struct pci_devinst *const pi, nvlist_t *const nvl __unused)
}
static void
gvt_d_deinit(struct pci_devinst *const pi __unused)
gvt_d_deinit(struct pci_devinst *const pi)
{
struct passthru_softc *sc;
struct passthru_mmio_mapping *opregion;
sc = pi->pi_arg;
opregion = passthru_get_mmio(sc, GVT_D_MAP_OPREGION);
/* HVA is only set, if it's initialized */
if (opregion->hva)
munmap((void *)opregion->hva, opregion->len);
}
static struct passthru_dev gvt_d_dev = {