Merge pull request #1340 from hreinecke/tls-key

nvme: fixup 'check-tls-key'

Merge pull request #1339 from igaw/drop-nvme_msg

fabrics: Stop using nvme_msg from libnvme

nvme: fixup 'check-tls-key'

The offsets into the key are wrong, causing the check to fail on
valid keys.

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

fabrics: Stop using nvme_msg from libnvme

nvme_msg() is logging function from libnvme. The cli should not use
the logging infrastructure from the library instead it should use it's
own infrastructure. At this point nvme-cli doesn't have one, so just
use the default stderr for all error messages.

This is a preperation step to unexport the nvme_msg() from libnvme.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1337 from hreinecke/disable-sqflow

build: Update libnvme wrap

build: Update libnvme wrap

Get updated nvmf_add_ctrl() argument list (libnvme PR#195)
and fixup callers.

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

Merge pull request #1336 from hreinecke/ctrl-init

build: Update libnvme wrap

build: Update libnvme wrap

Get fix for incorrect controller display (libnvme PR#195)

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

build: Update libnvme wrap

Get compile fix for latest nvme_get_log_args changes (libnvme PR#192)

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Update zlib to verison 1.2.11-6

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1330 from Arunpandian15/Media_Unit_Status_Log_Page

nvme: Add Media Unit Status log page(LID: 0x10)

nvme: Add Media Unit Status log page(LID: 0x10)

Reviewed-by: Steven Seungcheol Lee <sc108.lee@samsung.com>
Signed-off-by: Arunpandian J <apj.arun@samsung.com>

build: Sync libnvme wrap to lastest version

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1335 from hreinecke/tls-key

nvme: add 'gen-tls-key' and 'check-tls-key' options

Merge pull request #1333 from sc108-lee/fix/#1324

fix #1324 using explicit casting, using libnvme macro for cap

Merge pull request #1329 from samiWaheed/master

list-ctrl should support CNS values of 12h and 13h

Merge pull request #1332 from hreinecke/check-ns

nvme-print: check namespace in nvme_ctrl_for_each_path()

nvme: add 'gen-tls-key' and 'check-tls-key' options

Add options to generate and check an NVMe TLS PSK.

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

Merge pull request #1307 from sc108-lee/fix/flbas

nvme: modify flbas rsvd bits to be used

fix #1324 using explicit casting, using libnvme macro for cap

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

nvme-print: check namespace in nvme_ctrl_for_each_path()

We should check if the namespace is available when scanning all
paths in nvme_ctrl_for_each_path(); a controller might be resetting
and might not be able to access the namespaces here.

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

Merge pull request #1327 from igaw/remove-lightnvm-doc

doc: Remove lightnvm documentation

nvme: modify flbas rsvd bits to be used

flbas Bits 6:5 indicate the most significant 2 bits of
the Format Index of the supported LBA Format indicated
in this data structure that was used to format the namespace

using libnvme util function nvme_id_ns_flbas_to_lbaf_inuse
to extract currently used lba index

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

Merge pull request #1328 from jeffreyalien/compiler-warnings2

[nvme-cli] Fix build warnings in Micron Plugin - Issue 1317

Update nvme.c

list-ctrl should support CNS values of 12h and 13h.  The support for CNS=13h was dropped from 2.0 version

[nvme-cli] Fix build warnings in Micron Plugin - Issue 1317

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

doc: Remove lightnvm documentation

Commit fa49249 ("nvme: remove lightnvm") removed the implementation but
missed to remove the documentation. Remove it now.

While at it also remove it from bash-completion.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1325 from jeffreyalien/compiler-warnings2

[nvme-cli] Fix build warnings in WDC Plugin from Issue #1309

[nvme-cli] Fix build warnings in WDC Plugin from Issue #1309

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

Merge pull request #1318 from hreinecke/host_traddr

Update to libnvme host_traddr changes

fabrics: Only disconnect discovery controllers during 'connect-all'

When calling 'connect-all' the 'persistent' argument only applies to
discovery controllers; normal I/O controllers should stay connected,
as this was kinda the point...

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

Remove 'nvme-private.h'

With the libnvme split this file is not used anymore.

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

fabrics: update for libnvme host_traddr and host_iface changes

Libnvme moved the 'host_traddr' and 'host_iface' entries into the
fabrics configuration, so update nvme-cli to follow suit.

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

Merge pull request #1316 from jeffreyalien/new-dev-support2

Add support for new PCI device ID for SN655 Drive

[nvme-cli] Update WDC plugin version to 1.16.0

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

[nvme-cli] Add support for new SN655 PCI ID - 0x2722

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

Merge pull request #1314 from igaw/api-rework

nvme: Adapt to API rework

nvme: Adapt to API rework

libnvme changed the calling API, which uses an an argument structure
instead of passing all arguments one by one. Update all call sites
accordingly.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1313 from igaw/fix-unitialized-variable

Fix unitialized variable

nvme: Initialize variable in passthru

Always initialize huge so that we avoid reading the unitialized
variable in nvme_free().

Fixes: fb15eee3d72b ("support a meatadata file redirection for passthru").
Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Add libhugetablefs to CI environment

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1310 from linux-nvme/pt-meta

Pt meta

update passthrough documentation

Adding the metadata file parameter.

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

support a meatadata file redirection for passthru

The passthrough command provided a way to specific a metadata size, but
no way to save or set the data from a file. Provide an option so this is
allowed.

While we're here, it looks like the passthrough command only supported
file redirection for data-out commands. Go ahead and fix that with this
commit since the metadata needs to go both ways too.

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

build: Add provide section in libnvme.wrap

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1297 from igaw/openssl-3-support

Add OpenSSL 3 support for dhchap-key generation

nvme: Use libnvme for generating DH-HMAC-CHAP host keys

libnvme learned to generate the keyes. Use it. With this we can also
drop the dependency on OpenSSL.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1290 from igaw/logical-block-size

 nvme: support ns generic device to submit_io

nvme: Use libnvme helper to retrieve logical block size

Use the libnvme helper nvme_get_logical_block_size to figure out the
logical block size. This way we have support for normal char
devices (nvme%d) and for generic devices (ng%d%d).

Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme: Remove unused function decleration

There is no implementation for these functions and no user.
Remove it.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1302 from igaw/fix-help-message-fid-support-effects-log

Fix help message fid support effects log

nvme: Fix entry short help text for fid-support-effects-log

The function is to retrieve the log page information. Update the help
text accordingly and moved the entry up into the group of get_log_page
functions.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Set maximum warning level

Without defining the default warning level the build system
will use the default level, which is implementation depended.
For example muon sets it to 3 which includes '-Wpendantic'.
This results in a lot of ISO-C non compliant warnings.

Let's define the warning level so that all build systems
are using the same values.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Ingore default .build directory

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1306 from igaw/build-replace-make-by-meson

build: Replacing make by meson.

build: Replacing make by meson.

This involves replacing all Makefiles by meson.build files.
Also the top-level configure script now simply invokes meson.
The top-level Makefile is just a wrapper for meson. It supports
the following operations:

make
make test
make install
make clean    # Remove build artifacts but keep configuration
make purge    # Remove build artifacts and configuration

Based on Martin Belanger's libnvme work.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Fix *.wrap exclude pattern in .gitignore

The current rules trigger git to to complain with

 $ git add subprojects/libnvme.wrap
 The following paths are ignored by one of your .gitignore files:
 subprojects

We need to keep the parent directory in the tracking
domain and just filter out all sub directories and files
except the wrap files.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1304 from jeffreyalien/master

[nvme-cli] Fix vs-smart-add-log bug to retrieve customer id

[nvme-cli] Fix vs-smart-add-log bug to retrieve customer id

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

Merge pull request #1301 from igaw/build-fix-include-config

build: cleanup and make the defines more consistent

build: Use _dep postfix for dependency objects

Use consistently _dep postfix for all dependency objects.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Prefix configuration options

Use the prefix CONFIG for all configuration option.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1300 from eli-schwartz/meson-fixups

Meson fixups

meson: fix incorrect substitutions in configured files

The custom Makefile rules performed this substitution:

@@VAR@@ -> VAL

However, meson did this substitution:

@@VAR@@ -> @VAL@

This produced incorrect behavior, such as configuring a udev file to
read:

install_items+=" @/usr/etc/udev/rules.d@/70-nvmf-autoconnect.rules "

muon, meanwhile, failed loudly with:

/tmp/nvme-cli/nvmf-autoconnect/dracut-conf/70-nvmf-autoconnect.conf.in:1:19: error: key of zero length not supported
  1 | install_items+=" @@UDEVRULESDIR@@/70-nvmf-autoconnect.rules "
                        ^
/tmp/nvme-cli/meson.build:161:1: error: in function configure_file
161 | configure_file(
      ^

Since meson does not support custom token rules for configuring
replacements, and there is no real benefit to doubling the @@, just fix
this to use one @.

meson: fix incorrect argument call to has_header()

It only accepts a string, but an array of strings was passed.

Due to meson internal implementation details, arguments are sometimes
flattened and this works anyway.

However, muon choked on it and errored out.

Merge pull request #1215 from olase23/open_namespace_exclusive

nvme: open namespace exclusive by default

nvme: Consolidate --force option and udpate documetation

Refactor the error handling for the --force option.
Drop support for the -f and use --force consistent in all commands.
Also update the documation accordingly.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme: open namespace exclusive by default

All data modifying functions open the namespace exclusive. With the --force
option this can be bypassed.

Signed-off-by: Olaf Schmerse <olaf.schmerse@ionos.com>

memblaze-nvme: remove power loss protection from smart-log-add (#1292)

* remove power loss protection from smart-log-add

Signed-off-by: jinhua2021 <jinhua.huang@memblaze.com>

Merge pull request #1291 from igaw/meson-feature-option

build: Use Feature option for OpenSSL

build: Use Feature option for OpenSSL

Simplify the OpenSSL logic by using meson's Feature option.

Suggested-by: Eli Schwartz <eschwartz93@gmail.com>
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1288 from igaw/gen-nvme-connect-doc

Generate nvme-connect documentation

fabrics: Remove unnecessary brackets

0ad49ade724e ("Print device name on connect") introduced the brackets
which were necessary in the orginal code base. Drop them.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

docs: Update nvme-connect

0ad49ade724e ("Print device name on connect") added new option,
update the generated documetation.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1287 from ikegami-t/nvme-connet

Print device name on connect

Print device name on connect

Commit 0ad49ad ("Print device name on connect")
Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

Merge pull request #1286 from igaw/add-ctrl-metadata-and-ns-metadata

Add ctrl metadata and ns metadata

nvme-print: Add Controller Metadata and Namespace Metadata

Port and extent the libnvme part from the commit b8a403b ("Add NVMe MI
Features: Controller Metadata (0x7E) and Namespace Metadata (0x7F).")
from the nvme-cli monolithic branch.

While at it add the missing definitions.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme-print: Resort nvme_features_ids usage

The order is not matching the definion order of the spec. Let's make
it consistent and order them according the value.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1285 from igaw/wrapdb-openssl-zlib

build: wrapdb for OpenSSL and zlib, detect libhugetblfs

build: Change wording from 'required' to 'available'

The help text is added to config.h which is a bit missleading. The
flag indicates if the library/header/feature is available not if it is
required.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Use compiler.find_library for libhugetlbfs

libhugetlbfs is not detected via dependency() because the library
neither provides a pkg-config() nor a cmake helper. Use
compiler.find_library to figure out if the library is available.

We do not use the has_header feature of compiler.find_library()
function as this would update our minimum version for meson to
0.50.0. Instead open code it by using compiler.has_header().

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Add wrapdb support for zlib

The meson wrapdb library supports also zlib. Let's add it as fallback
option to the build.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Add wrapdb support for OpenSSL

The meson wrapdb library has support for OpenSSL. Introduce a combo
option for full control if the user wants OpenSSL added to nvme-cli or
not. Only if the option is set to 'true' the fallback will be
considered when there is no system version of OpenSSL available.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Add subpackages contents to .gitignore

we just need to maintain .wrap files for dependencies

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

Merge pull request #1277 from igaw/add_get_log_page_fields_2.0

nvme: Add Command Set Idendifier and Offset Type to get_log

nvme: Add Command Set Idendifier and Offset Type to get_log

Add Command Set Identifier (CSI) and Offset Type (OT) support to
get_log as per the Spec 2.0.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1284 from sc108-lee/fixes

Fixes

Merge pull request #1282 from bjpaupor/report-zones-json

Fix JSON Report Zones Output

build: Add subpackages dirs to .gitignore

Add the two not tracked by git directories to .gitignore.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme-print: fix wrong log name, reorder case based on lid

Log name based on spec to be aligned other logs name

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

nvme: getrandom function should be defined when sys/random.h is exist

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

Fix JSON report zones output

Previously multiple JSON objects would be in the output of
zns-report-zones if there were more than 1024 zones requested because of
the chunked requests sent by nvme-cli.

Fix this to output a single JSON object with one zone_list. This is done
by passing around and appending to the zone_list, then outputting all of
it once after receiving all the reports.

Signed-off-by: Brandon Paupore <brandon.paupore@wdc.com>

Merge pull request #1276 from MatiasBjorling/zns-fixes

zns: small cleanups to zrwa

zns: Documentation update and whitespace fixes

Signed-off-by: Daniel Wagner <dwagner@suse.de>

zns: small cleanups to zrwa

A couple of cleanups for the ZRWA to align it with the existing
style.

Signed-off-by: Matias Bjørling <matias.bjorling@wdc.com>

Merge branch 'pel_fix_nvme_cli'

nvme-cli: Add support Telemetry CRT in PEL

Add "Telemetry CRT" event in PEL.

Signed-off-by: Wen Xiong <wenxiong@linux.ibm.com>

nvme-cli: Add support set feature event in PEL

Add "Set Feature" event in PEL.

Persistent Event Entries:
Event Number: 0
Event Type: Set Feature Event(0xb)
Event Type Revision: 1
Event Header Length: 21
Controller Identifier: 65
Event Timestamp: 564587204146155
Vendor Specific Information Length: 0
Event Length: 16
Set Feature Event Entry:
Set Feature ID  :0x7 (Number of Queues),  value:0x270027
        Number of IO Completion Queues Allocated (NCQA): 40
        Number of IO Submission Queues Allocated (NSQA): 40

Signed-off-by: Wen Xiong <wenxiong@us.ibm.com>

nvme-cli: Adds readable firmware level in persistent

For example, In Firmware Commit Event:

Old Firmware Revision: 3617604718875264338
New Firmware Revision: 3833777500989048146

changes to

Old Firmware Revision: 3617604718875264338 (REV.SP42)
New Firmware Revision: 3833777500989048146 (REV.SP45)

Signed-off-by: Wen Xiong <wenxiong@linux.ibm.com