Re: [PATCH 1/2] Simplify sysrq help

[Greg KH]

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