From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: Can't see more than 3.5GB of RAM / UEFI / no e820 memory map detected Date: Tue, 28 Aug 2012 23:31:49 +0100 Message-ID: References: <503D3D2E.70103@abpni.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <503D3D2E.70103@abpni.co.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jonathan Tripathy , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 28/08/2012 22:50, "Jonathan Tripathy" wrote: >>>>> I reversed the if block in setup.c and now my server can see the full >>>>> 32GB of RAM. I haven't submitted a patch yet as we have run into another >>>>> (possibly unrelated to xen) issue with this server build that we are >>>>> working on. Once we complete our full testing, a patch will be submitted >>>>> :) >>>> In this case, I will re-make the patch myself and check it in. Since it is >>>> a >>>> trivial one. >>>> >>>> Thanks, >>>> Keir >>> Thanks Keir >> Now done. xen-unstable:25786 >> >> -- Keir > Hi Keir, > > Thanks for doing that. However, the ordering of the if block in my code > that I used was slightly different from your commit. Perhaps this > doesn't make too much of a difference? Your ordering favours Multiboot-e820 over Xen-e820, which we do not want to do, and Multiboot-e801 over Xen-e820 which is definitely not sane. I see you tested the checked-in version and it worked okay for you, which is expected, and is a more sensible re-ordering. :) -- Keir