Messages in this thread | | | Date | Thu, 19 Feb 2015 23:45:48 -0500 | From | Steven Rostedt <> | Subject | Re: [PATCH v2] x86: fix output of show_stack_log_lvl() |
| |
On Fri, 20 Feb 2015 03:34:21 +0100 Adrien Schildknecht <adrien+dev@schischi.me> wrote:
> show_stack_log_lvl() does not set the log level after a new line, > the following messages printed with pr_cont are thus assigned to the > default log level.
This looks like a bug in printk(). Why doesn't pr_cont() continue? It shouldn't care if there's a newline or not. pr_cont() is supposed to continue whatever the last printk log level was.
If this is broken here, it's probably broken elsewhere. The fix is to fix printk, not to hunt and peck for the places with work arounds that are broken by it.
-- Steve
> This patch prepends the log level to the next message following a new > line. > > print_trace_address() uses printk(log_lvl). Using printk with just > a log level is ignored and thus has no effect on the next pr_cont. > We need to prepend the log level directly into the message. > > Signed-off-by: Adrien Schildknecht <adrien+dev@schischi.me>
| |