lkml.org 
[lkml]   [2020]   [Apr]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] mptcp: move pr_fmt defining to protocol.h
From
Date
Hi Geliang,

On 03/04/2020 09:57, Geliang Tang wrote:
> Some of the mptcp logs didn't print out the format string "MPTCP":
>
> [ 129.185774] DSS
> [ 129.185774] data_fin=0 dsn64=1 use_map=1 ack64=1 use_ack=1
> [ 129.185774] data_ack=5481534886531492085
> [ 129.185775] data_seq=15725204003114694615 subflow_seq=1425409 data_len=5216
> [ 129.185776] subflow=0000000093526a92 fully established=1 seq=0:0 remaining=28
> [ 129.185776] MPTCP: msk=00000000d5a704a6 ssk=00000000b5aabc31 data_avail=0 skb=0000000088f05424
> [ 129.185777] MPTCP: seq=15725204003114694615 is64=1 ssn=1425409 data_len=5216 data_fin=0
> [ 129.185777] MPTCP: msk=00000000d5a704a6 ssk=00000000b5aabc31 status=0
> [ 129.185778] MPTCP: msk ack_seq=da3b25b9a233c2c7 subflow ack_seq=da3b25b9a233c2c7
> [ 129.185778] MPTCP: msk=00000000d5a704a6 ssk=00000000b5aabc31 data_avail=1 skb=000000000caed2cc
> [ 129.185779] subflow=0000000093526a92 fully established=1 seq=0:0 remaining=28
>
> So this patch moves the pr_fmt defining from protocol.c to protocol.h, which
> is included by all the C files. Then we can get the same format string
> "MPTCP" in all mptcp logs like this:
>
> [ 141.854787] MPTCP: DSS
> [ 141.854788] MPTCP: data_fin=0 dsn64=1 use_map=1 ack64=1 use_ack=1
> [ 141.854788] MPTCP: data_ack=18028325517710311871
> [ 141.854788] MPTCP: data_seq=6163976859259356786 subflow_seq=3309569 data_len=8192
> [ 141.854789] MPTCP: msk=000000005847a66a ssk=0000000022469903 data_avail=0 skb=00000000dd95efc3
> [ 141.854789] MPTCP: seq=6163976859259356786 is64=1 ssn=3309569 data_len=8192 data_fin=0
> [ 141.854790] MPTCP: msk=000000005847a66a ssk=0000000022469903 status=0
> [ 141.854790] MPTCP: msk ack_seq=558ad84b9be1d162 subflow ack_seq=558ad84b9be1d162
> [ 141.854791] MPTCP: msk=000000005847a66a ssk=0000000022469903 data_avail=1 skb=000000000b8926f6
> [ 141.854791] MPTCP: subflow=00000000e4e4579c fully established=1 seq=0:0 remaining=28
> [ 141.854792] MPTCP: subflow=00000000e4e4579c fully established=1 seq=0:dcdf2f3b remaining=28

Good idea to uniform that.
I think it can be useful for MPTCP devs to add a different prefix in
each MPTCP .c files but this small improvement can be done later.

LGTM, thanks Geliang!

Reviewed-by: Matthieu Baerts <matthieu.baerts@tessares.net>

\
 
 \ /
  Last update: 2020-04-03 12:30    [W:0.079 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site