summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafał Miłecki <rafal@milecki.pl>2022-03-01 18:46:27 +0100
committerRafał Miłecki <rafal@milecki.pl>2022-03-15 10:15:39 +0100
commit6b8407c6da66bd2d42f9300f82532fd93dab56d9 (patch)
tree7c4788dc8695d67df0a6cd9dedccc7a3e61a5bc9
parent9ced994057ae8243445d3f22aa981753de4f7d34 (diff)
downloadopenwrt-6b8407c6da66bd2d42f9300f82532fd93dab56d9.tar.gz
openwrt-6b8407c6da66bd2d42f9300f82532fd93dab56d9.tar.bz2
openwrt-6b8407c6da66bd2d42f9300f82532fd93dab56d9.zip
base-files: call "sync" after initial setup
OpenWrt uses a lot of (b)ash scripts for initial setup. This isn't the best solution as they almost never consider syncing files / data. Still this is what we have and we need to try living with it. Without proper syncing OpenWrt can easily get into an inconsistent state on power cut. It's because: 1. Actual (flash) inode and data writes are not synchronized 2. Data writeback can take up to 30 seconds (dirty_expire_centisecs) 3. ubifs adds extra 5 seconds (dirty_writeback_centisecs) "delay" Some possible cases (examples) for new files: 1. Power cut during 5 seconds after write() can result in all data loss 2. Power cut happening between 5 and 35 seconds after write() can result in empty file (inode flushed after 5 seconds, data flush queued) Above affects e.g. uci-defaults. After executing some migration script it may get deleted (whited out) without generated data getting actually written. Power cut will result in missing data and deleted file. There are three ways of dealing with that: 1. Rewriting all user-space init to proper C with syncs 2. Trying bash hacks (like creating tmp files & moving them) 3. Adding sync and hoping for no power cut during critical section This change introduces the last solution that is the simplest. It reduces time during which things may go wrong from ~35 seconds to probably less than a second. Of course it applies only to IO operations performed before /etc/init.d/boot . It's probably the stage when the most new files get created. All later changes are usually done using smarter C apps (e.g. busybox or uci) that creates tmp files and uses rename() that is expected to be atomic. Signed-off-by: Rafał Miłecki <rafal@milecki.pl> Acked-by: Hauke Mehrtens <hauke@hauke-m.de> Acked-by: Sergey Ryazanov <ryazanov.s.a@gmail.com> (cherry picked from commit 9851d4b6ce6e89d164a04803817625a9041b060a)
-rwxr-xr-xpackage/base-files/files/etc/init.d/boot1
1 files changed, 1 insertions, 0 deletions
diff --git a/package/base-files/files/etc/init.d/boot b/package/base-files/files/etc/init.d/boot
index 09b7f44c79..b7e62ff187 100755
--- a/package/base-files/files/etc/init.d/boot
+++ b/package/base-files/files/etc/init.d/boot
@@ -45,6 +45,7 @@ boot() {
/bin/config_generate
uci_apply_defaults
+ sync
# temporary hack until configd exists
/sbin/reload_config