Page MenuHomeFreeBSD

sfxge(4): ignore error in completion event on MCDIv2 HW
ClosedPublic

Authored by arybchik on Nov 21 2018, 11:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 16, 9:33 PM
Unknown Object (File)
Tue, Apr 14, 11:39 PM
Unknown Object (File)
Mon, Apr 13, 11:35 PM
Unknown Object (File)
Sun, Apr 5, 10:19 PM
Unknown Object (File)
Sun, Apr 5, 4:52 PM
Unknown Object (File)
Sun, Apr 5, 8:58 AM
Unknown Object (File)
Fri, Apr 3, 8:23 PM
Unknown Object (File)
Fri, Mar 27, 6:33 PM
Subscribers

Details

Summary

With MCDIv2, the reponse length can be to big to fit into the
CMDDONE_DATALEN field in the MCDI completion event. But rather that
the length being truncated, it can overflow into the CMDDONE_ERRNO
field (this is a longstanding firmware bug). Hence the CMDDONE_ERRNO
field may not be valid.

It isn't necessary to use the value in the CMDDONE_ERRNO field though,
so it can be ignored. The actual error code is already read from the
response header on MCDIv2 capable hardware and stored in emr_rc, so
that can be used instead.

Submitted by: Mark Spender <mspender at solarflare.com>
Sponsored by: Solarflare Communications, Inc.

Test Plan

amd64 tinderbox kernel builds tested
driver loaded/unloaded, interfaces up/down, traffic works fine

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Nov 23 2018, 4:00 PM
This revision was automatically updated to reflect the committed changes.