From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36299) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fWMeS-0007UZ-3y for qemu-devel@nongnu.org; Fri, 22 Jun 2018 10:03:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fWMeN-0003nH-Vf for qemu-devel@nongnu.org; Fri, 22 Jun 2018 10:03:00 -0400 Date: Fri, 22 Jun 2018 15:02:44 +0100 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20180622140244.GK23296@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20180615142108.27814-1-kwolf@redhat.com> <20180615142108.27814-26-kwolf@redhat.com> <7a310b92-f8cb-b68b-d882-9b2959794347@de.ibm.com> <20180622125502.GF4366@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [libvirt] [PULL 25/26] block: Remove deprecated -drive option serial List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Christian Borntraeger Cc: Kevin Wolf , Peter Maydell , Boris Fiuczynski , qemu-block@nongnu.org, libvir-list@redhat.com, qemu-devel@nongnu.org, pkrempa@redhat.com On Fri, Jun 22, 2018 at 03:36:50PM +0200, Christian Borntraeger wrote: > > > On 06/22/2018 02:55 PM, 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) > >> > >> > >> > >> > >> > >> skel > >> > >>
> >> > >> > >> > >> -> > >> [...] > >> -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. > > To me it seems that this is also broken on x86. Correct, this is not architecture specific. > > Thanks for bringing this up. libvirt should fix this before QEMU 3.0 is > > released. > > I think this is definitely too short notice. We should not break existing > setups just by insisting that users have to update libvirt when they update > QEMU. Yes, this might be our policy, but doing so "just because we can" > is certainly a very bad attitude. I see no fundamental technical reason why > we should not revert this change. > > > Sadly, it also shows that deprecation warnings in log files go > > unnoticed. > > In fact whoever added the deprication notice should have followed up > with the libvirt team to implement that change. no? On libvirt side I thought we had already stopped using the deprecated syntax, but we clearly missed it :-( Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|