Thread (10 messages) flat view 10 messages, 2 authors, 2015-08-28

Re: [PATCH net-next] net: FIB tracepoints

From: David Miller <davem@davemloft.net>
Date: 2015-08-28 05:16:44

From: David Ahern <redacted>
Date: Thu, 27 Aug 2015 22:07:30 -0700
On 8/27/15 10:05 PM, David Miller wrote:
quoted
From: David Ahern <redacted>
Date: Thu, 27 Aug 2015 21:59:32 -0700
quoted
+		__array(	__u8,	src,	4	)
+		__array(	__u8,	dst,	4	)
  ...
quoted
+		__array(	__u8,	src,	4	)
Maybe there is something I don't understand about tracing, but why not
use __u32?  If endianness types are the issue, just force cast it as
needed.

Using a memcpy() on a 4-byte array is kinda excessive.
Silly trick need to use %pI4. ie., printing the addresses as strings
vs. hex.
%pI4 doesn't care what kind of pointer you give it, &__entry->src will
work just fine if you used __u32.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help