Page MenuHomeFreeBSD

Remove headers from iommu.h
ClosedPublic

Authored by br on Jul 29 2020, 1:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 20 2024, 10:46 PM
Unknown Object (File)
Nov 19 2024, 4:42 PM
Unknown Object (File)
Nov 19 2024, 8:38 AM
Unknown Object (File)
Oct 5 2024, 6:13 PM
Unknown Object (File)
Sep 16 2024, 8:16 PM
Unknown Object (File)
Sep 16 2024, 8:16 PM
Unknown Object (File)
Sep 16 2024, 8:16 PM
Unknown Object (File)
Sep 16 2024, 8:16 PM
Subscribers

Details

Summary

Remove headers from iommu.h

Test Plan

boot amd64

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

br requested review of this revision.Jul 29 2020, 1:43 PM
br created this revision.
sys/dev/iommu/iommu_gas.c
72

This is quite weird #ifdef block. Why pci and iommu are under ifdef ?

sys/x86/iommu/intel_drv.c
72

I propose to make the order of headers consistent. Put dev/pci right after vm/, then dev/iommu, then x86 headers.

sys/x86/iommu/intel_drv.c
72

dev/iommu/busdma_iommu.h depends on struct bus_dma_tag_common, which is defined in x86/include/busdma_impl.h.
so x86 header should go first ?

sys/x86/iommu/intel_drv.c
72

Might be. What provides a similar common structure on arm64 ?

Fix order of headers:
sys/vm/contrib/dev/machine/x86

sys/x86/iommu/intel_drv.c
72

arm64/include/bus_dma_impl.h

This revision is now accepted and ready to land.Jul 29 2020, 9:48 PM