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
Unknown Object (File)
Fri, Jun 13, 4:01 PM
Unknown Object (File)
Fri, Jun 13, 3:11 PM
Unknown Object (File)
Thu, Jun 12, 3:12 PM
Unknown Object (File)
Wed, Jun 11, 9:30 AM
Unknown Object (File)
Mon, Jun 9, 2:42 PM
Unknown Object (File)
Fri, Jun 6, 11:49 AM
Unknown Object (File)
Tue, Jun 3, 11:43 AM
Unknown Object (File)
Mon, Jun 2, 8:24 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.