From patchwork Wed May 11 04:09:09 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sasha Levin X-Patchwork-Id: 777642 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.3) with ESMTP id p4BHTJhB022077 for ; Wed, 11 May 2011 17:29:19 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753555Ab1EKR3O (ORCPT ); Wed, 11 May 2011 13:29:14 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:45457 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752174Ab1EKR3N (ORCPT ); Wed, 11 May 2011 13:29:13 -0400 Received: by wya21 with SMTP id 21so558408wya.19 for ; Wed, 11 May 2011 10:29:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:from:to:cc:subject:date:message-id:x-mailer; bh=0adKCPmfskxPBbWOHsQr/VpcRSt4Ku/CxfaXLsNQ6ao=; b=vC1g4MitsOYDdtKO63M93745T2SusTBnoqRfN6+R5isCWEvycMGCe2+5viq9t98sqI koeXnE9YWVNVnlwj3UGc1RbPEH0jX6OLZuGh1+6qR+MZEoLel0qvc+henb0t7NvAVqC9 65xMmW8gDMk7edPECEPzqvVLHeLbuofxbziUw= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer; b=xEGkv8gfDEPROwdoHn9s3S/YCKeobhZThzS3mgAEjCGmUaiN6ADUiOudnRcBS4Q/B5 INFQtnT5f90TWobG7mbv4FDuDKCOesEpl/yY1IKGzTG/XGtL80olVzwlKmO6NLJ9B74O j7bQ+klrxJ/mBba7VsKyVs/H0N6zCqSyx7WnM= Received: by 10.216.136.67 with SMTP id v45mr4759604wei.106.1305086971911; Tue, 10 May 2011 21:09:31 -0700 (PDT) Received: from localhost.localdomain ([188.120.134.217]) by mx.google.com with ESMTPS id d54sm3838919wej.34.2011.05.10.21.09.29 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 10 May 2011 21:09:30 -0700 (PDT) From: Sasha Levin To: penberg@kernel.org Cc: mingo@elte.hu, asias.hejun@gmail.com, prasadjoshi124@gmail.com, avi@redhat.com, gorcunov@gmail.com, kvm@vger.kernel.org, Sasha Levin Subject: [PATCH 1/3] kvm tools: Add memory gap for larger RAM sizes Date: Wed, 11 May 2011 07:09:09 +0300 Message-Id: <1305086951-31698-1-git-send-email-levinsasha928@gmail.com> X-Mailer: git-send-email 1.7.5.rc3 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Wed, 11 May 2011 17:29:19 +0000 (UTC) Add a memory gap between 0xe0000000 and 0x100000000 when using more than 0xe0000000 bytes for guest RAM. This space is used by several things, PCI configuration space for example. This patch updates the e820 table, slot allocations used for KVM_SET_USER_MEMORY_REGION, and the address translation. Signed-off-by: Sasha Levin --- tools/kvm/bios.c | 27 +++++++++++++++++++++------ tools/kvm/include/kvm/e820.h | 2 +- tools/kvm/include/kvm/kvm.h | 9 ++++++++- tools/kvm/kvm.c | 22 ++++++++++++++++------ 4 files changed, 46 insertions(+), 14 deletions(-) diff --git a/tools/kvm/bios.c b/tools/kvm/bios.c index 2199c0c..cd417fa 100644 --- a/tools/kvm/bios.c +++ b/tools/kvm/bios.c @@ -61,7 +61,7 @@ static void e820_setup(struct kvm *kvm) size = guest_flat_to_host(kvm, E820_MAP_SIZE); mem_map = guest_flat_to_host(kvm, E820_MAP_START); - *size = E820_MEM_AREAS; + mem_map[i++] = (struct e820_entry) { .addr = REAL_MODE_IVT_BEGIN, @@ -78,13 +78,28 @@ static void e820_setup(struct kvm *kvm) .size = MB_BIOS_END - MB_BIOS_BEGIN, .type = E820_MEM_RESERVED, }; - mem_map[i++] = (struct e820_entry) { - .addr = BZ_KERNEL_START, - .size = kvm->ram_size - BZ_KERNEL_START, - .type = E820_MEM_USABLE, - }; + if (kvm->ram_size < 0xe0000000) { + mem_map[i++] = (struct e820_entry) { + .addr = BZ_KERNEL_START, + .size = kvm->ram_size - BZ_KERNEL_START, + .type = E820_MEM_USABLE, + }; + } else { + mem_map[i++] = (struct e820_entry) { + .addr = BZ_KERNEL_START, + .size = 0xe0000000 - BZ_KERNEL_START, + .type = E820_MEM_USABLE, + }; + mem_map[i++] = (struct e820_entry) { + .addr = 0x100000000ULL, + .size = kvm->ram_size - 0xe0000000 - BZ_KERNEL_START, + .type = E820_MEM_USABLE, + }; + } BUILD_BUG_ON(i > E820_MEM_AREAS); + + *size = i; } /** diff --git a/tools/kvm/include/kvm/e820.h b/tools/kvm/include/kvm/e820.h index 252ae1f..e0f5f2a 100644 --- a/tools/kvm/include/kvm/e820.h +++ b/tools/kvm/include/kvm/e820.h @@ -8,7 +8,7 @@ #define E820_MEM_USABLE 1 #define E820_MEM_RESERVED 2 -#define E820_MEM_AREAS 4 +#define E820_MEM_AREAS 5 struct e820_entry { u64 addr; /* start of memory segment */ diff --git a/tools/kvm/include/kvm/kvm.h b/tools/kvm/include/kvm/kvm.h index 3dab78d..e9c16ea 100644 --- a/tools/kvm/include/kvm/kvm.h +++ b/tools/kvm/include/kvm/kvm.h @@ -60,7 +60,14 @@ static inline u32 segment_to_flat(u16 selector, u16 offset) static inline void *guest_flat_to_host(struct kvm *self, unsigned long offset) { - return self->ram_start + offset; + /* + * We have a gap between 0xe0000000 and 0x100000000. + * Consider it when translating an address above 0x100000000. + */ + if (offset < 0xe0000000) + return self->ram_start + offset; + else + return self->ram_start + 0xe0000000 + (offset - 0x100000000); } static inline void *guest_real_to_host(struct kvm *self, u16 selector, u16 offset) diff --git a/tools/kvm/kvm.c b/tools/kvm/kvm.c index 65793f2..976b099 100644 --- a/tools/kvm/kvm.c +++ b/tools/kvm/kvm.c @@ -153,23 +153,33 @@ static bool kvm__cpu_supports_vm(void) return regs.ecx & (1 << feature); } -void kvm__init_ram(struct kvm *self) +static void kvm_register_mem_slot(struct kvm *kvm, u32 slot, u64 guest_phys, u64 size, u64 userspace_addr) { struct kvm_userspace_memory_region mem; int ret; mem = (struct kvm_userspace_memory_region) { - .slot = 0, - .guest_phys_addr = 0x0UL, - .memory_size = self->ram_size, - .userspace_addr = (unsigned long) self->ram_start, + .slot = slot, + .guest_phys_addr = guest_phys, + .memory_size = size, + .userspace_addr = userspace_addr, }; - ret = ioctl(self->vm_fd, KVM_SET_USER_MEMORY_REGION, &mem); + ret = ioctl(kvm->vm_fd, KVM_SET_USER_MEMORY_REGION, &mem); if (ret < 0) die_perror("KVM_SET_USER_MEMORY_REGION ioctl"); } +void kvm__init_ram(struct kvm *self) +{ + if (self->ram_size < 0xe0000000) { + kvm_register_mem_slot(self, 0, 0, self->ram_size, (u64)self->ram_start); + } else { + kvm_register_mem_slot(self, 0, 0, 0xe0000000, (u64)self->ram_start); + kvm_register_mem_slot(self, 1, 0x100000000ULL, self->ram_size - 0xe0000000, (u64)self->ram_start + 0xe0000000); + } +} + int kvm__max_cpus(struct kvm *self) { int ret;