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>

nvme-print-json: update register print code to use libnvme definition

Use libnvme register definitions instead of the shift operator.

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

util: clean up json.h file

Since some json macros can be used without json as common.

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

dapustor: avoid sprinkling ifdefs

Use empty json functions instead without json.

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

netapp: avoid sprinkling ifdefs

Use empty json functions instead without json.

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

intel: avoid sprinkling ifdefs

Use empty json functions instead without json.

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

micron: avoid sprinkling ifdefs

Use empty json functions instead without json.

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

util: cast to use empty function json_object_array_add parameter k

This is to prevent the build warning without json.

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

wdc: enable wdc-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

ssstc: build plugin not depended on json without json

ssstc plugin does not use json so no need to check the build dependency.

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

seagate: enable seagate-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

scaleflux: enable sfx-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

scaleflux: use PRIu64 printf format string instead of llu

To fix the cross compiler build fails and some indentation errors also.

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

nvidia: build plugin not depended on json without json

nvidia plugin does not use json so no need to check the build dependency.

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

netapp: enable netapp-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

nbft: enable nbft-plugin.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

plugins: introduce OPT_FLAG_JSON for command option json

This is to not need ugly ifdefs everywhere.

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

amzn: avoid sprinkling ifdefs and use flags

Use flags instead of cfg.output_format.

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

Fixed latency_monitor_feature_enable to be 01h as per OCP spec v2.0

ocp: add ocp_get_log_simple() to set UUID index

The index is required by the OCP specification.

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

build: bump wrap

Update build dependencies.

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

plugin/lm: Introduce Live Migration plugin

Implementation of TP 4159 PCIe Infrastructure for Live Migration plugin.
Includes command support for Track Send, Migration Receive, Migration
Send, and Controller Data Queue; Identify Controller LM related fields;
Bash and ZSH completions.

Changes are isolated to the User Data Migration subset, with Track
Memory functionality deferred to a future commit.

Signed-off-by: Nate Thornton <n.thornton@samsung.com>

ocp-nvme: Add a different formatting for JSON output

The current OCP JSON format for the SMART extended log page is not
condusive to metric collection via tools like Prometheus. So we add a
new output mode that uses all lower case and underscores (instead of
spaces). This should help with metric collection. At the same time we
clean up some of the field names. We add a new argument
(--output-format-version) to allow us to select which output version
we want. Documentation updated to reflect this change and mark this as
experimental.

Fixes #2577.

Signed-off-by: Stephen Bates <sbates@raithlin.com>

Plugin: ScaleFlux Adding Status Command

Adding ScaleFlux status command, equivalent to vendor unique utility.

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

Plugin: ScaleFlux Use NVME_NSID_ALL

Using NVME_NSID_ALL macro instead of 0xffffffff magic number

Signed-off-by: Jayden Stenfort <127808534+A-Fellow-Comrade@users.noreply.github.com>

Plugin: ScaleFlux Spelling&Typo Fixes

Fixing spelling and typos

Signed-off-by: Jayden Stenfort <127808534+A-Fellow-Comrade@users.noreply.github.com>

Plugin: ScaleFlux Seperating Types

Moving types associated with ScaleFlux into new file sfx-types.h

Signed-off-by: Jayden Stenfort <127808534+A-Fellow-Comrade@users.noreply.github.com>

micron: enable micron-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

intel: enable intel-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

huawei: enable huawei-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

fdp: build plugin not depended on json without json

fdp plugin does not use json so no need to check the build dependency.

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

dell: build plugin not depended on json without json

dell plugin does not use json so no need to check the build dependency.

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

dapustor: enable dapustor-nvme.c build without json-c dependencies checkin

Only build json print codes with CONFIG_JSONC build option instead.

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

amzn: use PRIu64 printf format string instead of llu

To fix the cross compiler build fails.

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

amzn: enable amzn-nvme.c build without json-c dependencies checking

Only build json print codes with CONFIG_JSONC build option instead.

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

nvme: disable output-format option json value without json-c

Since only json print codes work with the json-c library.

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

doc: Add write zeroes command namespace-zeroes option

Add Namespace Zeroes (NSZ) option of Write Zeroes command.

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

ocp: add set-enable-ieee1667-silo command

The command sets enable IEEE1667 silo.

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

ocp: add ocp-types.h file for OCP requirements definitions

This is for the OCP requirement fields and bits definitions.

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

nvme-wrap: add nvme_cli_set_features() API function

The wrap API supported for MI interface.

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

doc: remove TLS documentation

We started to collect these types of documetation in the associated
nvme-cli wiki. Thus remove this file.

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

sed: add '--read-only' to 'sed initialize'

By default 'sed initialize' will set the entire disk to read/write
locked, ie no access is possible. That is all fine if the BIOS supports
TCG Opal, as then the BIOS can unlock the drive before access.
But for BIOS _not_ supporting TCG Opal this results in an inaccessible
drive, and requires the user to use a different drive to boot and
unlock the SED device.
This patch adds a flag '--read-only' to 'nvme sed initlialize' such
that the 'read-lock enable' flag is not set for the locking range,
and the device continues to be readable by the BIOS for booting.

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

plugins: build plugins not depended on json without json

Some plugins do not use json so no need to check the build dependency.

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

ocp: update internal-log command telemetry type description

Add host0 and host1 help documentation description.

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

ocp: delete internal-log command telemetry type none value definition

Since the type none value never set by the command handling.

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

nvme: plugins/fdp, check endurance group identifier

For FDP options such as stats and events, default endurance identifier 0
is used which results in error.
This patch returns error for missing endurance group identifier from
user.

Signed-off-by: Nitesh Shetty <nj.shetty@samsung.com>

print: OACS Host Managed Live Migration Support (HMLMS)

Additional HMLMS bit in OACS field (stdout)

Signed-off-by: Nate Thornton <n.thornton@samsung.com>

build: bump libnvme wrap

Update libnvme wrap to latest to pick up Live Migration types & ioctls

Signed-off-by: Nate Thornton <n.thornton@samsung.com>

nbft: Report proper error codes from discover_from_nbft()

In case of a connection failure of any SSNS records that are not
marked as 'unavailable', report proper error code so that nvme-cli
callers may retry the operation.

The semantics of most functions is to return 0 zero for success
and negative errno numbers in case of an error.

Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>

tests: Use container image to run nightly tests

Using the pre-build linux-nvme/debian.python container to avoid
installing dependencies every time which fails occasionally.

Also drop clean up steps as this is part of the self-hosted runner
now.

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

tests: Fix `subprosess.call` calls

Don't use stdout=PIPE with subprocess.call as noted in the documentation:
https://docs.python.org/3/library/subprocess.html#subprocess.call

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

tests: Refactor prints

Remove unused prints and add some informative prints.

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

tests: Remove unneeded sleep statements

Removing sleep statements, which improves the test cases runtime.

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

tests: use current LBA format as I/O block size

When running I/O use the current LBA format as default block size.

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

tests: Use json output for parsing

To make the parsing of nvme cmd output easier use the json output when
possible.

Refactoring code snippets that needed to be touched when adjusing for
json output.

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

tests: Introduce configurable nvme binary path

Introducing the `nvme_bin` config to point to a different nvme-cli
binary.

With this change all nvme cmd strings are reworked for uniformity.
This includes using f-strings and expanding short options into long
options.

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

nvme: add Namespace Zeroes support to Write Zeroes command

Added the Namespace Zeroes (NSZ) support to Write Zeroes command.
Also, fixed the dtype limitation (bit 22:20).
TP4160 - WZSL Limit Modification

Signed-off-by: Francis Pravin <francis.p@samsung.com>
Reviewed-by: Steven Seungcheol Lee <sc108.lee@samsung.com>

print: id-ctrl print updated based on NVMe v2.1

new fields, missing fields print added

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