diff options
| author | Mathieu Desnoyers <mathieu.desnoyers@efficios.com> | 2011-01-06 12:53:19 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2011-01-07 20:55:42 -0500 |
| commit | 881fe4bdcdc899674524e30a76c76d298b447008 (patch) | |
| tree | ce463d1c92168af7f127bb706b4a868ef8ca3ad2 /include | |
| parent | ec6e7c3ae39cb1dc279b5274aaaadd09ff8e224b (diff) | |
tracing: remove duplicate null-pointer check in skb tracepoint
The check for NULL skb in the kfree_skb trace event is a duplicate from the
check already done in its only caller, kfree_skb(). Remove this duplicate check.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
LKML-Reference: <20110106175319.GA30610@Krystal>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: David S. Miller <davem@davemloft.net>
CC: Steven Rostedt <rostedt@goodmis.org>
CC: Frederic Weisbecker <fweisbec@gmail.com>
CC: Ingo Molnar <mingo@elte.hu>
CC: Thomas Gleixner <tglx@linutronix.de>
CC: Zhaolei <zhaolei@cn.fujitsu.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/trace/events/skb.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/trace/events/skb.h b/include/trace/events/skb.h index 75ce9d500d8e..f10293c41b1e 100644 --- a/include/trace/events/skb.h +++ b/include/trace/events/skb.h | |||
| @@ -25,9 +25,7 @@ TRACE_EVENT(kfree_skb, | |||
| 25 | 25 | ||
| 26 | TP_fast_assign( | 26 | TP_fast_assign( |
| 27 | __entry->skbaddr = skb; | 27 | __entry->skbaddr = skb; |
| 28 | if (skb) { | 28 | __entry->protocol = ntohs(skb->protocol); |
| 29 | __entry->protocol = ntohs(skb->protocol); | ||
| 30 | } | ||
| 31 | __entry->location = location; | 29 | __entry->location = location; |
| 32 | ), | 30 | ), |
| 33 | 31 | ||
