From: Alexander Dahl <ada@thorsis.com>
To: "Pali Rohár" <pali@kernel.org>
Cc: u-boot@lists.denx.de
Subject: Re: [PATCH 0/9] Nokia RX-51: Small cleanups and UBI boot test case
Date: Mon, 19 Sep 2022 13:12:41 +0200 [thread overview]
Message-ID: <YyhOqSbzRotTty9h@ada.ifak-system.com> (raw)
In-Reply-To: <20220904012906.17718-1-pali@kernel.org>
Hello Pali,
Am Sun, Sep 04, 2022 at 03:28:57AM +0200 schrieb Pali Rohár:
> Do various small fixup/cleanups and extend test script to boot kernel
> image from UBI volume. This test verifies that U-Boot UBI implementation
> is working and U-Boot can read volume with bootable kernel code
> correctly. And therefore CI prevents UBI breakage.
>
> Note that U-Boot UBIFS code on ARM is currently somehow broken and
> trying to mount UBIFS from UBI volume fails :-( I have already tried to
> debug this issue but I have no idea why it is failing. Function
> check_lpt_crc in unpack_ltab is failing. Volume is for sure correct and
> valid because Linux kernel can successfully mount it. And to make it
> more suspicious, U-Boot UBIFS is working fine on big endian powerpc
> platform. So UBIFS issue is probably endian or arch specific.
> (This is UBIFS related, not UBI related.)
I had some trouble with UBIFS volumes created by Linux not mountable
by U-Boot myself recently. Problem was Linux gained zstd compression
support for UBIFS with version 5.3 and that was made default with
kernel 5.13. U-Boot does not support zstd compression at least up to
version 2022.04 (did not try later ones), and simply refuses to mount
those volumes. The error message however was different than yours,
but maybe you get an idea what might be wrong. I would certainly
check with what features the UBIFS is created and if U-Boot supports
that.
HTH & Greets
Alex
>
> Pali Rohár (9):
> Nokia RX-51: Remove label copy_kernel_start from lowlevel_init.S
> Nokia RX-51: Do not clear unknown memory in lowlevel_init.S
> Nokia RX-51: Set default SYS_LOAD_ADDR to 0x80008000
> Nokia RX-51: Change UBIFS volume size to 1870 LEBs in test script
> Nokia RX-51: Call bootm in test script only when image is valid
> Nokia RX-51: Fix documentation how to enable UBI support
> Nokia RX-51: Do not set useless ARCH= in test script
> Nokia RX-51: Add comment describing kernel image type into test script
> Nokia RX-51: Add booting from UBI into test script
>
> board/nokia/rx51/lowlevel_init.S | 7 +--
> configs/nokia_rx51_defconfig | 2 +-
> doc/board/nokia/rx51.rst | 3 +-
> test/nokia_rx51_test.sh | 97 +++++++++++++++++++++++++-------
> 4 files changed, 82 insertions(+), 27 deletions(-)
>
> --
> 2.20.1
>
next prev parent reply other threads:[~2022-09-19 11:12 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-04 1:28 [PATCH 0/9] Nokia RX-51: Small cleanups and UBI boot test case Pali Rohár
2022-09-04 1:28 ` [PATCH 1/9] Nokia RX-51: Remove label copy_kernel_start from lowlevel_init.S Pali Rohár
2022-09-15 21:54 ` Tom Rini
2022-09-04 1:28 ` [PATCH 2/9] Nokia RX-51: Do not clear unknown memory in lowlevel_init.S Pali Rohár
2022-09-15 21:54 ` Tom Rini
2022-09-04 1:29 ` [PATCH 3/9] Nokia RX-51: Set default SYS_LOAD_ADDR to 0x80008000 Pali Rohár
2022-09-15 21:54 ` Tom Rini
2022-09-04 1:29 ` [PATCH 4/9] Nokia RX-51: Change UBIFS volume size to 1870 LEBs in test script Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 1:29 ` [PATCH 5/9] Nokia RX-51: Call bootm in test script only when image is valid Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 1:29 ` [PATCH 6/9] Nokia RX-51: Fix documentation how to enable UBI support Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 1:29 ` [PATCH 7/9] Nokia RX-51: Do not set useless ARCH= in test script Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 1:29 ` [PATCH 8/9] Nokia RX-51: Add comment describing kernel image type into " Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 1:29 ` [PATCH 9/9] Nokia RX-51: Add booting from UBI " Pali Rohár
2022-09-15 18:34 ` Tom Rini
2022-09-15 18:57 ` Pali Rohár
2022-09-15 18:59 ` [PATCH v2] " Pali Rohár
2022-09-15 21:55 ` Tom Rini
2022-09-04 3:01 ` [PATCH 0/9] Nokia RX-51: Small cleanups and UBI boot test case Tony Dinh
2022-09-04 9:37 ` Pali Rohár
2022-09-04 19:28 ` Tony Dinh
2022-09-04 19:56 ` Tony Dinh
2022-09-04 21:58 ` Daniel Golle
2022-09-04 22:10 ` Pali Rohár
2022-09-05 21:35 ` Tony Dinh
2022-09-19 11:12 ` Alexander Dahl [this message]
2022-09-19 11:23 ` Pali Rohár
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YyhOqSbzRotTty9h@ada.ifak-system.com \
--to=ada@thorsis.com \
--cc=pali@kernel.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox