ar71xx: Create /var/lock directory in platform_do_upgrade_*
authorSven Eckelmann <sven.eckelmann@openmesh.com>
Mon, 1 Oct 2018 09:48:04 +0000 (11:48 +0200)
committerMathias Kresin <dev@kresin.me>
Sun, 7 Oct 2018 13:44:19 +0000 (15:44 +0200)
The sysupgrade_pre_upgrade hook was removed with 5e1b4c57ded7 ("base-files:
drop fwtool_pre_upgrade") while there were still scripts using it:

* target/linux/ar71xx/base-files/lib/upgrade/allnet.sh
* target/linux/ar71xx/base-files/lib/upgrade/openmesh.sh
* target/linux/ipq40xx/base-files/lib/upgrade/openmesh.sh

Not running the hooks can either prevent a successful upgrade or brick the
device because the fw_setenv program cannot be started correctly.

Instead of adding this hook again, the directory /var/lock for fw_setenv
can also just be created directly before fw_setenv is called.

Fixes: 5e1b4c57ded7 ("base-files: drop fwtool_pre_upgrade")
Signed-off-by: Sven Eckelmann <sven.eckelmann@openmesh.com>
target/linux/ar71xx/base-files/lib/upgrade/allnet.sh
target/linux/ar71xx/base-files/lib/upgrade/openmesh.sh

index c55e7bc062ad20956e1a1232cd819e310e79a67c..5295d8cfe0f2ed34a06ac106254d5aafbd5addc6 100644 (file)
@@ -3,14 +3,6 @@
 # In case the check fails during boot, a failsafe-system is started to provide
 # a minimal web-interface for flashing a new firmware.
 
-# create /var/lock for the lock "fw_setenv.lock" of fw_setenv
-# the rest is copied using ar71xx's RAMFS_COPY_BIN and RAMFS_COPY_DATA
-platform_add_ramfs_ubootenv()
-{
-       mkdir -p $RAM_ROOT/var/lock
-}
-append sysupgrade_pre_upgrade platform_add_ramfs_ubootenv
-
 # determine size of the main firmware partition
 platform_get_firmware_size() {
        local dev size erasesize name
@@ -152,6 +144,8 @@ rootfs_size $rootfs_hexsize
 rootfs_checksum $rootfs_md5
 bootcmd bootm $vmlinux_hexaddr
 EOF
+
+       mkdir -p /var/lock
        fw_setenv -s /tmp/fw_env_upgrade || {
                echo "failed to update U-Boot environment"
                return 1
index e24a6590263473bd31972ba6c1e3c2a7015540ea..65c624d86cf9d36790b73c354ea0392d10e086a6 100644 (file)
@@ -26,14 +26,6 @@ cfg_value_get()
                done
 }
 
-# create /var/lock for the lock "fw_setenv.lock" of fw_setenv
-# the rest is copied using ar71xx's RAMFS_COPY_BIN and RAMFS_COPY_DATA
-platform_add_ramfs_ubootenv()
-{
-       mkdir -p $RAM_ROOT/var/lock
-}
-append sysupgrade_pre_upgrade platform_add_ramfs_ubootenv
-
 platform_check_image_target_openmesh()
 {
        img_board_target="$1"
@@ -232,6 +224,7 @@ platform_do_upgrade_openmesh()
        printf "rootfs_size %s\n" $rootfs_checksize >> $uboot_env_upgrade
        printf "rootfs_checksum %s\n" $rootfs_md5 >> $uboot_env_upgrade
 
+       mkdir -p /var/lock
        fw_setenv -s $uboot_env_upgrade || {
                echo "failed to update U-Boot environment"
                return 1