build: Update release version rules

Include the release candidate version string into the project
version. This allows to upload every release candidate to PyPI instead
just the final release.

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

Release v2.1-rc0

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

Regenerate all documentation

Regenerate documentation for v2.1-rc0 release

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

build: Update libnvme wrap

Update to v1.1-rc0 of libnvme.

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

Merge pull request #1610 from jeffreyalien/master

[wdc] Fix capabilities settings for SN650 and SN655

wdc: Fix capabilities settings for SN650 and SN655

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

Merge pull request #1606 from Ssd5513146/master

ymtc: fix ymtc additional smart info display

ymtc: fix ymtc additional smart info display

Merge pull request #1603 from igaw/fix-return-code-from-effects-log

nvme: Return status/error code for effects-log command

Merge pull request #1605 from martin-gpy/get_feature_H_option

nvme: fix nvme get-feature with -H option

Merge pull request #1604 from CodeConstruct/wdc-close-fix

plugins/wdc: telemetry log fixes

Merge pull request #1607 from igaw/add-release-script

Add release script

build: Add release automation script

Add a script for generating a release.

Note, meson's project version is updated only once, either at the
begin of a RC phase or directly when there was no RC.

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

build: Add workflow to create release from tags

Let's automate the release process. The trigger to build a release is
the release tag.

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

nvme: fix nvme get-feature with -H option

Currently, nvme get-feature with the -H option fails to print detailed
info for specific feature ids such as number of IO queues (fid=0x07),
async event config (fid=0x0b), KATO value (fid=0x0f), etc. This is
because nvme_feature_show_fields() is invoked only if the corresponding
'buf' pointer is valid, which is not necessary for the above feature
ids. So fix this by removing this restriction while invoking
nvme_feature_show_fields(). At the same time, ensure this 'buf' pointer
is appropriately validated before use in this function.

Fixes: bcab212 ("nvme: coverity fixes")
Signed-off-by: Martin George <marting@netapp.com>

plugins/wdc: don't write invalid log data

In wdc_do_cap_telemetry_log, we currently continue to write log data if
any of the telemetry queries return with < 0.

This change aborts the output write on these errors instead.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>

plugins/wdc: prevent double close in wdc_do_cap_telemetry_log

Currently, wdc_do_cap_telemetry_log will close the NVMe fd before
returning, but so it all of its call sites. Since we'd be better off
making the fd open/close symmetrical, drop the close from
wdc_do_cap_telemetry_log.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>

nvme: Return status/error code for effects-log command

collect_effects_log eats up the return code from
nvme_get_log_cmd_effects. Forward it to the shell..

Fixes: 155fbebfe7b7 ("Update effects-log to handle multiple command sets")
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1599 from ikegami-t/timestamp

nvme: Check timestamp format result and increase string size

Merge pull request #1600 from igaw/replace-util-json

    nvme: Use correct json serializing for long doubles

nvme: Use correct json serializing for long doubles

Fixup all places which use json_object_add_value_float() to serialize
long double types with json_object_add_value_double(). The float
version cuts off the upper 64 bits of the 128 bit long double type.

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

json: Add helper to serialize long double types

Introduce a helper to create a struct json_object from a long double
type.

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

json: Remove own json implementation

We have a hard dependency on json-c, so let's remove our own json
implementation.

While at it move the json-c wrappers to util dir.

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

build: Cleanup json-c dependency

nvme-cli has hard dependency on json-c. Cleanup the leftovers from the
time where json-c was optional.

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

build: Explicitly add required on hard dependency

Make it explicit which library nvme-cli is depending on.

While at it also resort the section so that we list all
required dependencies.

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

docs: Correct spelling check error

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

nvme-print: Correct spelling check error

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

nvme: Correct spelling check error

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

nvme: Output timestamp value as thousands grouping characters

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

nvme: Make default timestamp format buffer size and test strftime retrun code

Since the current buffer size is exceeded for ja_JP.UTF-8 locale time.

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

Merge pull request #1594 from ikegami-t/docs

docs: Add nvme command RETURNS section for #1560 requirement

docs: Add nvme command RETURNS section for #1560 requirement

Required information on how an invoking process might determine success or
failure.

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

Merge pull request #1597 from ikegami-t/io

nvme: Fix submit_io to use nvme_io for #1565

nvme: Use opcode directly in submit_io

submit_io is an internal functions and all callers pass the correct
opcode in. Thus no need to reparse and select the right wrapper
function (nvme_read, nvme_write, ...). Instead just pass the opcode to
the generic nvme_io function.

Note, the compare was fixed in 9b6d3fca4289 ("submit_io(): fix "nvme
compare" to pass correct command") but missed the opportunity to
cleanup this part.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
[dwagner: reworded commit message]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1596 from igaw/remove-auto-review

build: Remove auto review bot

build: Remove auto review bot

Unfortunately, I was not able to figure out how to set the correct
permissions for necojackarc/auto-request-review to run.

The runner logs:

  Error: HttpError: Resource not accessible by integration

The documentation says that a PAT would work, but I really don't feel
comfortable to create such a powerful token.

So let's drop it and make the review assignment per manual
for the time being.

Links: https://github.com/necojackarc/auto-request-review#optional-github-personal-access-token

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

Merge pull request #1591 from igaw/update-json-c

build: Update json-c.wrap

Merge pull request #1592 from igaw/auto-review-bot-perm

build: Give auto review bot write permission on pull request

build: Give auto review bot write permission on pull request

Currently setting the reviewer from the bot fails with:
  Error: HttpError: Resource not accessible by integration

The GITHUB_TOKEN needs enough permission to update the PR.

Links: https://docs.github.com/en/actions/security-guides/automatic-token-authentication#permissions-for-the-github_token
Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Update json-c.wrap

Update to the latest the json-c wrap version.

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

udev: re-read the discovery log page when a discovery controller reconnected

When using persistent discovery controllers, if the discovery
controller loses connectivity and manage to reconnect after a while,
we need to retrieve again the discovery log page in order to learn
about possible changes that may have occurred during this time as
discovery log change events were lost.

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

Merge pull request #1590 from sc108-lee/rpm

rpmbuild: Do docs build to avoid error from missing .1, .html files

rpmbuild: Do docs build to avoid error from missing .1, .html files

Program asciidoc found: YES (/usr/bin/asciidoc)
Documentation/meson.build:253:8: ERROR: File nvme-wdc-cloud-boot-SSD-version.1 does not exist.

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

Merge pull request #1588 from lgdacunha/average_latency_tracking

Added average latency field to solidigm latency-tracking-log.

solidigm: Add average latency field

Add average latency field to solidigm latency-tracking-log.

While at it also remove the -b flag option from solidigm plug-in, in
favor of "-o binary".

ci: Add missing per_author reviewer entry

"per_author" is not set; returning no reviewers for the author.

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

Merge pull request #1583 from lgdacunha/Solidigm_Latency_Tracking

Added support to Solidigm Latency Tracking.

Added support to Solidigm Latency Tracking.

[dwagner: whitespace fixes]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1585 from igaw/add-autor-review

ci: Add auto review bot

Merge pull request #1584 from glimchb/dell

plugins: Add Dell plugin with basic id-ctrl

Merge pull request #1578 from zwhfly/master

docs: Document nvme-id-ns behavior when nsid is 0xFFFFFFFF

docs: Document nvme-id-ns behavior when nsid is 0xFFFFFFFF

as documented in NVM Command Set Specification Revision 1.0b,
chapter 4.1.5.1 - NVM Command Set Identify Namespace Data Structure (CNS 00h).

ci: Add auto review bot

Arthur has asked to be added as reviewer to the OCP plugin changes.
Let's setup an auto review bot. We might also want to hook up this for
the other plugins in the future.

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

plugins: Add Dell plugin with basic id-ctrl

Signed-off-by: Boris Glimcher <Boris.Glimcher@emc.com>

Merge pull request #1581 from AndyHouHit/master

create-ns: fix create-ns fail when -b 4096 while id-ns return nlbaf=1…

create-ns: fix create-ns fail when -b 4096 while id-ns return nlbaf=1 and 0 for 512, 1 for 4096

Fail case:
    sudo nvme create-ns /dev/nvme0 --nsze=1875366486   --ncap=937684566  --block-size=4096  --timeout=120000
        FLBAS corresponding to block size 4096 not found
        Please correct block size, or specify FLBAS directly

Signed-off-by: Wei Hou <wei.hou@scaleflux.com>

Merge pull request #1579 from igaw/add-innogrid-plugin

plugins: Add innogrit plugin

plugins: Add innogrit plugin

Add support for the innogrit plugin. It supports following commands:

   smart-log-add
   get-eventlog
   get-cdump

Code provided by innogritkenny
[dwagner: Whitespace fixes and small code refactorings.
          Added a commit message]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1575 from Karlson2k/completion_fixes_02

bash-nvme-completion.sh: a better fix for "readonly" with bash 5.x

bash-nvme-completion.sh: a better fix for "readonly" with bash 5.x

The error with one-line "readonly" with assignment on bash 5.x was fixed
by 25fd8c707b106ca0763402566ad657ef710bf09e. This commit restores "readonly"
functionality and adds some uniformity.

Merge pull request #1574 from Karlson2k/completion_fixes_01

Bash Completion fixes

bash-nvme-completion.sh: fixed typo with seagate plugin

Obvious typo resulted in broken completion for seagete plugin

bash-nvme-completion.sh: fixed error when sourced twice

One-line "readonly" with assignment produce error if completion file
is sourced for the second time (/etc/profile reload, su etc.)

Merge pull request #1572 from jeffreyalien/stc-control-bit-fix

Fix to set correct bit  for Storage Tag Check in the IO Control Flags

Merge pull request #1573 from igaw/tests-port-nose2

tests: Port to nose2 and unittest

tests: Port to nose2 and unittest

The test are still using nose which has been superseeded by
nose2. Update the tests to use the nose2 and unittest.

To minimize errors in most of the porting is done using sed, e.g
's/TestNVMeIO.__init__(self)/super().setUp()/g'.

While at it also update the build system and move the test files to
the build dir. This avoids in tree exectution of the Python files
which leaves the __pycache__ dir in the source tree.

This also fixes the problem, that the config file is not found when
'meson test -C .build' is used. We have the prober dir structure for
Python and nose2 to find all resources in the build root. Furthermore,
when running the tests the output is captured in the nvmetests
dir. This dir is also created under the build root dir.

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

nvme: Fix Storage Tag Check IO Control Flag

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

build: Update libnvme.wrap revision

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

Merge pull request #1571 from jeffreyalien/sn560-capabilities

wdc: Increment WDC Plugin Version

wdc: Increment WDC Plugin Version

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

Merge pull request #1568 from jeffreyalien/sn560-capabilities

wdc: Fix SN560 Device Capabilities

Merge pull request #1570 from igaw/sfx-get_lat_stats_log-csd3000

Sfx get lat stats log csd3000

sfx: remove unsupport ouptut option for query_cap and change_cap cmd

Signed-off-by: Wei Hou <wei.hou@scaleflux.com>

sfx: get_lat_stats_log support for csd3000

Signed-off-by: Wei Hou <wei.hou@scaleflux.com>

Merge pull request #1567 from linux-nvme/dependabot/github_actions/actions/setup-python-4

build(deps): bump actions/setup-python from 3 to 4

build(deps): bump actions/setup-python from 3 to 4

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 3 to 4.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #1569 from igaw/ci-add-python-version

ci: add python version to setup-python environment

ci: add python version to setup-python environment

v4 of the setup-python helper wants to know the python version number.

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

wdc: Fix SN560 Device Capabilities

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

Merge pull request #1566 from igaw/add-kernel-dep-note

README: add note about min kernel version

README: add note about min kernel version

The min kernel version is v4.15 kernel because libnvme depends on the
/sys/class/nvme-subsystem interface.

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

Merge pull request #1564 from jlevon/fix-compare

submit_io(): fix "nvme compare" to pass correct command

submit_io(): fix "nvme compare" to pass correct command

Since "18de3a6d Convert to libnvme", an "nvme compare" was being
converted incorrectly into an "nvme write" in submit_io().

Signed-off-by: John Levon <john.levon@nutanix.com>

Merge pull request #1561 from ankit-sam/master

libnvme wrap update, update power descriptor fields

print: Update power descriptor fields

1. Add missing active power workload field.
2. Use enums for power scale.
3. Fix indentation.
4. Update json entry for active power workload and
   fix idle scale.

Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com>

build: update libnvme wrap

Fetch power state descriptor enums from libnvme

Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com>

Merge pull request #1554 from hanumanthuh/master

Micron plugin version 1.0.14

Micron plugin version 1.0.14

Merge pull request #1550 from igaw/log-connect-errors

Log connect errors

fabrics: Already connected uses a different error code

The libnvme library reports connection errors with it's own error
codes, e.g. ENVME_CONNECT_ALREADY stands for already connected. Update
the connect-all call so that it prints 'already connected' again.

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

build: Update libnvme wrap

Get connect error codes feature from libnvme

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

Merge pull request #1549 from igaw/list-subsys-ana

nvme-print: Show ANA state only for one namespace

Merge pull request #1552 from ankit-sam/master

print: Update nvme id-ctrl ctratt fields and small cleanup

print: Update nvme id-ctrl ctratt fields and small cleanup

Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com>

nvme-print: Show ANA state only for one namespace

'nvme list-subsys' shows the state of all controllers belonging to a
subsystem. The ANA state is a per namespace attribute hence it only
makes sense to show it if the user lists the subsystem for a
namespace.

Fixes: 7435ed9ae6a6 ("nvme-print: Show paths from the first namespace only")
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1532 from jeffreyalien/tp-4068-4084-master

Changes to support TP-4068  and TP-4084

doc: Update to verify and write-zeroes for TP-4068

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

nvme: 64-bit Reference Tags and TP-4068 changes

Signed-off-by: Brandon Paupore <brandon.paupore@wdc.com>
Signed-off-by: Jeff Lien <jeff.lien@wdc.com>

print: Update show-regs for TP-4084

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

print: Fix json_ctrl_registers 64 bit fields

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