Re: [uml-user] [PATCH] um: Fix _print_addr()

From: Richard Weinberger
Date: Wed May 03 2017 - 16:55:40 EST


On Sun, Dec 25, 2016 at 11:11 PM, Richard Weinberger <richard@xxxxxx> wrote:
> Recent changes to printk() broke UML's stack trace
> output. Kill the root of the problem by using a single
> printk() statement.
>
> Signed-off-by: Richard Weinberger <richard@xxxxxx>
> ---
> arch/um/kernel/sysrq.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/arch/um/kernel/sysrq.c b/arch/um/kernel/sysrq.c
> index aa1b56f5ac68..18eddf677ec6 100644
> --- a/arch/um/kernel/sysrq.c
> +++ b/arch/um/kernel/sysrq.c
> @@ -17,10 +17,8 @@
>
> static void _print_addr(void *data, unsigned long address, int reliable)
> {
> - pr_info(" [<%08lx>]", address);
> - pr_cont(" %s", reliable ? "" : "? ");
> - print_symbol("%s", address);
> - pr_cont("\n");
> + pr_info(" [<%08lx>] %s%pF\n", address, reliable ? "" : "? ",
> + (void *)address);
> }
>
> static const struct stacktrace_ops stackops = {

Applied.

--
Thanks,
//richard