| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 11/11] xfsprogs: add dummy mremap for OS X, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 21/22] xfs: %pF is only for function pointers, Steven Rostedt |
| Previous by Thread: | Re: [PATCH 21/22] xfs: %pF is only for function pointers, Dave Chinner |
| Next by Thread: | Re: [PATCH 21/22] xfs: %pF is only for function pointers, Steven Rostedt |
| Indexes: | [Date] [Thread] [Top] [All Lists] |