mbox series

[v2,0/2] Add command to display or save Linux PStore dumps

Message ID 20200226094211.20392-1-frederic.danis@collabora.com
Headers show
Series Add command to display or save Linux PStore dumps | expand

Message

Frédéric Danis Feb. 26, 2020, 9:42 a.m. UTC
This serie of patches adds a new pstore command allowing to display or save
ramoops logs (oops, panic, console, ftrace and user) generated by a previous
kernel crash.
PStore parameters can be set in U-Boot configuration file, or at run-time
using "pstore set" command. Records size should be the same as the ones
used by kernel, and should be a power of 2.

Since v1:
- Fix 64bit mode build warnings
- Add documentation
- Add function description comments
- Replace calls to pr_debug() by debug()
- Add CONFIG_CMD_PSTORE to sandbox and sandbox64
- Add unit tests

Fr?d?ric Danis (2):
  cmd: Add command to display or save Linux PStore dumps
  test: Add PStore command tests

 cmd/Kconfig                        |  63 ++++
 cmd/Makefile                       |   1 +
 cmd/pstore.c                       | 505 +++++++++++++++++++++++++++++
 configs/sandbox64_defconfig        |   1 +
 configs/sandbox_defconfig          |   1 +
 doc/index.rst                      |   7 +
 doc/pstore.rst                     |  68 ++++
 test/py/tests/test_pstore.py       |  82 +++++
 test/py/tests/test_pstore_data.hex | Bin 0 -> 1048576 bytes
 9 files changed, 728 insertions(+)
 create mode 100644 cmd/pstore.c
 create mode 100644 doc/pstore.rst
 create mode 100644 test/py/tests/test_pstore.py
 create mode 100644 test/py/tests/test_pstore_data.hex

Comments

Frédéric Danis March 17, 2020, 5:07 p.m. UTC | #1
Hi,

On 26/02/2020 10:42, Fr?d?ric Danis wrote:
> This serie of patches adds a new pstore command allowing to display or save
> ramoops logs (oops, panic, console, ftrace and user) generated by a previous
> kernel crash.
> PStore parameters can be set in U-Boot configuration file, or at run-time
> using "pstore set" command. Records size should be the same as the ones
> used by kernel, and should be a power of 2.
>
> Since v1:
> - Fix 64bit mode build warnings
> - Add documentation
> - Add function description comments
> - Replace calls to pr_debug() by debug()
> - Add CONFIG_CMD_PSTORE to sandbox and sandbox64
> - Add unit tests
>
> Fr?d?ric Danis (2):
>    cmd: Add command to display or save Linux PStore dumps
>    test: Add PStore command tests
>
>   cmd/Kconfig                        |  63 ++++
>   cmd/Makefile                       |   1 +
>   cmd/pstore.c                       | 505 +++++++++++++++++++++++++++++
>   configs/sandbox64_defconfig        |   1 +
>   configs/sandbox_defconfig          |   1 +
>   doc/index.rst                      |   7 +
>   doc/pstore.rst                     |  68 ++++
>   test/py/tests/test_pstore.py       |  82 +++++
>   test/py/tests/test_pstore_data.hex | Bin 0 -> 1048576 bytes
>   9 files changed, 728 insertions(+)
>   create mode 100644 cmd/pstore.c
>   create mode 100644 doc/pstore.rst
>   create mode 100644 test/py/tests/test_pstore.py
>   create mode 100644 test/py/tests/test_pstore_data.hex
>

Are those patches acceptable?

Best Regards,

Fr?d?ric Danis