Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Jul 2021 12:00:31 GMT
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 1b36b3869f3b - main - mlx5en: eliminate magic constant
Message-ID:  <202107121200.16CC0Vhp008361@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by kib:

URL: https://cgit.FreeBSD.org/src/commit/?id=1b36b3869f3be722dd15760ddcb4598b6dfe7cf7

commit 1b36b3869f3be722dd15760ddcb4598b6dfe7cf7
Author:     Konstantin Belousov <konstantinb@nvidia.com>
AuthorDate: 2021-04-06 03:30:09 +0000
Commit:     Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2021-07-12 09:34:37 +0000

    mlx5en: eliminate magic constant
    
    Use sizeof(struct udphdr) where appropriate
    
    Reviewed by:    hselasky
    Sponsored by:   Mellanox Technologies/NVidia Networking
    MFC after:      1 week
---
 sys/dev/mlx5/mlx5_en/mlx5_en_tx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/dev/mlx5/mlx5_en/mlx5_en_tx.c b/sys/dev/mlx5/mlx5_en/mlx5_en_tx.c
index 124d2afe414c..1ba47e44ae96 100644
--- a/sys/dev/mlx5/mlx5_en/mlx5_en_tx.c
+++ b/sys/dev/mlx5/mlx5_en/mlx5_en_tx.c
@@ -277,7 +277,7 @@ mlx5e_get_full_header_size(const struct mbuf *mb, const struct tcphdr **ppth)
 			goto tcp_packet;
 		case IPPROTO_UDP:
 			ip_hlen = ip->ip_hl << 2;
-			eth_hdr_len += ip_hlen + 8;
+			eth_hdr_len += ip_hlen + sizeof(struct udphdr);
 			th = NULL;
 			goto udp_packet;
 		default:
@@ -293,7 +293,7 @@ mlx5e_get_full_header_size(const struct mbuf *mb, const struct tcphdr **ppth)
 			eth_hdr_len += sizeof(*ip6);
 			goto tcp_packet;
 		case IPPROTO_UDP:
-			eth_hdr_len += sizeof(*ip6) + 8;
+			eth_hdr_len += sizeof(*ip6) + sizeof(struct udphdr);
 			th = NULL;
 			goto udp_packet;
 		default:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202107121200.16CC0Vhp008361>