From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH v5 2/3] test: Add PStore command tests
Date: Tue, 13 Oct 2020 11:48:56 -0400 [thread overview]
Message-ID: <20201013154856.GA23861@bill-the-cat> (raw)
In-Reply-To: <20200320095924.523-3-frederic.danis@collabora.com>
On Fri, Mar 20, 2020 at 10:59:23AM +0100, Fr?d?ric Danis wrote:
> Add PStore command to sandbox and sandbox64 defconfigs.
> Add test checking:
> - 'pstore display' of all records
> - 'pstore display' only the 2nd dump record
> - 'pstore save' of all records
>
> Signed-off-by: Fr?d?ric Danis <frederic.danis@collabora.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Cc: Wolfgang Denk <wd@denx.de>
> Cc: Heiko Schocher <hs@denx.de>
> ---
> Changes in v5:
> - Fix test_pstore.py license
>
> Changes in v4:
> - Fix PStore memory address in sandbox defconfig files for tests
>
> Changes in v3:
> - Replace 1M test file by 3 * 4K files and build pstore memory during test
>
> New in v2:
> - Add unit tests
>
> configs/sandbox64_defconfig | 2 +
> configs/sandbox_defconfig | 2 +
> test/py/tests/test_pstore.py | 73 +++++++++++++++++++++
> test/py/tests/test_pstore_data_console.hex | Bin 0 -> 4096 bytes
> test/py/tests/test_pstore_data_panic1.hex | Bin 0 -> 4096 bytes
> test/py/tests/test_pstore_data_panic2.hex | Bin 0 -> 4096 bytes
> 6 files changed, 77 insertions(+)
> create mode 100644 test/py/tests/test_pstore.py
> create mode 100644 test/py/tests/test_pstore_data_console.hex
> create mode 100644 test/py/tests/test_pstore_data_panic1.hex
> create mode 100644 test/py/tests/test_pstore_data_panic2.hex
Sorry for the delay here. When I run the tests I see:
------------------------------------ Captured stdout call -------------------------------------
=>
=> => host load hostfs - 0x3000000 test/py/tests/test_pstore_data_panic1.hex
4096 bytes read in 0 ms
=> => host load hostfs - 0x3001000 test/py/tests/test_pstore_data_panic2.hex
0 bytes read in 0 ms
=> => host load hostfs - 0x30fd000 test/py/tests/test_pstore_data_console.hex
4096 bytes read in 0 ms
=> => pstore set 0x3000000 0x100000
=> => pstore save hostfs - /tmp/tmp1xx0rahx
3798 bytes written in 1 ms (3.6 MiB/s)
4084 bytes written in 0 ms
=> => load hostfs - 1000008 /tmp/tmp1xx0rahx/dmesg-ramoops-0
3798 bytes read in 0 ms
=> => printenv filesize
filesize=ed6
=> => md5sum 1000008 $filesize
md5 for 01000008 ... 01000edd ==> 8059335ab4cfa62c77324c491659c503
=> => setenv filesize
=> => load hostfs - 1000008 /tmp/tmp1xx0rahx/dmesg-ramoops-1
Failed to load '/tmp/tmp1xx0rahx/dmesg-ramoops-1'
=> => printenv filesize
## Error: "file
=================================== short test summary info ===================================
So I'm not sure why the second one fails to load.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20201013/fdd8b78c/attachment.sig>
next prev parent reply other threads:[~2020-10-13 15:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-20 9:59 [PATCH v5 0/3] Add command to display or save Linux PStore dumps Frédéric Danis
2020-03-20 9:59 ` [PATCH v5 1/3] cmd: " Frédéric Danis
2020-10-14 17:43 ` Tom Rini
2020-03-20 9:59 ` [PATCH v5 2/3] test: Add PStore command tests Frédéric Danis
2020-10-13 15:48 ` Tom Rini [this message]
2020-10-13 16:35 ` Frédéric Danis
2020-10-13 18:03 ` Tom Rini
2020-10-13 18:33 ` Tom Rini
2020-10-14 17:44 ` Tom Rini
2020-03-20 9:59 ` [PATCH v5 3/3] cmd: Fixup DT to pass PStore Ramoops parameters Frédéric Danis
2020-10-14 17:44 ` Tom Rini
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=20201013154856.GA23861@bill-the-cat \
--to=trini@konsulko.com \
--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