[PATCH V2 0/5] generalize panic_print's dump function to be used by other kernel parts

Feng Tang posted 5 patches 3 months, 3 weeks ago
There is a newer version of this series
.../admin-guide/kernel-parameters.txt         |  13 ++
Documentation/admin-guide/sysctl/kernel.rst   |  18 +++
include/linux/kernel.h                        |   1 +
include/linux/sys_info.h                      |  27 ++++
kernel/panic.c                                |  66 +++++----
lib/Makefile                                  |   2 +-
lib/sys_info.c                                | 127 ++++++++++++++++++
7 files changed, 217 insertions(+), 37 deletions(-)
create mode 100644 include/linux/sys_info.h
create mode 100644 lib/sys_info.c
[PATCH V2 0/5] generalize panic_print's dump function to be used by other kernel parts
Posted by Feng Tang 3 months, 3 weeks ago
When working on kernel stability issues, panic, task-hung and 
software/hardware lockup are frequently met. And to debug them, user
may need lots of system information at that time, like task call stacks,
lock info, memory info etc. 

panic case already has panic_print_sys_info() for this purpose, and has
a 'panic_print' bitmask to control what kinds of information is needed,
which is also helpful to debug other task-hung and lockup cases.

So this patchset extract the function out to a new file 'lib/sys_info.c',
and make it usable for other cases which also need system info for
debugging. 

Also as suggested by Petr Mladek, add 'panic_sys_info=' interface to
take human readable string like "tasks,mem,lock,timer,ftrace,....", 
and eventually obsolete the current 'panic_print' bitmap interface.

In RFC and V1 version, hung_task and SW/HW watchdog modules are enabled
with the new sys_info dump interface. In v2, they are kept out for
better review of current change, and will be posted later. 

Locally these have been used in our bug chasing for stabilility issues
and was helpful.

Please help to review, thanks!

- Feng

Changelog:

  Sinc V1:
     * Separate the 'sys_show_info' related code to new file sys_info.[ch] 
       (Petr Mladek)
     * Clean up the code for panic console replay (Petr Mladek)
     * Add 'panic_sys_info=' cmdline and sysctl interface for taking
       human readable parameters (Petr Mladek)
     * Add note about the obsoleting of 'panic_print' (Petr Mladek)
     * Hold the changes to hungtask/watchdog 

  Since RFC:
     * Don't print all cpu backtrace if 'sysctl_hung_task_all_cpu_backtracemay'
       is 'false' (Lance Yang)
     * Change the name of 2 new kernel control knob to have 'mask' inside, and
       add kernel document and code comments for them (Lance Yang)
     * Make the sys_show_info() support printk msg replay and all CPU backtrace. 


Feng Tang (5):
  panic: clean up code for console replay
  panic: generalize panic_print's function to show sys info
  sys_info: add help to translate sys_info string to bitmap
  panic: add 'panic_sys_info=' setup option for sysctl and kernel
    cmdline
  panic: add note that panic_print interface is deprecated

 .../admin-guide/kernel-parameters.txt         |  13 ++
 Documentation/admin-guide/sysctl/kernel.rst   |  18 +++
 include/linux/kernel.h                        |   1 +
 include/linux/sys_info.h                      |  27 ++++
 kernel/panic.c                                |  66 +++++----
 lib/Makefile                                  |   2 +-
 lib/sys_info.c                                | 127 ++++++++++++++++++
 7 files changed, 217 insertions(+), 37 deletions(-)
 create mode 100644 include/linux/sys_info.h
 create mode 100644 lib/sys_info.c

-- 
2.39.5 (Apple Git-154)