I think it's fine to iterate on this in the tree since it is not active by default (with static int amdiommu_enable = 0;). There are a handful of XXXs remaining but I think it's reasonable to be able to cast a wider net for testing.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 2 2024, 11:47 PM

Closed by commit rG0f5116d7efe3: AMD IOMMU driver (authored by kib). · Explain Why

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG0f5116d7efe3: AMD IOMMU driver.

Revision Contents
Changeset List

Path

Size

sys/

conf/

files.x86

6 lines

x86/

iommu/

360 lines

639 lines

1205 lines

323 lines

396 lines

391 lines

243 lines

Diff 145910

View Options

sys/conf/files.x86

View Options

sys/x86/iommu/amd_cmd.c

View Options

sys/x86/iommu/amd_ctx.c

View Options

sys/x86/iommu/amd_drv.c

View Options

sys/x86/iommu/amd_event.c

View Options

sys/x86/iommu/amd_idpgtbl.c

View Options

sys/x86/iommu/amd_intrmap.c

View Options

AMD IOMMU driverClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 145910

sys/conf/files.x86

sys/x86/iommu/amd_cmd.c

sys/x86/iommu/amd_ctx.c

sys/x86/iommu/amd_drv.c

sys/x86/iommu/amd_event.c

sys/x86/iommu/amd_idpgtbl.c

sys/x86/iommu/amd_intrmap.c

sys/x86/iommu/amd_iommu.h

AMD IOMMU driver
ClosedPublic
Actions

Revision Contents
Changeset List