subr_devmap: Reduce the use of the static devmap

[andrew]

Summary

We only create the static devmap on arm. Stop building this code on
other architectures.

Sponsored by: Arm Ltd

[mhorne]

Thanks, this is great.

To me it is preferable to replace the repeated #ifdef __arm__ instances with a named flag, e.g. #ifdef DEVMAP_HAS_STATIC. If it is annoying to achieve then no problem.

sys/kern/subr_devmap.c

236

Best to indicate the condition at the end of a long section.

[jhb]

In D50016#1141998, @mhorne wrote:

Thanks, this is great.

To me it is preferable to replace the repeated #ifdef __arm__ instances with a named flag, e.g. #ifdef DEVMAP_HAS_STATIC. If it is annoying to achieve then no problem.

We could add a __HAVE_STATIC_DEVMAP in <machine/param.h>. Similar to what we do for __HAVE_ACPI.

[imp]

In D50016#1142030, @jhb wrote:

In D50016#1141998, @mhorne wrote:

Thanks, this is great.

To me it is preferable to replace the repeated #ifdef __arm__ instances with a named flag, e.g. #ifdef DEVMAP_HAS_STATIC. If it is annoying to achieve then no problem.

We could add a __HAVE_STATIC_DEVMAP in <machine/param.h>. Similar to what we do for __HAVE_ACPI.

I love it. I wish we did this more often, honestly, but in the early days that was shouted down. While it may
cause some churn as we may eventually have all of our architectures on or off, that should be minimal.

[andrew]

Add __HAVE_STATIC_DEVMAP