Re: [PATCH V11 4/5] vsprintf: add printk specifier %px

From: Alexey Dobriyan
Date: Tue Dec 05 2017 - 23:20:55 EST


> more %p grepping [filtering out all `%ps %pf %pb' variants] gives
> a huge number of print outs that potentially can be broken now

Because people who introduce this stupid %p hashing can't be bothered
to actually audit users:

static int show_timer(struct seq_file *m, void *v)
{
...
seq_printf(m, "signal: %d/%p\n",
timer->sigq->info.si_signo,
timer->sigq->info.si_value.sival_ptr);

Overall, this %px thing doesn't matter. Developers will quickly learn
than %p gives some useless irreversible values and start using %px
everywhere. Soon someone will use %px in the wrong place and new
non-standard format specifier will be added.