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

Note: Still the remaining warning messages are not fixed as same with nvme.c.

WARNING: quoted string split across lines
+       const char *log_specific = "1 - controller shall capture Data representing the internal "
+               "state of the controller at the time the command is processed. "

WARNING: quoted string split across lines
+               "state of the controller at the time the command is processed. "
+               "0 - controller shall not update the Telemetry Host Initiated Data.";

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

plugins/scaleflux: Fix sfx-nvme.c linux kernel check patch errors and warnings

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

plugins/netapp: Fix netapp-nvme.c linux kernel check patch errors and warnings

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

plugins/nbft: Fix nbft-plugin.c linux kernel check patch warnings

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

plugins/micron: Fix micron-nvme.c linux kernel check patch errors and warnings

Note: Still the remaining warning messages are not fixed as same with nvme.c.

WARNING: quoted string split across lines
+               "This performs a selective firmware download, which allows the user to "
+               "select which firmware binary to update for 9200 devices. This requires "

WARNING: quoted string split across lines
+               "select which firmware binary to update for 9200 devices. This requires "
+               "a power cycle once the update completes. The options available are:\n\n"

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

plugins/micron: Fix micron-nvme.c space indentation to tab indentation

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

plugins/memblaze: Fix memblaze-nvme.c linux kernel check patch errors and warnings

Note: Still the remaining warning messages are not fixed as same with nvme.c.

WARNING: quoted string split across lines
+               "This performs a selective firmware download, which allows the user to "
+               "select which firmware binary to update for 9200 devices. This requires a power cycle once the "

WARNING: quoted string split across lines
+               "select which firmware binary to update for 9200 devices. This requires a power cycle once the "
+               "update completes. The options available are:\n\n"

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

plugins/memblaze: Fix memblaze-utils.h space indentation to tab indentation

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

plugins/memblaze: Fix memblaze-nvme.c space indentation to tab indentation

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

plugins/intel: Fix intel-nvme.c linux kernel check patch errors and warnings

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

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

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

plugins/inspur: Fix inspur-nvme.c space indentation to tab indentation

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

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

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

plugins/huawei: Fix huawei-nvme.c linux kernel check patch errors and warnings

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

plugins/fdb: Fix fdb.c linux kernel check patch warnings

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

plugins/dera: Fix dera-nvme.c linux kernel check patch errors and warnings

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

plugins/amzn: Fix amzn-nvme.c linux kernel check patch error and warnings

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

plugins/wdc: Fix wdc-utils.c linux kernel check patch errors and warnings

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

plugins/wdc: Refactor wdc-nvme.c for linux kernel check patch errors and warnings

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

util: Move falltrhough feature check to meson.build

Instead cluttering the header file, let's add it to meson.build.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Signed-off-by: Daniel Wagner <dwagner@suse.de>

nvme: Use correct metadata buffer size

The submit_io command is using the metadata buffer size provided by the
user. But this size argument might be updated and the correct length
for the mbuffer is mbuffer_size. Thus use this value and not the
initial length.

Fixes: 18de3a6d61a7 ("Convert to libnvme")
Signed-off-by: Daniel Wagner <dwagner@suse.de>

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

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

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

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>