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>

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>

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

nvme: fix segfault in nvme telemetry-log error handling

After the upgrade to libnvme, the nvme telemetry-log command
segfaults for error scenarios as shown below:

NVMe status: Invalid Log Page: The log page indicated is invalid(0x4009)
Failed to acquire telemetry log 16393!
Segmentation fault (core dumped)

Freeing the telemetry log page header for error scenarios is already
handled in the libnvme. So avoid freeing this again here, which led to
this segfault.

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

Merge pull request #1397 from Arunpandian15/nvme_txt_01

cmds-main: Add media unit status log and supported capacity config list log pages in main commands list

cmds-main: Add media unit status log and supported capacity config list log pages in main commands list

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

Merge pull request #1395 from igaw/port-dsc-kato

fabrics: ensure zero kato for non-persistent controllers

fabrics: ensure zero kato for non-persistent controllers

Add a helper function that sets the default kato value for the
discovery controllers. Along with ensuring a non-zero kato value
(i.e. NVMF_DEF_DISC_TMO) is passed to a persistent discovery
controller, it also ensures a zero kato value is passed to a
non-persistent discovery controller, as mandated by the NVMe spec.

Signed-off-by: Martin George <marting@netapp.com>
[dwagner: ported from monolithic branch]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Use boolean type for docs-build default

muon complains about the type used for the default value. As it a
boolean type use a boolean as value instead of a string value.

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

build: Update libnvme wrap

Update to v1.0-rc4 of libnvme.

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

Merge pull request #1393 from igaw/fix-intel-nvme-compile-bug

intel: Workaround initializer element is not constant

Merge pull request #1392 from martin-gpy/ontap_free_nsdescs

netapp-nvme: free the nsdescs pointer after use

Merge pull request #1391 from Arunpandian15/doc_21

parser: Streamline documentation

intel: Workaround initializer element is not constant

Older gcc do not like the constant initializer expression in the intel
plugin:

../plugins/intel/intel-nvme.c: In function ‘lat_stats_log_scale’:
../plugins/intel/intel-nvme.c:662:47: error: initializer element is not constant
  static const int LATENCY_STATS_V4_BASE_VAL = (

Use defines as workaround.

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

netapp-nvme: free the nsdescs pointer after use

Ensure the nsdescs pointer is freed after use in
netapp_ontapdevices_get_info().

Fixes: 476d019 ("netapp-nvme: fix nvme ns desc uuid handling
for ontapdevices")

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

parser: Streamline documentation

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

Merge pull request #1388 from martin-gpy/get_ns_id_status

nvme: get_ns_id command fails on nvme device

Merge pull request #1390 from martin-gpy/json_simple_list_crash

nvme-print: fix 'nvme list -o json' segfault

nvme-print: fix 'nvme list -o json' segfault

After the recent upgrade to libnvme, 'nvme list -o json' segfaults
due to incorrect json object type passed here. So fix this by passing
the appropriate json object type and invoking it outside the
nvme_for_each_host loop.

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

nvme: get_ns_id command fails on nvme device

The get-ns-id command currently prints the perror status for a
given nvme device rather than its nsid. Fix this by properly
checking the return status of the nvme_get_nsid() function.

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

Merge pull request #1386 from AkhileshRN/SN_telemetry_type_desc

updated products list for telemetry (--type) argument in the wdc vs-internal-log command

wdc: updated products list for telemetry (--type) argument

Updated products list for telemetry (--type) argument in the wdc
vs-internal-log command.

Merge pull request #1385 from nobodyatandnothing/patch-1

fix typo

docs: fix typo in Data Set Management section

Merge pull request #1384 from paulentzel/master

Fix ctrlist for attach-ns and detach-ns

Fix ctrlist for attach-ns and detach-ns

Merge pull request #1379 from hanumanthuh/master

Micron plugin version 1.0.8

Merge pull request #1381 from igaw/revert-force-exclusive

Revert force exclusive

Merge pull request #1383 from martin-gpy/ontap_uuid_handling

netapp-nvme: fix nvme ns desc uuid handling for ontapdevices

netapp-nvme: fix nvme ns desc uuid handling for ontapdevices

The nvme ns desc uuid handling in the NetApp plugin for
ontapdevices is broken after the recent upgrade to libnvme. The
uuid hanlding logic here was flawed to begin with, so rework this
code to ensure the respective uuid descriptor itself is stored in
the ontapdevice_info structure and processed appropriately.

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

Merge pull request #1382 from jeffreyalien/pr-1375

[nvme-cli] Additional changes for PR 1375

wdc: Fix use-after-free access of cbs_data

Fix incorrect access after memory is already freed up. Also couple
of minor fixes to use correct cpu endianness while accessing data
read from drive.

Create a new function wdc_get_fw_cust_id that will simplfy the code
and handle the free of the allocated data

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
Co-Authored-by: Indraneel-M
[dwagner: updated commit message and merged everything into one patch]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme: Update help text for --force

The short flag version --force has been removed in
90592d5188fe ("nvme: Consolidate --force option and udpate
documetation") but the help text didn't get updated.

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

nvme: Open device exclusive for write command

Commit 2100609ddefb ("nvme: open namespace exclusive by default")
tried to make the write command a bit safer by asking to open the
device exclusive. Because the commit got reverted let's add the main
change back which is to use O_EXCL for write commands on default.

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

nvme: Only open device once

open_exclusive() takes care with opening the device. Do not open a
second time the device.

open_exclusive() has an overwrite flag. Document it a bit better by
renaming it what it actually means. The idea here is that the device
is openened with O_EXCL and when forced is used, open without this
flag.

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

nvme: Rename force variable to force_unit_access

Distinguish clearly between 'force' and 'force_unit_access' help text
by renaming the variable accordingly.

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

Revert "nvme: open namespace exclusive by default"

This reverts commit 2100609ddefbe5f2c93a6ba8fcf82e9adc3abcb9.

update libnvme commit-id

Merge pull request #1378 from samiWaheed/master

Update nvme-print.c:  Only print FW slots that contain  a valid FW image revision number

Merge pull request #1377 from martin-gpy/ontapdevices_json_segfault

netapp-nvme: fix ontapdevices segfault in json output

Micron plugin version 1.0.8

Update nvme-print.c

netapp-nvme: fix ontapdevices segfault in json output

After the upgrade to libnvme, the NetApp plugin for ontapdevices
now 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
ontapdevices. Include a couple of minor changes here 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>

build: Update libnvme wrap

Update to v1.0-rc3 of libnvme.

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

Merge pull request #1373 from sc108-lee/fix/io_cmd

nvme: fix dsm, dspec wrong usage

nvme: No need to get nsid twice

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

nvme: io dry-run command meant to be show command based on help

even if show-command arg is not given, it should show command

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

nvme: fix dsm, dspec wrong usage

dsm is 8bits on Command Dword13 [07:00]
dspec should issued on nvme_io_args.dspec

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

subproject: libnvme.wrap update

nvme_io_args.dspec changed as 16bits

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

Merge pull request #1371 from jeffreyalien/sn650-fix

[nvme-cli] Fix sn650 capabilities setting

[nvme-cli] Fix sn650 capabilities setting
  Change sn450 name to sn560
  Update wdc plugin version to 1.16.3

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

Merge pull request #1370 from igaw/fix-json-output-list-subsys

nvme-print: Fix json output for list-subsys

nvme-print: Fix json output for list-subsys

The root needs to be an array an not an object.

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

Merge pull request #1369 from igaw/fix-verbose-flag

argconfig: Increase the flag value

nvme: Allow --verbose flag to increase log level

Mark the option as an increase option to increase the log level if
needed.

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

argconfig: Increase the flag value

If the flag is provided several time increase the flag value. This is
a pretty ugly fix for how getopt_long_only() handles '--verbose' vs
'-v'. getopt_long_only() returns 0 when --verbose is used and 'v' for
-v hence we will not usually not take the short option path for
--verbose.

Break out in the short option parsing when non flag options are used.

Another uglyliness here is that it sets up long options differently
depending on short or long version is used. For the short version we
have to use value_addr.

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

Merge pull request #1368 from sc108-lee/remove/unused

common.h: remove unused macros

common.h: remove unused macros

round_up is not used
__stringify is defined in define_cmd and used
__fallthrough__ not used, occur error from lower version of GCC

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

Merge pull request #1363 from igaw/cleanup-libuuid-dep

nvme-print: Remove dead !LIBUUID code

Merge pull request #1361 from igaw/doc-build

docs: Introduce documentation build option

Merge pull request #1357 from igaw/fix-force-flag-parsing

nvme: Fix --force flag inversion

Merge pull request #1367 from jeffreyalien/sn650-fix

[nvme-cli] Fix vs-smart-add-log command for SN650 devices

[nvme-cli] Fix vs-smart-add-log command for SN650 devices

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

Merge pull request #1366 from AkhileshRN/SN_telemetry

Added telemetry log fetch support for WDC vs-internal-log command

Added telemetry log fetch support for SN810, SN530 and SN740 series NVMe SSDs through wdc vs-internal-log command

nvme-print: Remove dead !LIBUUID code

We have added a hard dependency to libuuid. Remove the ifdefs and the
dead code.

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

docs: Introduce documentation build option

Build the documentation only if explicitly asked to build it. If not
fallback to install the precompiled versions.

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

Merge pull request #1358 from igaw/fix-matching-nvmf-connect

nvmf: Remove --matching from systemd service file

nvmf: Remove --matching from systemd service file

'--matching' is not a great concept. The whole point of a discovery
log is to return different port addresses from the one we're using.

Thus the '--matching' logic was dropped when switching over the
libnvme but we forgot to remove it from the systemd service file.

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