Keith Busch [Mon, 8 Apr 2019 20:51:59 +0000 (14:51 -0600)]
Release version 1.8
There are some important fixes pending a release tag, so now seems as
good as any to tag the release.
Thank you again to everyone who's contribute to this round. Shortlog
below.
Breno Leitao (2):
Fix compilation with GCC-8
Fix CFLAGS parameter
Chaitanya Kulkarni (3):
nvme-cli: fix compilation with uint64_t cast
nvme-cli: minor cleanup for submit_io()
nvme-cli: align all the macros
Dong Ho (11):
wdc: Change device capability checking
wdc: Add additional device for vs-internal-log
wdc: Add device unit info log extraction
wdc: Add get drive status command
wdc: Add clear assert dump status command
wdc: Add drive resize command
wdc: Update device capabilities for clear pcie corr
wdc: Update file parameter checking
wdc: Add command to extract NAND statistics
wdc: Add data area extraction for DUI command
wdc: Additional device capabilities check
James Smart (1):
nvme-cli: fix fc trtype string
Jeffrey Lien (3):
NVMe-CLI WDC Plugin - add support for additional SN840 device ID.
WDC plugin Change vs-internal-log Command to Save Partial E6 Log Data.
WDC Plugins vs-internal-log command unsupported on SN340.
Keith Busch (8):
generate new documentation
Regen docs
Fix print format specifiers
Regen documentation
Add new docs to repository
Generate new documentation
Fix format block size usage
Regen documentation
Kenneth Heitke (4):
Add reporting of Virtual Management support to id-ctrl
Fix possible NULL pointer dereferences
Fix possible memory leak on get-internal-log
Print timestamp as a date and time string
Martin George (2):
nvme-cli: add netapp ontapdevices command
nvme-cli: add netapp ontapdevices man page
Maxim (1):
Adding a helper to format command to enable the user to specify desired block size and have nvme-cli determine the correct LBAF and add format helper to nvme-create-ns command
Minwoo Im (15):
fabrics: Return negative errno when asprintf() fails
fabrics: Return negative errno to check remove_ctrl() is failed
doc: Update nvme-discover by describing options
doc: Update nvme-connect by describing options
doc: Update nvme-connect-all by describing options
doc: Update nvme-disconnect by describing options
doc: Add missing commands to main document entry
nvme: Support raw_binary and json format for show-regs
doc: Update show-regs document with output-format
ioctl: Free memory if get_property for fabrics failed
nvme: Use ARRAY_SIZE() macro in common.h
nvme: Unify min(), max() macro as a common one
nvme: Simplify description for virt-mgmt
nvme: Simplify virt-mgmt options to lowercase
tests: Update print to print() for python3
Roy Shterman (1):
nvme-print: Fix evaluation of ctratt on pretty print
Sagi Grimberg (1):
nvme: print I/O error to stderr
Signed-off-by: Keith Busch <keith.busch@intel.com>
Minwoo Im [Sun, 7 Apr 2019 11:24:55 +0000 (20:24 +0900)]
tests: Update print to print() for python3
The following issue has been reported that README in "tests" seems not
correct because it will not be able to be invoked in case python3
because print() has to be taken instead of print which had been used for
python2.
Minwoo Im [Sun, 7 Apr 2019 10:32:53 +0000 (19:32 +0900)]
nvme: Simplify virt-mgmt options to lowercase
All the other subcommand options are supported with lowercase, but
virt-mgmt subcommand has long argument name started with uppercase.
This can be simplified to a lowercase with being the same with the names
on spec document.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Sun, 7 Apr 2019 10:30:25 +0000 (19:30 +0900)]
nvme: Simplify description for virt-mgmt
The options of virt-mgmt subcommand can be simplified for a good
readability. This patch not only adds necessary whitespace between
sentences, but remove unnecessary empty line and words which might cause
too long line to describe options.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Sat, 6 Apr 2019 21:45:16 +0000 (06:45 +0900)]
ioctl: Free memory if get_property for fabrics failed
The given *pbar is allocated inside this function without freeing it
will be freed from the caller in case of show-regs subcommand for NVMe-oF
controller.
If Get Property is failed because it has PCIe transport, then it needs
to be freed here instead of caller.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Sat, 6 Apr 2019 21:01:17 +0000 (06:01 +0900)]
nvme: Support raw_binary and json format for show-regs
Somebody might want to compare entire controller registers in a raw
binary format to check controller status or some of configurations. It
also can be needed to print out JSON format for easily parsing.
This patch introduces JSON and RAW_BINARY print format with a general
--output-format argument. This will have "normal" value by default, and
also support "json" and "binary".
We don't care about whether the given "fabrics" argument in
json_ctrl_registers() because all the reserved registers and fields are
to be returned as 0x0 as specified in NVMe-oF "Property Definitions"
section.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Wed, 3 Apr 2019 16:30:33 +0000 (01:30 +0900)]
doc: Add missing commands to main document entry
cmds-main.txt has the entire command sets provided by nvme-cli. This
patch added the following commands which has already had corresponding
documents already.
1. nvme-connect-all
2. nvme-disconnect
2. nvme-disconnect-all
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Wed, 3 Apr 2019 16:27:17 +0000 (01:27 +0900)]
doc: Update nvme-connect-all by describing options
We have had newly added command options for Connect to all NVMe-oF subsystems.
This patch updates descriptions of command options.
+ This patch also fixes an trivial mis-aligned paragraph issue which can
be in this commit to avoid any confusions in document update which might
show too many diffs.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Wed, 3 Apr 2019 16:24:07 +0000 (01:24 +0900)]
doc: Update nvme-discover by describing options
We have had newly added command options for Discover controller. This
patch updates descriptions of command options.
+ This patch also fixes an trivial mis-aligned paragraph issue which can
be in this commit to avoid any confusions in document update which might
show too many diffs.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Wed, 3 Apr 2019 15:02:05 +0000 (00:02 +0900)]
fabrics: Return negative errno to check remove_ctrl() is failed
remove_ctrl() might fail due to a reason such as memory allocation
failure. Once it fails, discovery should not proceed, but return the
errorno to the caller immediately.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Minwoo Im [Wed, 3 Apr 2019 14:53:33 +0000 (23:53 +0900)]
fabrics: Return negative errno when asprintf() fails
asprintf() will return -1 when it fails to allocate memory for the
string given with setting errno to ENOMEM. For the uniformity, we can
return errno with negative value.
Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
Chaitanya Kulkarni [Wed, 6 Feb 2019 22:59:56 +0000 (14:59 -0800)]
nvme-cli: align all the macros
Due to the addition of a lot of code in the WDC Plugin over a period
of time it has non-aligned macro definitions. This is purely cleanup
patch which makes all the macros aligned nicely which makes code
readable.
James Smart [Mon, 4 Feb 2019 21:39:54 +0000 (13:39 -0800)]
nvme-cli: fix fc trtype string
Commit 55b4ec12ffe4
("fabrics: use trtype_str when parsing a discovery log entry")
changed to use the trtypes string names in a generic fashion for
the "transport=?" string passed in the connect string to the kernel.
For FC, this meant the hardcoding of "transport=fc" was removed and
the generic code would now create "transport=fibre-channel", which
doesn't match any transport name anymore once the string is attempted
to be matched in the kernel.
Change the string used by for trtype for FC to use "fc" to restore
operation.
Maxim [Wed, 23 Jan 2019 23:12:36 +0000 (15:12 -0800)]
Adding a helper to format command to enable the user to specify desired block size and have nvme-cli determine the correct LBAF and add format helper to nvme-create-ns command
Breno Leitao [Mon, 21 Jan 2019 12:03:24 +0000 (07:03 -0500)]
Fix CFLAGS parameter
Currently nvme-cli 1.7 is not compiling on Debian because the Debian helper
(compilation toolkit) does pass a CFLAGS variable, thus, avoiding the
initial definition (CFLAGS ?= -O2 -g -Wall -Werror -I.)
The problem is that -I should not be removed, otherwise the code does not
compile, with the following bug:
cc -Wdate-time -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -D__CHECK_ENDIAN__ -g -O2 -fdebug-prefix-map=/home/breno/nvme/nvme-cli-1.7=. -fstack-protector-strong -Wformat -Werror=format-security -std=gnu99 -DLIBUUID -DNVME_VERSION='"1.7"' -o plugins/intel/intel-nvme.o -c plugins/intel/intel-nvme.c
plugins/intel/intel-nvme.c:10:18: fatal error: nvme.h: No such file or directory
compilation terminated.
This patch just moves the -I parameter to part of the CFLAGS that is not
replaced by dh's CFLAGS.
Breno Leitao [Mon, 21 Jan 2019 11:31:15 +0000 (06:31 -0500)]
Fix compilation with GCC-8
Printf() is being called to print an unsigned long int using the long long
identifier. This causes the following error with GCC-8:
nvme.c:3056:5: error: format ‘%llu’ expects argument of type ‘long long unsigned int’, but argument 3 has type ‘__u64 {aka long unsigned int}’ [-Werror=format=]
"Invalid value for block size (%llu), must be a power of two\n",
^
nvme.c:3091:6: error: format ‘%llu’ expects argument of type ‘long long unsigned int’, but argument 3 has type ‘__u64 {aka long unsigned int}’ [-Werror=format=]
"LBAF corresponding to block size %llu (LBAF %u) not found\n",
Adjusting the printf arguments to print the proper format.
Dong Ho [Fri, 11 Jan 2019 00:37:06 +0000 (00:37 +0000)]
wdc: Change device capability checking
Changes the way device VID and type are checked for compatibility
purposes. Other minor changes for command names, update the respective
documentation file names, and remove deprecated C1 log functionality.
Keith Busch [Fri, 11 Jan 2019 00:25:03 +0000 (17:25 -0700)]
nvme: Release 1.7
Thank you to all the contributions in this release. Lots of new options,
spec features, output updates, and bug fixes. Shortlog below.
Austin J Eberle (1):
nvme-cli: prevent resv action field overflow
Benreese0 (1):
nvme: Adding a block size selector to format
Chaitanya Kulkarni (23):
nvme-cli: remove unnecessary assignments
nvme-cli add ns write protect feature support
nvme-cli: add minimal ana-log page support
nvme-cli: add ana-log documentation
nvme-cli: make show_registers_cap() static
nvme-cli: wdc-plugin mark array static declaration
nvme-cli: fix sparse warnings for declaration
nvme-cli: don't use unsigned for fd in lnvm-nvme.c
nvme-cli: remove always true condition for u8
nvme-cli: fix dead-code nvmf log page discovery
nvme-cli: code cleanup for passthru() handlers
nvme-cli: fix scaandir() return value
nvme-cli: use uniform error message for open
nvme-cli: fix indentation
nvme-cli: make wdc plug-in functions static
nvme-cli: mark functions static
nvme-cli: make malloc error handling uniform
nvme-cli: fix indentation for dir_[send/receive]
nvme-cli: make read error handling uniform
nvme-cli: fix sanitize status endianness
nvme-cli: fix ana-log endianness
nvme-cli: fix array len to hold string of size 4
nvme-cli: fix endianness for the structure field
Dan Bond (1):
README: update Arch Linux install instructions
David Milburn (1):
nvme-cli: report subsystem-reset not supported by controller
Dong Ho (6):
nvme-cli: tests: fix fw_log test coding style
nvme-cli: tests: fix id_ctrl test coding style
nvme-cli: tests: fix id_ns test coding style
nvme-cli: tests: Add get_id_ctrl function
nvme-cli wdc plugin: Add get pfail dump command.
nvme-cli wdc plugin: Add documentation for pfail dump.
Eyal Ben David (8):
nvme-print.c : fix ns-descs JSON output
nvme-get-property : Add documentation
nvme-models.c - search for pci.ids
nvme list : fix nvme list output if identify failed on device
attach-ns : fix segfault for ctrl list wrong input
nvme.spec.in : fix bash completion path, require
Refactor plugins in a file hierarchy
connect-all: special treatment to EALREADY afetr write to /dev/nvme-fabrics
Hannes Reinecke (13):
nvme-ana-log: fixup compiler warning in show_ana_log()
nvme-vendor: fixup c99 declaration in huawei plugin
nvme-discover: sanitize options
nvme-discover: Retry discovery log if the generation counter changes
nvme-ioctl: retrieve log pages in 4k chunks
nvme-discover: Re-check generation counter after log page transfer
seagate-nvme: fixup coding style
nvme-cli: Add 'rae' argument to nvme_get_log()
nvme-cli: kill nvme_passthru_admin()
nvme: fixup ANA group descriptor offset
nvme: introduce get_nvme_ctrl_attr()
nvme: print out controller state for 'list-subsys'
nvme-list-subsys: Add device name argument and print out ANA state
Harry Mallon (3):
micron-nvme: Replace direct use of ioctl
Fix typos in docs
Fix typos in tests and utils
Jeff Lien (1):
nvme-cli: Fix the displayed metadata length value on Big Endian systems.
Jeffrey Lien (5):
Add PCI device id checking for the WDC plugin commands.
Fix nvme test device selection to use controller value defined in the config.json file. Before this fix, the value was hard coded to nvme0.
NVMe-CLI WDC Plugin - Simplify the device id checking routines.
nvme-cli: Add wdc-plugin command vs-internal-log and update wdc-plug cap-diag command with transfer-size parameter.
nvme-cli: Add help text for wdc-plugin command vs-internal-log. Update help text for wdc-plugin command cap-diag.
Johannes Thumshirn (2):
fabrics: write ctrl_loss_tmo to fabrics device on connect
fabrics: write keep-alive-timeout to fabrics device on connect
Keith Busch (15):
Fix print format specifier for 64-bit
Fix nvme id namespace struct
Regen documentation
Update gitignore for generated test files
Update documentation
Re-gen documentation for new files
wdc: Fix strncpy warning
wdc: Use const string for suffix
nvme: Handle supported capabilities result for get features
list-subsys: Preserve line breaks and spaces
Regen documentation
nvme-cli: Don't print result on passthru error
Update documentation
nvme: Regenerate documentation
nvme: Remove duplicate error message
M. Warner Losh (2):
Initialize line.
Use standard interfaces to get timezone offsets.
Minwoo Im (2):
nvme-cli: fix typo in json_print_list_items
nvme-cli: readme: fix typo in Alpine distro support
Narasimhan V (1):
Use tnvmcap register for creating namespace
Peter Dave Hello (1):
Enable code block syntax highlight in README.md
Ralph Corderoy (1):
Enlarge local char[] so snprintf(3) can't truncate path.
Rami Rosen (2):
nvme-cli: fix a typo
nvme-cli: fix a typo
Sagi Grimberg (13):
fabrics: make some arguments integers
fabrics: don't fail empty discovery log page
nvme: commonize subsystems info in a helper
fabrics: add disconnect-all command
fabrics: support fabrics sq flow control disable
fabrics: use trtype_str when parsing a discovery log entry
nvme: Add TCP transport
fabrics: add tcp port tsas decoding
fabrics: add transport header and data digest
nvme-ioctl: clear rae for discovery log pages
fabrics: pass in number of write queues
fabrics: pass in number of polling queues
fabrics: allow nr_io/write/poll_queues and queue size to discovery
Schremmer, Steven (1):
nvme-cli: netapp plugin handle empty strings
Stefan Wiehler (1):
Makefile: fix bash completion install path
Sushma Kalakota (1):
Added support for virtualization-management command
Tal Shorer (1):
don't execute a command if --dry-run is supplied
Warner Losh (7):
Use sys/ioctl.h to pull in definitions.
Remove unnecessary asm/byteorder.h
Remove unnecessary linux/types.h include.
Include libgen.h for basename prototype.
Include sys/stat.h
nvme-cli: remove endian.h
nvme-cli: fix ioctl argument
Warner.Losh (1):
We don't need to include linux/types.h here.
Yi Zhang (1):
nvme-cli: print "\n" after nvme list-subsys -o json
beomsooo.kim (1):
In json output, if the commands add zero length string, it occur allocation fail & abort (cannot display empty value)
chrono (1):
Added gentoo to README.md
freezer5 (2):
Fix printing of large ANAGRPID values
Fix printing of large ANAGRPID values
jaredeh (1):
timeout option added to ns_create/delete (#440)
lijie (2):
nvme-cli:fix a typo in SELF-TEST log
nvme-cli: fix a typo
puneet kandhari (1):
Remove trailing "\n" so that the output is valid json
Signed-off-by: Keith Busch <keith.busch@intel.com>
Benreese0 [Tue, 8 Jan 2019 22:59:19 +0000 (14:59 -0800)]
nvme: Adding a block size selector to format
Provide format option to allow the user to specify desired block size
and have nvme-cli determine the correct LBAF.
Signed-off-by: Benjamin Reese <benjamin.reese@intel.com>
[changelog, minor formatting, shift lbaf instead of bs, reference id_ns
only if we read it] Signed-off-by: Keith Busch <keith.busch@intel.com>
Sagi Grimberg [Thu, 13 Dec 2018 06:07:49 +0000 (22:07 -0800)]
fabrics: pass in number of write queues
nr_write_queues specifies the number of additional queues that
will be connected. These queues will host write I/O (host to target
payload) while nr_io_queues will host read I/O (target to host payload).
Sagi Grimberg [Thu, 24 May 2018 11:45:30 +0000 (14:45 +0300)]
fabrics: support fabrics sq flow control disable
If the discovery log entry indicates that the subsystem supports
disabling sq flow control, we ask the host to connect and disable sq
flow control (omit sq_head pointer updates). Also add a connect option
to explicitly ask for disable_sqflow.
If we failed to add_ctrl in connect-all command, we retry without it
to be backward compatible with a driver that cannot accept the new
flag.
projects / users / hch / nvme-cli.git / log