On Mon, Aug 28, 2023 at 12:27:53PM +0200, Andras Sebok wrote: > This patch simplyfies sysrq help menu by moving each command to a > separate line and by moving the key from brackets to after a semicolon. > > before: > sysrq: HELP : loglevel(0-9) reboot(b) crash(c) terminate-all-tasks(e) memory-full-oom-kill(f) kill-all-tasks(i) thaw-filesystems(j) sak(k) show-backtrace-all-active-cpus(l) show-memory-usage(m) nice-all-RT-tasks(n) poweroff(o) show-registers(p) show-all-timers(q) unraw(r) sync(s) show-task-states(t) unmount(u) force-fb(v) show-blocked-tasks(w) dump-ftrace-buffer(z) > > after: > sysrq: HELP : > loglevel : 0-9 > reboot : b > crash : c > terminate-all-tasks : e > memory-full-oom-kill : f > kill-all-tasks : i > thaw-filesystems : j > sak : k > show-backtrace-all-active-cpus : l > show-memory-usage : m > nice-all-RT-tasks : n > poweroff : o > show-registers : p > show-all-timers : q > unraw : r > sync : s > show-task-states : t > unmount : u > force-fb : v > show-blocked-tasks : w > dump-ftrace-buffer : z So now the options all scroll off the screen? Are you sure this is wise? What problem are you trying to solve here, it just looks like you are changing one "help text format for another", and doesn't actually change much of anything. Note the: sysrq: HELP : line format being different from all the others makes this not even look correct? So I really do not see the point of this change here, why is it needed? thanks, greg k-h
