Add memblaze get/set feature support

Signed-off-by: Wenwei Tao <wenwei.tao@memblaze.com>

Add memblaze vendor specific SMART Log support

Signed-off-by: Wenwei Tao <wenwei.tao@memblaze.com>

Merge pull request #100 from samiWaheed/master

Display NVMe command status in both string and hex format (write_zeroes & write_uncor)

Display NVMe command status in both string and hex format for write_uncorr and write_zeroes commands

Merge pull request #99 from samiWaheed/master

Enhanced error reporting for invalid commands and/or extensions.

Merge pull request #98 from safl/master

plugin-lnvm: Fixes to argument parsing.

Enhanced error reporting for invalid commands and/or extensions.  List reason why command line failed before displaying the usage message

plugin-lnvm: Added check for mandatory arg in lnvm_factory_init.

plugin-lnvm: Fixed argument parsing in lnvm_init.

The option 'mediamgr-name' had "no_argument" causing parsing failure,
changed to "required_argument".

Added check that option 'device-name' is provided since it is mandatory.

plugin-lnvm: Fixed argument parsing in lnvm_create_tgt.

The options lun-begin and lun-end had "no_argument" causing parsing failure,
changed to "required_argument".

The options lun-begin and lun-end parsed to invalid destination (cfg.tgt_type),
changed to cfg.lun_begin and cfg.lun_end respectfully.

Release nvme-cli v0.8

Lots of updates in this release!

 * Code re-organized to support new extensions. While these are called
   "plugins", they currently have to be built into the executable. Support
   for dynamically loadable plugins is under consideration.

 * LightNVM is added as a built-in plugin.

 * Intel vendor specific commands and structure decodings are supported
   in a new intel plugin.

 * Support for NVMe-over-Fabrics

 * New output format "json" for a subset of commands that decode returned
   structures.

 * Lots of various documentation and minor code fixes.

Seems stable, so tagging a release.

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

Merge pull request #94 from samiWaheed/master

Added "-s" option to the set-feature command to save the specified feature

Changed save variable from bool to int
Added "--save/-s" option to man page and documentation
Modified set-feature documentation (Removed invalid command line options, removed references to get-feature, and corrected examples to use set-feature instead of get-feature

Added "-s" option to the set-feature command to save the specified feature

Merge pull request #93 from samiWaheed/master

Minor fixes to printed output of set/get feature

In set_feature:  print value instead of status
In nvme_feature_show_fields:  Correct printed output of Arbitration Burst

Merge pull request #92 from samiWaheed/master

Enhanced both reset and subsystem-reset to return an error message

Enhanced both reset and subsystem-reset to return an error message on older kernels with no ioctl support
Added "zeroes based" to the block count documentation for write-uncorr and write-zeroes

Update help on IO command parameters

The block count has always been 0's based, but you'd never know that
without reading the code.

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

doc: mention 'list' requires udev

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

fabrics: compile when libudev not installed

libudev is optional to use this repository. Packages can require it if
desired, but others downloading the repo should still be able to use
most of the functionality this tool provides without it.

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

Fix documentation

Documentation fixes:

  Added the intel specific man pages
  Fixed the execlp man page name
  Fixed the help text in some of the intel commands.

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

Add intel vendor unique log support

This adds retrieval for Intel vendor specific logs.

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

Add some developer documentation to the README

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

Use simpler plugin definition

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

Make building plug-ins easier

This adds a level to the code generation to handle defining and
registering plug-ins.

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

Add Intel IO statistics log page

Defined as per public P3xxx data sheet.

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

Add Intel temperature statistics

Defined as per public P3xxx datasheet.

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

Add intel marketing log page

Added as defined by public Intel datasheet on P3xxx series.

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

Execute plugin when name is combined with command

Let a user specify plug-in sub commands in the same string. For example,
this patch will have the following two mean the same thing:

  # nvme intel id-ctrl
  # nvme intel-id-ctrl

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

Make lnvm sub-commands an extension

We want to keep the nvme top level commands strictly to spec. Anything
outside the spec can go in an extension.

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

Add intel vendor specific id-ctrl decoding

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

Add vendor specific callback to id-ctrl print

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

And intel vendor specific extensions

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

Make help and version generic commands

Plugins don't need to implement these.

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

Move program execution to plugin handler

The command list definition is moved to a header file that generates the
structures and prototypes when compiled. This makes it easy to define
new extensions that are not part of the NVMe standard.

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

Execute sub-commands out of plugin structure

This encasulates sub-commands into a structure. This is the beginning
to support extensions and plugins.

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

Add json output format

Adds new control option for json ouput support. The new option has
parameters for json, binary and normal output options.

The older control flag for raw binary still exists and can be used to
override the output format.

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

Seperate kernel uapi from user structures

This pulls in the latest kernel's exported uapi. Including it here
so that it can be used when the kernel headers are either outdated or
not installed.

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

Use pattern rule for compiling objects

All the object files had the same pattern, though some of the targets
were missing the correct dependency list. This fixes them all with a
pattern rule.

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

Use int type for CFG_NONE

GNU long option requires flag be an int pointer. We were potentially
corrupting other parameters when using the 8-byte type.

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

Move parsing to top directory

We've diverged significantly from the argconfig program that nvme-cli
inherited, so no point in trying to keep it separate. Move it to the
top-level directory to make build rules easier.

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

print friendly name for trtype and adrfam

Without this patch:
=====Discovery Log Entry 0======
trtype:  1
adrfam:  1

With this patch:
=====Discovery Log Entry 0======
trtype:  ipv4
adrfam:  rdma

Signed-off-by: Ming Lin <ming.l@samsung.com>

implement the connect-all command

Connects to all controllers pointed to by the discovery service.
Includes following referrals.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Ming Lin <ming.l@samsung.com>
Tested-by: Ming Lin <ming.l@samsung.com>

Implement disconnect command

Example usage:

Disconnect by NQN,
nvme disconnect -n nqn.testiqn

or by device name,
nvme disconnect -d nvme0

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>
Tested-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Jay Freyensee <james.p.freyensee@intel.com>

Implement connect command

Example usage:

root@host: nvme connect -t rdma -a 192.168.2.2 -p 1023 -n nqn.testiqn

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>
Tested-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Jay Freyensee <james.p.freyensee@intel.com>

Add option to save discovery raw log to a file

root@host:~# nvme discover -t rdma -a 192.168.2.2 -p 1023 -r disc.raw
Discovery log is saved to disc.raw

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>
Tested-by: Sagi Grimberg <sagi@grimberg.me>

Implement discover command

Example output:

root@host:~# nvme discover -t rdma -a 192.168.2.2 -p 1023
Discovery Log Number of Records 1, Generation counter 3
=====Discovery Log Entry 0======
trtype:  1
adrfam:  1
nqntype: 2
treq:    0
portid:  1
trsvcid: 1023

subnqn:  nqn.testiqn
traddr:  192.168.2.2

rdma_prtype: 0
rdma_qptype: 0
rdma_cms:    0
rdma_pkey: 0x0000

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>
Tested-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Jay Freyensee <james.p.freyensee@intel.com>

Adds parser library

This is imported from the Linux kernel 4.5.

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-and-tested-by: Sagi Grimberg <sagi@grimberg.me>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>

Adds nvme_discovery_log()

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-and-tested-by: Sagi Grimberg <sagi@grimberg.me>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>

Add macro helpers to common.h

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-and-tested-by: Sagi Grimberg <sagi@grimberg.me>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>

Add NVMeoF definition to nvme.h

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-and-tested-by: Sagi Grimberg <sagi@grimberg.me>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>

Support Get Log Page extended Number of Dwords

This supports 32 bits(rather than 12 bits) extended Number of Dwords
in nvme_get_log().

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-and-tested-by: Sagi Grimberg <sagi@grimberg.me>
Tested-by: Armen Baloyan <armenx.baloyan@intel.com>

Makefile: introduce OBJS variable

Instead of listing all the objects twice.

Signed-off-by: Christoph Hellwig <hch@lst.de>

Merge pull request #91 from ColinIanKing/master

 Fix leak of mbuffer on error exit path

 Fix leak of mbuffer on error exit path

building with clang scan-build picked up a memory leak:
nvme.c:2176:3: warning: Potential leak of memory pointed to by 'mbuffer'
   fprintf(stderr, "failed to read data buffer from input file\n");
   ^~~~~~~

instead of returning, jump to free_and_return which performs the
correct cleanup.  Also fix the indentation of the return at the end of
the function.

Signed-off-by: Colin Ian King <colin.king@canonical.com>

Genarate lnvm documentation

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

Merge pull request #90 from ColinIanKing/master

minor fix and code clean up

Remove extraneous and unused mode parameter from show_nvme_id_ctrl_power

mode is not used, so we may as well remove it

Signed-off-by: Colin Ian King <colin.king@canonical.com>

Merge pull request #87 from MatiasBjorling/for-keith

Add LightNVM adminstration and diagnose support

Fix null pointer dereference on strlen()

If string is NULL, then the proceeding strlen() on the string
will lead to a NULL pointer dereference error (segmentation fault).
Fix this by swapping the order of the comparisons, first check for
a NULL pointer; if that's not true then do the strlen check.

Signed-off-by: Colin Ian King <colin.king@canonical.com>

Merge pull request #89 from ColinIanKing/master

Fix segmentation fault using nvme show-regs with no device name

Fix segmentation fault using nvme show-regs with no device name

Add check_arg_dev() to check if a device name has been provided
and use this in show_registers to avoid the following segfault:

Segmentation fault (core dumped)

Signed-off-by: Colin Ian King <colin.king@canonical.com>

Add documentation for LightNVMcommands

Adds detailed documentation on each of these commands:

 - lnvm-list
 - lnvm-info
 - lnvm-id-ns
 - lnvm-init
 - lnvm-create
 - lnvm-remove
 - lnvm-factory
 - lnvm-diag-bbtbl
 - lnvm-diag-set-bbtbl

Add support for updating the bad block table

Introduce lnvm-diag-set-bbtbl. It takes the channel, lun, plane, block
and the value to update and sends it to the device. Afterwards, the
status can be retrieved with lnvm-diag-get-bbtbl.

Add LightNVM adminstration and diagnose support

Add support for identify geometry, get bad block interface and
administration interface for LightNVM. This patch adds the following
commands:

 - (lnvm-id-ns) Identify geometry of a LightNVM device.
 - (lnvm-diag-bbtbl) Retrieve bad block table from LightNVM device.
 - (lnvm-info) Get information about modules loaded, and available
               devices.
 - (lnvm-init) Initialize LightNVM device with a media manager.
 - (lnvm-create) Create targets on top of registered devices.
 - (lnvm-remove) Remove a registered target.
 - (lnvm-factory) Factory initialize a LightNVM device.

Signed-off-by: Matias Bjørling <m@bjorling.me>

Merge pull request #76 from maksim-teqniksoft/add-completion-to-packages

Install bash autocompletion by default

Merge pull request #88 from morbidrsa/for-keith

bash completion: Remove shebang line

bash completion: Remove shebang line

Remove shebang line from bash completion file as rpmlint on openSUSE/SLE
complains about a script ment to be sourced not executed containing a shebang
line.

[   38s] nvme-cli.x86_64: W: sourced-script-with-shebang /etc/bash_completion.d/nvme /usr/bin/env
[   38s] This text file contains a shebang, but is meant to be sourced, not executed.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>

Release v0.7

Just a few minor updates and documentation fixes. There is also a new
option for passthrough commands that can be used to help detect data
over/under run.

Tagging a stable release so we can consider features that bring in other
transports and third party extensions.

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

Align and clear user buffers

Avoid the bounce buffers a kernel may allocate by using aligned memory,
and clear the memory in case the user's input doesn't fill the entire
buffer.

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

Align passthrough buffers

Some versions of the kernel will create bounce buffers instead of using
directly if the user buffer is not aligned.

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

Fix create-ns man page size option

It's -s, not -n.

Issue: https://github.com/linux-nvme/nvme-cli/issues/86
Signed-off-by: Keith Busch <keith.busch@intel.com>

Pre-fill user buffers on passthrough commands

Reported that user data may be shorter than the buffer length, and need
to pad the data with a fixed byte value.

Also requested that we provied a way to prefill data in buffers so we
can know if a device is corrupting memory.

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

Use correct power scale bits

The active and idle power scales occupy the upper 2 bits of their byte
fields. Shift the scales to use these bits when decoding their values.

Issue: https://github.com/linux-nvme/nvme-cli/issues/85
Signed-off-by: Keith Busch <keith.busch@intel.com>

Decode all firmware activate reset codes

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

Add all error codes from 1.2 spec

This adds enums and prints for all error codes. Since the names are
getting longer, all the prints have white space code shift to align
the same.

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

Merge pull request #79 from minggr/master

Rename nvme_log() to nvme_get_log()

Merge pull request #82 from hrasiq/ppc64-fixes

fix data_size and metadata_size types

fix data_size and metadata_size types

The data_size and metadata_size fields in the config struct are declared as
__u32, but should actually be __u64. The current types cause issues in the
argument parsing functions depending on machine architecture and endianness. In
ppc64, using __u32 causes "data size not provided" errors due to the way writes
are being done by argconfig_parse() at src/argconfig.c. Changing the types to
__u64 should fix this behaviour and is in accordance to the CFG_LONG_SUFFIX
flag.

Signed-off-by: Heitor Ricardo Alves de Siqueira <halves@linux.vnet.ibm.com>

Release 0.6

Fixes a couple missing namespace parameteters to newer sub-commands,
and various printing updates.

Merge pull request #80 from samiWaheed/master

Master

Fix nsid print in show_error_log

nsid in error log page could be set to 0xffffffff and should be printed
as unsigned instead of int

Rename nvme_log() to nvme_get_log()

This kills off the old nvme_get_log() and rename
nvme_log() to nvme_get_log().

Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>

Use nsid parameter for security commands

Controllers optionally may use nsid.

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

Merge pull request #75 from asciiprod/debian-fixes

sync with actual debian package

Fix namespace attachment

The nsid was dropped from the command.

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

Code reuse: Namespace identification

Combining common code into single function.

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

Use enum instead of hard-coded values

Define constants to make clear what a device is doing.

Noticing that one identify mode was never used, so adding an option.

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

Namespace attach controller list endian fix

Convert for host endian.

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

Code reuse: parsing options and arguments

A common code pattern is to parse the options, then open the first
non-option argument as the device parameter.

This patch combines that pattern in a reusable single function. The
savings is not significant, but every little bit helps.

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

Code reuse: character device check

Moving character device sanity check to common function to reduce
duplication.

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

Merge pull request #77 from seehearfeel/master

Fix the bug of Model Number and display the info of Serial Number

Fix the bug of Model Number and display the info of Serial Number

OK, I see, thanks for your suggestion.

I have modified the related code of Serial Number, Model Number and Firmware Revision,
other fields are not be changed because the length can not be properly figured out by
the print format specifier, (int)sizeof(field) is too big and (int)strlen(field) is random.
In addition, fix a minor spelling mistake "Namepace" to "Namespace".

Please review, thanks!

Fix the bug of Model Number and display the info of Serial Number

According to the NVMe Specification, there is an error when display Model Number.
In addition, currently it does not display the info of Serial Number, so add it.

Install bash autocompletion by default

sync with actual debian package
drop all patches as they have all been fixed upstream

Release v0.5

The v0.4 breakage should all be resolved now. Tagging release v0.5,
and back to stability.

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

Add reset and subsystem reset commands

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

Print format consistency

Use same format for both get and set features when displaying results.

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