net: fixup for tracepoint napi:napi_poll
authorJesper Dangaard Brouer <brouer@redhat.com>
Fri, 15 Jul 2016 21:55:20 +0000 (23:55 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 15 Jul 2016 22:55:01 +0000 (15:55 -0700)
The recent change to tracepoint napi:napi_poll changed the order of
the parameters that perf scripts sees, the printk was correct.  The
problem was that the new parameters (work and budget) were pushed
in front of dev_name.

The new parameters obviously need to be appended to keep backward
compatible.

Fixes: 1db19db7f5ff ("net: tracepoint napi:napi_poll add work and budget")
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/trace/events/napi.h

index 118ed776763941ea7fd0a28aa4dd7d31648fedf2..0b9e5136a2a3a074dac3fa66bbc4ce4f07304eaf 100644 (file)
@@ -18,16 +18,16 @@ TRACE_EVENT(napi_poll,
 
        TP_STRUCT__entry(
                __field(        struct napi_struct *,   napi)
+               __string(       dev_name, napi->dev ? napi->dev->name : NO_DEV)
                __field(        int,                    work)
                __field(        int,                    budget)
-               __string(       dev_name, napi->dev ? napi->dev->name : NO_DEV)
        ),
 
        TP_fast_assign(
                __entry->napi = napi;
+               __assign_str(dev_name, napi->dev ? napi->dev->name : NO_DEV);
                __entry->work = work;
                __entry->budget = budget;
-               __assign_str(dev_name, napi->dev ? napi->dev->name : NO_DEV);
        ),
 
        TP_printk("napi poll on napi struct %p for device %s work %d budget %d",