Differential D36386
libexec/rc: Add var_run rc script Authored by cy on Aug 28 2022, 1:00 PM. Tags None Referenced Files
Details Users with a tmpfs /var/run will lose the directory tree state of /var/run at reboot. This rc script will optionally (by default) capture the state of the directory structure in /var/run prior to shutdown and recreate it at system boot. Alternatively a user can save the state of the /var/run directories manually using service var_run save and disable the autosaving of /var/run state using the var_run_autosave variable, for those paranoid SSD users. Running here
Diff Detail
Event TimelineComment Actions This should probably be documented somewhere. I wonder if users who are interested in preserving /var/run across reboots shouldn't simply use a persistent filesystem rather than tmpfs though ...
Comment Actions /var/run is already in /var which is persistent on UFS/ZFS. Persistence is the default. This script is there to support those who have added tmpfs /var/run to their fstab making it ephemeral. The problem with ephemeral /var/run is that ports assume that /var/run is persistent. The other alternative is to convince ports maintainers to add mkdirs to their rc scripts and if the pkg doesn't provide an rc script, create one simply to make sure the directory their package needs is in /var/run. In short, this is to support those who have opted to make their FreeBSD systems non-standard. Maybe some documentation in the handbook showing how to set this up should a person choose to. But that would be a separate review. BTW, I did this same thing a week ago to maybe avoid a couple of writes here or there to my new SSD on my laptop. In real terms it probably won't make much difference though. Comment Actions The use case definitely makes sense! I think all that's needed is a couple of lines in share/man/man5/rc.conf.5. Maybe also add this (default NO) to libexec/rc/rc.conf with a one-line comment. Other than that, this looks good to me. Thank you! Comment Actions Looks good to me! Thank you. Last remaining question: why the REQUIRE: zfs? As I understand it, this will only work if zfs_enable="YES". I don't see anything here that would only work on ZFS.
Comment Actions Removed local quirks I had thought I'd already removed. REQUIRE and BEFORE are more specific: need mountcrictlocal and must be run before cleanvar because cleanvar touches /var/run.
| ||||||||||||||||||||||||||||||||||||||||||||||||||||