meson: Install udev rules to /usr/lib

Signed-off-by: Morten Linderud <morten@linderud.pw>

doc: Regenerate all documentation

Prepare for first release of the 2.0 major version.

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

docs: Fix title for ocp plugin commands

asciidoc complains due to correct title markdown.

  asciidoc: FAILED: manpage document title is mandatory

While at it also stream line the documention slightly.

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

build: Update libnvme wrap

Update to v1.0 of libnvme.

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

build: Update zlib wrap

Update to zlib-1.2.12

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

Merge pull request #1490 from igaw/connect_unique_ctrl

Create persistent controller using unique subsystem NQN

fabrics: Create persistent controller using unique subsystem NQN

Prefer unique subsystem NQN discovery controllers over well-known
subsystem NQN when a persistent controller is created.

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

fabrics: Refactor discover controller creation

Move the discover controller creation code into one place.

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

fabrics: Refactor argument passing for transport configuration

We passing all the attributes of the transport configuration
individually. Group them together to reduce the number of arguments
one has to pass into the functions.

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

fabrics: Set default trsvcid before lookups and match

Set the default trsvcid before trying to lookup or match against
existing persistent discovery controllers.

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

Merge pull request #1489 from igaw/fix-persistent-kato-tmo2

fabrics: Set KATO for discovery controller when connecting

Merge pull request #1488 from igaw/fix-discover-argument

fabrics: Switch connect and persistent argument for __disover

fabrics: Set KATO for discovery controller when connecting

In case we are connecting to the discovered controllers we should also
set the KATO value for persistent discocovery controllers correctly.

Unfortunatly, the compiler seems to get confused by only conditionally
setting tmo and warns about uninitialized variable. Hence we always
store and restore keep_alive_tmo.

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

fabrics: Do no modify default config for discovery controller

When a discovery controller is setup we are currently modifying the
default config. As this configuration might also be used for normal
I/O controllers don't modify it permanently.

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

fabrics: Refactor discover controller setup

Move the setup of the discovery controller into a helper
function. This in preperation for handling KATO value updates
centrally.

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

Merge pull request #1485 from igaw/scan-filter-update

nvme: update to nvme_scan_filter_t modifications

Merge pull request #1486 from jeffreyalien/master

build: Update meson tips in README.md

nvme: update to nvme_scan_filter_t modifications

The scan filter in libnvme has been updated, so update the code
to make use of it.

Signed-off-by: Hannes Reinecke <hare@suse.de>
[dwagner: list also the paths for non multipath namespaces]
Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Update libnvme wrap

Get the changes in the filter API and the removal of
ctrl_get_ana_state.

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

Merge pull request #1483 from tbzatek/sysfs-fixes-2

nvme-print: Fix json array object instantiation

nvme-print: Fix json array object instantiation

In the case of multiple hosts records the array objects
gets corrupted.

Merge pull request #1484 from wubo009/master

Fix memory leak when nvme_scan_toplopy return failed

Merge pull request #1405 from arthurshau/add/ocp-plugin

Add new OCP plugin

Fix memory leak when nvme_scan_toplopy return failed

doc: Add doc files for new ocp plugin commands

Add new OCP plugin
New OCP plugin that consolidates some of the functionality from the other vendor plugins. For now, implements getting of C0 and C3 log pages.

Signed-off-by: Arthur Shau <arthurshau@fb.com>
Signed-off-by: Wei Zhang <wzhang@fb.com>
Signed-off-by: Venkat Ramesh <venkatraghavan@fb.com>

build: Update meson tips in README.md

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

Merge pull request #1480 from igaw/fix-default-ports

fabrics: Set default trsvcid ports for TCP and RDMA

Merge pull request #1474 from hreinecke/intel-json

plugins/intel: make 'buckets' a json array

Merge pull request #1471 from jeffreyalien/OCP-2.0-final

WDC plugin changes to support OCP 2.0 Spec

fabrics: Set default trsvcid ports for TCP and RDMA

libnvme doesn't know at controller creation if the new controller is
going to be discovery controller. But nvme-cli knows it, so just set
defaults on the caller level.

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

Merge pull request #1478 from igaw/fix-empty-hostnqn-file

fabrics: Do not free static string

fabrics: Do not free static string

38f5a54a7306 ("fabrics: Support connect even when no /etc/nvme/hostnqn
file exists") introduce a regression. arg_parser() might assign a
static string to hostnqn and/or hostid. Can't free this.

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

fabrics: Switch connect and persistent argument for __disover

The order of the connect and persistent argument for the recursive
call to __discover is in the wrong order:

  __discover(.., bool connect, bool persistent, enum nvme_print_flags flags)

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

Merge pull request #1477 from igaw/fix-empty-hostnqn-file

fabrics: Support connect even when no /etc/nvme/hostnqn file exists

fabrics: Support connect even when no /etc/nvme/hostnqn file exists

The connect call will fail if there is no /etc/nvme/hostnqn file
available. The 1.x version did have a fallback mechanisme in place
when the config file was missing.

Let's add this feature back by calling nvmf_hostnqn_generate() when
there is no /etc/nvme/hostnqn file.

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

plugins/intel: make 'buckets' a json array

The 'buckets' elements in the statistics output are an array, not
a list of (identically named) objects.

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

plugin: Update WDC capabilities command with new commmands

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

documentation: Add doc files for new wdc plugin commands

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

plugin: Add new WDC plugin commands for OCP 2.0
  Add command to display error recovery (id 0xC1) log page
  Add command to display device capabilities (id 0xC4) log page
  Add command to display unsupported requirements (id 0xC5) log page
  Update WDC plugin version to 1.16.4

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

build: Update libnvme wrap

Update to v1.0-rc8 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 #1468 from martin-belanger/dim-cmd

fabrics: Add DIM command

Merge pull request #1470 from hreinecke/config-fix

build: Update libnvme.wrap

build: Update libnvme.wrap

Get "tree: always allocate config file in nvme_read_config()" from upstream.

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

fabrics: Add DIM command

TP8010 defines the DIM command (Discovery Information Management)
as a way to Register-with or Deregister-from a Discovery
Controller. This is used for NVMe-over-Fabrics when connecting to
Central Discovery Controllers (CDC).

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

Merge pull request #1467 from rnaveentec/fix_spell

Fixed spelling for Partition

nvme-rpmb: Fix spelling for 'Partition'

Merge pull request #1455 from jeffreyalien/nvme-2.0

Nvme 2.0 Changes

nvme-print: Add human readable print for nsattr field

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

nvme-print: Update Persistent Event log fields

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

docs: Add Documentation files for get_mi_cmd_support_effects_log

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

nvme: add get_mi_cmd_support_effects_log command

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

Merge pull request #1460 from Arunpandian15/tests_lba

tests: Add test for get lba status command

Merge pull request #1457 from 0xRZ/nvme_copy_fix

nvme-copy: add missing field to the command

Merge pull request #1453 from igaw/pdc-force

fabrics: Introduce force flag to overwrite persistence logic

fabrics: Introduce force flag to overwrite persistence logic

The persistence flag will be overwritten by the built in logic. Allow
the user to overwrite this by providing a force command line option.

While at it make the 'discover_from_config_file' path behave as the
'nvmf_discover' path. The idea here is that whenever a matching
persistent discovery controller is present just use this one. Only
create a new discovery controller when none matching found or
we are asked to do so via --force.

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

fabrics: Free non-matching controller during discovery

Release all resources for the non-matching controller.

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

Merge pull request #1466 from hreinecke/ns-filter-fix

nvme: Fixup namespace filtering yet again

nvme: Fixup namespace filtering yet again

strcmp() is tricky, so rather check for something with a given
length.

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

Merge pull request #1464 from hreinecke/nvme-config

Add 'nvme config' command

fabrics: add 'nvme config' command

Add a command to display and modify the JSON configuration.

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

build: Update libnvme.wrap

Get "fabrics: add 'nvmf_update_config()'" from upstream.

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

Merge pull request #1463 from hreinecke/meson-stringify

fabrics: Correctly stringify discovery.conf and config.json paths

fabrics: Correctly stringify discovery.conf and config.json paths

The C preprocessor will not evaluate tokens in strings.

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

Merge pull request #1462 from igaw/fix-argconfig-parser-opt-flag

argconfig: Do not use default value loading by getopt_long_only

argconfig: Rename CFG_NONE to CFG_FLAG

CFG_NONE is used to for OPT_FLAG. All other OPT_ macros have the
matching CFG_ enum. Let's rename CFG_NONE to CFG_FLAG to match and
make it consistent.

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

plugins: Use type bool for OPT_FLAG

We switched the arg parser over to using the bool type for
OPT_FLAG. Update all OPT_FLAG user to new type.

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

nvme: Use type bool for OPT_FLAG

We switched the arg parser over to using the bool type for
OPT_FLAG. Update all OPT_FLAG user to new type.

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

argconfig: Do not use default value loading by getopt_long_only

getopt_long_only() is able to load the default values directly. Though
the type has to be 'int'. If the target type of argument not 'int'
getopt_long_only will overwrite adjacent memory location.

Reduce the complexity by not using this feature.

Instead use the type bool because libnvme uses bool in 'struct
nvme_fabrics_config'.

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

Merge pull request #1461 from hreinecke/filter-ns

nvme: use filter for 'list-subsys <devname>'

nvme: use filter for 'list-subsys <devname>'

When a device name is specified for 'nvme list-subsys' we should
be printing out only those parts of the tree which relate to the
specified device name, not the entire tree.

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

build: Update libnvme.wrap

Get 'tree: do not return error when filtering out subsystems' from
upstream.

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

build: Add nvme_get_lba_status_test in tests list

updated test list in meson.build with newly added
test nvme_get_lba_status_test

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

tests: Add test for get lba status command

test added for get lba status command

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

build: Add missing comma to file list in tests meson.build

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

Merge pull request #1454 from Arunpandian15/tests_lba

tests: Add tests for lba status log page

nvme-copy: add missing field to the command

Merge pull request #1452 from bjpaupor/lbstm

Add lbstm option to create-ns

Add lbstm option to create-ns

Signed-off-by: Brandon Paupore <brandon.paupore@wdc.com>

build: Update libnvme wrap

Get 'Add lbstm support for create-ns' changes from libnvme.

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

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>