Merge pull request #40 from ossobv/ossobv-make-deb-fixes

Improve debian package build process.

Improve debian package build process.

* The changes from 3686759 make the debian/rules a lot simpler.
* Fix so ld doesn't leave a.out turds.
* Add dist-orig make target that creates a reproducible orig.tar.gz.
  A reproducible source archive is a first step towards reproducible
  builds.
* Fix typo in debian package description.
* Add dist and pkg to the PHONY targets.

Merge pull request #39 from pmmccorm/master

Makefile cleanup.

Merge pull request #38 from pmmccorm/master

Factor out ID printing code into common.c

Makefile cleanup.

Don't link with unused libs: -lm, -lpthread
Don't hardcode in 64 bit compile, this will break cross compilation
Put preprocessor flags in CPPFLAGS
Add to existing variables instead of var = $var pattern

Signed-off-by: Patrick McCormick <patrick.m.mccormick@intel.com>

Factor out ID printing code into common.c

The printing output should be identical. The id structure
printing functions can be more or less used by lsnvme.

Signed-off-by: Patrick McCormick <patrick.m.mccormick@intel.com>

Merge pull request #37 from ossobv/ossobv-add-debian-files

Create repository ready `make deb` build.

LGTM. Thanks!

Create repository ready `make deb` build.

When running `make deb` on `wheezy` with version `0.2.5.ga6ce` it
creates these files:

    nvme-cli_0.2.5.ga6ce-1~wheezy_amd64.changes
    nvme-cli_0.2.5.ga6ce-1~wheezy_amd64.deb
    nvme-cli_0.2.5.ga6ce-1~wheezy.debian.tar.gz
    nvme-cli_0.2.5.ga6ce-1~wheezy.dsc
    nvme-cli_0.2.5.ga6ce.orig.tar.gz

These can be included in your package archive, so they can be fetched
through apt-get.

Remove uapi header dependency

This wasn't even correct since the source includes the local header,
and is almost always ahead of the uapi version anyway.

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

fix off by one error

Need to use a larger int type because 256 doesn't fit in 8 bits, and
subtract 1 becase number of ranges is 0's based.

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

Add data-set management command

By popular demand...

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

Update documetatation

Fixing the command in delete-ns and re-running asciidoc to flush all
changes to the repo.

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

Fix 32 bit compilation

Addressing issue:

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

Unfortunetly no testing that I'm aware of uses 32-bit by default, so I
suspect we've not seen the last 32-bit issue in this project's life.

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

Debian package fixes

Set the correct architecture and library dependencies.

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

Add deb package target

Note the make clean looks a little dangerous here. Just to be aware of
if we ever name a file or directory "nvme-<something" in the future.

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

Provide rpm build target

Makes creating an RPM package easy.

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

Fixup version file

Some mixups from trying to copy other projects led to incorrect
description.

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

Ignore generated version file

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

Print status to stderr on r/w commands

The option exists to pipe the data through stdout, so we don't want to
potentially contaminate user data with program output.

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

Different file handle for metadata

Allow a user to specify a different file for metadata rather than assume
it is contigous with the data.

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

Add a version

Starting with v0.1.

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

Updated license on bash scripts

Two bash scripts were missed for the GPLv2 licensing. Fixing that with this commit.

Merge pull request #34 from ajorg/master

Makefile fixes for packagers (DESTDIR / PREFIX)

Makefile fixes for packagers (DESTDIR / PREFIX)

Adds DESTDIR and PREFIX in appropriate places
Adds share/ to the man path
Lets the packager override PREFIX

Merge pull request #33 from lsgunth/master

Modified the license of the code pulled from libargconfig to be GPLv2

Modified the license of the code pulled from libargconfig to be GPLv2

There have been legal concerns of linking Apache licensed code with
GPLv2 code. This change modifies the license of the code originating
from libargconfig. All of the copyright holders (including myself and
PMC Sierra represented by Stephen Bates <Stephen.Bates@pmcs.com>) have
signed off on this change.

Fix nvme.c licence to GPLv2

Fixing comment to match the source's appropriate license.

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

Merge pull request #31 from samiWaheed/master

 Change default NSID in get-feature function. Fixed format of LBA range output

Corrected IEEE byte order in id-cntrl

Fixed byte order of IEEE in show_nvme_id_ctrl

Changed default NSID in get-feature function.  Fixed format of LBA range output

- Changed default NSID to 1 in get-feature function.   Some features
require a valid NSID (i.e. lba range type)
- Fixed format of LBA range output
- Added description to the attributes field in LBA range output
- Changed default NSID to 0xffffffff in get_log (Matches help
documentation now)

Merge remote-tracking branch 'linux-nvme/master'

Merge pull request #29 from samiWaheed/master

Added a decode option for all mandatory and optional Feature Identifiers fields

Added a decode option for all mandatory and optional Feature Identifiers

- Using --human-readable option with the get-feature command will decode
the fields of the specified FID.

Add namespace list option for all namespaces

Add flag to toggle CNS from 2 to 0x10, showing either active namespaces
or all namespaces.

Signed-off-by: root <root@polestar>

Open device for namespace deletion

So you can actually delete the namespace.

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

Updated .gitignore

Improved list command formatting

Fix data size on controller list

The parsing is corrupting data on a list.

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

Fix unsigned integers

Reported that using valid 32 bit integers causes errors do to type
signedness.

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

Use device maximum for error log entries

It's been reported that some controllers return error if more logs were
requested than the device's max. This patch overrides the user's request,
or the default if the option was not given, with the device's max if
the setting exceeds it.

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

Make timeout a parameter for format

Allows a user to override the timeout value, defaults to 2 minutes.

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

Fix compiler warning on format string

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

Merge pull request #25 from paul-grabinar/master

Allow for firmware activation that needs a reset

Allow for firmware activation that needs a reset

On a firmware activate, some drives require a reset before the new
firmware can be used. Such drives return a status 0x10b to the activate
request.
Rather than treat this as an error, tell this user the activation was
successful, but a reset is required.

Use hex for sgls print format

It's a bit field, better reprsented by hex.

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

add help strings and submenus

Add relevant help to nvme commands and command options.
Don't seg fault when trying to print help strings.
Don't declare "-h" an unrecognized option; print help menu instead.

Signed-off-by: kaoudis <kaoudis@colorado.edu>

Merge pull request #24 from kaoudis/master

[v2] add bash + zsh autocompletions

LGTM

add bash and zsh autocompletion files and readme

Merge pull request #23 from paul-grabinar/master

Allow longer timeout for format

This seems okay for now. Long term, we should make every command accept a 'timeout' parameter.

Allow longer timeout for format

Some NVMe drives take longer than 60 seconds to perform a format on a
namespace. Increase the timeout for the format to 120 seconds.

Merge pull request #21 from mihu/asmart

add additional smart log

The "pragma" usage works for me, so making the assumption it's portable.

remove usage of typedef

add additional smart log

derived from:
Intel Solid-State Drive Data Center P3700 Series Product Specification

Merge pull request #19 from mihu/makefile

prepare Makefile for easier packaging

prepare Makefile for easier packaging

Fixed data-size type and bug in return value for submit_io

The data-size and meta data input arguments should accept suffixes and
also fixed a bug where the returned value for submit_io is always 0.

Exit on bogus data

Notify of bad paramter provided and exit the program.

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

Set nvme bar to mapped address

Instead of copying the data, point the bar to the mapped address.

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

Fix parameter size

Fixing corruption on parameter parsing.

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

Makefile: install documentation without building

Fixes dependency timestamp problems on cloning that 'make' rules
require it rebuild manpages even though this is not needed. We don't
want an 'asciidoc' dependency to build and install the program unless
you explicitly want to build the documentation, so changing the make
target's dependencies to not rebuild it.

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

Documenation: fix heading on nvme-read

Finding lots of these documentation errors.

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

Update man pages

Pushing updated man pages to remove asciidoc dependency on install.

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

Support for controller list namespace modes

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

Fix namespace id usage in attach namespace

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

Add namespace attach/detach subcommands

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

Add delete namespace sub command

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

Add create namespace sub command

This regenerated the entire contents of the manpages as well, and I
didn't feel like splitting these commits.

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

Add nvme identify list controller sub command

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

Consistent error reporting

Prints the decoded error string and status in hex for all nvme errors.

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

White space fixing

Just some white space fixes to keep consistent formatting. The
original style set out to use same as the linux kernel, so sticking
with convention.

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

Printf platform fixes

Changing the 64-bit printf formats to use an appropriate inttype macro
for the given architecture, and converting for appropriate endianess for
"__le64" types.

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

Remove html install target

Not meant to be installed at this time. This previously polluted '/'
with undesriable (but harmless) artifacts.

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

Merge branch 'master' of https://github.com/linux-nvme/nvme-cli

Reverted to GPL licensing

Mixed licensing is not distributable. Based on this and on discussions
between Stephen, Keith and various legal people we are aligning with
GPL.

Merge branch 'master' of https://github.com/linux-nvme/nvme-cli

Remove build artifact

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

Add html documentation

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

Merge pull request #13 from stevegalbraith/master

Found another instance of flbas being used as an index into list_item.

Found another instance of flbas being used as an index.
Same fix was applied as in pull request #12.

Merge pull request #12 from stevegalbraith/master

Fix to determine initial value for lba in print_list_item

Thanks, LGTM.

Fix to determine initial value for lba in print_list_item
The index used to determine lba value takes into accout the entire FLBAS field.  Bit 4 determines whether an extended or separate is used to send metadata and if set to 1'b,
throws off the index value.  Masked out the top 4 bits to only take into account Bits 3:0 which indicate the LBA format type used.

Updated licensing

Added the GPL license to cover the nvme.h file and moved both licenses
into a new sub-folder.

Signed-off-by: Stephen Bates <sbates@raithlin.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>

Merge pull request #11 from jderrick/master

Human-readable decode for id-ctrl and id-ns

LGTM

Added a human-readable format option to id-ctrl and id-ns. This will
partially decode several relevant bitfields in the ID Controller and
Namespace commands. Reserved fields are displayed if not 0.

Signed-off-by: Jon Derrick <jonathan.derrick@intel.com>

Fix f/w download

This needs to read the image file into it's buffer for this to work
correctly, and use an appropriate default transfer size.

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

Minor typos in two man pages

Cut and paste fix

Fixes a cut and paste error in the passthru command.

Switch license from GPLv2 to Apache

Signed-off-by: Keith Busch <keith.busch@intel.com>
Signed-off-by: Stephen Bates <sbates@raithlin.com>

Merge pull request #8 from zevweiss/master

Compile with -std=gnu99.

Compile with -std=gnu99.

struct nvme_command contains an anonymous union (a GNU extension).

Merge pull request #7 from zevweiss/master

Fix over-long loop in show_fw_log().

Fix over-long loop in show_fw_log().

nvme_firmware_log_page.frs has 7 entries, not 8.

Register read fix for kernel 4.0

The nvme sysfs path was moved in kernel 4.0 to a new class type, so just
opening that one first before falling back to the previous location.

This also is printing an error if the user requested an invalid command.

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

Add data input option for set features

Some NVMe features perform a data transfer and this provides a user option
to specify where that data comes from when setting.  Documentation updated
as well.

This also includes an unrelated fix to hush a warning for unused symbols
under precompile libudev dependencies.

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

Improved Makefile to use kernel uapi header.

Until now the CLI has relied on the header in the local Linux folder.
This is not safe as the kernel header may have differences that can
cause severe issues (e.g. new/different structure members). This
commit tries to use the uapi for the current kernel. Note this only
fixes at compile time.

Fixed versioning for pre 1.2 devices

Prior to NVMe 1.2 there was no version field in controller ID and the
version had to come from the BAR register. This change ensures the
version is obtained from the right place for all devices.

Improved text display for nvme list command.

Fixed whitespace damage.

Fixed formatting

Major improvements in list output

Added improved output processing of the nvme list command. Improved
formatting and added fields for the output.