Page MenuHomeFreeBSD

Make camcontrol identify work with nda devices
ClosedPublic

Authored by imp on May 9 2018, 7:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 3, 5:49 AM
Unknown Object (File)
Thu, Nov 27, 3:35 AM
Unknown Object (File)
Tue, Nov 25, 11:15 AM
Unknown Object (File)
Tue, Nov 25, 12:51 AM
Unknown Object (File)
Sat, Nov 22, 12:17 PM
Unknown Object (File)
Fri, Nov 21, 1:46 AM
Unknown Object (File)
Fri, Nov 21, 1:41 AM
Unknown Object (File)
Fri, Nov 21, 1:41 AM
Subscribers

Details

Summary

Make it possible to use print_controller from another program

Rename print_controller to nvme_print_controller. Put it in its
own file for easy inclusion. Move util.c to be nc_util.c to not
conflict with camcontrol. add nvecontrol_ext.h to define shared
interfaces.

Both ATA and NVME have an identify command. They are completely
different, but to the user they are the same. Leverage nvmecontrol's
print_controller code to provide that functionality to camcontrol
identify. Query the path to see what kind of protocol it supports, and
send the most appropriate command down. Refactor nvme_print_dev a
little to make it easy to get the nvme cdata.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

the "new" nvme_print_controller() function appears to be missing from your diff. Missing a file?

This revision is now accepted and ready to land.May 9 2018, 8:26 PM
This revision was automatically updated to reflect the committed changes.