Re: [PATCH] hrtimer: Printing timer info when hitting BUG_ON()

[Peter Zijlstra]

On Mon, 2012-10-29 at 19:02 +0800, Chuansheng Liu wrote:
> +/*
> + * dump_hrtimer_callinfo - print hrtimer information including:
> + * state, callback function, pid and start_site.
> +*/
> +static void dump_hrtimer_callinfo(struct hrtimer *timer)
> +{
> +
> +       char symname[KSYM_NAME_LEN];
> +
> +       if (lookup_symbol_name((unsigned long)(timer->function), symname) < 0) {
> +               pr_err("timer info: state/%lx, func/%pK\n",
> +                       timer->state, timer->function);
> +       } else {
> +               pr_err("timer info: state/%lx, func/%s",
> +                       timer->state, symname);
> +       }
> +
> +#ifdef CONFIG_TIMER_STATS
> +       if (lookup_symbol_name((unsigned long)(timer->start_site),
> +                                               symname) < 0) {
> +               pr_err("timer stats: pid/%d(%s), site/%pK\n",
> +                       timer->start_pid, timer->start_comm, timer->start_site);
> +       } else {
> +               pr_err("timer stats: pid/%d(%s), site/%s\n",
> +                       timer->start_pid, timer->start_comm, symname);
> +       }
> +#endif
> +} 

What's wrong with %pf ?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/