Page MenuHomeFreeBSD

nvme: honor CRTO for the controller ready timeout
Needs ReviewPublic

Authored by seuros on Sun, Sep 13, 9:15 AM.
Referenced Files
Unknown Object (File)
Fri, Oct 2, 12:57 PM
Unknown Object (File)
Fri, Oct 2, 11:12 AM
Unknown Object (File)
Thu, Oct 1, 8:33 PM
Unknown Object (File)
Thu, Oct 1, 2:21 PM
Unknown Object (File)
Thu, Oct 1, 7:09 AM
Unknown Object (File)
Wed, Sep 30, 8:16 PM
Unknown Object (File)
Wed, Sep 30, 1:07 PM
Unknown Object (File)
Tue, Sep 29, 6:01 PM
Subscribers

Details

Reviewers
imp
adrian
dab
Summary

CAP.TO is a 8b field and saturate at 127.5 seconds; a controller
that needs longer reports the real worst case in CRTO.CRWMT.
Waiting only CAP.TO can reset a controller that is still within its declared
recovery time after e.g. an abrupt shutdown.
Taking the larger of CRTO.CRWMT and CAP.TO when CAP.CRWMS show CRTO support.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 76821
Build 73704: arc lint + arc unit

Event Timeline

seuros created this revision.

@seuros : I won't be able to provide a ship-it this weekend, but could you please provide the spec link for this and other related items in the future so @adrian and I can help provide you quicker ship-its?

(to be fair I have the attention span of Doug from Up and the short-term memory of a Dory from Finding Nemo: having the info helps future me track down what I was thinking if I need to look it up later :)..)

@seuros : I won't be able to provide a ship-it this weekend, but could you please provide the spec link for this and other related items in the future so @adrian and I can help provide you quicker ship-its?

(to be fair I have the attention span of Doug from Up and the short-term memory of a Dory from Finding Nemo: having the info helps future me track down what I was thinking if I need to look it up later :)..)

CAP.TO Figure 36, p.57
CRTO Figure 57, offset 68h, page 73

Everything is in NVM-Express-Base-Specification-Revision-2.4-Ratified-2026.07.31.pdf .

The new diffs have references.