From: Alexander Graf <agraf@suse.de>
To: Pekka Enberg <penberg@kernel.org>
Cc: "Avi Kivity" <avi@redhat.com>,
"Linus Torvalds" <torvalds@linux-foundation.org>,
"Ingo Molnar" <mingo@elte.hu>,
"linux-kernel@vger.kernel.org List"
<linux-kernel@vger.kernel.org>,
"kvm@vger.kernel.org list" <kvm@vger.kernel.org>,
"qemu-devel Developers" <qemu-devel@nongnu.org>,
"Américo Wang" <xiyou.wangcong@gmail.com>,
"Blue Swirl" <blauwirbel@gmail.com>
Subject: Re: [PATCH] KVM: Add wrapper script around QEMU to test kernels
Date: Sun, 6 Nov 2011 09:15:58 -0800 [thread overview]
Message-ID: <877C82F4-F07C-44AA-8722-3AF57CFC4597@suse.de> (raw)
In-Reply-To: <CAOJsxLFpNG1iZt8=XU2xH3x9dFCqjM=+i8y4EMPg0Ah47bYfqg@mail.gmail.com>
On 06.11.2011, at 05:06, Pekka Enberg wrote:
> On Sun, Nov 6, 2011 at 2:43 PM, Avi Kivity <avi@redhat.com> wrote:
>> You say that kvm-tool's scope is broader than Alex's script, therefore
>> the latter is pointless.
>
> I'm saying that Alex's script is pointless because it's not attempting
> to fix the real issues. For example, we're trying to make make it as
> easy as possible to setup a guest and to be able to access guest data
> from the host. Alex's script is essentially just a simplified QEMU
> "front end" for kernel developers.
>
> That's why I feel it's a pointless thing to do.
It's a script tailored to what Linus told me he wanted to see. I merely wanted to prove the point that what he wanted can be achieved without thousands and thousands of lines of code by reusing what is already there. IMHO less code is usually a good thing.
In fact, why don't you just provide a script in tools/testing/ that fetches KVM Tool from a git tree somewhere else and compiles it? It could easily live outside the kernel tree - you can even grab our awesome "fetch all Linux headers" script from QEMU so you can keep in sync with KVM header files.
At that point, both front ends would live in separate trees, could evolve however they like and everyone's happy, because KVM Tools would still be easy to use for people who want it by executing said shell script.
>
> On Sun, Nov 6, 2011 at 2:43 PM, Avi Kivity <avi@redhat.com> wrote:
>> You accept that qemu's scope is broader than kvm-tool (and is a
>> superset). That is why many people think kvm-tool is pointless.
>
> Sure. I think it's mostly people that are interested in non-Linux
> virtualization that think the KVM tool is a pointless project.
> However, some people (including myself) think the KVM tool is a more
> usable and hackable tool than QEMU for Linux virtualization.
Sure. That's taste. If I think that tcsh is a better shell than bash do I pull it into the kernel tree just so "it lies there"? It definitely does use kernel interfaces too, so I can make up just as many reasons as you to pull it in.
> The difference here is that although I feel Alex's script is a
> pointless project, I'm in no way opposed to merging it in the tree if
> people use it and it solves their problem. Some people seem to be
> violently opposed to merging the KVM tool and I'm having difficult
> time understanding why that is.
It's a matter of size and scope. Write a shell script that clones, builds and executes KVM Tool and throw it in testing/tools/ and I'll happily ack it!
Alex
next prev parent reply other threads:[~2011-11-06 17:16 UTC|newest]
Thread overview: 171+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-06 1:35 Alexander Graf
2011-11-06 1:14 ` [Qemu-devel] " Andreas Färber
2011-11-06 10:04 ` Pekka Enberg
2011-11-06 10:07 ` Avi Kivity
2011-11-06 10:12 ` Pekka Enberg
2011-11-06 10:23 ` Avi Kivity
2011-11-06 11:08 ` Pekka Enberg
2011-11-06 11:50 ` Avi Kivity
2011-11-06 12:14 ` Pekka Enberg
2011-11-06 12:27 ` Avi Kivity
2011-11-06 12:32 ` Pekka Enberg
2011-11-06 12:43 ` Avi Kivity
2011-11-06 13:06 ` Pekka Enberg
2011-11-06 15:56 ` Avi Kivity
2011-11-06 16:35 ` Pekka Enberg
2011-11-06 16:50 ` Avi Kivity
2011-11-06 17:08 ` [Qemu-devel] " Anthony Liguori
2011-11-06 18:09 ` Pekka Enberg
2011-11-07 1:38 ` Anthony Liguori
2011-11-07 6:45 ` Pekka Enberg
2011-11-06 18:31 ` Ted Ts'o
2011-11-06 18:54 ` Pekka Enberg
2011-11-06 18:58 ` Pekka Enberg
2011-11-06 23:19 ` Ted Ts'o
2011-11-07 6:42 ` Pekka Enberg
2011-11-07 17:03 ` Vince Weaver
2011-11-07 17:59 ` Ingo Molnar
2011-11-07 20:03 ` Frank Ch. Eigler
2011-11-07 20:09 ` Pekka Enberg
2011-11-07 20:35 ` Ted Ts'o
2011-11-08 10:22 ` [F.A.Q.] perf ABI backwards and forwards compatibility Ingo Molnar
2011-11-08 10:32 ` Peter Zijlstra
2011-11-08 11:34 ` Ingo Molnar
2011-11-08 10:41 ` Theodore Tso
2011-11-08 11:20 ` Pekka Enberg
2011-11-08 11:25 ` Theodore Tso
2011-11-08 11:29 ` Pekka Enberg
2011-11-08 11:31 ` Frank Ch. Eigler
2011-11-08 11:39 ` Pekka Enberg
2011-11-08 12:15 ` Ingo Molnar
2011-11-08 12:20 ` Peter Zijlstra
2011-11-08 12:59 ` Ingo Molnar
2011-11-09 10:05 ` Peter Zijlstra
2011-11-08 5:29 ` [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels Vince Weaver
2011-11-08 12:07 ` Ingo Molnar
2011-11-08 13:08 ` Arnaldo Carvalho de Melo
2011-11-09 6:04 ` Vince Weaver
2011-11-07 19:53 ` Pekka Enberg
2011-11-07 20:32 ` Ted Ts'o
2011-11-07 21:36 ` Pekka Enberg
2011-11-07 22:19 ` Anthony Liguori
2011-11-07 23:42 ` Theodore Tso
2011-11-08 9:32 ` [F.A.Q.] the advantages of a shared tool/kernel Git repository, tools/perf/ and tools/kvm/ Ingo Molnar
2011-11-08 10:21 ` Theodore Tso
2011-11-08 12:55 ` Ingo Molnar
2011-11-08 16:33 ` Ted Ts'o
2011-11-08 17:14 ` Anca Emanuel
2011-11-08 19:24 ` Ted Ts'o
2011-11-09 8:28 ` Ingo Molnar
2011-11-08 21:15 ` John Kacur
2011-11-09 8:38 ` Ingo Molnar
2011-11-09 8:23 ` Ingo Molnar
2011-11-10 1:41 ` Alexander Graf
2011-11-10 8:14 ` Ingo Molnar
2011-11-08 12:56 ` Arnaldo Carvalho de Melo
2011-11-08 13:40 ` Gerd Hoffmann
2011-11-08 14:32 ` Arnaldo Carvalho de Melo
2011-11-08 15:38 ` Gerd Hoffmann
2011-11-08 16:13 ` Arnaldo Carvalho de Melo
2011-11-09 8:55 ` Ingo Molnar
2011-11-09 8:51 ` Ingo Molnar
2011-11-09 10:40 ` Gerd Hoffmann
2011-11-09 10:50 ` Hagen Paul Pfeifer
2011-11-09 11:55 ` Arnaldo Carvalho de Melo
2011-11-09 12:26 ` Gerd Hoffmann
2011-11-09 12:30 ` Arnaldo Carvalho de Melo
2011-11-09 12:33 ` Arnaldo Carvalho de Melo
2011-11-09 12:46 ` Peter Zijlstra
2011-11-09 12:51 ` Arnaldo Carvalho de Melo
2011-11-09 13:17 ` Ingo Molnar
2011-11-09 19:25 ` Jim Paris
2011-11-09 20:13 ` Arnaldo Carvalho de Melo
2011-11-09 22:32 ` Anca Emanuel
2011-11-10 8:00 ` Ingo Molnar
2011-11-10 8:12 ` Anca Emanuel
2011-11-10 8:39 ` Gerd Hoffmann
2011-11-08 15:43 ` Steven Rostedt
2011-11-09 9:21 ` Ingo Molnar
2011-11-09 12:03 ` Arnaldo Carvalho de Melo
2011-11-09 13:40 ` Américo Wang
2011-11-10 7:47 ` Ingo Molnar
2011-11-07 10:31 ` [Qemu-devel] [PATCH] KVM: Add wrapper script around QEMU to test kernels Kevin Wolf
2011-11-07 11:38 ` Pekka Enberg
2011-11-07 11:59 ` Kevin Wolf
2011-11-06 16:19 ` Jan Kiszka
2011-11-06 16:30 ` Pekka Enberg
2011-11-06 16:39 ` Jan Kiszka
2011-11-06 17:11 ` Pekka Enberg
2011-11-06 17:23 ` Jan Kiszka
2011-11-06 17:55 ` Pekka Enberg
2011-11-06 16:39 ` Pekka Enberg
2011-11-07 10:11 ` Gerd Hoffmann
2011-11-07 10:18 ` Pekka Enberg
2011-11-06 17:10 ` [Qemu-devel] " Anthony Liguori
2011-11-06 17:15 ` Alexander Graf [this message]
2011-11-06 17:28 ` Pekka Enberg
2011-11-06 17:30 ` Alexander Graf
2011-11-06 18:05 ` Pekka Enberg
2011-11-06 19:14 ` Paolo Bonzini
2011-11-06 19:19 ` Pekka Enberg
2011-11-06 22:08 ` Frank Ch. Eigler
2011-11-07 6:58 ` Pekka Enberg
2011-11-06 19:11 ` Paolo Bonzini
2011-11-06 19:17 ` Pekka Enberg
2011-11-06 20:01 ` Paolo Bonzini
2011-11-06 20:17 ` Pekka Enberg
2011-11-07 8:00 ` Paolo Bonzini
2011-11-07 8:09 ` Pekka Enberg
2011-11-07 8:20 ` Paolo Bonzini
2011-11-07 8:45 ` Pekka Enberg
2011-11-07 8:52 ` Paolo Bonzini
2011-11-07 8:57 ` Pekka Enberg
2011-11-07 8:13 ` Pekka Enberg
2011-11-06 20:31 ` Pekka Enberg
2011-11-07 10:23 ` Gerd Hoffmann
2011-11-07 10:30 ` [Qemu-devel] " Sasha Levin
2011-11-07 11:02 ` Paolo Bonzini
2011-11-07 11:44 ` Pekka Enberg
2011-11-07 12:18 ` Gerd Hoffmann
2011-11-07 12:21 ` Pekka Enberg
2011-11-07 12:26 ` [Qemu-devel] " Avi Kivity
2011-11-07 12:29 ` Pekka Enberg
2011-11-07 12:43 ` Ted Ts'o
2011-11-07 12:44 ` Avi Kivity
2011-11-07 11:34 ` Pekka Enberg
2011-11-07 11:57 ` Ingo Molnar
2011-11-07 13:17 ` [Qemu-devel] " Anthony Liguori
2011-11-07 12:08 ` Gerd Hoffmann
2011-11-07 12:29 ` Ted Ts'o
2011-11-07 12:42 ` Pekka Enberg
2011-11-07 12:47 ` Ted Ts'o
2011-11-07 12:59 ` Pekka Enberg
2011-11-07 13:12 ` Pekka Enberg
2011-11-08 13:29 ` Karel Zak
2011-11-08 14:30 ` Pekka Enberg
2011-11-06 13:11 ` Pekka Enberg
2011-11-06 12:27 ` Pekka Enberg
2011-11-08 14:41 ` Avi Kivity
2011-11-08 14:52 ` Christoph Hellwig
2011-11-08 14:55 ` Sasha Levin
2011-11-08 14:57 ` Avi Kivity
2011-11-08 14:59 ` Christoph Hellwig
2011-11-08 17:34 ` Alexander Graf
2011-11-08 17:36 ` Avi Kivity
2011-11-08 15:04 ` Jan Kiszka
2011-11-08 15:26 ` Pekka Enberg
2011-11-08 15:28 ` Christoph Hellwig
-- strict thread matches above, loose matches on Subject: below --
2012-05-11 15:46 Alexander Graf
2011-08-24 21:38 Alexander Graf
2011-11-06 13:54 ` Jan Kiszka
2012-05-11 13:42 ` Alexander Graf
2012-05-11 14:05 ` Jan Kiszka
2011-08-23 22:16 [PATCH] KVM: Add wrapper script around Qemu " Alexander Graf
2011-08-24 5:19 ` Pekka Enberg
2011-08-24 5:31 ` Américo Wang
2011-08-24 20:35 ` Alexander Graf
2011-08-25 3:44 ` Américo Wang
2011-11-05 23:47 ` Alexander Graf
2011-08-24 8:25 ` Avi Kivity
2011-08-24 9:16 ` Jan Kiszka
2011-08-24 21:06 ` Alexander Graf
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=877C82F4-F07C-44AA-8722-3AF57CFC4597@suse.de \
--to=agraf@suse.de \
--cc=avi@redhat.com \
--cc=blauwirbel@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=penberg@kernel.org \
--cc=qemu-devel@nongnu.org \
--cc=torvalds@linux-foundation.org \
--cc=xiyou.wangcong@gmail.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®