From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: qemu-kvm: remove "boot=on|off" drive parameter compatibility Date: Mon, 1 Oct 2012 10:31:44 -0300 Message-ID: <20121001133144.GA4466@amt.cnet> References: <20120930191146.GA20012@amt.cnet> <50694EC1.8060006@siemens.com> <20121001093102.GA14797@amt.cnet> <50696E9E.7030302@siemens.com> <87zk468h3y.fsf@codemonkey.ws> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm , Scott Moser , Jan Kiszka , Michael Tokarev , qemu-devel , Cole Robinson , Andreas =?iso-8859-1?Q?F=E4rber?= To: Anthony Liguori Return-path: Content-Disposition: inline In-Reply-To: <87zk468h3y.fsf@codemonkey.ws> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On Mon, Oct 01, 2012 at 08:19:29AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > > On 2012-10-01 11:31, Marcelo Tosatti wrote: > > > > It's not just about default configs. We need to validate if the > > migration formats are truly compatible (qemu-kvm -> QEMU, the other way > > around definitely not). For the command line switches, we could provide > > a wrapper script that translates them into upstream format or simply > > ignores them. That should be harmless to carry upstream. > > qemu-kvm has: > > -no-kvm > -no-kvm-irqchip > -no-kvm-pit > -no-kvm-pit-reinjection > -tdf <- does nothing > > There are replacements for all of the above. If we need to add them to > qemu.git, it's not big deal to add them. At the moment the only purpose of this command line options is for compability with scripts. My view is that scripts are easily fixed, so we can just drop them. No need to carry this to QEMU. > -drive ...,boot= <- this is ignored > > cpu_set command for CPU hotplug which is known broken in qemu-kvm. > > testdev which is nice but only used for development > > Default nic is rtl8139 vs. e1000. Config file (as suggested earlier on this thread). > Some logic to move change the default VGA ram size to 16mb for pc-1.2 > (QEMU uses 16mb by default now too). Config file. > I think at this point, none of this matters but I added the various > distro maintainers to the thread. > > I think it's time for the distros to drop qemu-kvm and just ship > qemu.git. Is there anything else that needs to happen to make that > switch? > > Regards, > > Anthony Liguori > > > > > But I would really stop worrying about the qemu-kvm code base. > > > > Jan > > > > -- > > Siemens AG, Corporate Technology, CT RTC ITP SDP-DE > > Corporate Competence Center Embedded Linux > > -- > > To unsubscribe from this list: send the line "unsubscribe kvm" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:47490) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TIg6N-00061t-IE for qemu-devel@nongnu.org; Mon, 01 Oct 2012 09:32:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TIg6J-0005t4-7R for qemu-devel@nongnu.org; Mon, 01 Oct 2012 09:32:03 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56200) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TIg6I-0005su-Ur for qemu-devel@nongnu.org; Mon, 01 Oct 2012 09:31:59 -0400 Date: Mon, 1 Oct 2012 10:31:44 -0300 From: Marcelo Tosatti Message-ID: <20121001133144.GA4466@amt.cnet> References: <20120930191146.GA20012@amt.cnet> <50694EC1.8060006@siemens.com> <20121001093102.GA14797@amt.cnet> <50696E9E.7030302@siemens.com> <87zk468h3y.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87zk468h3y.fsf@codemonkey.ws> Subject: Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: kvm , Scott Moser , Jan Kiszka , Michael Tokarev , qemu-devel , Cole Robinson , Andreas =?iso-8859-1?Q?F=E4rber?= On Mon, Oct 01, 2012 at 08:19:29AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > > On 2012-10-01 11:31, Marcelo Tosatti wrote: > > > > It's not just about default configs. We need to validate if the > > migration formats are truly compatible (qemu-kvm -> QEMU, the other way > > around definitely not). For the command line switches, we could provide > > a wrapper script that translates them into upstream format or simply > > ignores them. That should be harmless to carry upstream. > > qemu-kvm has: > > -no-kvm > -no-kvm-irqchip > -no-kvm-pit > -no-kvm-pit-reinjection > -tdf <- does nothing > > There are replacements for all of the above. If we need to add them to > qemu.git, it's not big deal to add them. At the moment the only purpose of this command line options is for compability with scripts. My view is that scripts are easily fixed, so we can just drop them. No need to carry this to QEMU. > -drive ...,boot= <- this is ignored > > cpu_set command for CPU hotplug which is known broken in qemu-kvm. > > testdev which is nice but only used for development > > Default nic is rtl8139 vs. e1000. Config file (as suggested earlier on this thread). > Some logic to move change the default VGA ram size to 16mb for pc-1.2 > (QEMU uses 16mb by default now too). Config file. > I think at this point, none of this matters but I added the various > distro maintainers to the thread. > > I think it's time for the distros to drop qemu-kvm and just ship > qemu.git. Is there anything else that needs to happen to make that > switch? > > Regards, > > Anthony Liguori > > > > > But I would really stop worrying about the qemu-kvm code base. > > > > Jan > > > > -- > > Siemens AG, Corporate Technology, CT RTC ITP SDP-DE > > Corporate Competence Center Embedded Linux > > -- > > To unsubscribe from this list: send the line "unsubscribe kvm" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html