The TCP header fields seq and ack_seq are 32-bit values in network
byte order as (__be32). these fields were earlier printed using
ntohs(), which converts only 16-bit values and produces incorrect
results for 32-bit fields. This patch is changeing the conversion
to ntohl(), ensuring correct interpretation of these sequence numbers.
Notably, the format specifier is updated from %d to %u to reflect the
unsigned nature of these fields.
improves the accuracy of debug log messages for TCP sequence and
acknowledgment numbers during TX timeouts.
Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20250717193552.3648791-1-alok.a.tiwari@oracle.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
                                                 ntohs(tcphdr->source));
                                        dev_info(dev, "TCP dest port %d\n",
                                                 ntohs(tcphdr->dest));
-                                       dev_info(dev, "TCP sequence num %d\n",
-                                                ntohs(tcphdr->seq));
-                                       dev_info(dev, "TCP ack_seq %d\n",
-                                                ntohs(tcphdr->ack_seq));
+                                       dev_info(dev, "TCP sequence num %u\n",
+                                                ntohl(tcphdr->seq));
+                                       dev_info(dev, "TCP ack_seq %u\n",
+                                                ntohl(tcphdr->ack_seq));
                                } else if (ip_hdr(skb)->protocol ==
                                           IPPROTO_UDP) {
                                        udphdr = udp_hdr(skb);