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>

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

Merge branch 'master' into fix_nvme_list

nvme-cli: mark functions static

Mark nvme_property() and get_nvme_subsystem_info() static.

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

nvme-cli: make wdc plug-in functions static

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

nvme-cli: fix indentation

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

nvme-cli: use uniform error message for open

This patch adds errno based error message after open() system call
when open() system call fails.

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

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

changed files:
nvme.c        function list()
nvme-ioctl.c  function nvme_get_nsid() : return -errno if fstat fails

nvme-cli: Fix the displayed metadata length value on Big Endian systems.

On BE systems, the metadata length value displayed as part of the
format in the nvme list command is byte swapped.  So a value of 0
appears just fine but a value of 8 bytes is displayed as 0x0800.
This patch fixes that is the value is correct for both LE and BE
systems.

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

nvme-cli: netapp plugin handle empty strings

Handle controller fields with strings of length 0.

Signed-off-by: Steve Schremmer <steve.schremmer@netapp.com>

nvme-cli: fix scaandir() return value

Don't use unsigned int for scandir() return value as it can return -1.

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

nvme-cli: code cleanup for passthru() handlers

Get rid of the duplicate free of the metadata pointer in case memory
allocation fails for the data pointer.

Close wfd file in case of any resource allocation failure if
applicable.

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

nvme-cli: fix dead-code nvmf log page discovery

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

nvme-cli: remove always true condition for u8

Get rid of the always true condition of (cfg.owpass >= 0) since
cfg.owpass is defined as __u8.

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

nvme-cli: don't use unsigned for fd in lnvm-nvme.c

Change the data type of the fd from unsigned int -> int.

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

nvme-cli: fix sparse warnings for declaration

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

nvme-cli: wdc-plugin mark array static declaration

The array deFeatureIdList is only used in wdc-nvme.c,
use static declaration.

nvme-cli: make show_registers_cap() static

The function show_registers_cap() us only used in the nvme-print.c,
mark this function static.

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

Fix nvme id namespace struct

Off by one in the reserved field.

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

nvme-discover: sanitize options

The discovery controller doesn't accept any KATO or number of I/O
queue changes, but we do want to change the reconnect delay and
the controller loss timeout.

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

nvme-vendor: fixup c99 declaration in huawei plugin

We can't use c99 declaration with older compilers.

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

nvme-ana-log: fixup compiler warning in show_ana_log()

Add the correct type cast to keep the compiler happy.

Signed-off-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>

Merge pull request #398 from bdeyal/master

nvme-get-property : Add documentation

Merge pull request #403 from bdeyal/pci_ids

nvme-models.c - search for pci.ids

Fix print format specifier for 64-bit

For cross platform compatibility, we use PRI macros instead of hard
coding 64-bit specifiers.

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

nvme-cli: add ana-log documentation

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>

nvme-cli: add minimal ana-log page support

This patch adds a new command to retrieve the ANA Log page.
We update identify ctrl/ns data structure to support this command.
We also add ana based error codes and different identifiers to the
linux/nvme.h header file in order to support this command.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Hannes Reinecke <hare@suse.com>

nvme-cli add ns write protect feature support

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