Uploaded image for project: 'RHEL'
  1. RHEL
  2. RHEL-3542

Don't silently truncate (SCSI) disk serial number

    • Minor
    • sst_virtualization_storage
    • ssg_virtualization
    • False
    • Hide

      None

      Show
      None
    • If docs needed, set a value

      Description of problem:

      Silent truncation of the serial can lead to a guest ABI regression when the limit is changed such as happened in:

      commit 48b6206305b8d56524ac2ee347b68e6e0a528559
      Author: Rony Weng <ronyweng@synology.com>
      Date: Mon Aug 29 15:52:18 2016 +0800

      scsi-disk: change disk serial length from 20 to 36

      Openstack Cinder assigns volume a 36 characters uuid as serial.
      QEMU will shrinks the uuid to 20 characters, which does not match
      the original uuid.

      Note that there is no limit to the length of the serial number in
      the SCSI spec. 20 was copy-pasted from virtio-blk which in turn was
      copy-pasted from ATA; 36 is even more arbitrary. However, bumping it
      up too much might cause issues (e.g. 252 seems to make sense because
      then the maximum amount of returned data is 256; but who knows there's
      no off-by-one somewhere for such a nicely rounded number).

      Signed-off-by: Rony Weng <ronyweng@synology.com>
      Message-Id: <1472457138-23386-1-git-send-email-ronyweng@synology.com>
      Cc: qemu-stable@nongnu.org
      Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

      or alternatively when libvirt started using 'device_id' in addition to 'serial' which has a different truncation length.

      Rather than silently truncating, qemu should reject values which would lead to truncation (ideally after fixing truncation limits: https://bugzilla.redhat.com/show_bug.cgi?id=1967666 ).

      Libvirt will be okay with using an additional property to enable the feature or to allow querying of the truncation limit.

            virt-maint virt-maint
            pkrempa@redhat.com Peter Krempa
            virt-maint virt-maint
            qing wang qing wang
            Votes:
            0 Vote for this issue
            Watchers:
            10 Start watching this issue

              Created:
              Updated: