tracing: %pF is only for function pointers

Use %pS for actual addresses, otherwise you'll get bad output
on arches like ppc64 where %pF expects a function descriptor.

Link: http://lkml.kernel.org/r/1426130037-17956-22-git-send-email-scottwood@freescale.com
Signed-off-by: Scott Wood <scottwood@freescale.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

diff --git a/include/trace/events/btrfs.h b/include/trace/events/btrfs.h
index 1faecea101f3e85a84c247ec997af8216de2523d..572e6503394a4c77bbfadfe1ce33bca134600724 100644 (file)
--- a/include/trace/events/btrfs.h
+++ b/include/trace/events/btrfs.h
@@ -962,7 +962,7 @@ TRACE_EVENT(alloc_extent_state,
                __entry->ip     = IP
        ),
 
-       TP_printk("state=%p; mask = %s; caller = %pF", __entry->state,
+       TP_printk("state=%p; mask = %s; caller = %pS", __entry->state,
                  show_gfp_flags(__entry->mask), (void *)__entry->ip)
 );
 
@@ -982,7 +982,7 @@ TRACE_EVENT(free_extent_state,
                __entry->ip = IP
        ),
 
-       TP_printk(" state=%p; caller = %pF", __entry->state,
+       TP_printk(" state=%p; caller = %pS", __entry->state,
                  (void *)__entry->ip)
 );
 

diff --git a/include/trace/events/ext3.h b/include/trace/events/ext3.h
index 6797b9de90edeac929c5c234558434cc16d1d877..7f20707849bbaa76a75f18c9fd99ea78314af249 100644 (file)
--- a/include/trace/events/ext3.h
+++ b/include/trace/events/ext3.h
@@ -144,7 +144,7 @@ TRACE_EVENT(ext3_mark_inode_dirty,
                __entry->ip     = IP;
        ),
 
-       TP_printk("dev %d,%d ino %lu caller %pF",
+       TP_printk("dev %d,%d ino %lu caller %pS",
                  MAJOR(__entry->dev), MINOR(__entry->dev),
                  (unsigned long) __entry->ino, (void *)__entry->ip)
 );

diff --git a/include/trace/events/ext4.h b/include/trace/events/ext4.h
index 6e5abd6d38a2b7839aca39f961df6818b9330e42..47fca36ee426abb4cf359ceb47a3741fd9ba1393 100644 (file)
--- a/include/trace/events/ext4.h
+++ b/include/trace/events/ext4.h
@@ -240,7 +240,7 @@ TRACE_EVENT(ext4_mark_inode_dirty,
                __entry->ip     = IP;
        ),
 
-       TP_printk("dev %d,%d ino %lu caller %pF",
+       TP_printk("dev %d,%d ino %lu caller %pS",
                  MAJOR(__entry->dev), MINOR(__entry->dev),
                  (unsigned long) __entry->ino, (void *)__entry->ip)
 );
@@ -1762,7 +1762,7 @@ TRACE_EVENT(ext4_journal_start,
                __entry->rsv_blocks      = rsv_blocks;
        ),
 
-       TP_printk("dev %d,%d blocks, %d rsv_blocks, %d caller %pF",
+       TP_printk("dev %d,%d blocks, %d rsv_blocks, %d caller %pS",
                  MAJOR(__entry->dev), MINOR(__entry->dev),
                  __entry->blocks, __entry->rsv_blocks, (void *)__entry->ip)
 );
@@ -1784,7 +1784,7 @@ TRACE_EVENT(ext4_journal_start_reserved,
                __entry->blocks          = blocks;
        ),
 
-       TP_printk("dev %d,%d blocks, %d caller %pF",
+       TP_printk("dev %d,%d blocks, %d caller %pS",
                  MAJOR(__entry->dev), MINOR(__entry->dev),
                  __entry->blocks, (void *)__entry->ip)
 );

diff --git a/include/trace/events/module.h b/include/trace/events/module.h
index 81c4c183d348b03793a2c9d6253846ce54ce4601..28c45997e451f4b2cbfcd3aabab15b3e41e13e45 100644 (file)
--- a/include/trace/events/module.h
+++ b/include/trace/events/module.h
@@ -84,7 +84,7 @@ DECLARE_EVENT_CLASS(module_refcnt,
                __assign_str(name, mod->name);
        ),
 
-       TP_printk("%s call_site=%pf refcnt=%d",
+       TP_printk("%s call_site=%ps refcnt=%d",
                  __get_str(name), (void *)__entry->ip, __entry->refcnt)
 );
 
@@ -121,7 +121,7 @@ TRACE_EVENT(module_request,
                __assign_str(name, name);
        ),
 
-       TP_printk("%s wait=%d call_site=%pf",
+       TP_printk("%s wait=%d call_site=%ps",
                  __get_str(name), (int)__entry->wait, (void *)__entry->ip)
 );
 

diff --git a/include/trace/events/random.h b/include/trace/events/random.h
index 805af6db41cc6091a877d8ddf58742cb2e3fce55..4684de344c5d96d7fb86260d37cadd3d7c76a1fe 100644 (file)
--- a/include/trace/events/random.h
+++ b/include/trace/events/random.h
@@ -22,7 +22,7 @@ TRACE_EVENT(add_device_randomness,
                __entry->IP             = IP;
        ),
 
-       TP_printk("bytes %d caller %pF",
+       TP_printk("bytes %d caller %pS",
                __entry->bytes, (void *)__entry->IP)
 );
 
@@ -43,7 +43,7 @@ DECLARE_EVENT_CLASS(random__mix_pool_bytes,
                __entry->IP             = IP;
        ),
 
-       TP_printk("%s pool: bytes %d caller %pF",
+       TP_printk("%s pool: bytes %d caller %pS",
                  __entry->pool_name, __entry->bytes, (void *)__entry->IP)
 );
 
@@ -82,7 +82,7 @@ TRACE_EVENT(credit_entropy_bits,
        ),
 
        TP_printk("%s pool: bits %d entropy_count %d entropy_total %d "
-                 "caller %pF", __entry->pool_name, __entry->bits,
+                 "caller %pS", __entry->pool_name, __entry->bits,
                  __entry->entropy_count, __entry->entropy_total,
                  (void *)__entry->IP)
 );
@@ -207,7 +207,7 @@ DECLARE_EVENT_CLASS(random__get_random_bytes,
                __entry->IP             = IP;
        ),
 
-       TP_printk("nbytes %d caller %pF", __entry->nbytes, (void *)__entry->IP)
+       TP_printk("nbytes %d caller %pS", __entry->nbytes, (void *)__entry->IP)
 );
 
 DEFINE_EVENT(random__get_random_bytes, get_random_bytes,
@@ -242,7 +242,7 @@ DECLARE_EVENT_CLASS(random__extract_entropy,
                __entry->IP             = IP;
        ),
 
-       TP_printk("%s pool: nbytes %d entropy_count %d caller %pF",
+       TP_printk("%s pool: nbytes %d entropy_count %d caller %pS",
                  __entry->pool_name, __entry->nbytes, __entry->entropy_count,
                  (void *)__entry->IP)
 );

diff --git a/kernel/trace/trace_entries.h b/kernel/trace/trace_entries.h
index e2d027ac66a2d0f79bc1b3959b028a081f1fbc5d..ee7b94a4810af8c85aa4baa246203dc52f588eb6 100644 (file)
--- a/kernel/trace/trace_entries.h
+++ b/kernel/trace/trace_entries.h
@@ -223,7 +223,7 @@ FTRACE_ENTRY(bprint, bprint_entry,
                __dynamic_array(        u32,    buf     )
        ),
 
-       F_printk("%pf: %s",
+       F_printk("%ps: %s",
                 (void *)__entry->ip, __entry->fmt),
 
        FILTER_OTHER
@@ -238,7 +238,7 @@ FTRACE_ENTRY(print, print_entry,
                __dynamic_array(        char,   buf     )
        ),
 
-       F_printk("%pf: %s",
+       F_printk("%ps: %s",
                 (void *)__entry->ip, __entry->buf),
 
        FILTER_OTHER
@@ -253,7 +253,7 @@ FTRACE_ENTRY(bputs, bputs_entry,
                __field(        const char *,   str     )
        ),
 
-       F_printk("%pf: %s",
+       F_printk("%ps: %s",
                 (void *)__entry->ip, __entry->str),
 
        FILTER_OTHER

diff --git a/tools/lib/traceevent/event-parse.c b/tools/lib/traceevent/event-parse.c
index afe20ed9fac83e81d1887b86ad5cf01dd1726bad..2c0bd8f2aad0880c18eb4788a46559fe7611e3f4 100644 (file)
--- a/tools/lib/traceevent/event-parse.c
+++ b/tools/lib/traceevent/event-parse.c
@@ -3976,7 +3976,7 @@ static struct print_arg *make_bprint_args(char *fmt, void *data, int size, struc
        if (asprintf(&arg->atom.atom, "%lld", ip) < 0)
                goto out_free;
 
-       /* skip the first "%pf: " */
+       /* skip the first "%ps: " */
        for (ptr = fmt + 5, bptr = data + field->offset;
             bptr < data + size && *ptr; ptr++) {
                int ls = 0;