nvme cli utility clips Leaf model # to 20 characters

Bug #1595274 reported by bugproxy
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nvme-cli (Ubuntu)
Fix Released
Undecided
Taco Screen team
Xenial
Fix Released
Medium
Steve Langasek

Bug Description

nvme cli only shows the first 20 characters of the Leaf model number (mn) i.e "PCIe3 3.2TB NVMe Fla"
    root@gdx1x152:~# nvme id-ctrl /dev/nvme0 | more
      NVME Identify Controller:
      vid : 0x1c58
      ssvid : 0x1014
      sn : CJH001001830
      mn : PCIe3 3.2TB NVMe Fla
      fr : KMIPP107

When I check data from lspci it shows correct name
      Product Name: PCIe3 3.2TB NVMe Flash Adapter

checking block device data also shows correct model name
      # cat /sys/block/nvme0n1/device/model
      PCIe3 3.2TB NVMe Flash Adapter
      #

== Comment: #5 - Guilherme Guaglianoni Piccoli - 2016-06-21 16:07:02 ==
Not too much commits between v0.5 and v0.7. The following 2 commits are the fix to this issue:

ab43012 ("Fix the bug of Model Number and display the info of Serial Number")
a304435 ("Fix the bug of Model Number and display the info of Serial Number")

Probably the 1st was merged too early, instead of waiting v2..so it's easier to backport both, in order.

bugproxy (bugproxy)
tags: added: architecture-ppc64le bugnameltc-142912 severity-medium targetmilestone-inin16041
Changed in ubuntu:
assignee: nobody → Taco Screen team (taco-screen-team)
affects: ubuntu → nvme-cli (Ubuntu)
Steve Langasek (vorlon)
Changed in nvme-cli (Ubuntu Xenial):
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nvme-cli - 0.7-1ubuntu1

---------------
nvme-cli (0.7-1ubuntu1) yakkety; urgency=low

  * Merge from Debian unstable.
    - Includes upstream fix for truncated Leaf model number (LP: #1595274).
  * Remaining changes:
    - fix segfault when using show-regs command with no device
      name
    - fix null pointer deference segfault on strcmp() on a
      null string

nvme-cli (0.7-1) unstable; urgency=medium

  * New upstream release
  * Bump Standard-Version to 3.9.8, no changes required.

 -- Steve Langasek <email address hidden> Thu, 23 Jun 2016 12:45:21 -0700

Changed in nvme-cli (Ubuntu):
status: New → Fix Released
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2016-07-28 08:27 EDT-------
I want to ask Canonical is there's any news on the backport of the aforementioned patches to Xenial's nvme-cli.

Thanks in advance,

Guilherme

Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2016-08-26 17:27 EDT-------
.

Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2016-09-30 13:45 EDT-------
Hello Canonical.

What is the status on the request to backport this to xenial?

Thanks.

Steve Langasek (vorlon)
Changed in nvme-cli (Ubuntu Xenial):
assignee: nobody → Steve Langasek (vorlon)
status: Triaged → In Progress
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello bugproxy, or anyone else affected,

Accepted nvme-cli into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nvme-cli/0.5-1ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nvme-cli (Ubuntu Xenial):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2016-10-28 09:29 EDT-------
Verification done using the proposed package.
Thanks very much Canonical.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nvme-cli - 0.5-1ubuntu0.1

---------------
nvme-cli (0.5-1ubuntu0.1) xenial; urgency=medium

  * d/p/0001-Fix-the-bug-of-Model-Number-and-display-the-info-of-.patch,
    d/p/0002-Fix-the-bug-of-Model-Number-and-display-the-info-of-.patch:
    cherry-pick patches from upstream to fix display of information for
    devices with long names. LP: #1595274.

 -- Steve Langasek <email address hidden> Mon, 17 Oct 2016 17:21:59 -0700

Changed in nvme-cli (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Chris J Arges (arges) wrote : Update Released

The verification of the Stable Release Update for nvme-cli has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.