Merge pull request #296 from minwooim/v1.6/return_neg_val_get_nsid

nvme-cli: make return negative value in nvme_get_nsid() when not blkdev

Merge pull request #295 from minwooim/v1.6/update_doc

nvme-cli: update documentations to sync up with subcommands

Merge pull request #294 from minwooim/v1.6/move_print_to_nvme_print

nvme-cli: move print-function to nvme-print

Merge pull request #293 from minwooim/v1.6/fix_leaks_in_commands

nvme-cli: fix leaks in subcommands

Merge pull request #292 from minwooim/v1.6/add_print_cname_effects_log

nvme-cli: add print for command name to effects-log

Merge pull request #291 from minwooim/v1.6/bpid_cdw10_fw_commit

nvme-cli: make it have bpid in cdw10 for fw-commit

Merge pull request #290 from minwooim/v1.6/add_cs_field_to_json_error_log

nvme-cli: add command specific field to json output of error-log

flush: Use device NSID if issuing through block device

Signed-off-by: Keith Busch <keith.busch@intel.com>

NVMe-CLI Fix fw_log Command json output

Currently, the json output for the fw_log command slot number
is zero based while the normal output is one based.  This patch
will change the json output to one based.  It will also only
print the revisions for the valid slots.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
[fixed whitespace formatting]
Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #303 from pborzenkov/intel-smart

nvme-cli: fix typo in Intel SMART json output

nvme-cli: fix typo in Intel SMART json output

Signed-off-by: Pavel Borzenkov <pavel.borzenkov@gmail.com>

nvme-cli: update documentations to sync up with subcommands

Update all documentations to sync up with built-in plugin subcommands.
Not only options supported by subcommands, But few descriptions have not
been updated with implementations.

Sync-up documentation with subcommands implementations.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: make return negative value in nvme_get_nsid() when not blkdev

If caller invokes get_nsid() with character device without checking
whether a given device is block device or not, ENOTBLK is returned from
nvme_get_nsid() which is a positive value, so that caller may treat this
value as a valid nsid.

Make it return a negative vaelue with -ENOTBLK and add error checking to
callers.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: move print-function to nvme-print

Code clean-up by moving print-functions for list and list-subsys
commands to nvme-print.c with no functional changes.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix leaks in subcommands

Fix leaks(a single memory leak, file descriptors leaks) in subcommands
by adding explicit close for opened things.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: add print for command name to effects-log

Commands Supported and Effects log page describes commands supported and
effects of commands. It would be better if command names are printed out
with their opcodes.

So, Add the command names following NVMe 1.3a specification to
effects-log command to both normal mode and json mode.

Example of commands output
========
Admin Command Set
ACS0     [Delete I/O Submission Queue     ] 00000001
ACS1     [Create I/O Submission Queue     ] 00000001
ACS2     [Get Log Page                    ] 00000001
ACS4     [Delete I/O Completion Queue     ] 00000001
ACS5     [Create I/O Completion Queue     ] 00000001
ACS6     [Identify                        ] 00000001
ACS8     [Abort                           ] 00000001
ACS9     [Set Features                    ] 00000001
ACS10    [Get Features                    ] 00000001
ACS12    [Asynchronous Event Request      ] 00000001
ACS16    [Firmware Commit                 ] 00000001
ACS17    [Firmware Image Download         ] 00000001
ACS128   [Format NVM                      ] 00020003
ACS129   [Security Send                   ] 00020003
ACS130   [Security Receive                ] 00010001

NVM Command Set
IOCS0    [Flush                           ] 00010001
IOCS1    [Write                           ] 00000003
IOCS2    [Read                            ] 00000001
IOCS4    [Write Uncorrectable             ] 00000003
IOCS5    [Compare                         ] 00000001
IOCS8    [Write Zeroes                    ] 00000003
IOCS9    [Dataset Management              ] 00010003
=======

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: make it have bpid in cdw10 for fw-commit

_bpid_ was added as a parameter in commit d52c9988 without any being
used.
Make it have inside of cdw10 before submitting a command.

Fixes: d52c9988 ("nvme-cli: update Firmware Commit with boot partition
feature")
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: add command specific field to json output of error-log

Add command specific field to json output of error-log command.
This field was added in commit 662c1618 without considering json output
format.

Fixes: 662c1618("nvme-cli: add print for command specific info field of
error log page")
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #289 from tim-oleksii/master

nvme-cli: Fixed incosistency of error handling in intel plugin

nvme-cli: Fixed incosistency of error handling in intel plugin
Signed-off-by: Oleksii Timofieiev <tim.oleksii@gmail.com>

Use '-' for compounded word telemetry-log subcommand

Just for consistency.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Use NSID all for error log

At least some controllers return error if using 0 for the namespace. This
is essentially reverting to the previous behavior before 3ec5ae78c.

Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: Implement nvme_get_log13

This fixes a portion of 722c71bb:
nvme/vendor: Add get log LSP/LSO fields from 1.3 spec

This fixes up the code so it can use the same interface
log we've had for awhile. People who wish to pass log
specific parameters can use the new API.

Signed-off-by: Scott Bauer <scott.bauer@intel.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #286 from tim-oleksii/master

nvme-cli: wdc: Removed superfluous output in WDC plugin

nvme-cli: handle long commands in help message by adjusting minimal padding
Signed-off-by: Oleksii Timofieiev <tim.oleksii@gmail.com>

nvme-cli: wdc: Removed superfluous output in WDC plugin

Signed-off-by: Oleksii Timofieiev <tim.oleksii@gmail.com>

Merge pull request #283 from tim-oleksii/master

nvme-cli: Add support of unsigned long long to json routines

NVMe-CLI WDC-Plugin Update drive essentials code with review comments.

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

nvme-cli: Add support of unsigned long long to json routines
Signed-off-by: Oleksii Timofieiev <tim.oleksii@gmail.com>

Merge pull request #279 from minwooim/for-1.6

nvme-cli: memblaze: update memblaze plugin

NVMe-CLI WDC-Plugin Add clear-pcie-corr documentation.

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

NVMe-CLI WDC-Plugin Add clear-pcie-corr command

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
[Merged to account for recent API changes]
Signed-off-by: Keith Busch <keith.busch@intel.com>

NVMe-CLI WDC-Plugin Add drive-essentials documentation

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

NVMe-CLI WDC-Plugin Add drive-essentials command

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

Merge branch 'v1.6/update_memblaze' of https://github.com/minwooim/nvme-cli into for-1.6

nvme-cli: memblaze: add error return after parse_and_open()

Add returning error when it fails.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: memblaze: fix memory leak in memblaze-nvme.c

Once _buf_ is allocted by posix_memalign(), all return statement
should check if _buf_ is allocated and, if so, free it.

Make it jump to free: to free _buf_ if allocted.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #276 from minwooim/v1.6/update_wdc

nvme-cli: wdc: update wdc plugin

Merge pull request #275 from minwooim/v1.6/fix_memory_leak

nvme-cli: fix memory leak in nvme.c

Merge pull request #274 from minwooim/v1.6/handle_error_for_malloc

nvme-cli: add error handling for a failure of malloc

Merge pull request #278 from ScottyBauer/implement_telemetry3

nvme-cli: Implement get_telemetry_log from 1.3 spec

Merge pull request #277 from minwooim/v1.6/update_lnvm

nvme-cli: lnvm: Make it return error status when argconfig_parse fails

nvme/vendor: Add get log LSP/LSO fields from 1.3 spec

Signed-off-by: Scott Bauer <scott.bauer@intel.com>
[Fixed param descriptions and short opt]
Signed-off-by: Keith Busch <keith.busch@intel.com>

nvme-cli: Implement get_telemetry_log from 1.3 spec

Signed-off-by: Scott Bauer <scott.bauer@intel.com>

nvme-cli: lnvm: Make it return error status when argconfig_parse fails

The return value of argconfig_parse() should be returned if it fails.
Make lnvm commands return when it fails.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in passthru()

If _metadata_ is allocated with metadata_len in fail case of
posix_memalign() for _data_, it should be freed properly.

Add freeing _metadata_ if _data_ is failed to be allocated.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: wdc: fix mismatch in document for smart-add-log

Fix mistype in document for smart-add-log in _wdc_ plugin.
Human readable output mode is nor supported by this command.

Replace it with normal.

Reported in github issue #270.

Reported-by: Oleksii Timofieiev <tim.oleksii@gmail.com>
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: wdc: add const to descriptions

Add const to descriptions.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in dir_receive()

Fix memory leak for _buf_.

==10014== HEAP SUMMARY:
==10014==     in use at exit: 16 bytes in 1 blocks
==10014==   total heap usage: 52 allocs, 51 frees, 15,638 bytes
allocated
==10014==
==10014== Searching for pointers to 1 not-freed blocks
==10014== Checked 70,656 bytes
==10014==
==10014== 16 bytes in 1 blocks are definitely lost in loss record 1 of 1
==10014==    at 0x4C2FFC6: memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==10014==    by 0x4C300D1: posix_memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==10014==    by 0x402733: dir_receive (nvme.c:3531)
==10014==    by 0x4192A7: handle_plugin (plugin.c:150)
==10014==    by 0x401890: main (nvme.c:3825)
==10014==
==10014== LEAK SUMMARY:
==10014==    definitely lost: 16 bytes in 1 blocks
==10014==    indirectly lost: 0 bytes in 0 blocks
==10014==      possibly lost: 0 bytes in 0 blocks
==10014==    still reachable: 0 bytes in 0 blocks
==10014==         suppressed: 0 bytes in 0 blocks
==10014==
==10014== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in sec_recv()

Fix memory leak for _sec_buf_.

==7670== HEAP SUMMARY:
==7670==     in use at exit: 4,096 bytes in 1 blocks
==7670==   total heap usage: 31 allocs, 30 frees, 8,046 bytes allocated
==7670==
==7670== Searching for pointers to 1 not-freed blocks
==7670== Checked 70,656 bytes
==7670==
==7670== 4,096 bytes in 1 blocks are definitely lost in loss record 1 of
1
==7670==    at 0x4C2FFC6: memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7670==    by 0x4C300D1: posix_memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7670==    by 0x4056F3: sec_recv (nvme.c:3391)
==7670==    by 0x419587: handle_plugin (plugin.c:150)
==7670==    by 0x401890: main (nvme.c:3799)
==7670==
==7670== LEAK SUMMARY:
==7670==    definitely lost: 4,096 bytes in 1 blocks
==7670==    indirectly lost: 0 bytes in 0 blocks
==7670==      possibly lost: 0 bytes in 0 blocks
==7670==    still reachable: 0 bytes in 0 blocks
==7670==         suppressed: 0 bytes in 0 blocks
==7670==
==7670== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in dir_send()

Fix memory leak for _buf_.

==9566== HEAP SUMMARY:
==9566==     in use at exit: 1,024 bytes in 1 blocks
==9566==   total heap usage: 52 allocs, 51 frees, 16,681 bytes allocated
==9566==
==9566== Searching for pointers to 1 not-freed blocks
==9566== Checked 70,656 bytes
==9566==
==9566== 1,024 bytes in 1 blocks are definitely lost in loss record 1 of
1
==9566==    at 0x4C2FFC6: memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9566==    by 0x4C300D1: posix_memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9566==    by 0x402174: dir_send (nvme.c:2535)
==9566==    by 0x419317: handle_plugin (plugin.c:150)
==9566==    by 0x401890: main (nvme.c:3818)
==9566==
==9566== LEAK SUMMARY:
==9566==    definitely lost: 1,024 bytes in 1 blocks
==9566==    indirectly lost: 0 bytes in 0 blocks
==9566==      possibly lost: 0 bytes in 0 blocks
==9566==    still reachable: 0 bytes in 0 blocks
==9566==         suppressed: 0 bytes in 0 blocks
==9566==
==9566== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in sec_send()

Fix memory leak for _sec_buf_.

==9303== HEAP SUMMARY:
==9303==     in use at exit: 0 bytes in 1 blocks
==9303==   total heap usage: 31 allocs, 30 frees, 3,915 bytes allocated
==9303==
==9303== Searching for pointers to 1 not-freed blocks
==9303== Checked 70,664 bytes
==9303==
==9303== 0 bytes in 1 blocks are definitely lost in loss record 1 of 1
==9303==    at 0x4C2FFC6: memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9303==    by 0x4C300D1: posix_memalign (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9303==    by 0x405A4C: sec_send (nvme.c:2415)
==9303==    by 0x419527: handle_plugin (plugin.c:150)
==9303==    by 0x401890: main (nvme.c:3812)
==9303==
==9303== LEAK SUMMARY:
==9303==    definitely lost: 0 bytes in 1 blocks
==9303==    indirectly lost: 0 bytes in 0 blocks
==9303==      possibly lost: 0 bytes in 0 blocks
==9303==    still reachable: 0 bytes in 0 blocks
==9303==         suppressed: 0 bytes in 0 blocks
==9303==
==9303== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in set_feature()

Fix memory leak for _buf_.

==9183== HEAP SUMMARY:
==9183==     in use at exit: 1,024 bytes in 1 blocks
==9183==   total heap usage: 52 allocs, 51 frees, 16,576 bytes allocated
==9183==
==9183== Searching for pointers to 1 not-freed blocks
==9183== Checked 70,688 bytes
==9183==
==9183== 1,024 bytes in 1 blocks are definitely lost in loss record 1 of 1
==9183==    at 0x4C2FFC6: memalign (in
                          /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9183==    by 0x4C300D1: posix_memalign (in
                          /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9183==    by 0x407721: set_feature (nvme.c:2305)
==9183==    by 0x419407: handle_plugin (plugin.c:150)
==9183==    by 0x401890: main (nvme.c:3813)
==9183==
==9183== LEAK SUMMARY:
==9183==    definitely lost: 1,024 bytes in 1 blocks
==9183==    indirectly lost: 0 bytes in 0 blocks
==9183==      possibly lost: 0 bytes in 0 blocks
==9183==    still reachable: 0 bytes in 0 blocks
==9183==         suppressed: 0 bytes in 0 blocks
==9183==
==9183== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in fw_download()

Fix memory leak for _fw_buf_ without any free.

==8786== HEAP SUMMARY:
==8786==     in use at exit: 0 bytes in 1 blocks
==8786==   total heap usage: 31 allocs, 30 frees, 3,810 bytes allocated
==8786==
==8786== Searching for pointers to 1 not-freed blocks
==8786== Checked 70,664 bytes
==8786==
==8786== 0 bytes in 1 blocks are definitely lost in loss record 1 of 1
==8786==    at 0x4C2FFC6: memalign (in
                           /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8786==    by 0x4C300D1: posix_memalign (in
                           /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8786==    by 0x406963: fw_download (nvme.c:1767)
==8786==    by 0x419407: handle_plugin (plugin.c:150)
==8786==    by 0x401890: main (nvme.c:3810)
==8786==
==8786== LEAK SUMMARY:
==8786==    definitely lost: 0 bytes in 1 blocks
==8786==    indirectly lost: 0 bytes in 0 blocks
==8786==      possibly lost: 0 bytes in 0 blocks
==8786==    still reachable: 0 bytes in 0 blocks
==8786==         suppressed: 0 bytes in 0 blocks
==8786==
==8786== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in ns_descs()

Memory has been allocated before parsing command options by
parse_and_open() and validate_output_format().
Make it after the setup of options.

_nsdescs_ changes to (void *) for usability.

_nsdescs_ has been allocated without any free.
Add free statement for _nsdescs_. Please refer below memory leak.

==8533== HEAP SUMMARY:
==8533==     in use at exit: 4,096 bytes in 1 blocks
==8533==   total heap usage: 31 allocs, 30 frees, 7,906 bytes allocated
==8533==
==8533== Searching for pointers to 1 not-freed blocks
==8533== Checked 70,624 bytes
==8533==
==8533== 4,096 bytes in 1 blocks are definitely lost in loss record 1 of 1
==8533==    at 0x4C2FFC6: memalign (in
                           /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8533==    by 0x4C300D1: posix_memalign (in
                           /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8533==    by 0x409D2D: ns_descs (nvme.c:1466)
==8533==    by 0x419567: handle_plugin (plugin.c:150)
==8533==    by 0x401890: main (nvme.c:3803)
==8533==
==8533== LEAK SUMMARY:
==8533==    definitely lost: 4,096 bytes in 1 blocks
==8533==    indirectly lost: 0 bytes in 0 blocks
==8533==      possibly lost: 0 bytes in 0 blocks
==8533==    still reachable: 0 bytes in 0 blocks
==8533==         suppressed: 0 bytes in 0 blocks
==8533==
==8533== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: fix memory leak in list_ctrl()

Fix memory leak of _cntlist_ cause a memory leak below.

==7724== HEAP SUMMARY:
==7724==     in use at exit: 4,096 bytes in 1 blocks
==7724==   total heap usage: 31 allocs, 30 frees, 7,871 bytes allocated
==7724==
==7724== Searching for pointers to 1 not-freed blocks
==7724== Checked 70,624 bytes
==7724==
==7724== 4,096 bytes in 1 blocks are definitely lost in loss record 1 of 1
==7724==    at 0x4C2FFC6: memalign (in
                          /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7724==    by 0x4C300D1: posix_memalign (in
                          /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7724==    by 0x408354: list_ctrl (nvme.c:573)
==7724==    by 0x419587: handle_plugin (plugin.c:150)
==7724==    by 0x401890: main (nvme.c:3799)
==7724==
==7724== LEAK SUMMARY:
==7724==    definitely lost: 4,096 bytes in 1 blocks
==7724==    indirectly lost: 0 bytes in 0 blocks
==7724==      possibly lost: 0 bytes in 0 blocks
==7724==    still reachable: 0 bytes in 0 blocks
==7724==         suppressed: 0 bytes in 0 blocks

==7724== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: add error handling for a failure of malloc

Make it return ENOMEM when malloc of metadat is failed with some error
message.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #268 from minwooim/v1.6/fix_typo_in_get_feature_test

nvme-cli: tests: fix typo in get_feature_test

nvme-cli: tests: fix typo in get_feature_test

Comment and function name typo will directly reflect document which
will be generated by Makefile.

Fix typos like below.
    1. Typo in feature lists in the global comment.
    2. Mandetory -> Mandatory

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #262 from minwooim/v1.6/intel_smart_log_add

nvme-cli: intel: fix smart-log-add documentation

Merge pull request #263 from minwooim/v1.6/intel_add_return_fd

nvme-cli: intel: add check for fd after open to intel commands

Merge pull request #264 from minwooim/v1.6/intel_add_const_to_desc

nvme-cli: intel: add const to descriptions of subcommand option

Merge pull request #266 from minwooim/v1.6/effects_log_update

nvme-cli: add support for options to effect-log subcommand

Merge pull request #267 from minwooim/v1.6/remove_nsid_in_error_log

nvme-cli: tests: remove nsid parameter in a testcase of error-log

Merge pull request #261 from javigon/master

lightnvm: enable to set OP on target creation

nvme-cli: tests: remove nsid parameter in a testcase of error-log

Remove nsid parameter in a testcase of error-log subcommand.

This parameter has been removed by a commit 98ffd4b on the master branch
of linux-nvme/nvme-cli.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: update documentation of effects-log

Update documentation of effects-log subcommand with descriptions of
option. Also add an example of --raw-binary option to the document.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: add support for options to effects-log subcommand

Add support for options below to to effects-log subcommand.
    --output-format=<fmt>, -o <fmt>    normal|json|binary
    --human-readable, -H
    --raw-binary, -b

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: intel: add const to descriptions of subcommand option

Add const to descriptions of subcommand option.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: intel: add check for fd after open to intel commands

Make it return negative error value when file open is failed to intel
commands.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: intel: fix smart-log-add documentation

tim-oleksii has reported that intel smart-log-add help differs from man
page.

Update man, html, and txt documentation to follow output of below
command.
    ./nvme intel smart-log-add

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

lightnvm: enable to set OP on target creation

On target creation, allow to define the over-provision (OP). Since OP
area has a direct impact on write amplification, users can optimize the
OCSSD behavior for their workloads.

Support in the kernel has been upstreamed in 4.16

Signed-off-by: Javier González <javier@cnexlabs.com>

Merge pull request #259 from minwooim/v1.6/intel_smart_log_add_doc_update

nvme-cli: intel: update smart-log-add command documentation

Merge pull request #258 from minwooim/v1.6/lnvm_id_ns

nvme-cli: lnvm: update ns-id subcommand

Merge pull request #257 from minwooim/v1.6/add_bps_field_to_cap

nvme-cli: add BPS bit field to CAP controller register

Merge pull request #255 from minwooim/v1.6/add_sanitize_log_json_print

nvme-cli: add support for json output format of sanitize-log command

Merge pull request #254 from minwooim/v1.6/remove_nsid_in_error_log

nvme-cli: Remove nsid param in error-log

Merge pull request #256 from minwooim/v1.6/update_alias_desc_readme_md

nvme-cli: update subcommand alias description in README.md

nvme-cli: intel: update smart-log-add command documentation

Update smart-log-add command documentation in intel plugin with
output-format parameter description.

In github issue #175, tim-oleksii reported that descriptions of
output-format whose implementation was added in Pull request #178 was
not added in documentation.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: update error-log documentation

Update error-log documentation with removing nsid parameter.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: lnvm: add human-readable print format to ns-id subcommand

Add human-readable print format to _ns-id_ subcommand of lnvm.
-H option had been used nowhere. Make it possible to see in this format.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: lnvm: add check for fd after open operation

Make it return negative error value when file open operation is failed.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: lnvm: remove unnecessary force parameter of ns-id in lnvm

If identify geometry is failed, then nvme-cli is not to print out
information of namespace.
Also no one uses this _force_ flag in this subcommand.
So, remove this unnecessary parameter.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: lnvm: add whitespace between words in a description

Add an whitespace between words in a description for ns-id of lightnvm
subcommand.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: add BPS bit field to CAP controller register

NVMe 1.3 spec introduced BPS(Boot Partition Support) bit field in CAP of
controller register.
Add this bit field to "struct nvme_bar_cap" by changing
"css_nssrs_dstrd" field to "bps_css_nssrs_dstrd".
Also add a print for BPS bit field to print CAP properly when show-regs
command with human-readable.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Update subcommand alias description in README.md

Update description of alias for subcommand.

nvme-cli: add support for json output format of sanitize-log command

Add support for json output format of sanitize-log command.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: Remove unnecessary nsid field in error-log

Error information log page is global to controller.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Merge pull request #251 from minwooim/v1.6/update_print_sanitize

nvme-cli: update print for sanitize-log command with output-format

Merge pull request #250 from minwooim/v1.6/make_sanitize_desc_const

nvme-cli: make sanitize description const

Merge pull request #249 from minwooim/v1.6/add_log_page_field_to_error_log

nvme-cli: add print for command specific info field of error log page

nvme-cli: add print for command specific info field of error log page

Add print line for Command Specific Information[39:32] in Error
Information Log Entry.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: update print for sanitize-log command with output-format

Update print for sanitize-log command with normal and binary
output-format. Also move origin print code to nvme-print.c. with
human-readable option.

Make sanitize-log command call nvme_sanitize_log() so that
it looks more consistent with other XXX-log commands.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

nvme-cli: make sanitize description const

Make description in sanitize() const.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>

Fix verbose decoding for coalescing aggregation time

https://github.com/linux-nvme/nvme-cli/issues/246

Signed-off-by: Keith Busch <keith.busch@intel.com>

Allow hierarchy of similiarly named plugins

For historical reasons, we allow the user to write their command line
executing "vendor" plugin "command" as one of two ways:

  nvme vendor command

And

  nvme vendor-command

This was conflicting if a hierarchy of vendor additions exists, like
"vendor-oem".

So, this patch fixes that by first searching the entire builtin vendor
plugins before searching for matching substrings.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Release v1.5

It's been a while since the previous release, and we've accumulated
enough updates to warrant a new tag. Shortlog below.

There are printing fixes and improvements to output, documentation
updates, enhancements to vendor plugins, updates for the NVMe 1.3
Specification, a new command for listing subsystems, a new way to alias
sub-commands in case old ones need to be deprecated, fabrics fixes,
and resource leak fixes. Thank you to everyone who contributed!

Andrew Maier (2):
      Nitpicked a printing alignment issue with the id-ns vendor specific field
      Updated the human readable printing for the SGLS field in the id-ctrl command to be compliant with NVMe 1.3

David Manouchehri (1):
      Avoid unnecessary abbreviation.

Guan Junxiong (2):
      nvme-cli: add support to get properties for NVMe over Fabric
      nvme-cli: support to set the property for NVMe over Fabric

James Smart (2):
      nvme-cli: add duplicate_connect parameter support
      nvme-cli: add duplicate_connect parameter support

Jeff Lien (2):
      NVMe-CLI : WDC-Plugin Updated Smart Log Command
      NVMe-CLI : WDC-Plugin Documentation for Updated Smart Log Command

Johannes Thumshirn (2):
      fabrics: also set queue_size and nr_io_queues on connect-all connects
      nvme: Introduce new 'list-subsys' command

Keith Busch (28):
      Add AEN decoder helper to get-log
      Regen documentation
      Merge pull request #228 from primeos/nixos
      intel-nvme: Fix compiler errors and warnings
      Merge pull request #229 from steve-gh/master
      Add command effects log page
      Increase default format timeout
      Revert "nvme-cli: add duplicate_connect parameter support"
      Merge pull request #230 from liangxiao1/upstream
      Fix typo in previous commit
      Fix long suffix parse on 32-bit machines
      Merge pull request #232 from Manouchehri/verbose
      nvme/flush: explicitely close file descriptor
      Merge pull request #233 from cleandever/master
      Add 'DEAC' support to write-zeroes command
      Rebuild documentation
      Regenerate documentation
      Update get-feature help text
      Improve code alignment on intel plugin
      Update intel internal log man page.
      nvme-cli/intel: Return if arg parsing returns error
      nvme: add options for write command directives
      Update documentation
      Merge pull request #238 from Eideticom/andrew-dev
      Reorder command list
      Documentation fixes
      Update documentation to use aliases
      Merge pull request #243 from ScottyBauer/fix_mem_leak

Michael Weiss (2):
      Add Nix(OS) install instructions to the README
      Fix the Markdown code for the Alpine install instructions

Minwoo Im (11):
      nvme-cli: make address print format more human-readable
      nvme-cli: fix wrong bitmask in number of rpmb units.
      nvme-cli: make command description const
      nvme-cli: resync nvme.h with the kernel's
      nvme-cli: update show_registers_cc() to use nvme.h
      nvme-cli: replace values of all namespaces with NVME_NSID_ALL
      nvme-cli: add prints for new HMB identify fields of TP 4002
      nvme-cli: add support alias command with Firmware Activate
      nvme-cli: update Firmware Commit with boot partition feature
      nvme-cli: add command specific status of Firmware Commit
      nvme-cli: add prints of boot partition feature to show-regs

Scott Bauer (1):
      nvme-cli: Fix small mem leak.

Steve (2):
      Include sizing data in json output that's already included in the non-json results.
      Cleanup variable definitions

Xiao Liang (1):
      nvme/id-ns: do not try to get namespace id from non-block device.

cleandever (1):
      Fix memory leak

jeffreyalien (1):
      nvme-cli : wdc-plugin Add support for WDC SN100 and SN200 devices.

Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #243 from ScottyBauer/fix_mem_leak

nvme-cli: Fix small mem leak.

nvme-cli: Fix small mem leak.

Internally some people are building with asan which complains
that we're leaking that memory. So we'll fix it for them.

Signed-off-by: Scott Bauer <scott.bauer@intel.com>

nvme-cli: add prints of boot partition feature to show-regs

Add prints of Boot Partition feature.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>