ccr: Disable requests on port 1 when needed to workaround a firmware bug.
ClosedPublic
Actions

Authored by jhb on Sep 28 2020, 8:17 PM.

Details

Reviewers

Commits

rG5fe0cd6503d3: ccr: Disable requests on port 1 when needed to workaround a firmware bug.

Summary

Completions for crypto requests on port 1 can sometimes return a stale
cookie value due to a firmware bug. Disable requests on port 1 by
default on affected firmware.

Test Plan

was able to reproduce a panic when sending traffic over the second port of a T6 when port 1 was used for crypto requests

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 33863
Build 31074: arc lint + arc unit

Event Timeline

jhb created this revision.Sep 28 2020, 8:17 PM

Herald added a subscriber: ae. · View Herald TranscriptSep 28 2020, 8:17 PM

jhb requested review of this revision.Sep 28 2020, 8:17 PM

Harbormaster completed remote builds in B33863: Diff 77610.Sep 28 2020, 8:17 PM

jhb retitled this revision from Only enable port 0 by default. to ccr: Only enable port 0 by default..Mar 9 2021, 9:55 PM

jhb edited the summary of this revision. (Show Details)

np requested changes to this revision.Mar 10 2021, 5:46 AM

np added inline comments.

sys/dev/cxgbe/crypto/t4_crypto.c
2202

This revision now requires changes to proceed.Mar 10 2021, 5:46 AM

np added inline comments.Mar 10 2021, 5:51 AM

sys/dev/cxgbe/crypto/t4_crypto.c
2196–2202	Comment needs to be updated.

Update comment.

Herald added a subscriber: imp. · View Herald TranscriptMar 11 2021, 1:12 AM

Harbormaster completed remote builds in B37759: Diff 85537.Mar 11 2021, 1:12 AM

jhb retitled this revision from ccr: Only enable port 0 by default. to ccr: Disable requests on port 1 when needed to workaround a firmware bug..Mar 11 2021, 1:20 AM

jhb edited the summary of this revision. (Show Details)