From what I can tell in the code, padstr() is only coded in the IDE section, not SCSI/SATA?
> On Feb 19, 2015, at 11:42 PM, Markus Armbruster <arm...@redhat.com> wrote: > > John Snow <js...@redhat.com <mailto:js...@redhat.com>> writes: > >> On 02/19/2015 02:48 PM, Alan Latteri wrote: >>> I am virtualzing a physical server for which I need to set the SCSI/SATA >>> drive serial. It is comprised of 12 " " spaces then 8 letter/digits. If >>> I exclude the spaces, the drive serial is not accurate. If I include the >>> spaces I get the following error. >>> >>> error: Failed to start domain test1 >>> error: internal error: driver serial ' ABCD1234' contains unsafe characters >>> >>> virsh edit >>> Centos 7.0 >>> 3.19.0-1.el7.elrepo.x86_64 >>> QEMU emulator version 1.5.3 (qemu-kvm-1.5.3-60.el7.centos.7), Copyright >>> (c) 2003-2008 Fabrice Bellard >>> >> >> Are you sure you need to input the spaces? the standard inquiry reply >> that outputs the serial uses spaces as padding, so even if you use >> serial "ABCD1234" the inquiry command is going to tell you >> " ABCD1234". > > Are you sure it pads on the left? padstr() doesn't look like left... > > [...]