nvmf.h: New header defining ioctls for NVMe over Fabrics

[jhb]

Summary

This defines structures, ioctl commands, and related constants used
for both the Fabrics host and controller.

Sponsored by: Chelsio Communications

[imp]

One nit... also applies to all of this series of reviews: consider omitting the license boilerplate.

sys/dev/nvme/nvme.h

47

won't line numbers go stale quickly?

[jhb]

In D44706#1020670, @imp wrote:

One nit... also applies to all of this series of reviews: consider omitting the license boilerplate.

Hmm, are we ready to do that? I know we have some SPDX-only files in the tree and are kind of allowing it, but I wasn't sure we'd crossed the point to preferring that by default.

sys/dev/nvme/nvme.h

47

Not the line number, the constant 200 as the second argument to _IO*

[imp]

In D44706#1021491, @jhb wrote:

In D44706#1020670, @imp wrote:

One nit... also applies to all of this series of reviews: consider omitting the license boilerplate.

Hmm, are we ready to do that? I know we have some SPDX-only files in the tree and are kind of allowing it, but I wasn't sure we'd crossed the point to preferring that by default.

hence my suggesting that you consider it: it's an option....

sys/dev/nvme/nvme.h

47

Oh! That's a note for reserved future use...

[jhb]

Add NVMF_DISCONNECT_HOST and NVMF_DISCONNECT_ALL ioctls for /dev/nvmf

[jhb]

Add NVMF_DISCONNECT_HOST and NVMF_DISCONNECT_ALL ioctls for /dev/nvmf

[jhb]

Switch to SPDX-only license blocks for C files

[jhb]

Do we want to omit license blocks in manpages as well as C source? I've left them in the manpages for now.