[PATCH 00/11] imagemap: fixes from the v1 review
Daniel Golle
daniel at makrotopia.org
Mon Sep 28 17:04:16 PDT 2026
Simon Glass reviewed the v1 posting of the imagemap series
(cover.1787272661.git.daniel at makrotopia.org) after v2 had already been
applied, so his comments arrive against code that is now in next. This
series carries the correctness half of them; a second series follows
with the structural and cosmetic half.
Tested on sandbox: every commit builds, "ut imagemap" passes 13/13 and
survives "ut -r3", and a full test.py run shows the same 11 failures and
20 errors as unmodified next, all of them EFI-secboot tooling, ext4 and
config-rebuild tests unrelated to this code. Also built with
CONFIG_IMAGEMAP=n.
Signed-off-by: Daniel Golle <daniel at makrotopia.org>
Daniel Golle (11):
mtd: bind the block device without leaking on failure
cmd: ubi: report a failed UBI block device bind
cmd: ubi: unbind the block device when detaching UBI
boot: imagemap: free the reservation a replaced record owned
boot: imagemap: do not resize a region the caller owns
boot: imagemap: read only the bytes an extend adds
boot: fit: unmap the load address after a storage read
boot: fit: keep the load message for RAM-backed images
boot: bootm: release the imagemap on every failing exit
test: boot: imagemap: detach UBI when the test is done
test: boot: imagemap: reset driver-model state between tests
boot/bootm.c | 9 +++---
boot/image-fit.c | 7 +++--
boot/imagemap.c | 55 ++++++++++++++++++++++++---------
cmd/ubi.c | 67 +++++++++++++++++++++++++++--------------
drivers/mtd/mtdcore.c | 37 ++++++++++++++++-------
drivers/mtd/ubi/block.c | 4 ++-
include/ubi_uboot.h | 4 +--
test/boot/imagemap.c | 36 ++++++++++++++--------
8 files changed, 147 insertions(+), 72 deletions(-)
base-commit: 2b8902913c19eb581fb0d567201c09b8fa46ac74
--
2.55.0
More information about the openwrt-devel
mailing list