Page MenuHomeFreeBSD

vmm: Fix AMD-vi using wrong rid range
ClosedPublic

Authored by khng on Jun 20 2021, 11:39 AM.
Tags
None
Referenced Files
F80145678: D30827.id.diff
Thu, Mar 28, 1:25 PM
Unknown Object (File)
Feb 24 2024, 5:00 AM
Unknown Object (File)
Feb 24 2024, 5:00 AM
Unknown Object (File)
Feb 24 2024, 5:00 AM
Unknown Object (File)
Feb 24 2024, 5:00 AM
Unknown Object (File)
Feb 24 2024, 4:59 AM
Unknown Object (File)
Feb 24 2024, 4:59 AM
Unknown Object (File)
Feb 24 2024, 4:59 AM

Details

Summary

The ACPI parsing code around rid range was wrong on assuming there is
only one pair of start/end device id range. Besides, ivhd_dev_parse()
never work as supposed. The start/end rid info was always zero.

Restructure the code to build dynamic-sized tables for each IOMMU softc
holding device entries. The device entries are enumerated to find a
suitable IOMMU unit. Operations on devices not governed (e.g. the IOMMU
unit itself) are no-op from now on. There are also a minor fix on wrong
%b formatting string usage.

Tested on my EPYC 7282.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 39997
Build 36886: arc lint + arc unit

Event Timeline

khng requested review of this revision.Jun 20 2021, 11:39 AM

Handle ACPI_IVRS_TYPE_ALL as well.

sys/amd64/vmm/amd/amdvi_hw.c
1218

Hmmm, rev 3.* has been out for a while now: might be time to update spec references (just an fyi, not necessary for this review).

sys/amd64/vmm/amd/ivrs_drv.c
190

Test (or assignment) inverted ?

This revision is now accepted and ready to land.Jul 13 2021, 7:33 AM
This revision was automatically updated to reflect the committed changes.