xfs
[Top] [All Lists]

Re: [PATCH 21/22] xfs: %pF is only for function pointers

To: Scott Wood <scottwood@xxxxxxxxxxxxx>
Subject: Re: [PATCH 21/22] xfs: %pF is only for function pointers
From: Steven Rostedt <rostedt@xxxxxxxxxxx>
Date: Mon, 31 Aug 2015 15:45:21 -0400
Cc: Dave Chinner <david@xxxxxxxxxxxxx>, <trivial@xxxxxxxxxx>, <linux-kernel@xxxxxxxxxxxxxxx>, <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1441049065.4966.38.camel@xxxxxxxxxxxxx>
References: <1426130037-17956-1-git-send-email-scottwood@xxxxxxxxxxxxx> <1426130037-17956-21-git-send-email-scottwood@xxxxxxxxxxxxx> <20150831080657.GF26895@dastard> <1441049065.4966.38.camel@xxxxxxxxxxxxx>
On Mon, 31 Aug 2015 14:24:25 -0500
Scott Wood <scottwood@xxxxxxxxxxxxx> wrote:
 
> > Can you please look into why this change broke the tracing output
> > on x86-64 - if there is no obvious/easy fix for it, then I'm simply
> > going to revert it because having the tracing work correctly on
> > x86-64 is far more important to us than ppc64 or ia64....
> 
> It looks like the cause is that TP_printk() is not really printk() -- it 
> actually passes the format to userspace which has its own, not 100% 
> compatible implementation pretty_print() in tools/lib/traceevent/event-
> parse.c.  %pf in that function behaves like %ps in the kernel, and %ps is 
> absent.
> 

We can fix that with adding %ps to the traceevent library.

-- Steve

<Prev in Thread] Current Thread [Next in Thread>