Re: [PATCH] printk: clarify the documentation for plain pointer printing

From: Petr Mladek
Date: Fri Feb 26 2021 - 04:16:13 EST


On Thu 2021-02-25 17:46:39, Vlastimil Babka wrote:
> We have several modifiers for plain pointers (%p, %px and %pK) and now also
> the no_hash_pointers boot parameter. The documentation should help to choose
> which variant to use. Importantly, we should discourage %px in favour of %p
> (with the new boot parameter when debugging), and stress that %pK should be
> only used for procfs and similar files, not dmesg buffer. This patch clarifies
> the documentation in that regard.
>
> Signed-off-by: Vlastimil Babka <vbabka@xxxxxxx>

Nice!

Reviewed-by: Petr Mladek <pmladek@xxxxxxxx>

Best Regards,
Petr