Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 20 Apr 2023 19:22:19 +0000
From:      bugzilla-noreply@freebsd.org
To:        bugs@FreeBSD.org
Subject:   [Bug 270965] iflib/ice(4): invalid sized packet sent via netmap triggers MDD
Message-ID:  <bug-270965-227@https.bugs.freebsd.org/bugzilla/>

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

            Bug ID: 270965
           Summary: iflib/ice(4): invalid sized packet sent via netmap
                    triggers MDD
           Product: Base System
           Version: CURRENT
          Hardware: amd64
                OS: Any
            Status: New
          Severity: Affects Only Me
          Priority: ---
         Component: kern
          Assignee: bugs@FreeBSD.org
          Reporter: brian90013@gmail.com

Hello,

We recently saw a machine with Intel E810 cards stop transmitting and start
printing "Malicious Driver Detection Tx Descriptor check event 'Packet too
small or too big'" to the console. Some investigation showed a user applica=
tion
was incorrectly building packets over 9724 bytes and passing them via netma=
p to
the ice driver.

We quickly avoided the MDD event by fixing the user application. However, I=
 was
surprised there appears to be no length checking when using netmap with ifl=
ib
drivers.

Looking at iflib_netmap_txsync(), it assembles the set of fragments composi=
ng a
packet and passes them to ctx->isc_txd_encap() aka ice_ift_txd_encap(). That
function has an int return value and seems like an excellent place to compa=
re
the total packet length (in pi->ipi_len) against ICE_MAX_FRAME_SIZE. (In fa=
ct
the ixl driver does this with MPASS().)

The issue is iflib_netmap_txsync() ignores the return code and always
increments the nic_i pointer. I have made several attempts to modify this
function to handle failures from isc_txd_encap but none have passed all my
tests.

I wonder if there is a reasonable way to modify iflib_netmap_txsync() to dr=
op
fragments if isc_txd_encap returns an error code? I realize this code is in=
 the
fast path and should be kept to a minimum. However, this change would let t=
he
ice driver (and other iflib users) inspect packet sizes, avoid this class of
MDD events, and keep the interface up and running.

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



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