nvme-print-stdout: Refactor stdout print code to use print_ops

Use the newly introduced print_ops infrastructure to for the stdout
printing code.

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

nvme-print-json: Refactor json print code to use nvme_print_ops

Use the newly introduced print_ops infrastructure to for the json
printing code.

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

print: Introduce print ops structure

Introduce callbacks for printing the nvme data structures. With this we
are able to split the two printing types (stdout and json) and just use
the ops the user selects ('-o json' for example).

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

fabrics: Move printing functions to the print files

Move all printing functions to the corresponding print files.

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

nvme-print: Split add_bitmap implementation

Instead a common function for printing the bitmaps just split it
into the corresponding implementation.

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

nvme-print: Unify flag passing into functions

Use the 'enum nvme_print_flags' type consistently everywhere.
Furthermore, move the flag as last argument where needed.

While at it also do properly indent the functions arguments.

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

nvme-print: Export generic to string functions

These function do not print anything, thus can be shared between the
different output formats.

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

nvme-print: Double function definition

The json function is already defined in nvme-print-json.h header file.

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

plugins/sfx: sfx-nvme add expand-cap & dump evtlog cmd

Signed-off-by: yjin <yuekun.jin@scaleflux.com>

util/types: Add generic spinner

Signed-off-by: Wei Hou <wei.hou@scaleflux.com>

util/types: Move convert_ts to util/types

The time stamp converting function can be reused by various plugins.
Thus move it to the util section.

Signed-off-by: Wei Hou <wei.hou@scaleflux.com>
Signed-off-by: Daniel Wagner <dwagner@suse.de>

plugin: Change version and usage names for linux kernel check patch warnings

Also change usage function as a static function since not used externally.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

fabrics: prevent 'nvme connect' to well-known discovery NQN

Using 'nvme connect' to connect to the well-known discovery NQN
could result in a few problems with the current implementation:

1) It does not explicitly disconnect from the discovery subsystem
after usage, thereby leading to a spurious discovery controller
on the host.

2) Multiple 'nvme connect' invocations to the well-known discovery
NQN would result in duplicate discovery controllers on the host.

All this is already handled in 'nvme discover'. So it would be
better to avoid using 'nvme connect' to connect to the well-known
discovery NQN in the first place, and use 'nvme discover' instead
for the same.

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

util: Fix types.c linux kernel check patch warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Fix crc32.c linux kernel check patch errors and warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Fix base64.c linux kernel check patch errors and warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Fix argconfig.c linux kernel check patch error and warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/toshiba: Fix linux kernel check patch errors and warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugin: Fix files linux kernel check patch errors and warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/ocp: Fix remaining files linux kernel check patch warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/ocp: Fix ocp-nvme.c linux kernel check patch warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/ocp: Fix ocp-nvme.c linux kernel check patch errors

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/solidigm: Added NLog support to Telemetry parsing.
Telemetry Reason Identifier fields changed to start with lower case.

Signed-off-by: leonardo.da.cunha <leonardo.da.cunha@solidigm.com>

nbft: Add include util/types.h

util_uuid_to_string() is define in util/types.h, thus include it.

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

nbft: Add stdlib.h include

calloc() and free() are defined in stdlib.h, thus include the
definition.

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

nbft: Make json-c output depending on config flag

In case json-c is not available avoid compilation errors.

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

nvme-print-json: Update function sigs for !json-c builds

The json_output_* function got updated, but the !json-c case was not
updated accordingly. Update them now.

Fixes: e853da696ea4 ("nvme-print-json: Add json output result function")
Fixes: 6e3e1cc8d5b2 ("nvme: Add nvme_show_perror to use instead of perror for json output")
Fixes: 2b0bc4d824d0 ("nvme-printf: Add show error function for nvme list command json output")
Signed-off-by: Daniel Wagner <dwagner@suse.de>

build: Do not use json-c fallback for muon minimal build

Configure the muon minimal build without using the json-c
fallback. This ensures that the case where json-c is not
available is also compile tested.

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

ocp: Add eol plp failure mode json outputs

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme-print: Add nvme show result function

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme-print-json: Add json output result function

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme-print: Fix nvme_show_error() to call va_end() for json output format

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Fix nvme format block-size option handling

nlbaf returned by id-ns is a 0 based value, so the previous handling of
the loop skipped checking the last lbaf.

Signed-off-by: Mateusz Kozlowski <mateusz.kozlowski@solidigm.com>

Revert "nvme: Masks SSTAT in sanize-log output"

The NVME_SANITIZE_SSTAT_STATUS_MASK is used to mask the
3 least significant bits (representing the status of the most recent
sanitize operation) of the SSTAT field.

The SSTAT field is 16 bits wide and contains other information too, such
as the Global Data Erased bit and the number of completed passes.

Revert this commit so nvme-cli will print the entire SSTAT field

This reverts commit 7092ff55c5d9017231a68fa4dbb9a37b42d61d61.

fabrics: Fix NVMF_ARGS macro indentation error

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

Documentation: Add document for OCP Device capability(LID: 0xC4) log page

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

completions: Add bash completion for OCP Device capability(LID: 0xC4) log page

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

plugins/ocp: Add Device capability(LID: 0xC4) log page

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

udev: fix warning reported by udevadm verify

Fix the following warning reported by udevadm verify:

nvmf-autoconnect/udev-rules/70-nvmf-autoconnect.rules.in:18 Whitespace after comma is expected.
nvmf-autoconnect/udev-rules/70-nvmf-autoconnect.rules.in: udev rules check failed

Signed-off-by: Dmitry V. Levin <ldv@strace.io>

fabrics: fix conditions in nvmf-autoconnect.service

The unit should be started if either config.json or discovery.conf
exists. The current syntax would start it if either one does not
exist. Fix it.

Fixes: eec7634 ("fabrics: Trigger auto connect if config.json exists")
Signed-off-by: Martin Wilck <mwilck@suse.com>

fabrics: Change NVMF_OPTS as NVMF_ARGS to fix linux kernel check patch error

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Split passthru commands read output

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Fix io-passthru command read metadata output

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

build: Install missing s390 library

libjson-c-dev:s390x depends on the libgcc-s1 library.

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

plugin/ocp: Added implementation for 0xC1 log page

Reviewed-by: Karthik Balan <karthik.b82@samsung.com>
Reviewed-by: Arunpandian J <arun.j@samsung.com>
Signed-off-by: Jayakanthan Rajendran <jaya.ganthan@samsung.com>

util: Fix suffix_si_parse to parse no decimal point but suffix value correctly

For example create-ns command --ncap-si paramter value 800G parsed to 0 as invalid value.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add zsh option values handling for sanitize command action

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add ocp sub commands zsh completion

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add ocp commands zsh completion

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add ocp commands bash completion

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvmf-autoconnect: restart service to avoid dropping AEN

If another discovery log page change AEN is received while the
autoconnect service is already running for a previous AEN,
the autoconnect service is not being relaunched.
For example, if some of the entries in the log page are not reachable,
attempting to connect to them may take a long time before timing out,
so AENs received while waiting for the connections have no effect.
The AEN is effectively dropped: we ignore the new discovery log entries
and worse, the controller won't send more AENs because it's waiting
for the log page to be fetched in response to the previous AEN.

Use systemctl restart instead of systemctl start
to ensure nvme connect-all is run in response to each AEN.
If a previous instance of the service is already running,
it will be interrupted and the new log page will be fetched.

Fixes: 391d3b12248a ("nvme-cli: nvmf auto-connect scripts")
Signed-off-by: Caleb Sander <csander@purestorage.com>

solidigm: Reduce memory usage in supported log pages feature

solidigm: Changes to supported log pages feature requested by review

- Retrieve supported logs for all known UUIDs
- Update 'normal' output to table format

solidigm: Add supported log pages log feature to Solidigm plugin

plugins/ocp: Added ocp unsupported requirements log page

Reviewed-by: Karthik Balan <karthik.b82@samsung.com>
Reviewed-by: Arunpandian J <arun.j@samsung.com>
Signed-off-by: Vigneshwaran Saravanan <s.vignesh@samsung.com>

completions: Add bash option values handling for sanitize command action

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

doc: Correct nvme-show-hostnqn command NAME description

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add missing zsh commands descriptions

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/wdc: Extend SN530 support.

Extend SN530 SSD support for alternate PCI device IDs.

Fix incorrect article 'an' as 'a' to use for descriptions and message

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Set effects-log command csi option help message string

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: bash: Delete unsupported id-iocs command options

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: bash: Add id-domain command

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Fix nvm-id-ns-lba-format command zsh definition

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

completions: Add json command option

It was introduced by the change 6aab9e49e but not added for command completion.
  Note: The changes included some completion script files indentation fixes.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Fix get-feature command sel parameter check to allow value 8

The parameter was changed to use value 8 also by the change 30b89e5ef.
  Note: The value is used only for the command functionality to limit feature
        output changed from default but not passed to the driver.

Fixes: 63dcb68c6 ("nvme: Fix parameter limit range")
Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete unnecessary argconfig definitions OPT_VAL and CFG_VAL

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete argconfig value type to add option value as va lists instead

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Split to set argconfig option value

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete argconfig value length to calcurate instead

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Change sanitize command action option to use argconfig parse value

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Add argconfig parse value function

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete unused argconfig subopts type

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete unused argconfig file types

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

util: Delete unused argconfig help function

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme-wrap: do_admin_args_op should not clear args timeout

if set with NVME_DEFAULT_IOCTL_TIMEOUT, timeout parameter always ignored

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

nvme: Fix linux kernel check patch warnings

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Fix linux kernel check patch errors

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

plugins/ocp: Add telemetry-log command to OCP plugin

plugins/ocp: Add OCP set latency monitor feature command

plugins/ocp: Fix endianness in Latency Monitor Log output

OCP 2.0 spec changed the endianness for some of the fields in the C3 LM Log. Fixed this when outputting the values.

plugins/ocp: Add missing fields in Latency Monitor Log

Changes include:
- Added Debug Log Measured Latency, Debug Log Latency Time Stamp, Debug Log Pointer, Debug Counter Trigger Source, Debug Log Stamp Units, Log Page Version, and Log Page GUID, as required by OCP 2.0 spec.
- Reorganized some of the output so that it (mostly) matches the order in which they show up in the spec.

plugins/ocp: Fix typos in Latency Monitor Log

Changes include:
- Fixed the naming of some log page C3 constants that were incorrectly labelled as C0.
- Fixed a typo in the JSON output: 'lantency' -> 'latency'

plugins/ocp: Reorganize OCP plugin source code

Shifted the location of some functions around so they're grouped together better. Also added separators in the comments so it's easy to distinguish which block of functions pertain to what command.

tests: Add tests for controller reset

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

plugin/solidigm: Added vs-internal-log command.

nvme: Add nvme_show_perror to use instead of perror for json output

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme-print-json: Use vasprintf instead of vsnprintf to allocate string

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: fix rnlpt to_string() values.

"Reservation Notification Log Page Type" values do not start
from 0x1 but from 0x0.

Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>

build: Bump libnvme wrap to fix io_mgmt apis

TP4146 Flexible Data Placement 2022.11.30 Ratified Command Dword 10
Bits[31:16] : Management Operation Specific (MOS)
Bits[07:00] : Management Operation (MO)
https://github.com/linux-nvme/libnvme/pull/626

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

nvme: fix io_mgmt args parse error

Since mos defined __u16, it should be parsed with OPT_SHRT
if mos given right after mo, it cause parse error
mo always set as 0

Signed-off-by: Steven Seungcheol Lee <sc108.lee@samsung.com>
Co-authored-by: Minsik Jeon <hmi.jeon@samsung.com>
Reported-by: Youngjin Jung <yj4369.jung@samsung.com>

fdp: set-events feature is saveable

TP4146 Flexible Data Placement 2022.11.30 Ratified
5.27.1.TBD1 Flexible Data Placement Events (Feature Identifier 1Eh)
mentioned as below
This Feature shall be saveable (refer to section 4.2).

Signed-off-by: Steven Seungcheol Lee <sc108.lee@samsung.com>
Reported-by: Youngjin Jung <yj4369.jung@samsung.com>

fabrics: Return errno value as negative

Changes for linux kernel check patch warnings remained.
Change to use `//` instead for proper SPDX comment style.
And also change symbolic permissions `S_IRUSR | S_IWUSR' as '0600'.
  Note: Still remaining a check patch error.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

fabrics: Fix linux kernel check patch errors and warnings

Note: Still remaining some errors warnings.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nbft: add the nbft show plugin

Display contents of the ACPI NBFT files.

Usage: nvme nbft show <device> [OPTIONS]

Options:
  [  --output-format=<FMT>, -o <FMT> ]  --- Output format: normal|json
  [  --subsystem, -s ]                  --- show NBFT subsystems
  [  --hfi, -H ]                        --- show NBFT HFIs
  [  --discovery, -d ]                  --- show NBFT discovery controllers
  [  --nbft-path=<STR> ]                --- user-defined path for NBFT tables

Signed-off-by: Stuart Hayes <stuart_hayes@dell.com>
Signed-off-by: Martin Wilck <mwilck@suse.com>
Signed-off-by: John Meneghini <jmeneghi@redhat.com>

nbft: added NBFT v1.0 table support

Added support for parsing the contents of the NBFT table (per NVMe-oF
boot specification v1.0) with the connect-all and discover commands.

nvme discover/connect-all --nbft ignore /etc/nvme config and use NBFT tables
nvme discover/connect-all --no-nbft ignore NBFT tables and use /etc/nvme config
nvme discover/connect-all --nbft-path=<STR> user-defined path for NBFT tables

Signed-off-by: Stuart Hayes <stuart_hayes@dell.com>
Signed-off-by: Martin Belanger <martin.belanger@dell.com>
Use the new nbft public API.

Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>
Signed-off-by: Martin Wilck <mwilck@suse.com>
Signed-off-by: John Meneghini <jmeneghi@redhat.com>

nbft: make lookup_ctrl function public

To prepare for the addition of nbft functionality fixup the fabrics
declarations.

Signed-off-by: John Meneghini <jmeneghi@redhat.com>

nvme: Fix indentation errors around nvme show error API

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Check validate output format err value instead of flags

Since the nvme print flags enum value not set to the negative value.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>

nvme: Use nvme show error API to outptu stderr messages for json format

Multiple lines std error messages are remained yet since unsupported by the API.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>