From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 0/5] bios: >4G updates Date: Wed, 24 Sep 2008 14:07:22 +0300 Message-ID: <48DA1F6A.40105@redhat.com> References: <1222188118.8138.101.camel@2710p.home> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm-devel To: Alex Williamson Return-path: Received: from mx2.redhat.com ([66.187.237.31]:33410 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751602AbYIXLH3 (ORCPT ); Wed, 24 Sep 2008 07:07:29 -0400 In-Reply-To: <1222188118.8138.101.camel@2710p.home> Sender: kvm-owner@vger.kernel.org List-ID: Alex Williamson wrote: > As requested, here's the follow-on to the >4G MTRR changes split into > functional bits. > > [1/5] Rename variables to reflect what they're really reporting > [2/5] Reformat ram_probe() to match the rest of the code > [3/5] Add SMBIOS info for memory above 4G > [4/5] Fix the SMBIOS type 19 & 20 range end address > [5/5] Optional - switch default MTRR type to WB and only cover MMIO > > I've taken some liberties renaming and reformatting, if we'd rather not > introduce too many extraneous changes, I can drop those. The SMBIOS > changes seem to work up to 32767MB, then we hit a limitation in the type > 17 table only providing 15bits for the size. We might need to describe > multiple virtual DIMMs to get around that, but it's a separate issue. > > The final patch is optional and switches over to make the variable MTRRs > only describe the MMIO hole, leaving the rest of the address space > default to WB. I can't say I fully understand the implications of > hotplug memory for this scenario. Let me know if there are comments. > The patches all look good, however renaming and reformatting will lead to merge headaches later on. We haven't been good at working with bochs bios upstream. Can you peek in bochs upstream and see if it's worth merging? If not, I'll just merge these patches. -- error compiling committee.c: too many arguments to function