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>

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>

build: Update libnvme dependency revision

Get the TP-4068 and TP-4084 changes.

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

Merge pull request #1548 from jeffreyalien/new-dev-support

wdc:  Add support for new WDC Devices

Merge pull request #1547 from igaw/build-docs-on-arch

docs: Don't confuse asciidoc with asciidoctor

wdc:  Add support for new WDC Devices

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

docs: Don't confuse asciidoc with asciidoctor

Apparently, asciidoc and asciidoctor are two different
projects. nvme-cli depends on the former. Depend on asciidoc only and
make it a hard dependency when the user ask to build the doc.

Fixes: 6b18babf40d4 ("docs: Search for 'asciidoctor' as alternative")
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #1545 from igaw/build-docs-on-arch

docs: Search for 'asciidoctor' as alternative

docs: Search for 'asciidoctor' as alternative

There are alternative binary names for asciidoctor. Search for
'asciidoc' or 'asciidoctor' when building the docs.

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

Merge pull request #1540 from igaw/list-subsys-show-path-once

nvme-print: Show paths from the first namespace only

Merge pull request #1537 from lgdacunha/Solidigm_VU_garbage_collect_log

Added Vendor Unique Solidigm Garbage Collect Log.

Added Vendor Unique Solidigm Garbage Collect Log.

nvme-print: Show paths from the first namespace only

When listing the subsystem, show the path from the first namespace
only. Every namespace has the same paths for a subsystem. This avoids
listening the same controllers for each namespace.

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

Merge pull request #1531 from igaw/apply-json-config

fabrics: Honor config file for connect-all

fabrics: Honor config file for connect-all

The command line option -J config.json allows to provide a
configuration via a JSON file. We handle this correctly for the
'connect' case but fail to apply the configuration for 'connect-all'
because we missed to add nvme_read_config() in this path.

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

Merge pull request #1528 from igaw/fix-mec-field

nvme-print: Show mec field in human output instead of vwci

nvme-print: Show mec field in human output instead of vwci

Looks like a copy&past error for the human output.

Reported-by: mgerdts
Signed-off-by: Daniel Wagner <dwagner@suse.de>
Fix: 43460fcce21b ("nvme: add identify controller structure 2.0 spec. fields")

Merge pull request #1526 from igaw/clarify-lisence

treewide: Add SPDX identifier

build: Clarify current main project license

The original desired license for the project was GPL-2.0-or-later
but during the development a few files (e.g. fabrics.c and nvme.h)
got added which are licensed as GPL-2.0-only, which renders the
whole project to GPL-2.0-only.

Update the meson license field and the nvme.spec license field.
While at it mentions this in the contributions notes as well.

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

treewide: Add SPDX identifier

Add the default project wide license SPDX identifier GPL-2.0-or-later
to all files which do not have any copyright statement.

Add the GPL-2.0-or-later or GPL-2.0-only according the copyright
statement listed in the file.

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

ocp: Update license to GPL-2.0-or-later

When I reviewed the plugin I asked wrongly for GPL-2.0-only. The
project is using the GPL-2.0-or-later license. Relax the license
requirement to GPL-2.0-or-later.

Cc: Wei Zhang <wzhang@fb.com>
Cc: Venkat Ramesh <venkatraghavan@fb.com>
Signed-off-by: Daniel Wagner <dwagner@suse.de>
Signed-off-by: Arthur Shau <arthurshau@fb.com>
Link: https://lore.kernel.org/r/20220517060256.6733-1-dwagner@suse.de

nvme-rpmb: Update license to GPL-2.0-or-later

The project is using the GPL-2.0-or-later license. Relax the license
requirement to GPL-2.0-or-later.

Signed-off-by: Daniel Wagner <dwagner@suse.de>
Signed-off-by: Hanumanthu H <hanumanthuh@micron.com>
Link: https://lore.kernel.org/r/20220517062806.9448-1-dwagner@suse.de

Merge pull request #1525 from glimchb/master

ci: add static library compilation

ci: bump meson version from 0.53.2 to 0.61.2

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

ci: disable werror for uuid subproject

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

subprojects: libnvme.wrap update for static uuid

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

ci: add static library compilation

Now possible since we can use uuid for fallback

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

Merge pull request #1523 from jeffreyalien/sn810-2-master

Changes for WDC SN810-2 Drive

Merge pull request #1510 from lgdacunha/feature/SMART_VU_plugin

Feature/smart vu plugin

solidigm: Add Solidigm pluging for vendor specific SMART fields.

Merge pull request #1527 from igaw/remove-parser

util: Remove unused parser

util: Remove unused parser

There is no user for parser. Remove the implementation.

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

Merge pull request #1524 from glimchb/master

build: use uuid wrap for fallback build

build: use uuid wrap for fallback build

see util-linux/util-linux#1657

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

Merge pull request #1518 from bmouring/master

completions: Collapse declaration and attribute assignment

Merge pull request #1513 from Foxboron/morten/systemd-service

meson_options: Include system sub directory

Merge pull request #1522 from igaw/sfx-byte-align

sfx: align fixes + smartlog