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>

tests: correct to check blkdev for NS attached

Since currently only default NS blkdev checked for all NSs.

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

ocp-telemetry-decode: fix ascii string lookup

Lookup ascii string in VU Event Identifier String Table for vendor
unique event classes. Fix typo.

Signed-off-by: Paul Wood <p.wood@samsung.com>

nvmf-keys: udev rule should be using /etc/nvme

Append nvme to @SYSCONFDIR@ in udev rule.

I'd think the TLS keyfile should be located at /etc/nvme/tls-keys and
not just /etc/tls-keys.

Signed-off-by: Chris Leech <cleech@redhat.com>

nvme: set eds to true if controller supports 128 bit hostid

A controller that uses a 128-bit Host Identifier may result in the
"nvme resv-report" command failing with a "Host Identifier Inconsistent
Format" error that suggests the "simultaneous use of 64-bit and
128-bit Host Identifier values on different controllers".

This error can be avoided if the "--eds" option is used, to request
the extended data structure.  However, the controller's ctratt
value indicates whether the Host Identifier is 64 bits or 128 bits.
Therefore, check the ctratt flag, and set eds to true if the
controller indicates a 128-bit Host Identifier.

Signed-off-by: Bryan Gurney <bgurney@redhat.com>

nvme: add flush command chardev handle error message

Since IO commands rejected by the chardev if more than one namespace.

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

nvme: use blkdev for direct if namespace-id specified

Since IO commands rejected by the chardev if more than one namespace.

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

argconfig: add argconfig_get_value()

This is to get the option value by name.

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

nvme: add NS management commands error message output

Print it if the commands failed and NS management not supported.

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

build: remove thank you bot

Once again beaten by security.

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

build: fix REST API call and permissions for thanks you note

The github action needs to use the correct REST API and the write
permission to be able to comment on the issue/pull request.

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

nvme-print-stdout: use NVME_CTRL_OACS definitions

Still needed to add some remaining definitions in libnvme.

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

build: automatically add a thank you note on PR merge

Be friendly and add a thank you note to all merged PRs. Currently, I
type it always by hand.

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

nvme-print-json: add get feature pls mode print

This change required libnvme FEAT_PLS_MODE definition added.

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

nvme-print-stdout: add get feature pls mode print

This change required libnvme FEAT_PLS_MODE definition added.

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

doc: add mgmt-addr-list-log command

Added the nvme-mgmt-addr-list-log.txt file.

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

completions: add mgmt-addr-list-log command

Both bash and zsh completions updated for the command.

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

completions: fix ocp hardware-component-log zsh completion

Incorrectly the short option -v duplicated.

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

doc: fix ocp-hardware-component-log documentation errors

Since described as set-error-injection incorrectly.

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

nvme-print-binary: add binary_mgmt_addr_list_log()

Since added the NVMe 2.1 log page.

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

nvme-print-json: add json_mgmt_addr_list_log()

Since added the NVMe 2.1 log page.

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

nvme: add mgmt-addr-list-log command

Since added the NVMe 2.1 log page.

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

build: bump libnvme wrap

Include libnvme nvme_mi_admin_get_log_mgmt_addr_list().

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

nvme-print-json: add NVMe 2.1 identify power state descriptor fields

Since power loss signaling function introduced by NVMe 2.1.

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

nvme-print-stdout: add NVMe 2.1 identify power state descriptor fields

Since power loss signaling function introduced by NVMe 2.1.

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

types: move STR_LEN definition from json.h

Since to use the definition in nvme-print-stdout.c also.

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

nvme-print: add NVMe 2.1 admin commands strings

Also added some missed admin commands strings.

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

nvme-print-stdout: add human readable output for DSTO field

Add human readable output for DSTO field of Identify Controller Data
Structure.

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

nvme: add the new Self-test Code of Device Self-test command

Add the new Self-test Code (SCT) Host-Initiated Refresh operation support
to Device Self-test command. Also, add it in the Device Self-test Log Page.
TP4058 - Environmental Extremes Management.

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

nvme-print-stdout: split PEL functions

This is to reduce the PEL long function implementation.

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

nvme-print-stdout: split PEL set feature event function

This is to reduce the PEL long function implementation.

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

nvme-print-stdout: use libnvme FDP events feature definitions

This is to reduce to use the bitwise operators.

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

nvme-print-stdout: fix to print PEL set feature event FDP events

Since previously only FDP event type 0 output incorrectly.

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

nvme-print: add NVMe 2.1 get and set features FIDs string

Fixed also the indentation error.

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

build: bump libnvme wrap

Include libnvme NVMe 2.1 features FID definition addition.

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

ocp-nvme: Add missing data media_dies_offline to smart_extended_log.

related commit : https://github.com/linux-nvme/nvme-cli/commit/d4b5f77dc862530aed8523da3662ecab3850f0f4

Signed-off-by: Minsik Jeon <hmi.jeon@samsung.com>

ocp-nvme: Update ocp v2.6 Smart Information Extended

Log Identifier C0h.
Spec Documents:
https://www.opencompute.org/documents/datacenter-nvme-ssd-specification-v2-6-2-pdf

Signed-off-by: Minsik Jeon <hmi.jeon@samsung.com>
Co-authored-by: Steven Seungcheol Lee <sc108.lee@samsung.com>

plugins/ocp: Align ocp_smart_extended_log struct.

If the dssd point, minor version are declared as __le16, the alignment
will be broken. Remove __packed keyword.
Change parameter void to ocp_smart_extended_log.

Reported-by: Steven Seungcheol Lee <sc108.lee@samsung.com>
Signed-off-by: Minsik Jeon <hmi.jeon@samsung.com>

plugins/ocp: Use structure for ocp smart log

printed log tested (same), except below
Remove wrong duplicated prints from stdout
  NVMe base errata version
  NVMe command set errata version

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

ocp: Update ocp plugin version

Set plugin version to 2.11.0

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

ocp: Fixes for OCP 2.5 Telemetry Stats Parsing

The OCP 2.5 Telemetry log statistics parsing code was not
checking for the end of the stats data and therefore getting
stuck in an infinite while loop.
Added code to parse out the statistic specific data for the
Max Die, Max NAND Channel, and Min NAND Channel Bad Block.

Removed unneeded brackets
Fixed lines exceeding 100 chars

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

make: fix make rpm target

Not specifying 70-nvmf-keys.rules as a known spec file results in this
error from make rpm:
error: Installed (but unpackaged) file(s) found:
   /usr/local/%{_udevrulesdir}/70-nvmf-keys.rules

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Link: https://lore.kernel.org/r/20250107112711.192068-1-sagi@grimberg.me
Signed-off-by: Daniel Wagner <wagi@kernel.org>

nvme-print: add NVMe 2.1 get log page LIDs string

Added also Phy Rx EOM log page missed.

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

ocp: change hwcomp log size variable name to log_bytes

Since the size value calculated to bytes from dwords for version 1.

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

ocp: fix to check hwcomp log size if valid

Since the log size value needed to reduce the descriptor length.

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

ocp: add ocp dssd log id and feature id definitions

This is to reduce the hardcoded fid and lid values.

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

util: introduce int56_to_long() type conversion helper

This is to remove unnecessary mask with 0x00FFFFFFFFFFFFFF.

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

ocp-print: use int48_to_long() to print 48 bits values

This is to reduce unnecessary mask with 0x0000FFFFFFFFFFFF.

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

ocp: remove unnecessary 64 bits mask

Since the 64 bits variables not necessary to mask by 64 bits.

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

ocp: use struct nvme_timestamp type for FAHE-5 timestamp field

Since the field defined by the NVMe timestamp feature.

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

nvme-print: move obj_add_* definitions into util/json.h

Also use the same prefix json_object_add_*.

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

nvme: fix to use NVME_VAL() for CQ entry status field filter

Previously used incorrectly NVME_GET() but not shifted the masked values.

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

nvme-print-json: update id-ctrl print code to use libnvme definition

Use libnvme register definitions instead of the shift operator.

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