Page MenuHomeFreeBSD

iommu_unmap_msi() added
ClosedPublic

Authored by br on Oct 22 2020, 5:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 4:25 PM
Unknown Object (File)
Mon, Apr 22, 12:23 AM
Unknown Object (File)
Mar 20 2024, 10:53 PM
Unknown Object (File)
Mar 20 2024, 10:53 PM
Unknown Object (File)
Mar 20 2024, 10:53 PM
Unknown Object (File)
Mar 20 2024, 10:53 PM
Unknown Object (File)
Mar 20 2024, 10:53 PM
Unknown Object (File)
Mar 19 2024, 4:48 PM
Subscribers

Details

Summary

Add iommu de-initialization method for MSI.

This is needed to free up all the entries in GAS tree before iommu domain teardown and pmap release

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

br requested review of this revision.Oct 22 2020, 5:04 PM
sys/arm64/arm64/gicv3_its.c
1469 ↗(On Diff #78605)

This blank line is useless.

sys/dev/iommu/iommu_gas.c
738 ↗(On Diff #78605)

I think there should be a check entry != NULL and early return. Or assert that entry != NULL.

o Check if entry is not NULL, otherwise return early
o Remove gicv3_its.c changes from this diff. They are moved to https://reviews.freebsd.org/D26878

This revision is now accepted and ready to land.Oct 23 2020, 10:43 PM
sys/kern/subr_intr.c
1380 ↗(On Diff #78665)

BTW, what about providing default implementations for iommu_init/deinit, and then dropping #ifdef IOMMU there ?

Add default implementations of iommu_init/deinit

This revision now requires review to proceed.Oct 24 2020, 11:17 AM
This revision is now accepted and ready to land.Oct 24 2020, 4:59 PM
This revision was automatically updated to reflect the committed changes.