nvme-cli: smart-log: Fix memory leak for JSON.

This patch fixes the following memory leak when smart-log command
executed with JSON option:-
==7725== HEAP SUMMARY:
==7725==     in use at exit: 1,661 bytes in 53 blocks
==7725==   total heap usage: 99 allocs, 46 frees, 6,559 bytes allocated
==7725==
==7725== LEAK SUMMARY:
==7725==    definitely lost: 24 bytes in 1 blocks
==7725==    indirectly lost: 1,637 bytes in 52 blocks
==7725==      possibly lost: 0 bytes in 0 blocks
==7725==    still reachable: 0 bytes in 0 blocks
==7725==         suppressed: 0 bytes in 0 blocks

Fix:-
==8322==
==8322== HEAP SUMMARY:
==8322==     in use at exit: 0 bytes in 0 blocks
==8322==   total heap usage: 99 allocs, 99 frees, 6,559 bytes allocated
==8322==

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

nvme-cli: resv-report: Fix memory leak.

 This patch fixes the following memory leak for resv-report command:-
 ==7705==
 ==7705== HEAP SUMMARY:
 ==7705==     in use at exit: 4,096 bytes in 1 blocks
 ==7705==   total heap usage: 31 allocs, 30 frees, 7,941 bytes allocated
 ==7705==
 ==7705== LEAK SUMMARY:
 ==7705==    definitely lost: 4,096 bytes in 1 blocks
 ==7705==    indirectly lost: 0 bytes in 0 blocks
 ==7705==      possibly lost: 0 bytes in 0 blocks
 ==7705==    still reachable: 0 bytes in 0 blocks
 ==7705==         suppressed: 0 bytes in 0 blocks

 Fix:-
 ==8253==
 ==8253== HEAP SUMMARY:
 ==8253==     in use at exit: 0 bytes in 0 blocks
 ==8253==   total heap usage: 31 allocs, 31 frees, 7,941 bytes allocated
 ==8253==

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

nvme-cli: error-log: Fix memory leak for JSON.

This patch fixes the following memory leak when error-log command
executed with JSON option:-
==7690==
==7690== HEAP SUMMARY:
==7690==     in use at exit: 49,779 bytes in 1,708 blocks
==7690==   total heap usage: 2,242 allocs, 534 frees, 87,392 bytes allocated
==7690==
==7690== LEAK SUMMARY:
==7690==    definitely lost: 24 bytes in 1 blocks
==7690==    indirectly lost: 49,755 bytes in 1,707 blocks
==7690==      possibly lost: 0 bytes in 0 blocks
==7690==    still reachable: 0 bytes in 0 blocks
==7690==         suppressed: 0 bytes in 0 blocks

Fix:-
==8015==
==8015== HEAP SUMMARY:
==8015==     in use at exit: 0 bytes in 0 blocks
==8015==   total heap usage: 2,242 allocs, 2,242 frees, 87,392 bytes allocated
==8015==

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

nvme-cli: id-ns: Fix memory leak for JSON.

This patch fixes the following memory leak when id-ns command
executed with JSON option:-
==7642==
==7642== HEAP SUMMARY:
==7642==     in use at exit: 3,297 bytes in 120 blocks
==7642==   total heap usage: 182 allocs, 62 frees, 9,204 bytes allocated
==7642==
==7642== LEAK SUMMARY:
==7642==    definitely lost: 24 bytes in 1 blocks
==7642==    indirectly lost: 3,273 bytes in 119 blocks
==7642==      possibly lost: 0 bytes in 0 blocks
==7642==    still reachable: 0 bytes in 0 blocks
==7642==         suppressed: 0 bytes in 0 blocks
==7642== Rerun with --leak-check=full to see details of leaked memory
==7642==

Fix:-
==7949==
==7949== HEAP SUMMARY:
==7949==     in use at exit: 0 bytes in 0 blocks
==7949==   total heap usage: 182 allocs, 182 frees, 9,204 bytes allocated
==7949==

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

nvme-cli: id-ctrl: Fix memory leak for JSON.

This patch fixes the following memory leak when id-ctrl command
executed with JSON option:-
==7680==
==7680== HEAP SUMMARY:
==7680==     in use at exit: 9,796 bytes in 338 blocks
==7680==   total heap usage: 474 allocs, 136 frees, 24,729 bytes allocated
==7680==
==7680== LEAK SUMMARY:
==7680==    definitely lost: 24 bytes in 1 blocks
==7680==    indirectly lost: 9,772 bytes in 337 blocks
==7680==      possibly lost: 0 bytes in 0 blocks
==7680==    still reachable: 0 bytes in 0 blocks
==7680==         suppressed: 0 bytes in 0 blocks

Fix:-
==7971==
==7971== HEAP SUMMARY:
==7971==     in use at exit: 0 bytes in 0 blocks
==7971==   total heap usage: 474 allocs, 474 frees, 24,729 bytes allocated
==7971==

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

nvme-cli: fabrics: Fix to pass "host_traddr" to the kernel driver.

This fix adds the  "host_traddr" to the contructed string that is
passed to the kernel.

Signed-off-by: Darren Trapp <darren.trapp@cavium.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>

Merge pull request #158 from MatiasBjorling/lightnvm_default_target_parameters

lightnvm: default to -1 on lun begin and end

lightnvm: default to -1 on lun begin and end

The default behavior when initializing a target without lun end and
begin is to use only the first lun of the OCSSD. Going forward, the
user shall either explicit specific the begin and end, or if not defined
for kernel 4.11 and newer, the full SSD is initialized.

Signed-off-by: Matias Bjørling <matias@cnexlabs.com>

Merge pull request #157 from leitao/pr

Moving the bash completion files to the standard

Moving the bash completion files to the standard

Currently it is not recommend to put the bash completion files inside
/etc, but on the new /usr/share/completion, as stated in :

https://lintian.debian.org/tags/package-installs-into-obsolete-dir.html
https://bugs.debian.org/776954

Signed-off-by: Breno Leitao <breno.leitao@gmail.com>

Merge pull request #156 from leitao/master

Small changes detected during packaging version 1.1

Remove pyc files generated by 'make test'

When target 'test' is called, it generate some pyc files, that is not
removed by the target 'clean'.

This cause problem with the deb package, since the 'clean' target make
the source files equal to the original one.

Signed-off-by: Breno Leitao <breno.leitao@gmail.com>

Remove debian/ directory

Current debian/ directory is not used anymore. All the debian control
files are kept in the packages at Debian/Ubuntu/etc

This directory is, in fact, causing trouble when upgrading nvme-cli in
Debian.

Signed-off-by: Breno Leitao <breno.leitao@gmail.com>

Regen documentation

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

Documentation: fix location of device reset

Fix the location of the device reset sysfs path.

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

Merge pull request #155 from amluto/sel3

get-features: Don't try to read the data buffer for sel == 3

get-features: Don't try to read the data buffer for sel == 3

When sel == 3, the controller returns capabilities in the completion
entry and does not produce a feature value blob.  Stop allocating
space for the blob and displaying it -- the result is garbage.

Signed-off-by: Andy Lutomirski <luto@kernel.org>

Fix read/write man pages

Removing the second "block-count" parameter.

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

Regenerate Documentation

New man page added for gen-hostnqn.

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

Documentation: Add entry for gen-hostnqn

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>

nvme.spec/debian: Auto generate host nqn as part of install

The installation will generate a hostnqn and store it in
/etc/nvme/hostnqn file (in case it doesn't exist).
This file will be removed upon uninstallation (purge on for debian).

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvme-cli: Add nvme hostnqn generation option

Add option to generate a NVMe qualified name of a given host
(in the form of: nqn.2014-08.org.nvmexpress:NVMf:uuid:<some_uuid>).
This hostnqn will be used for fabrics discovery and connect functions.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

Merge pull request #153 from ncopa/fix-build-with-musl

Include endian.h for byte/host order functions

Include endian.h for byte/host order functions

Fix the following error when building with musl libc:

nvme.h:136:2: error: implicit declaration of function 'le16toh'
[-Werror=implicit-function-declaration]
  le16toh((__force __u16)(x))
  ^

Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>

Release v1.1

This fixes multiple compiler warnings for 32-bit and other environments
from v1.0, adds more documentation to fabrics, removes the libudev
dependency entirely from this project, adds a framework for unit testing
devices, and fixes usage for user defined install paths.

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

Merge pull request #149 from lsgunth/master

Fix bash completion script

Fix compiler warning, asprintf usage

Reported-by: Yachun Liu <D201577676@hust.edu.cn>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Fix bash completion script

The bash completion script has never worked correctly for me.
After reviewing the code, it didn't seem to make too much sense to me
either. Thus, I've rewrote parts of it so that it works. I've also done
a _quick_ audit of all the main commands to make sure they
have the correct arguments available.

fabrics: show number of controllers disconnected

Multiple controllers may have the same nqn, so we iterate multiple
times and don't want to return the status of just the last one to try to
disconnect. This patch has the status show the number of controllers that
were disconnected based on that nqn as that should be more informative.

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

nvme-cli: fabrics: remove libudev dependency

I get a lot of complaints about portability of the nvme-cli regarding the
libudev dependency. This patch removes the dependency from the fabrics
disconnect command when using the target nqn method. Tested on nvme
loop target.

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

Merge pull request #148 from vapier/master

Documentation: allow parent PREFIX to override

Documentation: allow parent PREFIX to override

The current Documentation/Makefile always sets PREFIX to /usr/local.
If the parent Makefile sets it to something else, then it still gets
clobbered.  Change the Makefile to look like the parent and allow the
cascading logic to work.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Merge pull request #147 from vapier/master

change default CFLAGS settings into the default

change default CFLAGS settings into the default

The way compiler settings are appended to CFLAGS breaks custom flags
that people are using to build.  So if someone does:
$ export CFLAGS='-O0 -ggdb'
$ make
The build will force -O2 instead.

Shift these defaults into a default setting rather than always appending.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Add error checking on all file opens

I'm missing a few of these, which is causing erroneous errors to be
printed when requesting help, or providing a bad file handle.

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

Decode an reservation errors code to strings

If reservation conflict happens with reservation related commands (e.g.,
resv_register, resv_acquire, and resv_release), show the error as a string.

[fixed compile error; reword commit message]
Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge branch 'apage43-jsonfixes'

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

Fix right trim() in JSON output

Was always trimming from the end of the format buffer, not the end of
the snprintf output. Also need to remove the initial decrement as
snprintf will leave fmt_sz pointing at the terminating NUL.

Output JSON device list in an array

When printing as an object with product name as key, multiple cards with
the same product name can be present as the same key, and most JSON
parsers will only see one.

Fix handling of Firmware field in JSON output

This was not checking the sizeof() of the .fr field and could read past
the end of it.

Allow bi-directional commands

The spec allows bi-directional commands so let's let a user submit
one. The kernel does not support this at the moment, but maybe it will
someday.

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

Fix segfault listing models

The models strings may not exist in the place this program thinks it
does. Check if the file is valid before reading from it, and return an
apporpriate string accordingly.

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

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

Remove intel specific log from top-level commands

Vendor specific plugins should be used for vendor specific logs, and
Intel already provides one.

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

Make code alignment great again

Some of these config options got mis-aligned somewhere along the way.

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

Make program have single exit location

All errors are returned rather than exiting the program. This requires
more return code error checking, but it makes it predictable where the
program may end and required if we ever implement an interactive mode.

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

Fix suffix shift overflow

From: Karsten Weiss <k.weiss@science-computing.de>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Fix intel read stats log page

Need to use the parameter. Was mistakenly using the defined string,
which makes no sense.

Reported-by: Karsten Weiss <k.weiss@science-computing.de>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Fix 48-bit conversion for 32-bit arch

The cast to 'long' was the wrong type for 32-bit.

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

Fix 32-bit warning on print

Use the arch specific print format macros.

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

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

Show negative temperatures in smart log.

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

Update man pages

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

add a disconnect man page

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

add a full blown connect man page

Mostly based on the wording in connect-all.

Signed-off-by: Christoph Hellwig <hch@lst.de>
[Removed older/duplicate option]
Signed-off-by: Keith Busch <keith.busch@intel.com>

update discover and connect-all man pages

Document the --host-traddr option, and add the missing description of
the --raw command to the connect-all man page.

Signed-off-by: Christoph Hellwig <hch@lst.de>
[Removed old/duplicate option entry]
Signed-off-by: Keith Busch <keith.busch@intel.com>
Conflicts:
Documentation/nvme-connect-all.txt
Documentation/nvme-discover.txt

nvmecli-tests : update the list of dependencies

README file is updated with complete list of dependencies
required to setup the environment for unitttests.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

Fix Debian rules

Usually when building a package we don't want to run tests
that require nvme devices. So until we can classify to tests
that have requirements to not run unless some env variables
are set we need this rule.

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

Unittest framework based on nvme-cli.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@hgst.com>

Release version 1.0

Been long enough, and lots of updates accumlated. Thank you to everyone
who contributed this round!

Just some highlights in this release:

 * Bug fixes on some of the lesser used parameters and commands
 * Support for fabrics discover and connect to targets
 * More 3rd party extensions have been added
 * New JSON output formats on some of the show routines
 * Documenation updates and fixes
 * Compiler warnings for some configurations

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

Update documentation

Fill in missing options in the IO commands.

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

Remove accidental print

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

Update nvme-write-uncor documentation.

Eluded to the write-zeroes by mistake.

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

get-feature updates

This fixes some of the printing options. We don't want to print text if
the request is to display the buffer in raw binary, and we don't want
to do a binary output if there is no buffer to write.

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

Re-add host-traddr option to connect-all manpage

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

Update documentation for discover

Previous commit mistakenly added the host-traddr option to
connect-all. This is for discover.

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

Fix coding alignment

Line wrapping argconfig options makes it harder to read.

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

Host-traddr documentation updates

Changed the parameter to use '-' instead of '_' to match consistency of
other options.

Added the option to the connect and connect all manpages.

Replaced the 's' option to 'w' (wwn) as 's' was already in use.

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

Document timeout value usage

Add to man page and clarify value is in milliseconds.

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

nvme-cli: endian swap mmio access

PCIe is always defined as little endian, so swap the data read from the
register to native endian.

Use the opportunity to also apply the no 64-bit access workaround to
all of the register accesses and not just some and dump struct nvme_reg.

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

nvme-cli: Add support for Fibre-channel host address

   Example:
nvme connect-all --transport=fc --traddr=pn-0x200100110d9f4100:nn-0x200100110d9f4100 \
--host_traddr=nn-0x20000024ff5ba062:pn-0x21000024ff5ba062

nvme connect --transport=fc --nqn=nqn.2014-08.org.nvmexpress:nvm-subsystem \
--traddr=pn-0x200100110d9f4100:nn-0x200100110d9f4100 --host_traddr=nn-0x20000024ff5ba062:pn-0x21000024ff5ba062

Signed-off-by: Darren Trapp <darren.trapp@cavium.com>

regenerate man pages

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

nvme-cli: fix nvme-connect-all using hostnqn

The example in the man pages:

nvme connect-all --transport=rdma --traddr=192.168.1.3 \
--hostnqn=host1-rogue-nqn

fails because nvme-cli fails to actually use hostnqn upon
connect.  This patch fixes that.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvme-cli: follow-on doc tweaks to nvme-connect-all

Minor massaging and typo fixes.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>

nvme-cli: follow-on discovery tweaks from thread

Follow-on doc patch tweaks from discovery documentation discussion.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>

Merge pull request #133 from weber-wenbo-wang/master

Add more memblaze specific smart info for FW 500

Add more memblaze specific smart info for FW 500

Merge pull request #131 from gwendalcr/fix_compil

Fix compilation errors

Fix compilation errors:

While compiling nvme-cli for ChromeOS, the compiler founds minor issues
in the code:
- unsigned and sizet_t can never be negative.
- use fabs instead of llabs when argument is a double.

Signed-off-by: Gwendal Grignou <gwendal@chromium.org>

Man page updates

Fix the connect-all manpage header section and regenerate all the
manpages.

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

add documentation for the connect-all command

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

fine tune the nvme-discover manpage

 - remove the device argument, which as far as I can tell does not
   exist.
 - mention the /etc/nvme/hostnqn file, the default host nqn and explain
   host nqns a bit more
 - mention the loopback transport
 - be a bit more specific about IP addressing as a concept separate from
   the RDMA transport
 - improve the introduction a bit.

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

consolidate reporting bugs information

Add the Linux-nvme list to the main reporting bugs section and remove
the separate ones in the connect and discovery pages.

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

Add json format to list command

Updating man page with new options.

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

argconfig: use strtoul for unsigned types

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

Return all errors on list command

Negative errors mean the command failed and we shouldn't try displaying
information if getting it wasn't succcessful.

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

Merge pull request #130 from jpfreyen/master

fix to argconfig.c for nvme-discover

nvme-cli: tweak discover man pages w/argconfig fix

With Sagi's fix to argconfig.c, the nvme-discover man
pages need to be tweaked to show the true intent of
using discovery.conf, a default with multiple discover
command-lines.

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvme-cli: have cfg var get set more than 1 execution

Without this fix, the struct config cfg variable only
gets set one time when argconfig_parse() is called. Any other
time it is attempted to be set, it acts as a read-only value
and continues to use the initial value it was set with.

This poses a problem for the discovery feature, discovery.conf file
as it is desired to make the file multiple line instead of a single
line, such as:

-t rdma -a 192.168.69.33 -s 4420 -q host2-proper-nqn
-t rdma -a 192.168.1.4   -s 4420 -q host2-proper-nqn
-t rdma -a 192.168.69.33 -s 99 -q host2-proper-nqn

I've tried the fix with a few existing commands and nothing
appears to be broken, but I haven't done an exhaustive test.

Reported-by: Jay Freyensee <james_p_freyensee@linux.intel.com>
Originally-by: Sagi Grimberg <sagi@lightbits.io>
Tested-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

Fix nvme-models in makefile

From: Scott Bauer <scott.bauer@intel.com>

We're not sure why our system tested ok without this...

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

Merge branch 'isdct-nvme-log' of https://github.com/ScottyBauer/nvme-cli

JSONify SMART-add command, and rework json for fw-info-slots

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

Implement json output for list that matches 'isdtc show -output json --intelssd'

    New command:
    nvme list -output-format json|normal
OR
    nvme list

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

Commit new man pages

This time actually adding the new man pages for fabrics.

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

Merge pull request #127 from narasimhan-v/latency_script_fix

latency script fixup

latency script fixup

This patch adds functionality to
*) Calculate and display the average latency
*) Make sure count is not 0

Fixes
*) in case of write, grep latency in case of dd command would
always fail. Fixed it.

Signed-off-by: Narasimhan V <sim@linux.vnet.ibm.com>

Merge pull request #125 from yongseokoh/fix_id_ctrl

Modified: print the subnqn (NVM Subsystem NVMe Qualified Name) field …

Change uuid_be type to __u8

From: Christoph Hellwig <hch@infradead.org>

This removes the linux header dependency for compiling.

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

Regenerate man pages

New text files added for fabrics documentation, so committing the
generated manpages.

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

nvme-cli: Add discover connect cmds to main list

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvme-cli: Add nvme-connect.txt shell for fill in

nvme-discover documentation refers to nvme-connect,
so add a starter file for nvme-connect for fill
in by a generous soul :-).

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>

nvme-cli: Tweak discover summary to match docs

Signed-off-by: Jay Freyensee <james_p_freyensee@linux.intel.com>