plugins/wdc: remove fd argument from print functions

These don't need the NVMe device file descriptor, they're just printing
pre-collected data.

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

plugins/wdc: fix memset() on the address of a pointer

vuLogInput is a pointer, we're currently memset()-ing it like it's an
aggregate type.

We've already initialised the pointer to NULL above, so this memset
isn't doing anything.

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

plugins/solidigm: fix return value on format parse failure

If validate_output_format() fails, there are a couple of paths in the
solidgm code that will return a (closed) file descriptor, rather than an
error status. This looks to be a couple of copy-and-paste errors.

This change fixes the return value for these failure path, ensuring that
we'll actually have an error return.

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

plugins/wdc: prevent duplicate close on NVMe device

wdc_crash_dump() will close the NVMe device file descriptor, but so do
all of its callers.

This change removes the unnecessary close(), as well as a stray
backslash on the previous line (which may cause unexpected indentation
warnings).

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

build: Update version to v2.1.1

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

build: Extend release script to support micro version releases

In case we need to fix up a a maj.min release extend the release
script to accept x.y.z version strings.

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

Merge pull request #1629 from igaw/fix-release-fallouts

Fix release fallouts

build: Add minimum build requirement on libnvme

We depend on added APIs in libnvme, hence we need to update our build
requirment accordingly. Note, the ABI of the library is stable.

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

wrapper: Add weak nvme_init_copy_range_f1 symbol

Avoid adding a hard dependency on LIBNVME_1_1 section.

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

Merge pull request #1614 from igaw/sync-release-steps

Sync release steps with libnvme

Regenerate all documentation

Regenerate documentation for v2.1 release

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

build: Update libnvme wrap

Update to v1.1 of libnvme.

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

Merge pull request #1627 from igaw/fix-double-error-log

nvme: Do not print error message in collect_effects_log helper

nvme: Do not print error message in collect_effects_log helper

We print error and status on the top level, so no need to print it a
helper.

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

Merge pull request #1626 from igaw/drop-auth-ctrl-key

fabrics: Remove dhchap-ctrl-secret from discover/connect-all

fabrics: Remove dhchap-ctrl-secret from discover/connect-all

The dhchap-ctrl-secret is per controller but discover and connect-all
operate on several controllers.

As this get's very fast complicated how to express this on the command
line, just drop the it. It was not implemented anyway and not listed
in the documentation.

Use the JSON config file in this case.

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

Merge pull request #1624 from martin-gpy/async_event_config

nvme-print: sanitize the get-feature async event config output

nvme-print: sanitize the get-feature async event config output

Trivial fix to sanitize the nvme get-feature async event config
output by ensuring the details printed are properly aligned.

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

Merge pull request #1621 from igaw/fix-rae-users

nvme: Set default rae value for nvme_get_nsid_log users

Merge pull request #1619 from jeffreyalien/id-uuid-fix

print: Fix nvme_id_uuid_list

nvme: Set default rae value for nvme_get_nsid_log users

libnvme 1.0 hard codes the rae value for all nvme_get_nsid_log to
false. The upcoming 1.1 release will pass the rae value through
to the kernel. This is causing a behavior change. To be on the safe
side set all rae=true to false.

At least for nvme_get_log_error and nvme_get_log_smart this seems also
what the spec recommends.

Reported-by: Jeremy Kerr <jk@codeconstruct.com.au>
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1617 from igaw/disconnect-wo-scan

fabrics: Avoid nvme_scan_ctrl when disconnecting

Merge pull request #1620 from Francis-Pravin/remove-duplicate-line

nvme: Remove duplicate line

nvme: Remove duplicate line

Remove the duplicate prinfow assignment

Signed-off-by: Francis Pravin Antony Michael Raj <francis.michael@solidigm.com>
Signed-off-by: Jonathan Derrick <jonathan.derrick@solidigm.com>

fabrics: Avoid nvme_scan_ctrl when disconnecting

In certain corner cases, the kernel will not populate a nvme subsystem
but only create a nvme controller. nvme_scan_ctrl will fail in such
scenarios to find the controller object. Let's do the lookup based on
the ctrl name only.

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

build: Update version before regenerating docs

The docs contain the version string. Let's set new version before
regenerating the documentation to keep it consistent.

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

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>

print: Fix nvme_id_uuid_list

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

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>