All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH iproute2-net] ss: mptcp: subflow: display seq counters as decimal
@ 2025-02-26 11:08 Matthieu Baerts (NGI0)
  2025-03-10 16:20 ` patchwork-bot+netdevbpf
  0 siblings, 1 reply; 2+ messages in thread
From: Matthieu Baerts (NGI0) @ 2025-02-26 11:08 UTC (permalink / raw)
  To: Stephen Hemminger
  Cc: David Ahern, netdev, MPTCP Upstream, Mat Martineau, Geliang Tang,
	Davide Caratti, Matthieu Baerts (NGI0)

This is similar to commit cfa70237 ("ss: mptcp: display seq related
counters as decimal") but for the subflow info this time. This is also
aligned with what is printed for TCP sockets.

That looks better to do the same with the subflow info (ss -ti), to
compare with the MPTCP info (ss -Mi), or for those who want to easily
count how many bytes have been exchanged between two runs without having
to think in hexa.

Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
 misc/ss.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/misc/ss.c b/misc/ss.c
index aef1a714b130900d6f3061c150639235c1eb13ca..6d5976501467842b1681d7dfc1ca56cc1b4a98d5 100644
--- a/misc/ss.c
+++ b/misc/ss.c
@@ -3055,16 +3055,16 @@ static void mptcp_subflow_info(struct rtattr *tb[])
 		    rta_getattr_u32(tb[MPTCP_SUBFLOW_ATTR_TOKEN_LOC]),
 		    rta_getattr_u8(tb[MPTCP_SUBFLOW_ATTR_ID_LOC]));
 	if (tb[MPTCP_SUBFLOW_ATTR_MAP_SEQ])
-		out(" seq:%llx",
+		out(" seq:%llu",
 		    rta_getattr_u64(tb[MPTCP_SUBFLOW_ATTR_MAP_SEQ]));
 	if (tb[MPTCP_SUBFLOW_ATTR_MAP_SFSEQ])
-		out(" sfseq:%x",
+		out(" sfseq:%u",
 		    rta_getattr_u32(tb[MPTCP_SUBFLOW_ATTR_MAP_SFSEQ]));
 	if (tb[MPTCP_SUBFLOW_ATTR_SSN_OFFSET])
-		out(" ssnoff:%x",
+		out(" ssnoff:%u",
 		    rta_getattr_u32(tb[MPTCP_SUBFLOW_ATTR_SSN_OFFSET]));
 	if (tb[MPTCP_SUBFLOW_ATTR_MAP_DATALEN])
-		out(" maplen:%x",
+		out(" maplen:%u",
 		    rta_getattr_u32(tb[MPTCP_SUBFLOW_ATTR_MAP_DATALEN]));
 }
 

---
base-commit: 48717184ba3700938b75542046c5108b21e95ea7
change-id: 20250226-iproute2-net-ss-ulp-mptcp-dec-92801e84ac6f

Best regards,
-- 
Matthieu Baerts (NGI0) <matttbe@kernel.org>


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH iproute2-net] ss: mptcp: subflow: display seq counters as decimal
  2025-02-26 11:08 [PATCH iproute2-net] ss: mptcp: subflow: display seq counters as decimal Matthieu Baerts (NGI0)
@ 2025-03-10 16:20 ` patchwork-bot+netdevbpf
  0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2025-03-10 16:20 UTC (permalink / raw)
  To: Matthieu Baerts
  Cc: stephen, dsahern, netdev, mptcp, martineau, geliang, dcaratti

Hello:

This patch was applied to iproute2/iproute2.git (main)
by Stephen Hemminger <stephen@networkplumber.org>:

On Wed, 26 Feb 2025 12:08:44 +0100 you wrote:
> This is similar to commit cfa70237 ("ss: mptcp: display seq related
> counters as decimal") but for the subflow info this time. This is also
> aligned with what is printed for TCP sockets.
> 
> That looks better to do the same with the subflow info (ss -ti), to
> compare with the MPTCP info (ss -Mi), or for those who want to easily
> count how many bytes have been exchanged between two runs without having
> to think in hexa.
> 
> [...]

Here is the summary with links:
  - [iproute2-net] ss: mptcp: subflow: display seq counters as decimal
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=e3f9681d4a77

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-03-10 16:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-26 11:08 [PATCH iproute2-net] ss: mptcp: subflow: display seq counters as decimal Matthieu Baerts (NGI0)
2025-03-10 16:20 ` patchwork-bot+netdevbpf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.