build: Include nvme_lba_status_log_test in meson.build

updated meson.build file with nvme_lba_status_log_test

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

tests: Add tests for lba status log page

Added test case for lba status log page

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1450 from Arunpandian15/tests_21_03

nvme_smart_log_test: call smart log for all ns based on LPA

nvme_smart_log_test: call smart log for all ns based on LPA

added check condition to check LPA value
before calling smart log for all namespace.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

nvme_test: Add supp_check_id_ctrl wrapper for support check

added supp_check_id_ctrl wrapper to check support
based on passed key value to make it generic.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1449 from Arunpandian15/tests_21_03

tests: print log info in next line

Merge pull request #1448 from martin-gpy/print_disc_async_event

nvme-print: print discovery async event support

tests: print log info in next line

printing log info as looks as below
nvme_fw_log_test.TestNVMeFwLogCmd.test_fw_log ...
Firmware Log for device:nvme0
afi  : 0x1
frs1 : xxxxxxx

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

nvme-print: print discovery async event support

Print the discovery log page change notices async event support
in nvme_feature_show_fields().

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

build: Update libnvme wrap

Update to v1.0-rc7 of libnvme.

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

doc: Regenerate all documentation

Prepare for next release candiate.

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

Merge pull request #1442 from igaw/bring-back-dev

nvme: Print full device path

Merge pull request #1445 from martin-gpy/eseries_json_segfault

netapp-nvme: fix smdevices segfault in json output

netapp-nvme: fix smdevices segfault in json output

After the upgrade to libnvme, the NetApp plugin for smdevices
segfaults due to the wrong json object type as shown below:

nvme: json_object.c:1194: json_object_array_add:
Assertion `json_object_get_type(jso) == json_type_array' failed.
Aborted (core dumped)

Fix this by passing the appropriate json array type for smdevices.
Also include a couple of minor changes as well - add a newline to
the end of the json output, and then free the json root device.

Signed-off-by: Martin George <marting@netapp.com>
Tested-by: Clayton Skaggs <claytons@netapp.com>

fabrics: keep the backward compatibility

Don't print the message while disconnecting it is breaking blktests,
see below without this patch :-

nvme/005 (reset local loopback target)                       [failed]
    runtime  6.799s  ...  6.814s
    --- tests/nvme/005.out 2021-08-29 01:09:20.287901783 -0700
    +++ /mnt/data/blktests/results/nodev/nvme/005.out.bad 2022-03-17 18:11:16.198191553 -0700
    @@ -1,2 +1,3 @@
     Running nvme/005
    +Disconnected nvme1
     Test complete

With this patch :-

blktests (master) # ./check nvme/005
nvme/005 (reset local loopback target)                       [passed]
    runtime  6.807s  ...  6.801s

Signed-off-by: Chaitanya Kulkarni <kch@nvidia.com>
Signed-off-by: Daniel Wagner <dwagner@suse.de>
Link: https://lore.kernel.org/r/20220318012408.24779-1-kch@nvidia.com

Merge pull request #1443 from Arunpandian15/plugin_16

cmd-plugins: Add ZNS plugin commands

Merge pull request #1441 from sc108-lee/build

rpm: use meson configuration instead of default macros

Merge pull request #1444 from igaw/fix-json-encoding

nvme: Do not slash escape strings in JSON output

cmd-plugins: Add ZNS plugin commands

updated cmd-plugins.txt file with ZNS commands.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Documentation: Fix typo in command description

updated nvme-zns-id-ns command description from
Identify controller to Identify Namespace.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

rpm: use meson configuration instead of default macros

meson:
remove unused variables (libdir, includedir, bindir)
add configuration data of installation directory

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

nvme: Do not slash escape strings in JSON output

nvme-cli 1.x didn't do any (optional) slash escaping in JSON
output. Don't break backwards compatibility unnecessary.

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

nvme: Print full device path

Restore nvme-cli 1.x behavior to print the full path to the device
file. We assume that the devtmpfs is mounted under /dev which is the
case for any systemd based distro and a de-facto standard.

But SPDK users see their device populated under /dev/spdk have their
device file appear. So instead blindly prefix /dev do a stat to check
if the file exist.

If our educated guesses fail, don't add the full path. This should be
really a corner case and as long no one complains don't overengineer.

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

Merge pull request #1248 from minwooim/print-generic-device-in-list

nvme-print: print generic device in list command

Merge pull request #1416 from sc108-lee/build

installation path modification

rpm: Add prefix for sysconfdir

rpm macro sysconfdir does not include prefix

bug fix for wrong macro use
%{_sysconfdir} defined as /etc
as is:
    /buildroot_somepath//etc/nvme/hostnqn

to be:
    /buildroot_somepath/etc/nvme/hostnqn

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

meson: installation prefix revert

make sbin dir using prefix as other installation dir
    nvme <= v1.16 SBINDIR has prefix like below
    SBINDIR = $(PREFIX)/sbin

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

meson: bash-completion installation path is reverted

orignal Makefile install in blow path
$(DESTDIR)$(PREFIX)/share/bash-completion/completions/nvme

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

Merge pull request #1440 from Arunpandian15/plugin_16

cmd-plugins: Add plugin commands in doc

cmd-plugins: Add plugin commands in doc

Added WDC plugin commands in cmd-plugins.txt file

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1437 from tbzatek/json-strings

nvme-print: Make JSON keys consistent with nvme-cli 1.x

nvme-print: Make JSON keys consistent with nvme-cli 1.x

nvme-print: print generic device in list command

Generic namespace device is one of I/O paths that needs to be listed up
in the `nvme list` command along with the blkdev.  Print it out in the
simple list command.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #1436 from hreinecke/disc-nqn

Fixup persistent discover controller usage

fabrics: check for discovery controller instead of subsystem NQN

Unique discovery subsystems have a unique subsystem NQN, to we cannot
check for the standard discovery NQN when checking for persistent
discovery controllers, but rather have to check if the subsystem is a
discovery subsystem.

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

build: Update libnvme wrap

Update to pull in fix for nvme_free_ctrl() and systemd paths.

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

Merge pull request #1434 from Arunpandian15/plugin_16

cmd-plugins: Add transcend, virtium plugin commands

cmd-plugins: Add transcend, virtium plugin commands

updated cmd-plugins.txt file with transcend, virtium
plugin commands

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1431 from igaw/fix-sysconfdir

Honor sysconfdir, sbindir and bindir

nvmf-autoconnect: Add option to specify path to systemctl

Many distros have adapted the usr merge and hence systemctl is install
in /usr/bin. Instead just hard coding the new path, let's introduce a
new build option to allow specifying the path to the systemclt
binary.

Note, the scripts also have hard coded paths to /bin/sh or
/bin/echo. None of these paths are covered by POSIX but in practice
all distros have those as symlinks when the usr merge have been
implemented. Let's address these issues when the pop up.

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

nvmf-autoconnect: Honor sysconfdir, sbindir and bindir

The autoconnect files use hard coded paths for the config files and
the binaries. Make these configurable.

Not all files need to run through the configure_files step but for
consistenty sake, let's treat all the same.

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

fabrics: Honor sysconfdir for hostnqn and hostid files

When sysconfdir is set during build the hostnqn and hostid file should
be installed under this path. Replace the hard coded path and also
update the documentation.

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

build: Copy include adoc files to build dir

asciidoctor wants all file to live under one common directory root,
thus we can't provide a search path for the include files. Copy
include files to the build dir.

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

build: Resort adoc_sources

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

Merge pull request #1428 from martin-belanger/fix-typo

Fix typo - hsot to host

Merge pull request #1429 from tbzatek/fixes-1

Minor compiler warning fixes

plugins/memblaze: Code style fixes

clang complains, purely cosmetic change:

 ../plugins/memblaze/memblaze-nvme.c:1119:2: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
         return err;
         ^
 ../plugins/memblaze/memblaze-nvme.c:1102:7: note: previous statement is here
     } else if (err > 0)
       ^

plugins/wdc: Fix a use-after-free

 ../plugins/wdc/wdc-nvme.c: In function ‘get_dev_mgment_cbs_data’:
 ../plugins/wdc/wdc-nvme.c:1714:31: warning: pointer ‘hdr_ptr’ used after ‘free’ [-Wuse-after-free]
  1714 |                 data = calloc(le32_to_cpu(hdr_ptr->length), sizeof(__u8));
       |                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
 ../plugins/wdc/wdc-nvme.c:1713:17: note: call to ‘free’ here
  1713 |                 free(data);
       |                 ^~~~~~~~~~

Fix typo - hsot to host

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

Merge pull request #1425 from tbzatek/connect-return-code

connect: Set errno to zero on nvmf_add_ctrl() success

connect: Set errno to zero on nvmf_add_ctrl() success

The libnvme's nvmf_add_ctrl() might return zero (success) while
having errno set to a non-zero value due to lots of calls inside.
And since nvmf_connect() returns errno primarily, make sure
it's zeroed on success.

See related https://github.com/linux-nvme/libnvme/pull/292

Merge pull request #1418 from Arunpandian15/tests_10

tests: Add tests for copy, verify, dsm commands

build: Add copy, DSM, verify command in meson build

meson.build file is updated with newly
added copy, DSM, verify tests

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

tests: verify command test case

test case added for verify command

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

tests: Add test for DSM command

Data Set Management command test case
has beed added.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

tests: Add test for copy command

test case added for copy command

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

nvme: constify variable in list_subsys

The compiler complains with

../nvme.c: In function ‘list_subsys’:
../nvme.c:2478:19: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
 2478 |                 n = strrchr(devicename, 'n');
      |                   ^

devicename is of type 'const char *' so the returning pointer is of
the same type. Thus constify the n variable.

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

doc: Regenerate all documentation

Prepare for next release candiate.

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

build: Enable docs build in update docs script

Commit 59263bbcd3e4 ("build: Use boolean type for docs-build default")
introduced a command line option to enable docs build. Let's enable
this in the update docs script.

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

build: Update libnvme wrap

Update to v1.0-rc6 of libnvme.

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

Merge pull request #1420 from hreinecke/ana-path

Fixup ANA path state display

nvme: print out ANA state for 'list-subsys'

Commit 00aeb92 ("nvme-list-subsys: Add device name argument
and print out ANA state") was not properly ported to libnvme/nvme-cli.
So port the missing bits.

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

build: Update libnvme wrap

Update to pull in changes for ANA path state display.

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

Merge pull request #1414 from igaw/init-args

nvme: Initialize all arguments for passthru

Merge pull request #1415 from Arunpandian15/plugin_netapp

cmd-plugins: plugin commands update in doc

tests: fix typo for format-python

changed the command to format-python as
per meson.build file

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

cmd-plugins: plugin commands update in doc

netapp, toshiba plugin commands has been
added in cmd-plugins.txt

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

nvme: Explicit initialize all command line options

Sort the command line options like the appear on the command line.
Also tabilize and initialize all command options. The consistent
aligment of the options makes the reviewing simpler.

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

nvme: Explicit initialize passthru command line options

The metadata config options needs to be initialized to an empty
string because we are using strlen to figure out if it is set or not.

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

Merge pull request #1413 from Arunpandian15/plugin_08

cmd-plugins: Add dera, micron plugin commands in doc

Add dera, micron plugin commands in doc

updated cmd-plugins.txt file with dera,
micron plugin commands.

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1410 from sc108-lee/list_ns

nvme: list_ns bug fix (csi option enable)

Merge pull request #1409 from sc108-lee/fix/write

nvme: nvme write bug fix (no parse for option)

Merge pull request #1408 from Arunpandian15/huawei_plugin

cmd-plugins: Add huawei plugin commands

nvme: list_ns bug fix (csi option enable)

recover functionallty of 7f8d194e2532ba5c8240d731e87c39186ba03764
list ns with CNS(0x07, 0x1A) enabled

Reported-by: Kyungsik Shin <ks1012.shin@samsung.com>
Signed-off-by: Steven Seungcheol Lee <sc108.lee@samsung.com>

build: Update libnvme wrap

bug fix for blow api included
nvme_identify_active_ns_list_csi
nvme_identify_allocated_ns_list_csi

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

nvme: nvme write bug fix (no parse for option)

occured from 0adb8c2c79df953bbef9e08ce684239a5650638f
write command always print below error
data size not provided

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

cmds-plugins: Add huawei plugin commands

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

Merge pull request #1407 from sc108-lee/master

build: Update libnvme wrap

Merge pull request #1406 from sc108-lee/build_assert

Add build_assert to fix debug build error

Merge pull request #1403 from Arunpandian15/intel_cmds

cmds-main: Add intel plugin commands

build: Update libnvme wrap

types: fix status code type bug (wrong masking)
https://github.com/linux-nvme/libnvme/pull/274/commits

Bug:
    NVMe status: unrecognized(0x13)
Meant to be:
    NVMe status: PRP Offset Invalid: The Offset field for a PRP entry is invalid(0x13)

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

Add build_assert to fix debug build error

../ccan/ccan/str/str.h:169:10: fatal error: ccan/build_assert/build_assert.h: No such file or directory
  169 | #include <ccan/build_assert/build_assert.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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

cmds-main: Add intel plugin commands

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

build: Update libnvme wrap

Update to v1.0-rc5 of libnvme.

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

Merge pull request #1404 from igaw/update-tests

tests: Add support for linting and formatting of Python code

tests: Refresh Python code

autopep3 and isort updated the code according the coding guidelines.

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

tests: Add support for linting and formatting of Python code

Shamelessly steal the linting and formatting tooling for Python code
form python-sdbus. Update the documentation accordingly.

Link: https://github.com/python-sdbus/python-sdbus
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1359 from igaw/mention-subproject-update

README: Mention meson subprojects update

README: Mention meson subprojects update

New meson users are not aware of this rather nifty update
mechanism. Mention it in the README.

While at it also mention the library dependency.

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

Merge pull request #1402 from sc108-lee/submit_io

passthru bugfix

Merge pull request #1401 from sc108-lee/meson

meson: remove rpm dependencies when build as static

nvme: passthru bugfix(wrong jump, wrong file descriptor)

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

nvme: remove unnecessary line from submit_io

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

meson: remove rpm dependencies when build as static

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

Merge pull request #1399 from igaw/traddr-case-insesitive

nvme-cli: Ignore traddr case

Merge pull request #1400 from Arunpandian15/cmds_main_2

cmds-main: Add all the admin, NVM commands in main commands list

cmds-main: Add all the admin, NVM commands in main commands list

Signed-off-by: Arunpandian J <apj.arun@samsung.com>

nvme-cli: Ignore traddr case

There was is no requirement in the NVME-FC specification that traddr
needs to be upper or lower case. Given the case difference, the
connect-all fails the match logic. Switch to strcasecmp for a
case insensitive compare.

Based on 1264c6323937 ("nvme-cli: Make connect-all matching be case
insensitive")

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

Merge pull request #1398 from martin-gpy/telemetry_error_crash

nvme: fix segfault in nvme telemetry-log error handling