Page MenuHomeFreeBSD

libvmmapi: Make vm_raise_msi() a common function
ClosedPublic

Authored by markj on Sep 6 2023, 12:29 AM.
Tags
None
Referenced Files
F85202316: D41752.diff
Mon, Jun 3, 12:25 AM
Unknown Object (File)
Sun, May 19, 7:23 PM
Unknown Object (File)
Apr 21 2024, 10:44 AM
Unknown Object (File)
Apr 11 2024, 11:40 AM
Unknown Object (File)
Apr 11 2024, 10:14 AM
Unknown Object (File)
Apr 10 2024, 4:13 PM
Unknown Object (File)
Apr 8 2024, 4:26 PM
Unknown Object (File)
Apr 8 2024, 3:21 PM
Subscribers

Details

Summary

Currently, bhyve PCI emulation uses vm_lapic_msi() to raise an MSI in
the guest. The arm64 port has a similar function, vm_raise_msi().
Add vm_raise_msi() on amd64 as well and have it simply call
vm_lapic_msi() so that bhyve can use a common, generically named
function.

For some reason the arm64 VM_RAISE_MSI ioctl takes a PCI device
identifier, but it is unused. Simply remove those parameters from the
arm64 prototype for now.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 53455
Build 50346: arc lint + arc unit

Event Timeline

markj requested review of this revision.Sep 6 2023, 12:29 AM
corvink added inline comments.
lib/libvmmapi/vmmapi.h
170

Nit: This doesn't belong to your change.

This revision is now accepted and ready to land.Sep 6 2023, 9:31 AM
lib/libvmmapi/aarch64/vmmapi_machdep.c
109

If we implement the GICv3 ITS we will likely need some of these fields.

Restore PCI b/s/f tuple parameters.

This revision now requires review to proceed.Sep 6 2023, 2:15 PM

Don't forget to drop the last paragraph from the commit log.

This revision is now accepted and ready to land.Sep 6 2023, 3:14 PM