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

Nvme 2.0 Changes

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>

util: Congregate status code values to return EINVAL

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

tree: Ignore traddr case in nvme_lookup_ctrl()

Some FC discovery controllers return traddr strings with upper case
hexadecimal. There was is no requirement in the NVME-FC specification
that it be upper or lower case. Switch to strcasecmp for case
insentive traddr comparison.

Based on nvme-cli change 1264c6323937 ("nvme-cli: Make connect-all
matching be case insensitive")

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

Merge pull request #264 from igaw/fix-system-uuid

fabrics: Do not swap bytes for system uuid

fabrics: Do not swap bytes for system uuid

The kernel is taking care of swapping the system uuid bytes since
f1d8e614d74b ("drivers/firmware/dmi_scan.c: check dmi version when get
system uuid"). Hence we should not swap them again. Instead just read
/sys/class/dmi/id/product_uuid and store the bytes directly into the
system_uuid buffer.

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

Merge pull request #266 from glimchb/actionbranch

CI: rename branch main to master

CI: rename branch main to master

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

Merge pull request #257 from glimchb/setuptools

 pyproject.toml instead of setup.py for new packaging

Workflow: temporary disable publish to PyPI, starting with TestPyPI

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

Merge pull request #259 from Arunpandian15/fix_sct

types: status code type value update as per NVMe specification

Merge pull request #262 from Arunpandian15/util_01

util: Add status code values in geneic status to errono conversion

util: Add status code values in generic status to errno conversion

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

Merge pull request #261 from martin-belanger/move-ccan-out-of-public-headers

Move ccan definitions out of public headers.

Move ccan definitions out of public headers.

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

Add workflow to publish to PyPI and TestPyPI

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

pyproject.toml instead of setup.py for new packaging

This allows to build with meson and package for Pypi

Build like this:
    python -m build

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

types: status code type value update as per NVMe specification

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

Merge pull request #258 from Arunpandian15/types_doc_28

types: Add missing doc for ZNS, NVMe OF, NVMe command set

types: Add missing doc for ZNS, NVMe OF, NVMe command set

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

Merge pull request #256 from Arunpandian15/doc_types_25

types: Add doc section for enum fields

types: Add doc section for enum fields

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

Merge pull request #255 from Arunpandian15/types_doc_24

types: Add doc field for admin opcodes, FIDs, LIDs, Identify commands

types: Add doc field for admin opcodes, FIDs, LIDs, Identify commands

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