Page MenuHomeFreeBSD
Differential D15162

switch nvme driver from make_dev_credf to make_dev_s
ClosedPublic
Actions

Authored by hordijk_netapp.com on Apr 23 2018, 4:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 26, 3:25 AM
Unknown Object (File)
Thu, Dec 25, 12:33 AM
Unknown Object (File)
Dec 11 2025, 12:29 PM
Unknown Object (File)
Nov 27 2025, 12:10 PM
Unknown Object (File)
Nov 21 2025, 2:44 AM
Unknown Object (File)
Nov 15 2025, 8:36 AM
Unknown Object (File)
Oct 27 2025, 4:09 PM
Unknown Object (File)
Oct 27 2025, 1:34 PM
View All 91 Files
Subscribers
john.a.meneghini_gmail.com

Details

Reviewers
jimharris
imp
Commits
rS332897: Migrate to make_dev_s interface to populate /dev/nvmeX entries
Summary
Leveraging the newer interface, removing deprecated API.  Also allows us
to remove the redundant check on the cdev entry being non-NULL.
Test Plan

verified /dev/nvme device gets created

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16260
Build 16208: arc lint + arc unit

Event Timeline

hordijk_netapp.com created this revision.Apr 23 2018, 4:27 AM
Harbormaster completed remote builds in B16260: Diff 41740.Apr 23 2018, 4:27 AM
hordijk_netapp.com added reviewers: jimharris, imp.Apr 23 2018, 4:28 AM
hordijk_netapp.com added a subscriber: john.a.meneghini_gmail.com.
jimharris accepted this revision.Apr 23 2018, 8:47 PM
This revision is now accepted and ready to land.Apr 23 2018, 8:47 PM
imp accepted this revision.Apr 23 2018, 9:26 PM

Saves us a check at the cost of extra lines :)
this looks good.

jimharris added a comment.Apr 23 2018, 9:44 PM

Warner - can you commit this to HEAD?

Closed by commit rS332897: Migrate to make_dev_s interface to populate /dev/nvmeX entries (authored by imp). · Explain WhyApr 23 2018, 10:30 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
dev/
nvme/
15 lines

Diff 41740

View Options

sys/dev/nvme/nvme_ns.c

Loading...