[Bug 1733276] Re: Cannot resize partitions on NVME devices due to bad device name parsing

[Eric Desrochers]

debdiff for xenial

** Patch removed: "lp1733276-partman-xenial.debdiff"
   https://bugs.launchpad.net/ubuntu/xenial/+source/partman-partitioning/+bug/1733276/+attachment/5049269/+files/lp1733276-partman-xenial.debdiff

** Patch removed: "lp1733276-partman-artful.debdiff"
   https://bugs.launchpad.net/ubuntu/xenial/+source/partman-partitioning/+bug/1733276/+attachment/5049270/+files/lp1733276-partman-artful.debdiff

** Patch added: "lp1733276-partman-xenial.debdiff"
   https://bugs.launchpad.net/ubuntu/xenial/+source/partman-partitioning/+bug/1733276/+attachment/5049272/+files/lp1733276-partman-xenial.debdiff

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1733276

Title:
  Cannot resize partitions on NVME devices due to bad device name
  parsing

Status in partman-partitioning:
  Fix Released
Status in debian-installer package in Ubuntu:
  Confirmed
Status in partman-partitioning package in Ubuntu:
  Fix Released
Status in debian-installer source package in Trusty:
  New
Status in partman-partitioning source package in Trusty:
  In Progress
Status in debian-installer source package in Xenial:
  New
Status in partman-partitioning source package in Xenial:
  In Progress
Status in debian-installer source package in Zesty:
  New
Status in partman-partitioning source package in Zesty:
  Won't Fix
Status in debian-installer source package in Artful:
  New
Status in partman-partitioning source package in Artful:
  In Progress
Status in debian-installer source package in Bionic:
  Confirmed
Status in partman-partitioning source package in Bionic:
  Fix Released

Bug description:
  [Impact]

  It is not possible to resize NVME partitions in the Ubiquity installer.
  NVME devices have an unusual device format, specifically e.g. /dev/nvme0n1p1 that has two 'sections' that potentially look like partitions.

  Based on the error message, it seems that this device name is being
  parsed incorrectly and it attempts to use the device path /dev/nvme0n1
  instead of /dev/nvme0n1p1

  ** See attachment (screenshot) took in action :
  - resize.png
  - resize_impossible.png

  [Test Case]

  You can test this using qemu and a virtual NVME device. If you toggle
  the same device/partition between a virtual SCSI/IDE and an NVME
  device - the issue appears and disappears.

  Commands you can use to replicate the issue - unpack netboot.tar.gz
  into the local directory and then

  dd if=/dev/zero of=nvme.disk bs=1M count=16384

  NVME:
  qemu-system-x86_64 -drive file=nvme.disk,if=none,id=drv0,format=raw -device nvme,drive=drv0,serial=foo -enable-kvm -smp 2 -m 1024 -kernel xenial/ubuntu-installer/amd64/linux -initrd xenial/ubuntu-installer/amd64/initrd.gz

  SCSI:
  qemu-system-x86_64 -drive file=nvme.disk,if=none,id=drv0,format=raw -device virtio-blk-pci,scsi=off,bus=pci.0,addr=0x7,drive=drv0,id=virtio-disk0,bootindex=1 -enable-kvm -smp 2 -m 1024 -kernel ubuntu-installer/amd64/linux -initrd ubuntu-installer/amd64/initrd.gz

  [Regression Potential]

   * Low risk of regression, with this patch, partman, will be more robust and align with what disk_name() in the kernel does (linux.git/tree/block/partition-generic.c). It is sufficient to check
  whether the last character is a digit.

  * The patch may also fixes (or at least prevent for the future) other
  potential similar devices issue and not explicitly fixing and/or
  benefit NVMe device types.

  [Other Info]

  * Debian upstream bug :
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=820818

  * Debian upstream fix :
  https://anonscm.debian.org/cgit/d-i/partman-partitioning.git/commit/?id=01087125e07a7b22da589e8116f9ef7a26275006

  * Commit [01087125]
  Make get_real_device() both simpler and more generic (See: #820818).
  Looking at the implementation of the disk_name() function in the kernel
  (linux.git/tree/block/partition-generic.c), it is sufficient to check
  whether the last character is a digit.

  * $ git describe --contains 01087125e07a7b22da589e8116f9ef7a26275006
  116~2

  * $ rmadison partman-partitioning
   partman-partitioning | 85ubuntu2    | precise/main/debian-installer
   partman-partitioning | 99ubuntu1    | trusty/main/debian-installer
   partman-partitioning | 110ubuntu4.1 | xenial-updates/main/debian-installer
   partman-partitioning | 114ubuntu2   | zesty/main/debian-installer
   partman-partitioning | 114ubuntu2   | artful/main/debian-installer
   partman-partitioning | 120ubuntu1   | bionic/main/debian-installer

  [Original Description]

  It is not possible to resize NVME partitions in the Ubiquity
  installer.  This appears to affect multiple filesystem types including
  NTFS and ext4.

  NVME devices have an unusual device format, specifically e.g.
  /dev/nvme0n1p1 that has two 'sections' that potentially look like
  partitions.

  Based on the error message, it seems that this device name is being
  parsed incorrectly and it attempts to use the device path /dev/nvme0n1
  instead of /dev/nvme0n1p1

  You can test this using qemu and a virtual NVME device.  If you toggle
  the same device/partition between a virtual SCSI/IDE and an NVME
  device - the issue appears and disappears.

  Commands you can use to replicate the issue - unpack netboot.tar.gz
  into the local directory and then

  dd if=/dev/zero of=nvme.disk bs=1M count=16384

  NVME:
  qemu-system-x86_64 -drive file=nvme.disk,if=none,id=drv0,format=raw -device nvme,drive=drv0,serial=foo -enable-kvm -smp 2 -m 1024 -kernel xenial/ubuntu-installer/amd64/linux -initrd xenial/ubuntu-installer/amd64/initrd.gz

  SCSI:
  qemu-system-x86_64 -drive file=nvme.disk,if=none,id=drv0,format=raw  -device virtio-blk-pci,scsi=off,bus=pci.0,addr=0x7,drive=drv0,id=virtio-disk0,bootindex=1  -enable-kvm -smp 2 -m 1024 -kernel ubuntu-installer/amd64/linux -initrd ubuntu-installer/amd64/initrd.gz

To manage notifications about this bug go to:
https://bugs.launchpad.net/partman-partitioning/+bug/1733276/+subscriptions