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
F103976904: D15162.diff
Sun, Dec 1, 11:17 PM
Unknown Object (File)
Oct 5 2024, 10:30 PM
Unknown Object (File)
Oct 4 2024, 3:45 PM
Unknown Object (File)
Oct 2 2024, 9:17 AM
Unknown Object (File)
Oct 2 2024, 5:25 AM
Unknown Object (File)
Sep 30 2024, 10:27 PM
Unknown Object (File)
Sep 16 2024, 10:29 PM
Unknown Object (File)
Sep 16 2024, 7:09 PM
View All 38 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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
head/
sys/
dev/
nvme/
15 lines

Diff 41784

View Options

head/sys/dev/nvme/nvme_ns.c

Loading...