Merge pull request #324 from hreinecke/subsystem-rework

Rework nvme_scan_subsystem()

tree: rework nvme_scan_subsystem()

The nvme subsystem does not have a 'hostnqn' sysfs entry, so we
cannot infer from the nvme subsystem sysfs entry to which host
it relates. And really, the subsystem should already have been
created by the previous call to nvme_scan_ctrl().
So do not call nvme_lookup_subsystem() in nvme_scan_subystem(),
but rather just validate the sysfs subsystem entries and create
any missing subsystems.

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

tree: make subsystem name mandatory in nvme_scan_ctrl()

nvme_scan_ctrl() is iterating over sysfs, so any controller must
have a corresponding subsystem; not finding one is an error.

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

tree: move nvme_init_subsystem() into nvme_lookup_subsystem()

We're always calling nvme_init_subsystem() when nvme_lookup_subsystem()
is called with a non-NULL 'name' parameter. So we might as well move
it into nvme_lookup_subsystem() and simplify the callers.

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

Merge pull request #323 from hreinecke/export-symbol

libnvme: export 'nvme_lookup_ctrl()'

libnvme: export 'nvme_lookup_ctrl()'

For some reason it has been missing from the map file.

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

Merge pull request #322 from hreinecke/update-config

fabrics: add 'nvmf_update_config()'

fabrics: add 'nvmf_update_config()'

Add a function to update the fabrics configuration values of an
existing controller.

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

Merge pull request #320 from hreinecke/json-scan

json: fix endless loop scanning for controllers

json: fix endless loop scanning for controllers

nvme_lookup_ctrl() will always create a new controller if none is
found, so checking for a NULL return value is not correct here.

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

Merge pull request #319 from hreinecke/scan-filter

Fixup scan filter usage

tree: do not return error when filtering out subsystems

The filter is set by the caller, and just modifies the tree.
It does not invalidate the scanning, so we shouldn't return
an error.

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

tree: add debugging messages during scanning

Add some debugging messages during scanning to aid debugging.

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

Merge pull request #318 from hreinecke/display-tree

meson: build display-tree example

meson: build display-tree example

The 'display-tree' example program wasn't build with meson.

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

Merge pull request #307 from bjpaupor/lbstm

Add lbstm support for create-ns

Merge pull request #315 from igaw/fix-host-traddr-free

fabrics: Free old traddr in nvmf_add_ctrl

Merge pull request #308 from Arunpandian15/tests_23_03

test: print test logs for pass case

test: print test logs for pass case

test log print for passed cases and aligning
log output.

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

fabrics: Free old traddr in nvmf_add_ctrl

We need to free the old traddr and not the newly translated traddr.

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

Merge pull request #306 from jeffreyalien/nvme-2.0-2

Nvme 2.0 Changes

Remove nvme_init_id_ns

Function was used only once in nvme-cli for the create-ns command,
instead just handle setting the fields in nvme-cli directly.

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

Add lbstm support for create-ns

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

types: Add support for get log - MI Command Supported

Add API to get NVMe-MI Commands Supported and Effects log page.

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

types: Add new Identify constant

Add IDENTIFY_CNS_CSI_ID_NS_DATA_STRUCTURE/0x1B define

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

types: Update persistent event entry struct added new fields

Add missing ehai and pelpid members.

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

types: Add Host Initiated Data Gen Number to telemetry log struct

Merge pull request #292 from tbzatek/errno-fixes-1

tree: nvme_configure_ctrl(): Set errno to zero on success

Merge pull request #310 from igaw/nvme_chomp

fabrics: Avoid out of bounds string chomping

Merge pull request #314 from glimchb/codecov

ci: add code coverage

ci: add code coverage

From https://mesonbuild.com/howtox.html#producing-a-coverage-report
Upload a report to https://codecov.io/

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

Merge pull request #313 from glimchb/badges

readme: Add github badges

Merge pull request #311 from glimchb/ci

CI: fixing dist packaging error

readme: Add github badges

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

CI: fixing dist packaging error

Fixes #303

python.platlibdir and python.install_env are mutually exclusive

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

Merge pull request #309 from tbzatek/init-subsystem-2

tree: Handle NULL subsysname in nvme_scan_ctrl()

fabrics: Avoid out of bounds string chomping

Update all callsides of nvme_chomp to use the correct max lenght of
the string to avoid out of bounds access.

While at it, nvme_chomp() is used by fabrics.c to sanitize the traddr
and trsvcid. There is no user outside of fabrics and clearly this is a
small helper function. Thus move it to fabrics and rename it to a more
comon name strchomp.

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

tree: Handle NULL subsysname in nvme_scan_ctrl()

As noted couple of lines earlier, the subsysname string might
be NULL. In that case, defer initialization of nvme_subsystem_t
in hope that it would get initialized while scanning subsystems.

Merge pull request #304 from tbzatek/init-subsystem-1

tree: Fix subsystem initialization in nvme_scan_ctrl()

tree: Fix subsystem initialization in nvme_scan_ctrl()

The subsystem instance would normally get properly initialized
in later stages of topology scanning, unless there's a host
mismatch and the instance would stand alone.

Merge pull request #302 from martin-gpy/update_log_level

fabrics: update log level for write failures

fabrics: update log level for write failures

Update the log level to LOG_ERR for write failures in
__nvmf_add_ctrl().

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

Merge pull request #299 from tbzatek/covscan-fixes-1

Misc. static analyzer fixes

Merge pull request #301 from Arunpandian15/doc_20

Streamlining documentation

ioctl: Align arguments indentation with braces

Aligned function arguments indentation with open braces

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

fabrics: Streamlining documentation

Aligning whitespace indentation

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

tree: Fix leaking 'name' in nvme_subsystem_lookup_namespace()

tree: Avoid dereferencing nvme_subsystem_t before its check for NULL

fabrics: Fix leaking ctrl in nvmf_connect_disc_entry()

fabrics: Add missing break in a switch

No functional change, just a static analyzer warning.

doc: Provide SYSCONFDIR without quotes

Unfortunately, we can't use the configuration data object defined in
the top meson.build file as this one is with quotes. Instead, making
the c code ugly, just provide a new configuration data object without
the quotes. This avoid generating strings suchs as

  "/etc"/nvme/hostnqn

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

tree: Clarify NULL return values from nvme_get_attr()

Returned NULL value means either an empty string or an error
reading the sysfs attribute. The only way to distinguish that
is by errno.

tree: nvme_configure_ctrl(): Set errno to zero on success

Merge pull request #297 from igaw/fix-stringify-hostnqn

fabrics: Correctly stringify default hostnqn and hostid paths

Merge pull request #298 from hreinecke/log-page-len

linux: fixup log page offset in nvme_get_log_page()

fabrics: Correctly stringify default hostnqn and hostid paths

Fixes: f356ab0ca74a ("fabrics: Allow to change sysconfdir for hostnqn and hostid file")
Reported-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Daniel Wagner <dwagner@suse.de>

Merge pull request #296 from igaw/set_default_trsvcid

tree: Add support for default trsvcid for all controllers

linux: fixup log page offset in nvme_get_log_page()

When resubmitting the log page with a different length in
nvme_get_log_page() we need to store the original length
of the log page to avoid the length being overwritten.

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

tree: Add support for default trsvcid for all controllers

When the user doesn't provide trsvcid always use the default values
for tcp/rdma transport.

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

Merge pull request #290 from glimchb/rpm

rpm: add makefile rule for rpms

Merge pull request #293 from igaw/fix-sysconfdir

fabrics: Allow to change sysconfdir for hostnqn and hostid file

Merge pull request #294 from hreinecke/disc-ttrsvcid

tree: fixup coredump during nvme discover

tree: fixup coredump during nvme discover

nvme_free_ctrl() expects the 'trsvcid' string to be dynamically
allocated; just calling 'stringify' will cause a coredump on exit.

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

fabrics: Allow to change sysconfdir for hostnqn and hostid file

Instead hard coding the sysconfdir allow the user to overwrite the
location of the hostnqn and hostid file. The default is "/etc" but
when configured with --prefix=DIR the files should be read from
"DIR/nvme/".

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

rpm: add makefile rule for rpms

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

Merge pull request #289 from glimchb/rpm

dist: add missing BuildRequires to spec file

dist: add missing BuildRequires to spec file

According to https://docs.fedoraproject.org/en-US/packaging-guidelines/Meson/

Fixes #288

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

doc: Regenerate all documentation

Prepare for next release candiate.

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

Merge pull request #283 from hreinecke/ana-path

tree: add nvme_ctrl_get_ana_state()

tree: add nvme_ctrl_get_ana_state()

Add a function to return the ANA state of a namespace on a given
controller.

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

tree: link paths to namespaces in nvme_subsystem_scan_namespace()

Depending on the order of scanning nvme_subsystem_scan_namespace()
might be called after nvme_scan_ctrl(), causing the links from
paths to namespaces to be missing.

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

Merge pull request #282 from glimchb/wheel2

CI: add python wheel build in addition to source package

Merge pull request #284 from hreinecke/filter-doc

Document filter functions

Document filter functions

Add documentation for the various filter functions.

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

CI: add python wheel build in addition to source package

When using source distribution,
python package will be compiled during pip install.

Wheels make the end-to-end installation of Python packages faster:
- wheels are typically smaller in size than source distributions
- pip fetches a prebuilt wheel and avoids the intermediate step of building

So why cibuildwheel and manylinux?

See https://github.com/pypa/manylinux and https://github.com/pypa/cibuildwheel

The goal of the manylinux project is to provide a convenient way to distribute binary Python extensions as wheels on Linux

See https://github.com/pypa/cibuildwheel/blob/main/examples/github-deploy.yml

Skipping musllinux and auditwheel repair.
Skipping MacOS and Windows for now.

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

Merge pull request #281 from glimchb/opensslci

CI: add test for subproject fallbacks compilation

CI: add test for subproject fallbacks compilation

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

Merge pull request #280 from glimchb/test

test: adding dependency on pynvme

test: adding dependency on pynvme

On few systems (i.e. Fedora) when running meson test,
the python library build is not invoked.

For example, on Ubuntu, this works fine.
That's why CI dodn't catch it.

So adding explicit dependency in tests.

Fixes #279

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

Merge pull request #278 from sc108-lee/list_ns

ns list bug fix (wrong cns value)

Merge pull request #277 from glimchb/openssl

openssl: update to patch-3 to fix installation issue

Merge pull request #275 from Arunpandian15/types_05

types: Add missing doc field description

ioctl.h: ns list bug fix (wrong cns value)

original cns is not using csi field
naming align
NVME_IDENTIFY_CNS_CSS_ALLOCATED_NS_LIST -> NVME_IDENTIFY_CNS_CSI_ALLOCATED_NS_LIST

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

types.h: Key Value Command Set Identifier added (NVME_CSI_KV)

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

openssl: update to patch-3 to fix installation issue

Fixes #268

Patch taken from https://mesonbuild.com/Wrapdb-projects.html
Wrap file used https://wrapdb.mesonbuild.com/v2/openssl_1.1.1l-3/openssl.wrap
Fixed by mesonbuild/wrapdb#322

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

types: Add missing doc field description

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

Merge pull request #274 from sc108-lee/status_code_type

types: fix status code type bug (wrong masking)

types: fix status code type bug (wrong masking)

the error comes from fc274c57c6b3063d35dd76e36b777adc4da29e0d
using NVME_GET for masked value #148

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

doc: Regenerate all documentation

Prepare for next release candiate.

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

linux: Add log.h back

We still need log.h for the various log level definitions.

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

linux: Include private.h

Replace log.h with private.h because we added log.h only for
nvme_msg() but this define has been moved to the private.h header
file.

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

Merge pull request #236 from birkelund/meson-fixes

meson: small fixes

Merge pull request #273 from igaw/fix-libssl-dep

build: Depend on libssl when using the fallback dependency

build: Depend on libssl when using the fallback dependency

libnvme doesn't depend on the openssl application. Update the fallback
dependency to libssl.

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

Merge pull request #272 from Arunpandian15/sc

util: Add status code values in order to return errno

util: Add status code values in order to return errno

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

Merge pull request #267 from igaw/traddr-case-insesitive

tree: Ignore traddr case in nvme_lookup_ctrl()

Merge pull request #269 from Arunpandian15/util_sc_02

util: Congregate status code values to return EINVAL

Merge pull request #270 from igaw/fix-pel

ioctl: Set lsp to action in nvme_get_log_persistent_event

ioctl: Set lsp to action in nvme_get_log_persistent_event

Retrieving PEL fails because the lsp is set to NVME_LOG_LSP_NONE
instead of action:

  # nvme -persistent-event-log  -a 1 /dev/nvme0
  NVMe status: Command Sequence Error: The command was aborted due to a protocol violation in a multi- command sequence(0xc)

Signed-off-by: Wen Xiong <wenxiong@linux.ibm.com>
[dwagner: updated commit message]
Signed-off-by: Daniel Wagner <dwagner@suse.de>