Page MenuHomeFreeBSD

nda: Only attach to devices whose progif if nvme storage
AcceptedPublic

Authored by imp on Sep 3 2025, 6:28 PM.
Tags
None
Referenced Files
F143178735: D52363.id.diff
Tue, Jan 27, 12:37 AM
Unknown Object (File)
Sun, Jan 25, 8:15 AM
Unknown Object (File)
Sat, Jan 24, 9:33 AM
Unknown Object (File)
Wed, Jan 21, 11:55 AM
Unknown Object (File)
Sat, Jan 10, 4:37 AM
Unknown Object (File)
Fri, Jan 9, 6:57 PM
Unknown Object (File)
Dec 27 2025, 12:15 PM
Unknown Object (File)
Dec 16 2025, 6:50 PM
Subscribers

Details

Reviewers
jhb
chuck
Group Reviewers
cam
Summary

There's a number of PCIe cards today that are not, strictly speaking
storage cards. For any PCIe card, make sure that it supports the proper
PCI ProgIF before attaching to it.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 66791
Build 63674: arc lint + arc unit

Event Timeline

imp requested review of this revision.Sep 3 2025, 6:28 PM
chuck added a subscriber: chuck.
chuck added inline comments.
sys/cam/nvme/nvme_da.c
678

Would it make sense to log this? No log message for the previous "device is not NVMe" is fine, but here, the code is noticing "this is the wrong kind of NVMe device" feels different (maybe).

This revision is now accepted and ready to land.Sep 8 2025, 5:12 PM
sys/cam/nvme/nvme_da.c
678

I think so, so long as we can have a good error message. It will be rare to see this, so we don't need to rate limit it, and it will happen only with weird, odd-ball cards, which is fine for those users.