On Sat, Dec 25, 2021 at 6:57 PM Pali Rohár pali@kernel.org wrote:
Hello! I'm using this patch for more than month and I have not seen any boot issue related to this patch A385 board.
FWIW I'm testing this on a Prestera board, so Armada XP based.
Just moving "main_hdr->checksum = image_checksum8(main_hdr, headersz);" after the "main_hdr->headersz_msb = (headersz & 0xFFFF0000) >> 16;" should be enough. Are you going to prepare a fixup for master branch?
I've actually tested this earlier -- csum8 works, but csum32 still doesn't pass in kwboot / mkimage. Note that I haven't checked whether the hardware accepts it (I know it didn't like the broken csum8), so this could be that the csum32 verification code is not tolerant to your modified headersz.
Thanks for the really quick response!
Best,