On Saturday 25 December 2021 19:06:13 Pierre Bourdon wrote:
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,
Ok!
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.
csum32 is checksum of data, not including header. If generated image does not pass kwboot verification then it is invalid.
Has it worked with some previous version? If yes, can you bisect git commit which broke it?
Thanks for the really quick response!
Best,
-- Pierre Bourdon delroth@gmail.com Software Engineer @ Zürich, Switzerland https://delroth.net/