From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39046) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fsmJm-0000pJ-Io for qemu-devel@nongnu.org; Thu, 23 Aug 2018 05:54:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fsmJj-0004rU-PW for qemu-devel@nongnu.org; Thu, 23 Aug 2018 05:54:18 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:46574 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fsmJh-0004lF-MA for qemu-devel@nongnu.org; Thu, 23 Aug 2018 05:54:13 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 76D46402211D for ; Thu, 23 Aug 2018 09:54:11 +0000 (UTC) Date: Thu, 23 Aug 2018 10:53:59 +0100 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20180823095359.GH23703@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20180822095421.11765-1-quintela@redhat.com> <20180822095421.11765-8-quintela@redhat.com> <0d3a215b-4916-b0a6-5654-5d64fcf6c996@redhat.com> <87y3cyo4ax.fsf@trasno.org> <20180823074321.wx7wix7jwsfrcbk5@sirius.home.kraxel.org> <9cb5f027-e49c-d943-7ffa-9e744537f174@redhat.com> <20180823094850.zycuwtjndmztgcm4@sirius.home.kraxel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180823094850.zycuwtjndmztgcm4@sirius.home.kraxel.org> Subject: Re: [Qemu-devel] [PATCH v3 07/10] check: Only test boot-serial when sga is compiled in List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann Cc: Thomas Huth , lvivier@redhat.com, Juan Quintela , "Michael S. Tsirkin" , qemu-devel@nongnu.org, peterx@redhat.com, dgilbert@redhat.com, Paolo Bonzini On Thu, Aug 23, 2018 at 11:48:50AM +0200, Gerd Hoffmann wrote: > > > I remember we even discussed that a while back ... > > > Oh yes: https://git.kraxel.org/cgit/qemu/commit/?h=work/no-sgabios&id=408690999429fdf44011c49b3af6efa50f369b1e > > > Somehow never ended up being merged. > > > > Sounds like a good idea! Could you repost the patch, with an update to > > qemu-deprecated.texi, please? Maybe also print the deprecation message > > in any case if !use_sgabios, e.g.: > > > > } else { > > /* > > * For this machine type we expect SeaBIOS to provide a > > * serial console for -machine graphics=off. No need to > > * add the sgabios rom here. > > */ > > warn_report("sgabios is deprecated and is not required anymore," > > " SeaBIOS now has the same functionality, too"); > > } > > Hmm, IIRC libvirt support is a bit tricky here as there is no easy way > to probe for serial console support in seabios. So the idea was to just > allow both -machine graphics=off and -device sga, by turning -device sga > into a nop in case -machine graphics=off is present. > > But given we meanwhile had some releases without this I suspect that > isn't a good idea any more. With both -machine graphics=off -device sga > seabios and sgabios will fight over the serial device ... > > So, maybe take a completely different route. Add a deprecation_reason > field to DeviceClass. Make it visible in qapi introspection, so libvirt > can see it and avoid adding it to the command line. Then we can > deprecate -device sga (and possibly other devices too) that way. That won't work IIUC. If we stop using "-device sga" based on some introspected deprecation value on the device, that means the guest will have a different ABI from POV of migration stream. So we would be unable to migrate to a QEMU without the deprecation, despite the XML config being unchanged. The only way we can stop using "-device sga" is if we either have something explicit in the XML saying whether to use it or not, or alternatively have some indicator associated with the next versioned machine type that we can hook off. 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 :|