Re: [RFC/PATCH 1/2] kvm tools, seabios: Add "--bios" option to "vm run"

2012-02-24 Thread ron minnich
I think you need to look at the change floren and I worked up,
assuming I can get him to release it to you.

Our change is a bit less complex and adds fewer lines of code. You've
added an extra option to load a bios, and it will be somewhat limited
as those ever-demanding users continue to demand more and more :-)

What we did instead is let users specify the load address of the
kernel, and the IP from which to start. So our command line ends up
with --kernel being a seabios image, and using switches to set the
load address at e and the initial IP at 0. Of course, now that
I know the memory is "reflected" to high memory too, the initial IP
could just as easily be fff0.

Anyway, John, can you give us a look at a patch please?

ron
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 1/2] kvm tools, seabios: Add "--bios" option to "vm run"

2012-02-24 Thread ron minnich
On Fri, Feb 24, 2012 at 10:00 AM, Pekka Enberg  wrote:

> I personally want a simple command line option that does everything for
> me automatically.

it's a great goal, and I like the swtich a lot. But if you want that
-firmware option why not have it turn into an automated setting of the
options that John's got in his patch? It really is less code overall
and I love the fact that kvm tool is so tiny, I'm anxious to keep it
that way :-)

ron
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 1/2] kvm tools, seabios: Add "--bios" option to "vm run"

2012-02-27 Thread ron minnich
On Mon, Feb 27, 2012 at 3:15 AM, Gleb Natapov  wrote:

> AFAIK kvm-tool initialize PCI by itself so it actually may work (and,
> very likely, the best thing to do).

we're going in circles. kvm-tool does not set the base address of some
of the BARs and seabios does not do it quite correctly. That's what
we're trying to decide how to fix. Probably the fastest way out is
just run coreboot+seabios for kvm-tool.

ron
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 1/2] kvm tools, seabios: Add "--bios" option to "vm run"

2012-02-27 Thread ron minnich
On Mon, Feb 27, 2012 at 8:38 AM, Gleb Natapov  wrote:

> Then make kvm-tool set it. Why do you need coreboot/seabios for that?

First thing we looked at. For a number of reasons it seems ugly.
Either we can pick a bunch of fixed base addresses for the kvm-tool
resources, and track them all as new devices are added to kvm-tool
over time (yuck), or we can replicate some of the pci BAR code in
kvm-tool that already exists in seabios/coreboot, which also seems
yucky.

kvm-tool provides a more realistic environment in some ways for guests
than qemu. Less is initialized. Given that coreboot does the things we
need done, and it's easy to build, I don't see a reason not to use it.

Of course due to the ongoing ld alignment bug issue, I can't build
seabios to test this idea, so have not. If I had time I'd put in a
patch to seabios to not depend on this feature, since it's broken on
every linux system I own, but ...

ron
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 1/2] kvm tools, seabios: Add "--bios" option to "vm run"

2012-02-27 Thread ron minnich
I'm less interested in this argument than getting something that
works, however it happens, so I'll let it stop with the comment that i
don't agree with you :-)

thanks

ron
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html