Thread (10 messages) flat view 10 messages, 6 authors, 2021-02-04

Re: [PATCH] net: tracepoint: exposing sk_family in all tcp:tracepoints

From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-02-03 18:45:36

On Wed, 3 Feb 2021 10:25:11 -0800 Hariharan Ananthakrishnan wrote:
On Wed, Feb 3, 2021 at 10:16 AM Jakub Kicinski [off-list ref] wrote:
quoted
On Wed, 3 Feb 2021 08:26:04 -0800 sdf@google.com wrote:  
quoted
They are not ABI and since we are extending tracepoints with additional
info (and not removing any existing fields) it shouldn't be a problem.  
Okay, but we should perhaps add the field at the end just to be on the
safe side (and avoid weird alignment of the IP addresses).  
I added it after dport to be consistent with the earlier patch to
sock:inet_sock_set_state
https://lore.kernel.org/patchwork/patch/870492/.
I see :(

I'll give it a few more hours and if there are no objections apply the
patch.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help