loader.conf/.local cleanup is a bit too aggressive
We have code that cleans up and eliminates duplicate settings in loader.conf and loader.conf.local to avoid foot-shooting and to make sure that hardware works properly.
The code that checks/matches these values is a bit too aggressive and over-matches, and removes some values from loader.conf.local that it makes sense to allow a user to override.
The matching could be improved, and it could handle the local case slightly differently so that some values can be overridden by users.
Rework loader.conf(.local) filtering. Fixes #8571
If this isn't aggressive enough, we could remove the "local" changes and only
keep the new matching method.
#2 Updated by James Dekker 9 months ago
On 2.4.4.a.20180705.0032 (SG-2440) added autoboot_delay=10 to /boot/loader.conf.local (which didn't exist before that), rebooted a couple of times with option 5 at console and the file remained with the contents unchanged. For the loader.conf part, saved settings on the System > Advanced page and after that the contents of /boot/loader.conf remained unchanged through a couple of reboots.