Re: [PATCH V2] riscv: Convert uses of REG_FMT to %p

[Andy Shevchenko]

On Sat, Jul 28, 2018 at 7:39 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> Use %p pointer output instead of REG_FMT and cast the unsigned longs to
> (void *) to avoid exposing kernel addresses.
>
> Miscellanea:
>
> o Convert pr_cont to printk(KERN_DEFAULT as these uses are
>   new logging lines and not previous line continuations
> o Remove the now unused REG_FMT defines

> +       pr_alert("Unable to handle kernel %s at virtual address %p\n",

> +                (addr < PAGE_SIZE) ? "NULL pointer dereference" :
> +                "paging request", (void *)addr);

Hmm... This is what printf() should take care of.

Petr, what is the status of your series to unify this kind of
addresses to be printed?

-- 
With Best Regards,
Andy Shevchenko