Re: MTRR setup in OVMF [was: PATCH v3 01/10 KVM: MMU: fix decoding cache type from MTRR]

2015-07-14 Thread Jordan Justen
On 2015-07-14 14:29:11, Laszlo Ersek wrote: > On 07/14/15 23:15, Paolo Bonzini wrote: > >> The long delay that Alex reported (for the case when all guest memory > >> was set to UC up-front) is due to the fact that the SEC phase of OVMF > >> decompresses an approximately 1712 KB sized, LZMA-compress

Re: KVM call agenda for 2013-05-28

2013-06-03 Thread Jordan Justen
On Sun, Jun 2, 2013 at 2:43 AM, Michael S. Tsirkin wrote: > On Fri, May 31, 2013 at 01:45:55PM +0200, Laszlo Ersek wrote: >> On 05/31/13 09:09, Jordan Justen wrote: >> >> > Why is updating the ACPI tables in seabios viewed as such a burden? >> > Either qemu does

Re: [PATCH uq/master] fix double free the memslot in kvm_set_phys_mem

2013-06-02 Thread Jordan Justen
Reviewed-by: Jordan Justen On Fri, May 31, 2013 at 1:52 AM, Xiao Guangrong wrote: > Luiz Capitulino reported that guest refused to boot and qemu > complained with: > kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument > > It is caused by commit 235e8982ad

Re: KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Fri, May 31, 2013 at 5:01 PM, Laszlo Ersek wrote: > On 05/31/13 23:03, Jordan Justen wrote: > >> Of course, the fact that the current FAT driver is exclusionary for >> free software projects is a point that is not lost on me. I just don't >> agree that the best re

Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Fri, May 31, 2013 at 2:32 AM, Gerd Hoffmann wrote: > Hi, > >> I guess -bios would load coreboot. Coreboot would siphon the data >> necessary for ACPI table building through the current (same) fw_cfg >> bottleneck, build the tables, > > Yes. So, this is really about making coreboot+seabios th

Re: KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Fri, May 31, 2013 at 1:27 PM, Anthony Liguori wrote: > Jordan Justen writes: > >> On Fri, May 31, 2013 at 7:38 AM, Anthony Liguori >> wrote: >>> In terms of creating a FAT module, the most likely source would seem to >>> be the kernel code and since tha

Re: KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Fri, May 31, 2013 at 11:35 AM, Anthony Liguori wrote: > As I think more about it, I think forking edk2 is inevitable. We need a > clean repo that doesn't include the proprietary binaries. I doubt > upstream edk2 is willing to remove the binaries. No, probably not unless a BSD licensed altern

Re: KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Fri, May 31, 2013 at 7:38 AM, Anthony Liguori wrote: > In terms of creating a FAT module, the most likely source would seem to > be the kernel code and since that's GPL, I don't think it's terribly > avoidable to end up with a GPL'd uefi implementation. Why would OpenBSD not be a potential sou

Re: KVM call agenda for 2013-05-28

2013-05-31 Thread Jordan Justen
On Thu, May 30, 2013 at 7:34 PM, Kevin O'Connor wrote: > On Tue, May 28, 2013 at 07:53:09PM -0400, Kevin O'Connor wrote: >> There were discussions on potentially introducing a middle component >> to generate the tables. Coreboot was raised as a possibility, and >> David thought it would be okay t

Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28

2013-05-30 Thread Jordan Justen
On Thu, May 30, 2013 at 9:41 AM, Laszlo Ersek wrote: > On 05/30/13 18:20, Jordan Justen wrote: >> I think ACPI table generation lives in firmware on real products, >> because on real products the firmware is the point that best >> understands the actual hardware layout for th

Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28

2013-05-30 Thread Jordan Justen
On Thu, May 30, 2013 at 5:19 AM, David Woodhouse wrote: > On Thu, 2013-05-30 at 13:13 +0200, Laszlo Ersek wrote: >> Where is CorebootPkg available from? > > https://github.com/pgeorgi/edk2/tree/coreboot-pkg Is the license on this actually BSD as the License.txt indicates? Is this planned to be u

[PATCH] kvm: always define KVM_CAP_READONLY_MEM

2013-04-30 Thread Jordan Justen
Don't use #ifdef __KVM_HAVE_READONLY_MEM when defining KVM_CAP_READONLY_MEM. Signed-off-by: Jordan Justen Cc: Xiao Guangrong Cc: Jan Kiszka --- include/uapi/linux/kvm.h |2 -- 1 file changed, 2 deletions(-) diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h index 3c

Re: [Qemu-devel] KVM/QEMU: Support executing from flash device

2012-03-04 Thread Jordan Justen
On Sun, Mar 4, 2012 at 02:46, Avi Kivity wrote: > On 03/04/2012 12:38 PM, Gleb Natapov wrote: >> On Sun, Mar 04, 2012 at 12:36:20PM +0200, Avi Kivity wrote: >> > On 03/04/2012 10:12 AM, Gleb Natapov wrote: >> > > On Sat, Mar 03, 2012 at 10:56:02PM -0800, Jordan Just

KVM/QEMU: Support executing from flash device

2012-03-03 Thread Jordan Justen
On Tue, Oct 25, 2011 at 00:47, Avi Kivity wrote: > The core issue that kvm (the kernel part) supports two styles of memory: > read/write RAM, and read/write MMIO.  ROM wants writes to be ignored, > and rom/device wants reads serviced from memory and writes serviced by > userspace (as MMIO). > > It

Re: [Qemu-devel] How does the QEMU load the binary files bios.bin and vgabios-cirrus.bin?

2011-04-07 Thread Jordan Justen
On Thu, Apr 7, 2011 at 12:24, Olivier Galibert wrote: > On Thu, Apr 07, 2011 at 12:03:41PM -0700, Jordan Justen wrote: >> But, Anthony's point ('there's never really been a pressing need to >> support SMM') is probably the most important here, as I can't

Re: [Qemu-devel] How does the QEMU load the binary files bios.bin and vgabios-cirrus.bin?

2011-04-07 Thread Jordan Justen
On Thu, Apr 7, 2011 at 12:21, Anthony Liguori wrote: > QEMU does the leg work already to support SMM.  It doesn't work with KVM > because we treat SMM as ROM memory and trap read/write access. > > To make it work with KVM, you'd have to hack things around to switch the VGA > space to RAM in order

Re: [Qemu-devel] How does the QEMU load the binary files bios.bin and vgabios-cirrus.bin?

2011-04-07 Thread Jordan Justen
On Thu, Apr 7, 2011 at 11:44, Gleb Natapov wrote: > On Thu, Apr 07, 2011 at 01:37:14PM -0500, Anthony Liguori wrote: >> The CS base starts out at 0xf and IP is 0xfff0.  That gives a >> real address of 0x0.  This is usually a trampoline to somewhere >> else in the space. >> >>>C

Re: [Qemu-devel] Re: [PATCH 2/2] Make sure to enable dirty log tracking for VMware VGA

2009-07-29 Thread Jordan Justen
On Wed, Jul 29, 2009 at 10:15 AM, Anthony Liguori wrote: > Jordan Justen wrote: >> >> On Mon, Jun 8, 2009 at 2:53 PM, Anthony Liguori >> wrote: >> >>> >>> Anthony Liguori wrote: >>> >>>> >>>> This is needed for V

Re: [Qemu-devel] Re: [PATCH 2/2] Make sure to enable dirty log tracking for VMware VGA

2009-07-29 Thread Jordan Justen
On Mon, Jun 8, 2009 at 2:53 PM, Anthony Liguori wrote: > Anthony Liguori wrote: >> >> This is needed for VMware VGA to work properly under KVM. >> >> Signed-off-by: Anthony Liguori >> --- >>  hw/vmware_vga.c |    4 >>  1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/hw/vmw

Re: [PATCH 1/3] Update BIOS INT15-E820 to allow a larger BIOS image

2009-07-28 Thread Jordan Justen
On Tue, Jul 28, 2009 at 7:40 AM, Marcelo Tosatti wrote: > On Sun, Jul 26, 2009 at 05:23:51PM -0700, Jordan Justen wrote: >> The bios will now reserve more memory via the E820 functions. >> >> Note that the standard KVM BIOS will most likely not make use of >> this

[PATCH 3/3] Move EPT identity mapping pages to allow a larger BIOS image

2009-07-26 Thread Jordan Justen
. Signed-off-by: Jordan Justen --- qemu-kvm-x86.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemu-kvm-x86.c b/qemu-kvm-x86.c index 0b47b57..65ba470 100644 --- a/qemu-kvm-x86.c +++ b/qemu-kvm-x86.c @@ -102,7 +102,7 @@ static int kvm_init_identity_map_page(kvm_context_t

[PATCH 2/3] Move TSS pages to allow a larger BIOS image

2009-07-26 Thread Jordan Justen
-off-by: Jordan Justen --- qemu-kvm-x86.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemu-kvm-x86.c b/qemu-kvm-x86.c index 492dbc5..0b47b57 100644 --- a/qemu-kvm-x86.c +++ b/qemu-kvm-x86.c @@ -62,7 +62,7 @@ static int kvm_init_tss(kvm_context_t kvm

[PATCH 1/3] Update BIOS INT15-E820 to allow a larger BIOS image

2009-07-26 Thread Jordan Justen
pages 0xfeffd000-0xfeff - 12KB - TSS Pages 0xff00-0x - 16MB - Max bios.bin Signed-off-by: Jordan Justen --- kvm/bios/rombios.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/kvm/bios/rombios.c b/kvm/bios/rombios.c index 6186199..2d0c153 100644

[PATCH 0/3] Allow larger BIOS image

2009-07-26 Thread Jordan Justen
From: Jordan Justen These changes are similar to my patches sent July 16, except they now are based on Yang Sheng's recent changes to enable a new ioctl for controlling the EPT identity mapping page location. -Jordan Jordan Justen (3): Update BIOS INT15-E820 to allow a larger BIOS

Re: [PATCH 3/3] Update VMX_EPT_IDENTITY_PAGETABLE_ADDR to synchronize with kernel code.

2009-07-16 Thread Jordan Justen
On Thu, 2009-07-16 at 11:18 -0700, Marcelo Tosatti wrote: > On Thu, Jul 16, 2009 at 11:02:22AM -0700, Jordan Justen wrote: > > Although VMX_EPT_IDENTITY_PAGETABLE_ADDR does not appear to be used within > > qemu-kvm, this change mirrors a similar change in the kernel kvm code. >

[PATCH 3/3] Update VMX_EPT_IDENTITY_PAGETABLE_ADDR to synchronize with kernel code.

2009-07-16 Thread Jordan Justen
ree up the 0xff00-0x (16MB) range for use with bios.bin. The KVM kernel change depends upon a change to kvm/bios/rombios.c so the bios INT15-E820 function will properly reserve the new location. Signed-off-by: Jordan Justen --- kvm/include/x86/asm/vmx.h |2 +- 1 files changed, 1

[PATCH 1/3] Update qemu-kvm bios to allow for a larger bios image.

2009-07-16 Thread Jordan Justen
used) Now we will reserve: 0xfeffc000-0xfeffcfff - 4KB - KVM kernel module, EPT identity pages tables 0xfeffd000-0xfeff - 12KB - KVM bios, VMC TSS Pages 0xff00-0x - 16MB - Max KVM bios.bin Signed-off-by: Jordan Justen --- kvm/bios/rombios.c |8 1 files

[PATCH 2/3] Move qemu-kvm 'VMC TSS Pages' to allow a larger BIOS image.

2009-07-16 Thread Jordan Justen
eserve the new location. Signed-off-by: Jordan Justen --- qemu-kvm-x86.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/qemu-kvm-x86.c b/qemu-kvm-x86.c index daf60b6..b5306aa 100644 --- a/qemu-kvm-x86.c +++ b/qemu-kvm-x86.c @@ -63,7 +63,7 @@ static int kvm_init_tss(kvm

Re: [PATCH] Update qemu-kvm to allow a larger BIOS image.

2009-07-16 Thread Jordan Justen
On Wed, Jul 15, 2009 at 10:37 PM, Sheng Yang wrote: > Make sense to me. So what's mattered here is not bios, but qemu-kvm and kvm > code. The user can replace bios binary by UEFI binary easily, but not with kvm > related part. > > I realized you still need separate the qemu-kvm patch into two: one

Re: [PATCH] Update qemu-kvm to allow a larger BIOS image.

2009-07-15 Thread Jordan Justen
On Wed, Jul 15, 2009 at 8:08 PM, Sheng Yang wrote: > On Thursday 16 July 2009 10:58:53 Jordan Justen wrote: >> On Wed, Jul 15, 2009 at 6:34 PM, Sheng Yang wrote: >> > On Thursday 16 July 2009 08:12:15 Jordan Justen wrote: >> > > >> > > Motivation for ma

Re: [PATCH] Update qemu-kvm to allow a larger BIOS image.

2009-07-15 Thread Jordan Justen
On Wed, Jul 15, 2009 at 6:34 PM, Sheng Yang wrote: > > On Thursday 16 July 2009 08:12:15 Jordan Justen wrote: > > Previously the KVM kernel module would allocate the address range > > of 0xfffbc000-0xfffbcfff for the EPT Indentity Page Tables. > > A separate patch is m

[PATCH] Update qemu-kvm to allow a larger BIOS image.

2009-07-15 Thread Jordan Justen
le UEFI firmware or coreboot firmware. Additionally, an alternative firmware might include a linux kernel+initrd payload, which would require several megabytes. Signed-off-by: Jordan Justen --- kvm/bios/rombios.c|8 kvm/include/x86/asm/vmx.h |2 +- qemu-kvm-x86.c

[PATCH] Update KVM kernel module to allow a larger BIOS image.

2009-07-15 Thread Jordan Justen
ges: A larger firmware image size allows alternative BIOS images to be used with KVM. Some possible uses are to enable UEFI firmware or coreboot firmware. Additionally, an alternative firmware might include a linux kernel+initrd payload, which would require several megabytes. Signed-off-by: Jor