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)
Mon, Jul 20, 1:22 AM
Unknown Object (File)
Sun, Jul 12, 5:16 PM
Unknown Object (File)
Mon, Jun 29, 10:14 PM
Unknown Object (File)
Mon, Jun 22, 2:04 AM
Unknown Object (File)
Jun 19 2026, 2:25 AM
Unknown Object (File)
May 19 2026, 9:01 AM
Unknown Object (File)
May 11 2026, 6:21 PM
Unknown Object (File)
May 11 2026, 9:46 AM
View All 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...