nvme: add error message for mmap_registers

To indicate that backward compatibility is not supported
since we drop the block device fallback

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

sed: improved contextual error messages

Improve quality of error messages but including context and
locking state.

Signed-off-by: Greg Joyce <gjoyce@linux.ibm.com>

nvme: use correct parser type for nvmsetid and endgid option

nvmsetid and endgid are both shorts, thus the OPT_SHRT parser
type needs to be used. Otherwise the result is written to the wrong
memory location.

Fixes: 972effdf884e ("nvme.c: create-ns arguments are added")
Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme-print: print new fields of Asynchronous Event Configuration

Print the newly added fields of Asynchronous Event Configuration
feature (FID - 0Bh)
NVM Express Base Specification 2.1.

Signed-off-by: Francis Pravin <francis.p@samsung.com>

completions: add pull-model-ddc-req-log command

Both bash and zsh completions updated for the command.

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

doc: add pull-model-ddr-req-log command

Added the nvme-pull-model-ddr-req-log.txt file.

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

nvme: add pull-model-ddc-req-log command

Since added the NVMe 2.1 log page.

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

nvme: fix ave-discovery-log command LID setting

Also fixed the function definition unnecessary line break.

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

Makefile: Add new targets and update meson command syntax

- For the "install" target, use "--skip-subprojects" so that only
  nvme-cli (and not the subprojects) get installed.
- For the "purge" target, add "meson subprojects purge".
- Added "update-subprojects" target
- Added "test-strict" target. This is to limit testing to nvme-cli
  and not all the subprojects.

Signed-off-by: Martin Belanger <martin.belanger@dell.com>

micron: Add missing newline

Add missing newline to error for cards that don't support the
vs-drive-info command.

Signed-off-by: Warner Losh <imp@bsdimp.com>

completions: add ave-discovery-log command

Both bash and zsh completions updated for the command.

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

doc: add ave-discovery-log command

Added the nvme-ave-discovery-log.txt file.

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

Documentation: Added solidigm plugin commands to main manual page.

Added most unique solidigm plugin commands to main manual page.

Signed-off-by: Leonardo da Cunha <leonardo.da.cunha@solidigm.com>

nvme-print: add human readable output for IOCS Data Structure

Add human readable output for Identify I/O Command Set Data
Structure (CNS 1Ch).

Signed-off-by: Francis Pravin <francis.p@samsung.com>

build: bump libnvme wrap

020b979 - types: add enum for Identify I/O Command Set Data Structure

Signed-off-by: Francis Pravin <francis.p@samsung.com>

sed: extended discovery feature code printing

Added ability to discover and print details of all specified
level 0 features.

Signed-off-by: Greg Joyce <gjoyce@linux.ibm.com>

netapp-ontapdev: correct the basestr header

Yet another correction to the basestr headers.

Signed-off-by: Martin George <marting@netapp.com>

nvme: add ave-discovery-log command

Since added the NVMe 2.1 log page.

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

nvme-print-json: delete json_host_discovery_log printf error code

Missed to delete the printf call since copied from nvme-print-stdout.

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

netapp-smdev: update err msg for no smdevices

Update the error message when no smdevices are detected on
the host.

Signed-off-by: Martin George <marting@netapp.com>

netapp-ontapdev: update err msg for no ontapdevices

Update the error message when no ontapdevices are detected
on the host.

Signed-off-by: Martin George <marting@netapp.com>

netapp-smdev: update basestr header in verbose output

Update the basestr headers to be more descriptive in the
verbose output.

Signed-off-by: Martin George <marting@netapp.com>

netapp-ontapdev: update basestr headers in verbose output

Update the basestr headers to be more descriptive in the
verbose output.

Signed-off-by: Martin George <marting@netapp.com>

nvme: enforce ns granularity on ns create

When the device supports the namespace granularity feature, update the
alignment requirements accordingly. Because this is likely to change the
capacity of the namespace also issue a info what is going to happen.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

doc: add host-discovery-log command

Added the nvme-host-discovery-log.txt file.

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

completions: add host-discovery-log command

Both bash and zsh completions updated for the command.

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

completions: fix zsh completions array parameter names

Since copied from the exsisting completion but missed to change.

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

nvme: add host-discovery-log command

Since added the NVMe 2.1 log page.

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

build: bump libnvme wrap

Include nvmf_exat_ptr_next export.

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

nvme-print: print Host Behavior Support feature fields

Print the fields of Host Behavior Support feature (FID - 16h).
NVM Express Base Specification 2.1

Signed-off-by: Francis Pravin <francis.p@samsung.com>

completions: Added fdp feature auto-completion.

Fixed syntax error on zsh auto-completion

Signed-off-by: Leonardo da Cunha <leonardo.da.cunha@solidigm.com>

nvme-wrap: fix get_log_reachability_associations parameters order

Since the len parameter should be moved before the log parameter.

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

nvme-print-binary: fix to output phy rx eom log length

Previously incorrectly the log length is calculated by the LE log data.
So fix to convert the value from LE to host endian for the calculation.

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

nvme-print-binary: fix to output dispersed ns participating nss log length

Previously only the log header output since the log length is changeable.
The changes to output the changeable log length correctly.

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

nvme-print-binary: fix to output reachability groups log length

Previously only the log header output since the log length is changeable.
The changes to output the changeable log length correctly.

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

nvme-print-binary: fix to output reachability associations log length

Previously only the log header output since the log length is changeable.
The changes to output the changeable log length correctly.

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

plugins/ocp: Added OCP 2.6 telemetry support
Updated the Telemetry Structure and Enum to support the OCP 2.6 spec

Reviewed-by: Karthik Balan <karthik.b82@samsung.com>
Reviewed-by: Arunpandian J <arun.j@samsung.com>
Signed-off-by: Vigneshwaran Saravanan/Vigneshwaran Saravanan <s.vignesh@samsung.com>

util/logging: fix print format string

Let's cast it to the longest type which works on 32bit and 64bit.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

plugins/sed: use platform depended cast for pointer type

On 32bit platforms the compiler complains with

../plugins/sed/sedopal_cmd.c:515:25: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  515 |         discover.data = (__u64)buf;

Signed-off-by: Daniel Wagner <wagi@kernel.org>

plugins/virtium: fix print format string

time_ms is of type time_t which is not really portable to use. Let's
cast it to the longest type which works on 32bit and 64bit.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

util/types: fix print format string

Let's cast it to the longest type which works on 32bit and 64bit.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

build: bump libnvme wrap

Update to get cross build fixes for the updated cross build containers.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme-print: print new fields of nvm_id_ctrl

Print the new fields of I/O Command Set Specific Identify Controller
Data Structure based on NVM Command Set Specification 1.1.

Signed-off-by: Francis Pravin <francis.p@samsung.com>

nvme: check for valid output format

Several commands such as nvme get-feature, get-property, id_iocs, etc.
don't check for a valid output format unlike other nvme commands.
Fix the same.

Signed-off-by: Martin George <marting@netapp.com>

nvme: make get-feature JSON output print everything

The nvme JSON outputs now print everything verbose. But this was
missed out for the get-feature command, so fix the same here.

Signed-off-by: Martin George <marting@netapp.com>

doc: Add nvme-fdp-feature command

Adds helper command to enable FDP configuration.

Signed-off-by: Leonardo da Cunha <leonardo.da.cunha@solidigm.com>

plugins/fdp: Added FDP feature subcommand.

Allows enabling and disabling FDP configuration.

Signed-off-by: Leonardo da Cunha <leonardo.da.cunha@solidigm.com>

tests: check if verify command is supported

Check if the optional NVM command 'verify' (NVMCPYS) is supported before
running tests for this command.

Signed-off-by: Dennis Maisenbacher <dennis.maisenbacher@wdc.com>

tests: fix hex to base-10 conversion in id-ctrl output

When doing bit checks on id-ctrl output fields we need to convert the
field numbers to a base-10 integer. However, the json output mostly
returns already base-10 integers, so we now first check for the '0x'
prefix before falsely converting the desired value.

Signed-off-by: Dennis Maisenbacher <dennis.maisenbacher@wdc.com>

tests: read LBA size for validation

Create the default namespace before running tests and validate IO in LBA
size granularity.

Signed-off-by: Dennis Maisenbacher <dennis.maisenbacher@wdc.com>

tests: fix LBA format calculation

We assumed that the test drive is always formatted with lbaf 0 which
leads to miscalculation of the maximum available size on namespace
creation.
Now we query the LBA size according to the selected `self.flbas`.

Signed-off-by: Dennis Maisenbacher <dennis.maisenbacher@wdc.com>

completions: add reachability-associations-log command

Both bash and zsh completions updated for the command.
Also fix the reachability-groups-log bash completion to use a single space.

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

doc: add reachability-associations-log command

Added the nvme-reachability-associations-log.txt file.

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

nvme: add reachability-associations-log command

Since added the NVMe 2.1 log page.

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

nvme-wrap: do_admin_op() get_log_reachability_groups parameters order

Since the len parameter should be moved before the log parameter.

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

build: bump libnvme wrap

Include nvme_reachability_association_desc member variable name fix.

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

nvme: make -v output consistent with -H output for nvme get-feature

Current -v verbose output of the nvme get-feature command is a no-op.
So update it to make it consistent with the -H human-readable output
for this same command.

Signed-off-by: Martin George <marting@netapp.com>

nvme: make -v output consistent with -H output for nvme get-property

Current -v verbose output of the nvme get-property command is a
no-op. So update it to make it consistent with the -H human-readable
output for this same command.

Signed-off-by: Martin George <marting@netapp.com>

nvme: fix reachability-groups-log command perror message

Since copied the command implementation but missed to change.

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

completions: add reachability-groups-log command

Both bash and zsh completions updated for the command.

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

doc: add reachability-groups-log command

Added the nvme-reachability-groups-log.txt file.

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

CI: add missing bind-mount option for /dev

The failing run-nightly-tests GitHub action is caused by a missing
bind-mount option of the /dev directory. Without the 'shared'
bind-propagation the device tree is not reliably updated within the
container. This causes newly created namespaces not to show up.

Signed-off-by: Dennis Maisenbacher <dennis.maisenbacher@wdc.com>

Revert "argconfig: add argconfig_get_value()"

This reverts commit cccefc48a9aca3c0b3f0f2a2a0fe5d01ad9eb59a.

Causes regression in nightly CI runs. The problem couldn't be figured out
thus revert it.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

Revert "nvme: use blkdev for direct if namespace-id specified"

This reverts commit 565b30e285934502e9fcaaf7d9b8bd81e5011d0f.

Causes regression in nightly CI runs. The problem couldn't be figured out
thus revert it.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

Revert "nvme: add flush command chardev handle error message"

This reverts commit 946029cf58d2b039a20c0c96dba7e6ef636070f9.

Causes regression in nightly CI runs. The problem couldn't be figured out
thus revert it.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme: add reachability-groups-log command

Since added the NVMe 2.1 log page.

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

nvme: do not output debug infos on info level

Do not clutter the info message level with debug information.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

build: checkpatch limit line length to 100 chars

The default is 100 chars.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme-print-stdout: add details in list-ns verbose output

Add identifier & NSID tags in the verbose output of the list-ns
command for better clarity. And while at it, print a common
"NVMe Namespace List" tag at the top too. For e.g.

nvme list-ns /dev/nvme1n1 -v
dev: /dev/nvme1n1, fd: 3
NVME Namespace List:
Identifier    0: NSID 0x1
Identifier    1: NSID 0x2
...

Signed-off-by: Martin George <marting@netapp.com>

nvme-print-stdout: add descriptions for nsze, ncap & nuse

Add descriptions for the nsze, ncap & nuse fields in the verbose
output of the id-ns command for better clarity. For e.g.

nvme id-ns /dev/nvme1n1 -v
dev: /dev/nvme1n1, fd: 3
NVME Identify Namespace 1:
nsze    : 0x280000      Total size in logical blocks
ncap    : 0x280000      Maximum size in logical blocks
nuse    : 0x13b3e       Current size in logical blocks
...

Signed-off-by: Martin George <marting@netapp.com>

completions: add and fix io-mgmt-recv and io-mgmt-send commands

Also fixed get-reg and set-reg commands completions.

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

completions: add dispersed-ns-participating-nss-log command

Both bash and zsh completions updated for the command.

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

doc: add dispersed-ns-participating-nss-log command

Added the nvme-dispersed-ns-participating-nss-log.txt file.

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

nvme: add dispersed-ns-participating-nss-log command

Since added the NVMe 2.1 log page.

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

completions: add changed-alloc-ns-list-log command

Both bash and zsh completions updated for the command.

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

completions: fix mgmt-addr-list-log command

Since missed to add the command into the command list.

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

doc: add changed-alloc-ns-list-log command

Added the nvme-changed-alloc-ns-list-log.txt file.

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

nvme: add NVMe 2.1 changed-alloc-ns-list-log command

The command functions combined with the changed-ns-list-log command.
The output almost same but different as lists attached or allocated NSs.

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

nvme-wrap: handle get log rotational media info correctly

Return error for the missing get log rotational media function
for non MI devices.

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

build: sync checkpatch config file

Sync nvme-cli checkpatch configuration with the one in libnvme.
There is no point in having different configurations for this.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

build: checkpatch ignore new/del/mov files

Do not warn about missing MAINTAINER file updates.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme-wrap: use rotational media information log libnvme-mi API

Since the libnvme-mi missed to add but will be added then use it.

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

build: bump libnvme wrap

Include libnvme-mi NVMe 2.1 log page APIs.

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

ocp: fix to free hwcomp log desc memory allocated

Since the desc pointer used _cleanup_free_ only set NULL value.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
[wagi: free buffer on error in get_hwcomp_log_data]
Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme: change attach-ns to error out if wrong device given

Also updating the documentation.

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

nvme-print: print new id-ns fields added in NVM Command Set Spec 1.1

Print the new fields of Identify Namespace Data Structure added in NVM
Command Set Specification 1.1.

Signed-off-by: Francis Pravin <francis.p@samsung.com>

build: bump libnvme wrap

dfc5cdf - types: Update id-ns fields based on NVM Command Set Spec 1.1
d8d9542 - types: Update nvm-id-ns fields based on NVM Command Set Spec 1.1

Signed-off-by: Francis Pravin <francis.p@samsung.com>

plugins/ocp: fix telemetry parser buffer overflow

The fixed 256-byte description_str will overflow for any reasonably
sized data_size >= 128.

Max data_size is for OCP VU Event Data is 0xFF Dwords, so the buffer
should be at least 1020 bytes + 1 for null. 1024 seems like a nice
number for OCP.

Reported-by: Nate Thornton <n.thornton@samsung.com>
Signed-off-by: Daniel Wagner <wagi@kernel.org>

sed: add '--read-only' to lock/unlock commands

This change allows a user to specify that a drive is to be locked
read-only rather than just read-write. Also a drive that is locked
read-write can be "unlocked" to locked read-only.

The flag '--read-only' is used so the locking type can be specified.

Signed-off-by: Greg Joyce <gjoyce@linux.ibm.com>

udev-rules-ontap: switch to queue-depth iopolicy

The queue_depth based iopolicy suits ONTAP NVMe controllers better.
So switch to the same.

Signed-off-by: Martin George <marting@netapp.com>

doc: add rotational-media-info-log command

Added the nvme-rotational-media-info-log.txt file.

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

completions: add rotational-media-info-log command

Both bash and zsh completions updated for the command.

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

tests: monitor /dev for new nvme block device

The attach operation takes a bit of time and is asynchronous, thus we
should monitor if the block device appears for a period, before
returning an error.

Signed-off-by: Daniel Wagner <wagi@kernel.org>

plugins/solidigm: Telemetry null check json object

ensure object is not null

Signed-off-by: hsdhillo <Hardeep.Dhillon@solidigm.com>

plugins/solidigm: Replace snprintf with asprintf

Replace snprintf with better memory management option

Signed-off-by: hsdhillo <Hardeep.Dhillon@solidigm.com>

Revert "tests: add blkdev checking after NS creation"

This reverts commit c276f3ddd780c55458e509b74d80633728828950.

The namespace will be created but not yet attached, thus the block
device is not created either.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
[wagi: added commit message]
Signed-off-by: Daniel Wagner <wagi@kernel.org>

fabrics: make 'disconnect' accepting only -n or -d

There is limited usability in specifying both -n and -d, so make
both exclusive.

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

nvme: add rotational-media-info-log command

Since added the NVMe 2.1 log page.

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

tests: add verbose options for NS management commands

This is for debugging output if test failed.

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

tests: add blkdev checking after NS creation

To make sure the blkdev as same with NS attachment.

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