Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 01 Aug 2020 09:41:08 +0000
From:      bugzilla-noreply@freebsd.org
To:        net@FreeBSD.org
Subject:   [Bug 248306] if_mvneta: Corrupts TX packets when TXCSUM is not used
Message-ID:  <bug-248306-7501-Prv2uSZMY1@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-248306-7501@https.bugs.freebsd.org/bugzilla/>
References:  <bug-248306-7501@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D248306

--- Comment #9 from commit-hook@FreeBSD.org ---
A commit references this bug:

Author: mw
Date: Sat Aug  1 09:40:19 UTC 2020
New revision: 363759
URL: https://svnweb.freebsd.org/changeset/base/363759

Log:
  Fix TX csum handling in if_mvneta

  The mvneta device requires MVNETA_TX_CMD_L4_CHECKSUM_NONE bit to be set in
the tx descriptor is checksum not required. However, mvneta_tx_set_csumflag=
()
is not setting this flag currently, causing the hardware to randomly corrup=
t IP
header during transmission.

  This affects injected IPv4 packets that skips kernel IP stack processing
(e.g. DHCP), as well as all IPv6 packets, since the driver currently does n=
ot
offload csum for IPv6.

  The fix is to remove all the early return paths from mvneta_tx_set_csumfl=
ag()
which do not set the MVNETA_TX_CMD_L4_CHECKSUM_NONE flag.

  PR: 248306
  Submitted by: Mike Cui <cuicui@gmail.com>
  Reported by: Mike Cui <cuicui@gmail.com>

Changes:
  head/sys/dev/neta/if_mvneta.c

--=20
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-248306-7501-Prv2uSZMY1>