]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
selftests: drv-net: gro: remove TOTAL_HDR_LEN
authorJakub Kicinski <kuba@kernel.org>
Thu, 2 Apr 2026 20:59:57 +0000 (13:59 -0700)
committerJakub Kicinski <kuba@kernel.org>
Fri, 3 Apr 2026 22:05:44 +0000 (15:05 -0700)
Willem points out TOTAL_HDR_LEN is identical to MAX_HDR_LEN.
This seems to have been the case ever since the test was added.
Replace the uses of TOTAL_HDR_LEN with MAX_HDR_LEN, MAX seems
more common for what this value is.

Reviewed-by: Willem de Bruijn <willemb@google.com>
Link: https://patch.msgid.link/20260402210000.1512696-6-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/lib/gro.c

index 57080ecc3df8857785bea0561ed69074e868e959..3616ecc42a01981697b970faa64fbad9cefa2b15 100644 (file)
@@ -94,7 +94,6 @@
 #define START_SEQ 100
 #define START_ACK 100
 #define ETH_P_NONE 0
-#define TOTAL_HDR_LEN (ETH_HLEN + sizeof(struct ipv6hdr) + sizeof(struct tcphdr))
 #define MSS (4096 - sizeof(struct tcphdr) - sizeof(struct ipv6hdr))
 #define MAX_PAYLOAD (IP_MAXPACKET - sizeof(struct tcphdr) - sizeof(struct ipv6hdr))
 #define NUM_LARGE_PKT (MAX_PAYLOAD / MSS)
@@ -526,9 +525,9 @@ static void send_data_pkts(int fd, struct sockaddr_ll *daddr,
  */
 static void send_large(int fd, struct sockaddr_ll *daddr, int remainder)
 {
-       static char pkts[NUM_LARGE_PKT][TOTAL_HDR_LEN + MSS];
-       static char last[TOTAL_HDR_LEN + MSS];
-       static char new_seg[TOTAL_HDR_LEN + MSS];
+       static char pkts[NUM_LARGE_PKT][MAX_HDR_LEN + MSS];
+       static char last[MAX_HDR_LEN + MSS];
+       static char new_seg[MAX_HDR_LEN + MSS];
        int i;
 
        for (i = 0; i < NUM_LARGE_PKT; i++)