Merge pull request #439 from Sushmakk/nvme-cli_virtualization_cmd

nvme-cli: Added support for virtualization-management command

Merge pull request #441 from minwooim/v1.7/fix_typo_in_readme

nvme-cli: readme: fix typo in Alpine distro support

Merge pull request #442 from chron0/patch-1

Added gentoo to README.md

Added gentoo to README.md

Working 1.6 build and functional output on current gentoo from mainline portage

nvme-cli: readme: fix typo in Alpine distro support

Fix typo 'akp' to 'apk'.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: Don't print result on passthru error

The kernel doesn't copy CQE DW0 to the ioctl result on error, so don't
print it.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Added support for virtualization-management command

Merge pull request #430 from Foxboron/foxboron/zsh-completion-make

Added zsh completion installation

Merge pull request #436 from hjmallon/spelling

Fix a host of typos in utils, tests and docs

nvme-ioctl: clear rae for discovery log pages

The discovery controller should now be able to send
us discovery log change events so we need to clear the
rae bit for that.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>

fabrics: add transport header and data digest

This setting is enabling header and data digest over NVMe/TCP
controllers.

Signed-off-by: Sagi Grimberg <sagi@lightbitslabs.com>

fabrics: add tcp port tsas decoding

tcp tsas include sectype indication for unsecured/tls ports.

Signed-off-by: Sagi Grimberg <sagi@lightbitslabs.com>

nvme: Add TCP transport

Signed-off-by: Sagi Grimberg <sagi@lightbitslabs.com>

fabrics: use trtype_str when parsing a discovery log entry

Reduce some code duplication.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>

fabrics: support fabrics sq flow control disable

If the discovery log entry indicates that the subsystem supports
disabling sq flow control, we ask the host to connect and disable sq
flow control (omit sq_head pointer updates). Also add a connect option
to explicitly ask for disable_sqflow.

If we failed to add_ctrl in connect-all command, we retry without it
to be backward compatible with a driver that cannot accept the new
flag.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>

Merge pull request #435 from hjmallon/micron

micron-nvme: Replace direct use of ioctl

Fix typos in tests and utils

Fix typos in docs

Fix printing of large ANAGRPID values

micron-nvme: Replace direct use of ioctl

Merge pull request #433 from freezer5/patch-2

id-ns: Fix printing of large ANAGRPID values

Merge pull request #431 from lightjd/master

In json output, if there is zero length string in command return data, it occur allocation failure & abort (cannot display empty value)

Fix printing of large ANAGRPID values

In json output, if the commands add zero length string, it occur allocation fail & abort (cannot display empty value)

Regen documentation

Update manpages with latest doc updates.

Signed-off-by: Keith Busch <keith.busch@intel.com>

list-subsys: Preserve line breaks and spaces

The manpage looked weird before. Make sure it preserves the desired format.

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-list-subsys: Add device name argument and print out ANA state

Update the 'nvme list-subsys' command to accept a device name and
print out the ANA state for all paths to that device.

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme: print out controller state for 'list-subsys'

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme: introduce get_nvme_ctrl_attr()

Use a generic function 'get_nvme_ctrl_attr()' instead of coding
the same function for each sysfs attribute.

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme: fixup ANA group descriptor offset

The ANA group descriptor offset changed between the specification
sent for ratification and the final ratified document, causing the
ANA log page not to be displayed correctly.

Signed-off-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Steve Schremmer <steve.schremmer@netapp.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: fix a typo

This patch fixes a typo in nvme-cli usage text.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Added zsh completion installation

Signed-off-by: Morten Linderud <morten@linderud.pw>

Merge pull request #428 from bdeyal/ealready_exit_0

connect-all: fix exit code on fail to write to /dev/nvme-fabrics

connect-all: special treatment to EALREADY afetr write to /dev/nvme-fabrics

nvme: Handle supported capabilities result for get features

When using the supported capabilities for get feature, SEL set to 3,
no data is transfered and the completion result is the controller
capabilities for the requested feature, not the actual feature. Handle
this accordingly.

Link: https://github.com/linux-nvme/nvme-cli/issues/426
Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: fix endianness for the structure field

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme-cli: fix array len to hold string of size 4

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme-cli: fix a typo

Signed-off-by: Rami Rosen <ramirose@gmail.com>

nvme-cli:fix a typo in SELF-TEST log

This patch fixes a typo in SELF-TEST log.

wdc: Use const string for suffix

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: fix ana-log endianness

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme-cli: fix sanitize status endianness

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

Merge pull request #425 from bdeyal/plugins_refactor

Refactor plugins in a file hierarchy

Refactor plugins in a file hierarchy

wdc: Fix strncpy warning

Some compilers were complaining copying from the constant source was
truncating it. The size is supposed to protect the destination buffer,
so this patch uses that size instead.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Re-gen documentation for new files

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli wdc plugin: Add documentation for pfail dump.

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli wdc plugin: Add get pfail dump command.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #420 from bsdimp/portable

Portability fixes

Merge pull request #421 from bdeyal/fix_rpm

nvme-cli make rpm:  fix bash completion and require in nvme.spec.in

nvme.spec.in : fix bash completion path, require

Update documentation

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: Add help text for wdc-plugin command vs-internal-log. Update help text for wdc-plugin command cap-diag.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

nvme-cli: Add wdc-plugin command vs-internal-log and update wdc-plug cap-diag command with transfer-size parameter.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

NVMe-CLI WDC Plugin - Simplify the device id checking routines.

This change will make one common function that will retrieve the pci
vendor and device id's and check them for a match of the passed in
id's.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

Merge pull request #418 from sephalon/oe-packaging

Makefile: fix bash completion install path

Merge pull request #417 from PeterDaveHelloKitchen/update-README.md

Enable code block syntax highlight in README.md

Makefile: fix bash completion install path

Change path to bash-completion upstream recommendation used by most
distributions.

Signed-off-by: Stefan Wiehler <stefan.wiehler@missinglinkelectronics.com>

Enable code block syntax highlight in README.md

nvme-cli: fix a typo

This patch fixes a typo in nvme-cli usage text.

Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Rami Rosen <ramirose@gmail.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: fix ioctl argument

The command argument for ioctl is unsigned long, not int. Fix all the
places that we pass cmd through to the ioctl call to handle it as an
unsigned long. This prevents a sign extension bug when IOCTL commands
encode with the top bit set.

Use standard interfaces to get timezone offsets.

Use the tm_gmtoff returned from localtime() rather than the global
timezone.  The formal is more portable, while the latter is specific
to glibc.  There's no good reason to use the glibc specific interface.

Initialize line.

Ensure line is initialized to NULL. This ensures that the return
expression doesn't use a uninitialized variable when we can't read sys
nodes.

We don't need to include linux/types.h here.

nvme-cli: remove endian.h

endian.h isn't needed, so remove it.

Include sys/stat.h

S_* mode bits are defined in sys/stat.h, per POSIX. Include this for
extra portability.

Include libgen.h for basename prototype.

Also, convert open_dev to take a char * and remove the cast from where
it's called. basenmae takes a char * and modifies the string. In this
case, such modifications are fine and only the const char * type was
in error.

Remove unnecessary linux/types.h include.

Remove unnecessary asm/byteorder.h

These includes aren't necessary, so remove them.

Use sys/ioctl.h to pull in definitions.

_IO* are part of the ioctl intereface, so use the more portable
sys/ioctl.h to get the definitions. Eliminate all uses of linux/fs.h
that were needed only for this.

nvme-cli: make read error handling uniform

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme-cli: fix indentation for dir_[send/receive]

This is a cleanup patch to get rid of the inconsistent indentation
for the dir_send() and dir_receive() functions.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme-cli: report subsystem-reset not supported by controller

Add a little more description for "nvme subsystem-reset"
failures, currently driver reports "Inappropriate ioctl
for device".

$ nvme subsystem-reset /dev/nvme0
Subsystem-reset: Inappropriate ioctl for device

With this change report controller doesn't support
NVM Subsystem Reset.

$ nvme subsystem-reset /dev/nvme0
Subsystem-reset: NVM Subsystem Reset not supported.

nvme-cli: prevent resv action field overflow

Signed-off-by: Austin Eberle <ajeberle@us.ibm.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Fix nvme test device selection to use controller value defined in the config.json file. Before this fix, the value was hard coded to nvme0.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

Add PCI device id checking for the WDC plugin commands.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

Merge branch 'master' of https://github.com/linux-nvme/nvme-cli

Update gitignore for generated test files

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: tests: Add get_id_ctrl function

Signed-off-by: Dong Ho <dong.ho@wdc.com>

nvme-cli: tests: fix id_ns test coding style

Simplified and fixed coding style to conform with the style of all other tests.

Signed-off-by: Dong Ho <dong.ho@wdc.com>

nvme-cli: tests: fix id_ctrl test coding style

Simplified and fixed coding style to conform with the style of all other tests.

Signed-off-by: Dong Ho <dong.ho@wdc.com>

nvme-cli: tests: fix fw_log test coding style

Simplified and fixed coding style to conform with the style of all other tests.

Signed-off-by: Dong Ho <dong.ho@wdc.com>

Merge pull request #413 from bdeyal/fix_attach_ns_segfault

attach-ns : fix segfault for ctrl list wrong input

nvme-cli: print "\n" after nvme list-subsys -o json

Signed-off-by: Yi Zhang <yi.zhang@redhat.com>

nvme-cli: kill nvme_passthru_admin()

The seagate plugin is the only one using it, and it's identical
to nvme_get_log13(). So replace the invocation with nvme_get_log13()
and remove the function.

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme-cli: Add 'rae' argument to nvme_get_log()

We should be setting the 'rae' bit when sending a log page; we don't
want to accidentally overwriting any outstanding AENs when calling
nvme-cli.

Signed-off-by: Hannes Reinecke <hare@suse.com>

seagate-nvme: fixup coding style

One really should try to adhere to one coding style in one file ...

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme-discover: Re-check generation counter after log page transfer

The log page transfer might have been split up in several chunks, so
it might happen that the generation counter changed inbetween these
transfer. Hence we need to re-fetch the header again to figure out
if there had been changes to the generation counter; if so we need
to fetch the entire page again.

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme-ioctl: retrieve log pages in 4k chunks

Some log pages increase with the scale of the subsystem, so the
overall size might be larger than the MDTS value for that controller.
In order to avoid having to pass in the MDTS value we should restrict
the transfer size to 4k and retrieve the log page in 4k chunks.

Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme-discover: Retry discovery log if the generation counter changes

If the generation counter changes we need to validate if the number
of records has changed, too.
If so we need to retry retrieving the discovery log to the most recent
values. The retry will be terminated after MAX_DISC_RETRIES (currently
set to 10) to avoid infinite recursion.

Signed-off-by: Hannes Reinecke <hare@suse.com>

attach-ns : fix segfault for ctrl list wrong input

Regen documentation

Signed-off-by: Keith Busch <keith.busch@intel.com>

fabrics: add disconnect-all command

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme: commonize subsystems info in a helper

We will want that to reuse for other ops that
will require to scan subsystems or controllers

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

fabrics: don't fail empty discovery log page

It can be possible that discovery subsystem will not return
any valid discovery records.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

fabrics: make some arguments integers

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #410 from RalphCorderoy/avoid-truncating-path

Enlarge local char[] so snprintf(3) can't truncate path.

Enlarge local char[] so snprintf(3) can't truncate path.

The path is in a char[310], contributing at most 309 chars for the ‘%s’.
‘/subsysnqn’ is a further 10.  Plus one for the NUL.
309 + 10 + 1 = 320 so sspath[319] is too short.

nvme-cli: make malloc error handling uniform

This patch makes malloc() memory handling uniform by using errno based
error message.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

Merge pull request #407 from bdeyal/fix_nvme_list

nvme list : fix nvme list output if identify failed on device