From: Sergey Ryazanov Date: Tue, 10 Oct 2017 22:07:08 +0000 (+0300) Subject: firmware-utils: mktplinkfw: fix JFFS2 EOF markers X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=1a51241cb1be87a45cac12753a437f786329afc6;p=openwrt%2Fstaging%2Fblogic.git firmware-utils: mktplinkfw: fix JFFS2 EOF markers mktplinkfw/mktplinkfw2 utilities put JFFS2 EOF market only at 64KB boundary, this could lead to current device configuration lost during the sysupgrade on a device, which is equpped with flash with the 4KB erase block size (e.g. TP-Link Archer C20). This happens when 64KB and 4KB alignments do not match, so the JFFS2 data is written not exactly at the partition beginnig and startup scripts can not find the JFFS2 during the first boot just after the sysupgrade. Fix this by placing additional JFFS2 EOF marker at a 4KB boundary. Also keep the marker at 64KB intact, so the utilities will produce images suitable for devices with both 4KB and 64KB erase blocks. Fixes: 29a2c2ea80441895a2ffe100d854d2b26d5fa606 (add ability to put jffs2 eof marker into the image) Signed-off-by: Sergey Ryazanov --- diff --git a/tools/firmware-utils/src/mktplinkfw-lib.c b/tools/firmware-utils/src/mktplinkfw-lib.c index b2b6de26db95..e3213274c19c 100644 --- a/tools/firmware-utils/src/mktplinkfw-lib.c +++ b/tools/firmware-utils/src/mktplinkfw-lib.c @@ -121,7 +121,7 @@ static int pad_jffs2(char *buf, int currlen, int maxlen) uint32_t pad_mask; len = currlen; - pad_mask = (64 * 1024); + pad_mask = (4 * 1024) | (64 * 1024); /* EOF at 4KB and at 64KB */ while ((len < maxlen) && (pad_mask != 0)) { uint32_t mask; int i;