From mboxrd@z Thu Jan 1 00:00:00 1970 From: Izik Eidus Subject: Re: [PATCH 1/5] Use correct types to enable > 2G support (v3) Date: Mon, 04 Feb 2008 17:29:45 +0200 Message-ID: <1202138985.18306.7.camel@localhost.localdomain> References: <1202137865-20232-1-git-send-email-aliguori@us.ibm.com> <1202137865-20232-2-git-send-email-aliguori@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org, Avi Kivity , Paul Brook , Andrzej Zaborowski , Fabrice Bellard To: Anthony Liguori Return-path: In-Reply-To: <1202137865-20232-2-git-send-email-aliguori-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org On Mon, 2008-02-04 at 09:11 -0600, Anthony Liguori wrote: > KVM supports more than 2GB of memory for x86_64 hosts. The following patch > fixes a number of type related issues where int's were being used when they > shouldn't have been. It also introduces CMOS support so the BIOS can build > the appropriate e820 tables. again, you need patch the bios for this, as the bochs bios doesnt know how to map memory above 4 giga, and it doesnt know about this CMOS addresses, (look at the patch that i sent) > > For v2 of this patch, I've moved ram_addr_t to cpu-all.h and switched > ram_size to be a ram_addr_t. I've also removed the memory limit check for > x86_64 (provided kqemu isn't enabled) and enabled the use of a 'M' or 'G' > suffix for the -m option. I've also tried to do a more thorough job of > updating the code to use the proper types. > > This patch also includes support for setting up > 2GB of memory for > TARGET_I386. KVM works quite happily with 5GB of ram but I suspect there > are still some uint32_t's in the non-KVM does not work when using more than > 3GB of RAM. yes, the patch for kvm was never desgiend to solve the problems qemu would have with above 4 giga memory. ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1JM3H9-0001LW-G5 for qemu-devel@nongnu.org; Mon, 04 Feb 2008 10:30:27 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1JM3H7-0001KI-Hq for qemu-devel@nongnu.org; Mon, 04 Feb 2008 10:30:26 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1JM3H7-0001KD-9u for qemu-devel@nongnu.org; Mon, 04 Feb 2008 10:30:25 -0500 Received: from bzq-179-150-194.static.bezeqint.net ([212.179.150.194] helo=il.qumranet.com) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1JM3H7-0001SE-AT for qemu-devel@nongnu.org; Mon, 04 Feb 2008 10:30:25 -0500 From: Izik Eidus In-Reply-To: <1202137865-20232-2-git-send-email-aliguori@us.ibm.com> References: <1202137865-20232-1-git-send-email-aliguori@us.ibm.com> <1202137865-20232-2-git-send-email-aliguori@us.ibm.com> Content-Type: text/plain Date: Mon, 04 Feb 2008 17:29:45 +0200 Message-Id: <1202138985.18306.7.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [kvm-devel] [PATCH 1/5] Use correct types to enable > 2G support (v3) Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: kvm-devel@lists.sourceforge.net, qemu-devel@nongnu.org, Paul Brook On Mon, 2008-02-04 at 09:11 -0600, Anthony Liguori wrote: > KVM supports more than 2GB of memory for x86_64 hosts. The following patch > fixes a number of type related issues where int's were being used when they > shouldn't have been. It also introduces CMOS support so the BIOS can build > the appropriate e820 tables. again, you need patch the bios for this, as the bochs bios doesnt know how to map memory above 4 giga, and it doesnt know about this CMOS addresses, (look at the patch that i sent) > > For v2 of this patch, I've moved ram_addr_t to cpu-all.h and switched > ram_size to be a ram_addr_t. I've also removed the memory limit check for > x86_64 (provided kqemu isn't enabled) and enabled the use of a 'M' or 'G' > suffix for the -m option. I've also tried to do a more thorough job of > updating the code to use the proper types. > > This patch also includes support for setting up > 2GB of memory for > TARGET_I386. KVM works quite happily with 5GB of ram but I suspect there > are still some uint32_t's in the non-KVM does not work when using more than > 3GB of RAM. yes, the patch for kvm was never desgiend to solve the problems qemu would have with above 4 giga memory.