Change DMAR sysctl node

[br]

Summary

Change all _hw_dmar to _hw_iommu

Test Plan

Boot on amd64

[kib]

sys/dev/iommu/iommu_gas.c

736 ↗

(On Diff #74933)

Mmm, I think the node should be declared unconditionally, regardless of the INVARIANTS build. Also I remember that nodes should be RW if any mib below them is rw.

sys/x86/iommu/intel_utils.c

658 ↗

(On Diff #74933)

This should not link in non-INVARIANTS kernels.

Also, I asked to put dmar-specific mibs under hw.iommu.dmar.

[br]

o _hw_iommu is RW
o _hw_iommu is declared regardless INVARIANTS support
o dmar sysctl options are under _hw_iommu_dmar, not _hw_iommu
o _hw_iommu_dmar are all INVARIANTS only

sys/dev/iommu/iommu_gas.c

736 ↗

(On Diff #74933)

All fixed. sorry

[kib]

sys/dev/iommu/iommu_gas.c

738 ↗

(On Diff #74935)

This one should be under INVARIANTS indeed.

sys/x86/iommu/intel_utils.c

671 ↗

(On Diff #74935)

None of these sysctls should be under INVARIANTS. They are all either informational or knobs to tune performance.

[br]

sysctl knobs present in all kernels, regardless of INVARIANTS

[br]

sys/x86/iommu/intel_utils.c

658 ↗

(On Diff #74933)

what do you mean that it should not link (in non-invariants kernel) ?

[kib]

sys/x86/iommu/intel_utils.c

658 ↗

(On Diff #74933)

I mean that for non-INVARIANTS kernels the node (or whatever, I do not remember already) was defined only under INVARIANTS (i.e. nowhere), while declared and used. Then linker should compain about undefined symbol.