From: Jan Kiszka <jan.kiszka@siemens.com> To: Anthony Liguori <anthony@codemonkey.ws> Cc: "Marcelo Tosatti" <mtosatti@redhat.com>, kvm <kvm@vger.kernel.org>, qemu-devel <qemu-devel@nongnu.org>, "Cole Robinson" <crobinso@redhat.com>, "Scott Moser" <smoser@ubuntu.com>, "Andreas Färber" <afaerber@suse.de>, "Michael Tokarev" <mjt@tls.msk.ru> Subject: Re: qemu-kvm: remove "boot=on|off" drive parameter compatibility Date: Mon, 01 Oct 2012 15:26:05 +0200 [thread overview] Message-ID: <506999ED.6010409@siemens.com> (raw) In-Reply-To: <87zk468h3y.fsf@codemonkey.ws> On 2012-10-01 15:19, Anthony Liguori wrote: > Jan Kiszka <jan.kiszka@siemens.com> 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. But I don't think we should add them to the source code. This can perfectly be handled my a (disposable) script layer on top of qemu-system-x86_64 - the namespace (qemu-kvm in most cases) is also free. > > -drive ...,boot= <- this is ignored > > cpu_set command for CPU hotplug which is known broken in qemu-kvm. Right, so nothing is lost when migrating to QEMU. > > testdev which is nice but only used for development > > Default nic is rtl8139 vs. e1000. > > Some logic to move change the default VGA ram size to 16mb for pc-1.2 > (QEMU uses 16mb by default now too). Also nicely manageable in a wrapper. > > 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. +1 Jan > Is there anything else that needs to happen to make that > switch? > > Regards, > > Anthony Liguori -- Siemens AG, Corporate Technology, CT RTC ITP SDP-DE Corporate Competence Center Embedded Linux
WARNING: multiple messages have this Message-ID (diff)
From: Jan Kiszka <jan.kiszka@siemens.com> To: Anthony Liguori <anthony@codemonkey.ws> Cc: kvm <kvm@vger.kernel.org>, "Scott Moser" <smoser@ubuntu.com>, "Marcelo Tosatti" <mtosatti@redhat.com>, "Michael Tokarev" <mjt@tls.msk.ru>, qemu-devel <qemu-devel@nongnu.org>, "Cole Robinson" <crobinso@redhat.com>, "Andreas Färber" <afaerber@suse.de> Subject: Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility Date: Mon, 01 Oct 2012 15:26:05 +0200 [thread overview] Message-ID: <506999ED.6010409@siemens.com> (raw) In-Reply-To: <87zk468h3y.fsf@codemonkey.ws> On 2012-10-01 15:19, Anthony Liguori wrote: > Jan Kiszka <jan.kiszka@siemens.com> 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. But I don't think we should add them to the source code. This can perfectly be handled my a (disposable) script layer on top of qemu-system-x86_64 - the namespace (qemu-kvm in most cases) is also free. > > -drive ...,boot= <- this is ignored > > cpu_set command for CPU hotplug which is known broken in qemu-kvm. Right, so nothing is lost when migrating to QEMU. > > testdev which is nice but only used for development > > Default nic is rtl8139 vs. e1000. > > Some logic to move change the default VGA ram size to 16mb for pc-1.2 > (QEMU uses 16mb by default now too). Also nicely manageable in a wrapper. > > 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. +1 Jan > Is there anything else that needs to happen to make that > switch? > > Regards, > > Anthony Liguori -- Siemens AG, Corporate Technology, CT RTC ITP SDP-DE Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2012-10-01 13:26 UTC|newest] Thread overview: 97+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-09-30 19:11 qemu-kvm: remove "boot=on|off" drive parameter compatibility Marcelo Tosatti 2012-10-01 1:02 ` Amos Kong 2012-10-01 8:05 ` Jan Kiszka 2012-10-01 9:31 ` Marcelo Tosatti 2012-10-01 10:21 ` Jan Kiszka 2012-10-01 13:03 ` Marcelo Tosatti 2012-10-01 13:14 ` Jan Kiszka 2012-10-01 13:19 ` Anthony Liguori 2012-10-01 13:19 ` [Qemu-devel] " Anthony Liguori 2012-10-01 13:26 ` Jan Kiszka [this message] 2012-10-01 13:26 ` Jan Kiszka 2012-10-03 9:55 ` Gleb Natapov 2012-10-03 9:55 ` Gleb Natapov 2012-10-03 10:06 ` Jan Kiszka 2012-10-03 10:06 ` Jan Kiszka 2012-10-03 10:11 ` Gleb Natapov 2012-10-03 10:11 ` Gleb Natapov 2012-10-03 10:57 ` Lucas Meneghel Rodrigues 2012-10-03 10:57 ` [Qemu-devel] " Lucas Meneghel Rodrigues 2012-10-03 13:19 ` Paolo Bonzini 2012-10-03 13:19 ` [Qemu-devel] " Paolo Bonzini 2012-10-03 13:27 ` Gleb Natapov 2012-10-03 13:27 ` [Qemu-devel] " Gleb Natapov 2012-10-04 10:48 ` Jan Kiszka 2012-10-04 10:48 ` [Qemu-devel] " Jan Kiszka 2012-10-04 12:10 ` Lucas Meneghel Rodrigues 2012-10-04 12:10 ` [Qemu-devel] " Lucas Meneghel Rodrigues 2012-10-04 12:27 ` Jan Kiszka 2012-10-04 12:27 ` [Qemu-devel] " Jan Kiszka 2012-10-04 17:21 ` Lucas Meneghel Rodrigues 2012-10-04 17:21 ` [Qemu-devel] " Lucas Meneghel Rodrigues 2012-10-04 17:24 ` Jan Kiszka 2012-10-04 17:24 ` [Qemu-devel] " Jan Kiszka 2012-10-01 13:31 ` Marcelo Tosatti 2012-10-01 13:31 ` [Qemu-devel] " Marcelo Tosatti 2012-10-01 13:36 ` Jan Kiszka 2012-10-01 13:36 ` [Qemu-devel] " Jan Kiszka 2012-10-01 13:44 ` Michael Tokarev 2012-10-01 13:44 ` [Qemu-devel] " Michael Tokarev 2012-10-01 13:48 ` Jan Kiszka 2012-10-01 13:48 ` [Qemu-devel] " Jan Kiszka 2012-10-01 13:38 ` Paolo Bonzini 2012-10-01 13:38 ` [Qemu-devel] " Paolo Bonzini 2012-10-01 13:46 ` Jan Kiszka 2012-10-01 13:46 ` [Qemu-devel] " Jan Kiszka 2012-10-01 14:04 ` Anthony Liguori 2012-10-01 14:04 ` [Qemu-devel] " Anthony Liguori 2012-10-01 14:34 ` [PATCH] kvm: Set default accelerator to "kvm" if the host supports it Jan Kiszka 2012-10-01 14:34 ` [Qemu-devel] " Jan Kiszka 2012-10-01 16:20 ` Anthony Liguori 2012-10-01 16:20 ` [Qemu-devel] " Anthony Liguori 2012-10-01 16:56 ` Aurelien Jarno 2012-10-01 16:56 ` [Qemu-devel] " Aurelien Jarno 2012-10-03 9:02 ` Jan Kiszka 2012-10-03 9:02 ` [Qemu-devel] " Jan Kiszka 2012-10-03 20:01 ` Blue Swirl 2012-10-03 20:01 ` [Qemu-devel] " Blue Swirl 2012-10-03 20:26 ` Peter Maydell 2012-10-03 20:26 ` Peter Maydell 2012-10-05 1:15 ` Alexander Graf 2012-10-05 1:15 ` Alexander Graf 2012-10-05 2:17 ` Anthony Liguori 2012-10-05 2:17 ` [Qemu-devel] " Anthony Liguori 2012-10-05 2:24 ` Alexander Graf 2012-10-05 2:24 ` Alexander Graf 2012-10-05 8:15 ` Peter Maydell 2012-10-05 8:15 ` Peter Maydell 2012-10-08 14:03 ` Andreas Färber 2012-10-08 14:03 ` Andreas Färber 2012-10-08 14:08 ` Alexander Graf 2012-10-08 14:08 ` Alexander Graf 2012-10-01 16:43 ` Andreas Färber 2012-10-01 16:43 ` Andreas Färber 2012-10-01 16:47 ` Daniel P. Berrange 2012-10-01 16:47 ` Daniel P. Berrange 2012-10-01 19:03 ` Anthony Liguori 2012-10-01 19:03 ` Anthony Liguori 2012-10-01 19:25 ` Paolo Bonzini 2012-10-01 19:25 ` [Qemu-devel] " Paolo Bonzini 2012-10-01 20:07 ` Anthony Liguori 2012-10-01 20:07 ` Anthony Liguori 2012-10-02 7:46 ` Markus Armbruster 2012-10-02 7:46 ` [Qemu-devel] " Markus Armbruster 2012-10-02 8:15 ` Aurelien Jarno 2012-10-02 8:15 ` Aurelien Jarno 2012-10-03 6:58 ` Michael Tokarev 2012-10-03 6:58 ` Michael Tokarev 2012-10-03 9:05 ` Jan Kiszka 2012-10-03 9:05 ` [Qemu-devel] " Jan Kiszka 2012-10-01 14:07 ` qemu-kvm: remove "boot=on|off" drive parameter compatibility Alexander Graf 2012-10-01 14:07 ` [Qemu-devel] " Alexander Graf 2012-10-01 14:20 ` Paolo Bonzini 2012-10-01 14:20 ` [Qemu-devel] " Paolo Bonzini 2012-10-01 15:39 ` Bruce Rogers 2012-10-01 15:39 ` Bruce Rogers 2012-10-02 8:14 ` Daniel P. Berrange 2012-10-02 8:14 ` [Qemu-devel] " Daniel P. Berrange
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=506999ED.6010409@siemens.com \ --to=jan.kiszka@siemens.com \ --cc=afaerber@suse.de \ --cc=anthony@codemonkey.ws \ --cc=crobinso@redhat.com \ --cc=kvm@vger.kernel.org \ --cc=mjt@tls.msk.ru \ --cc=mtosatti@redhat.com \ --cc=qemu-devel@nongnu.org \ --cc=smoser@ubuntu.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.