Tests: Hande string encoding and handling failures

Handling string encoding to utf-8 and also string handling in
tests.

Signed-off-by: Narasimhan V <sim@linux.vnet.ibm.com>

Makefile: use UDEVRULESDIR instead of UDEVDIR

The SUSE rpm macros only have UDEVRULESDIR, not UDEVDIR.
So introduce a UDEVRULESDIR variable which is preset to
UDEVDIR/rules.d

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

Makefile: generate Dracut config according to UDEVDIR

Signed-off-by: Haochen Tong <i@hexchain.org>

Add additional smart log critical warn decoding

Provide a user option to request additional smart log page decoding,
and print out detailed critical warning when requested.

Signed-off-by: Wendy Xiong<wenxiong@linux.vnet.ibm.com>
[merge up, rename json keys]
Signed-off-by: Keith Busch <kbusch@kernel.org>

New tag for missing docs

The previous release tag missed adding documentation required for a
successful install.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Add new wdc documentation

Signed-off-by: Keith Busch <kbusch@kernel.org>

Release v1.10

Signed-off-by: Keith Busch <kbusch@kernel.org>

Regen documentation

Signed-off-by: Keith Busch <kbusch@kernel.org>

Use asprintf for long names

It's too easy to observe snprintf truncation warnings when using long
strings.

Signed-off-by: Keith Busch <kbusch@kernel.org>

[NVME-CLI] Add documentation file for vs-error-reason-identifier WDC
           plugin command.
[NVME-CLI] Minor updates/corrections to the clear-fw-activate-history
           and vs-fw-activate-history command help text.

[NVME-CLI] Add support for WDC plugin command - vs-error-reason-identifier
[NVME-CLI] Reverse enable/disable bit on the vs-telemetry-controller-option
           WDC plugin command

Fix systemd service script use of CONNECT_ARGS env var

In testing, some environments are not properly expanding CONNECT_ARGS.

Change systemd service file use of CONNECT_ARGS to be encapsulated by
apostrophes and parens.

Signed-off-by: James Smart <jsmart2021@gmail.com>
Signed-off-by: Keith Busch <kbusch@kernel.org>

Shannon Plugin update.

Doc/Makefile: Introduce 'quiet' build option

Signed-off-by: Keith Busch <kbusch@kernel.org>

Makefile: use 'quiet' compile

Just using a simpler compile output.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Makefile: Use special variables

Signed-off-by: Keith Busch <kbusch@kernel.org>

micron: don't ignore 'system()' return value

Some compilers will generate warnings that this return value can't be
ignored.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Fix libhugetlbfs flags

Allocating a region uses GHR_ flags, not GHP_ types.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Regenerate documentation

Signed-off-by: Keith Busch <kbusch@kernel.org>

Micron plugin updates

README.md: Mention the buildroot package

Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com>

nvmf-autoconnect: add udev rules to set iopolicy for certain NetApp devices

Fix intel json latency statistics output format

Missing any support for the json output.

Link: https://github.com/linux-nvme/nvme-cli/commit/b7eb621f1a998f9b7a58501fdf6e6773ddc937ff#commitcomment-36448196
Signed-off-by: Keith Busch <kbusch@kernel.org>

intel: Add function lat-stats to cli
Adds the lat-stats function to intel plugin.
If lat-stats-tracking is enabled, lat-stats will print out latency
statistics according to the user's desired format.

Auto detects intel drives to print the correct format.

Can output as raw binary, a formatted histogram, or as formatted json.

Signed-off-by: Kevin Chau <kchaumein@gmail.com>

[NVMe-CLI] Update documentation for telemetry data retrieval to
vs-internal-log wdc plugin command.

[NVMe-CLI] Add support to retrieve telemetry data in vs-internal-log

[nvme-cli] Add documentation file for new  wdc plugin command -
vs-telemetry-controller-option

[nvme-cli] Add support for wdc plugin command - vs-telemetry-controller-option

Fix typo (bulit -> built)

Warn on admin-passthru opcode not matching read/write flag

Opcode bits 1:0 indicate the direction of data transfer. These
bits should agree with the read or write flags provided.

Signed-off-by: Kelly Ursenbach <kelly.ursenbach@eideticom.com>

MAINT: Fix format compile warning error on 32 bit system

Since the following %lx format warning error was caused on 64 bit system.

  error: format ‘%lx’ expects argument of type ‘long unsigned int’,
  but argument 2 has type ‘long long unsigned int’ [-Werror=format=]

So fix this by using %" PRIx64 " format instead as generic.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

Merge branch 'Dequan-Dera-dera-plugin'

Merge branch 'dera-plugin' of https://github.com/Dequan-Dera/nvme-cli into Dequan-Dera-dera-plugin

nvme-cli: Check for sysfs interface before NVMe discovery

This prevents an unnecessary error message in the case that the nvme-fc
kernel module is not loaded and the
  /sys/class/fc/fc_udev_device/nvme_discovery
handle is not available.

Merge branch 'dera-plugin' of github.com:Dequan-Dera/nvme-cli into dera-plugin

Merge branch 'master' of github.com:linux-nvme/nvme-cli into dera-plugin

Merge branch 'master' of github.com:linux-nvme/nvme-cli into dera-plugin

Revert "Add Dera plugin extension."

This reverts commit 8086ab1751f820e449b63932d286bc45e229f94b.

Revert "Add Dera plugin extension."

This reverts commit fb46db77ff65412ebc5568ff36aa5a2dd011834c.

Add Dera plugin extension.
Add stat sub-command to retrieve Dera device status and additional SMART log.

Add Dera plugin extension.
Add stat sub-command to retrieve Dera device status and additional SMART log.

Improve self test log command

Show the page's valid entries even if no test is actively running,
and be less verbose about the output unless requested.

Signed-off-by: Keith Busch <kbusch@kernel.org>

nvme-cli: Update err to 0 in get_telemetry_log

Err needs to be updated to 0 else err contains bs(no. of bytes written).
This gets passd onto nvme_status_to_errno, thus returning irrelevant errno.

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

fixup up self-test log when no test is in progress

Link: https://github.com/linux-nvme/nvme-cli/issues/612
Signed-off-by: Keith Busch <kbusch@kernel.org>

nvme-cli: Update err value to 0 in get_ns_id func.

Err value should be updated to 0 on success of nvme_get_nsid,
else the File Descriptor is passed on to nvme_status_to_errno
which returns an irrelevant errno.

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

Merge branch 'dera-plugin' of github.com:Dequan-Dera/nvme-cli into dera-plugin

Add Dera plugin extension.
Add stat sub-command to retrieve Dera device status and additional SMART log.

nvme.c: create-ns arguments are added

ANA Group Identifier, NVM Set Identifier
Its based on NVMe 1.4 Spec

Signed-off-by: Steven Seungcheol Lee <sc108.lee@samsung.com>

Add Dera plugin extension.
Add stat sub-command to retrieve Dera device status and additional SMART log.

[NVMe-CLI] Get DUI for SN-340 failing with Data Area 4 - Buffer Misalignment

Minor intel-nvme print refactor

Signed-off-by: Keith Busch <kbusch@kernel.org>

Use simpler nvme ioctl api for admin

Signed-off-by: Keith Busch <kbusch@kernel.org>

Relocate functions and constants to the files that use them

Make more things static helps reduce what exports need to be tracked.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Move all spec defined structs to single file

Signed-off-by: Keith Busch <kbusch@kernel.org>

Update build rules for .c with no .h

Not all .c files have a .h, but we still want to ensure the compile options are
the same.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Update my email address

Set to a working email address.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Makefile: fix cleaning subdirectories

Signed-off-by: Keith Busch <kbusch@kernel.org>

remove unused "paths" controller field

Signed-off-by: Keith Busch <kbusch@kernel.org>

reign in line lengths

Line lengths have started to get out of control. Try to get these to 80
characters.

Signed-off-by: Keith Busch <kbusch@kernel.org>

Add print for directive receive

Signed-off-by: Keith Busch <kbusch@kernel.org>

Unify print routines

Move all the binary/json/normal output handling into a single function
call for each structure that we print. This will reduce the API surface
that we have to maintain.

Signed-off-by: Keith Busch <kbusch@kernel.org>

regenerate documentation

fix fw-download reference to committing fw

Spec renamed the Firmware Download command to Firmware Commit.

Link: https://github.com/linux-nvme/nvme-cli/issues/601
Signed-off-by: Keith Busch <kbusch@kernel.org>

argconfig: Remove unused paramters

Signed-off-by: Keith Busch <kbusch@kernel.org>

fix huawei filter to search all namespaces

Signed-off-by: Keith Busch <kbusch@kernel.org>

nvme-cli: Support for hugetlbfs

Some commands require exceptionally large data transfers, and the
kernel driver supports only a limited number of phyiscal segments per
command. To help support this, try to allocate physically contiguous
space via hugetlbfs and fallback to a normal malloc if that fails.

Rather than deal with the nuances of allocating huge pages, use
libhugetlbfs as a dependency.

Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Reviewed-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Keith Busch <kbusch@kernel.org>

Remove all 'struct nvme_command' usage

This is a kernel only data structure, user space should use it's own
uapi defined passthrough command structure for all command formatting.

Signed-off-by: Keith Busch <kbusch@kernel.org>

nvme-cli: Update PMR Capability and PMR Status
1. Adding Controller Memory Space Supported(CMSS) in PMRCAP
2. Adding Persistent Memory Region Status(CBAI) in PMRSTS

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

nvme-cli: Update Controller Capabilities Register
1. Add Controller Memory Buffer Supported (CMBS)

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

nvme-cli: Update NVMe Registers
1. Add Controller Memory Buffer Memory Space Control (CMBMSC)
2. Add Controller Memory Buffer Status (CMBSTS)
3. Add Persistent Memory Region Memory Space Control (PMRMSC)

Update show registers

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

fixup attach-ns documentation

Signed-off-by: Keith Busch <kbusch@kernel.org>

Merge pull request #578 from rgirasegit/patch-1

Add an example for attach-ns

Add files via upload

Merge pull request #593 from jeffreyalien/master

[NVMe-CLI] Updates to WDC plugin command vs-internal-log

[NVMe-CLI] Updates and fixes to WDC plugin command vs-internal-log
for the SN340 drive.

Regen docs, add host-nqn man page

Signed-off-by: Keith Busch <kbusch@kernel.org>

Merge pull request #592 from jeffreyalien/new_branch_name

New branch name

[NVMe-ClI] Add Documentation files for clear-fw-activate-history command

[NVMe-CLI] Add clear-fw-activate-history WDC plugin command

Merge https://github.com/linux-nvme/nvme-cli

fix namespace checks for legacy list

First ensure the controller we're comparing with has been initialized, and then
compare the correct fields with each other. Previously had been comparing a
model number against the serial.

Link: http://lists.infradead.org/pipermail/linux-nvme/2019-October/027562.html
Reported-by: Jeff Lien <jeff.lien@wdc.com>
Signed-off-by: Keith Busch <kbusch@kernel.org>

Reset wdc_do_cap_dui default parameters to the proper values

Merge pull request #589 from sephalon/oe-recipe-readme

README.md: Mention the OpenEmbedded/Yocto recipe

README.md: Mention the OpenEmbedded/Yocto recipe

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

Merge pull request #584 from benreese0/master

Intel plugin: Adding fields to id-ctrl VU region

Intel plugin: Adding fields to id-ctrl VU region
Per https://github.com/linux-nvme/nvme-cli/pull/584/commits/f33510efbf5192116e5757d13a02463d446a1dc5#r332178975 removing pack pragma on vu_id_ctrl_field struct.

Signed-off-by: Ben Reese <5884008+benreese0@users.noreply.github.com>

Merge pull request #588 from amluto/hostnqn

Use a systemd app-specific machine ID for hostnqn

Merge pull request #587 from RevanthRajashekar/error_log

nvme-cli: Add 'transport type' error to the error log

Merge pull request #586 from RevanthRajashekar/endu_grp_log

nvme-cli: Update nvme_endurance_group_log structure

Merge pull request #583 from pszczerx/nvme_list_bugfix

Check return value of scandir

Merge pull request #579 from RevanthRajashekar/self_test_log

nvme-cli: Add namespace-id option to self-test-log

udev: convert the discovery event handler to the kernel support

The kernel will not send us a specific event for discovery but
rather the AEN result code. So expect NVME_AEN=0x70f002 for
discovery log change events.

Also, we don't get the NVME_CTRL_NAME env var anymore as this is
available from the device $kernel.

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

Use a systemd app-specific machine ID for hostnqn

If /etc/nvme/hostnqn is not present, the fabric commands will ask
systemd for an app-specific machine ID as a fallback.  This should
improve functionality if /etc/nvme/hostnqn is not present and should
allow packagers to avoid creating /etc/nvme/hostnqn.

Heavily based on an earlier patch from Tomasz Torcz.

Signed-off-by: Tomasz Torcz <tomek@pipebreaker.pl>
Signed-off-by: Andy Lutomirski <luto@kernel.org>

nvme-cli: Add 'transport type' error to the error log

* Update show_error_log
* Update json_error_log

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

nvme-cli: Update nvme_endurance_group_log structure

* Update show_endurance_log
* Update json_endurance_log

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

Adding fields to id-ctrl VU reigon for Intel DC drives

Signed-off-by: Ben Reese <5884008+benreese0@users.noreply.github.com>

Check return value of scandir

On error, scandir returns -1 and does not allocate memory
for namelist array. In some places in the code return value
of scandir call is not checked. This causes nvme-cli to
attempt to free() an uninitialized pointer, which subsequently
leads to segmentation fault.

To address this issue, check return value of scandir calls
throughout the code.

Signed-off-by: Szczerbik, Przemyslaw <przemyslawx.szczerbik@intel.com>

nvme-cli: Add namespace-id option to self-test-log

Instead of scanning all the namespaces for logs, the user can specify
the namespace from which he wants to retrieve the self-test log from.

Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>

Report invalid output format for 'list' command

Don't fail silently if user provdies invalid 'list' options.

Reported-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Keith Busch <kbusch@kernel.org>

Provide no-binary output format help text

The binary output is only for device returned structures. Remove this
help text for the 'list' command.

Signed-off-by: Keith Busch <kbusch@kernel.org>