[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive
From: |
Peter Krempa |
Subject: |
Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial |
Date: |
Mon, 25 Jun 2018 09:16:10 +0200 |
User-agent: |
Mutt/1.9.3 (2018-01-21) |
On Fri, Jun 22, 2018 at 14:55:02 +0200, Kevin Wolf wrote:
> Am 22.06.2018 um 13:38 hat Christian Borntraeger geschrieben:
> >
> > On 06/15/2018 04:21 PM, Kevin Wolf wrote:
> > > The -drive option serial was deprecated in QEMU 2.10. It's time to
> > > remove it.
> > >
> > > Tests need to be updated to set the serial number with -global instead
> > > of using the -drive option.
> >
> > libvirt 4.5 still creates those (at least on s390x)
> >
> > <disk type='file' device='disk'>
> > <driver name='qemu' type='qcow2' cache='none' io='native'
> > iothread='1'/>
> > <source file='/var/lib/libvirt/qemu/image.zhyp137'/>
> > <target dev='hda' bus='virtio'/>
> > <serial>skel</serial>
> > <boot order='1'/>
> > <address type='ccw' cssid='0xfe' ssid='0x0' devno='0x0000'/>
> > </disk>
> >
> >
> > ->
> > [...]
> > -drive
> > file=/var/lib/libvirt/qemu/image.zhyp137,format=qcow2,if=none,id=drive-virtio-disk0,serial=skel,cache=none,aio=native
> > -device
> > virtio-blk-ccw,iothread=iothread1,scsi=off,devno=fe.0.0000,drive=drive-virtio-disk0,id=virtio-disk0,bootindex=1,write-cache=on
> >
> > [...]
> >
> > 2018-06-22T11:25:20.946024Z qemu-system-s390x: -drive
> > file=/var/lib/libvirt/qemu/image.zhyp137,format=qcow2,if=none,id=drive-virtio-disk0,serial=skel,cache=none,aio=native:
> > Block format 'qcow2' does not support the option 'serial'
> > 2018-06-22 11:25:21.098+0000: shutting down, reason=failed
> >
> > So it seems that this breaks s390x.
>
> Thanks for bringing this up. libvirt should fix this before QEMU 3.0 is
> released.
So, I already extracted the code that formats the attributes which
should be actually done for the frontend few months ago, but did not
swithc to it since I did not want to check when everything was added.
The fix will be rather simple if we are certain that the disk serial,
geometry and error policies were supported in qemu 1.5. In that case we
can switch unconditionally to the new format.
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PULL 25/26] block: Remove deprecated -drive option serial, (continued)
- Re: [Qemu-devel] [PULL 25/26] block: Remove deprecated -drive option serial, Daniel P . Berrangé, 2018/06/22
- Re: [Qemu-devel] [PULL 25/26] block: Remove deprecated -drive option serial, Kevin Wolf, 2018/06/22
- Re: [Qemu-devel] [PULL 25/26] block: Remove deprecated -drive option serial, Thomas Huth, 2018/06/25
- Re: [Qemu-devel] [PULL 25/26] block: Remove deprecated -drive option serial, Markus Armbruster, 2018/06/22
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial, Daniel P . Berrangé, 2018/06/22
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial, Daniel P . Berrangé, 2018/06/22
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial, Eric Blake, 2018/06/22
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial,
Peter Krempa <=
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial, Thomas Huth, 2018/06/25
- Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial, Daniel P . Berrangé, 2018/06/25
[Qemu-devel] [PULL 08/26] qobject: Move block-specific qdict code to block-qdict.c, Kevin Wolf, 2018/06/15
Re: [Qemu-devel] [PULL 00/26] Block layer patches, Peter Maydell, 2018/06/15