Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-23 Thread Gleb Natapov
On Tue, Feb 23, 2010 at 07:50:16AM -0600, Anthony Liguori wrote: > On 02/23/2010 02:22 AM, Gleb Natapov wrote: > >On Mon, Feb 22, 2010 at 08:31:00PM -0500, Kevin O'Connor wrote: > >>On Mon, Feb 22, 2010 at 10:33:12AM +0200, Gleb Natapov wrote: > >>>On Sun, Feb 21, 2010 at 02:13:51PM -0500, Kevin O'

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-23 Thread Anthony Liguori
On 02/23/2010 02:22 AM, Gleb Natapov wrote: On Mon, Feb 22, 2010 at 08:31:00PM -0500, Kevin O'Connor wrote: On Mon, Feb 22, 2010 at 10:33:12AM +0200, Gleb Natapov wrote: On Sun, Feb 21, 2010 at 02:13:51PM -0500, Kevin O'Connor wrote: Are you thinking of moving qemu more torwa

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-23 Thread Gleb Natapov
On Mon, Feb 22, 2010 at 08:31:00PM -0500, Kevin O'Connor wrote: > On Mon, Feb 22, 2010 at 10:33:12AM +0200, Gleb Natapov wrote: > > On Sun, Feb 21, 2010 at 02:13:51PM -0500, Kevin O'Connor wrote: > > > Are you thinking of moving qemu more torwards what coreboot does, or > > > did you have a differe

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-22 Thread Kevin O'Connor
On Mon, Feb 22, 2010 at 10:33:12AM +0200, Gleb Natapov wrote: > On Sun, Feb 21, 2010 at 02:13:51PM -0500, Kevin O'Connor wrote: > > Are you thinking of moving qemu more torwards what coreboot does, or > > did you have a different idea in mind? > > > We shouldn't compare coreboot with qemu. Qemu is

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-22 Thread Avi Kivity
On 02/21/2010 09:13 PM, Kevin O'Connor wrote: I'd like to have QEMU handle more, I picked the TSS page because we changed the location of that in the past and it was the one that triggered my patch in the first place. Now we have the infrastructure, it will be easier to add more. What pa

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-22 Thread Gleb Natapov
On Sun, Feb 21, 2010 at 02:13:51PM -0500, Kevin O'Connor wrote: > On Sun, Feb 21, 2010 at 06:44:55PM +0100, Jes Sorensen wrote: > > On 02/19/10 22:02, Anthony Liguori wrote: > > >I noticed that you use this for the TSS page with EPT but you don't use > > >this interface for the rest of memory. I'm

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-21 Thread Kevin O'Connor
On Sun, Feb 21, 2010 at 06:44:55PM +0100, Jes Sorensen wrote: > On 02/19/10 22:02, Anthony Liguori wrote: > >I noticed that you use this for the TSS page with EPT but you don't use > >this interface for the rest of memory. I'm curious what you think the > >right long term split is? If QEMU is not m

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-21 Thread Jes Sorensen
On 02/19/10 22:02, Anthony Liguori wrote: Acked-by: Anthony Liguori Would be nice to use git-send-email in the future as your attachments are often rendered poorly by mail clients. Just switching over from quilt to git, and still having issues with git-send-email. It does some things nicely,

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-19 Thread Anthony Liguori
On 02/15/2010 11:33 AM, Jes Sorensen wrote: Hi, Kevin and I have agreed on the approach for this one now. So here is the latest version of the patch for QEMU, submitting e820 reservation entries via fw_cfg. Cheers, Jes Applied. Thanks. Regards, Anthony Liguori

Re: [Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-19 Thread Anthony Liguori
On 02/15/2010 11:33 AM, Jes Sorensen wrote: Hi, Kevin and I have agreed on the approach for this one now. So here is the latest version of the patch for QEMU, submitting e820 reservation entries via fw_cfg. Cheers, Jes Acked-by: Anthony Liguori Would be nice to use git-send-email in the fut

[Qemu-devel] [PATCH] QEMU e820 reservation patch

2010-02-15 Thread Jes Sorensen
Hi, Kevin and I have agreed on the approach for this one now. So here is the latest version of the patch for QEMU, submitting e820 reservation entries via fw_cfg. Cheers, Jes Use qemu-cfg to provide the BIOS with an optional table of e820 entries. Notify the BIOS of the location of the TSS+EPT