From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-x232.google.com (mail-wm0-x232.google.com [IPv6:2a00:1450:400c:c09::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 43EDF1A1E18 for ; Mon, 8 Aug 2016 04:34:37 -0700 (PDT) Received: by mail-wm0-x232.google.com with SMTP id i5so132626238wmg.0 for ; Mon, 08 Aug 2016 04:34:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=cL5/ltEAlnk8oTsCLP5IqL5HwrHgJtGohH3kpNdMAIM=; b=iUjyml86WpQ713bp+s5cPnrNg9ie4IUboWjrWx6+agiwlTqCjztKAc3J/1779767Px 5jCd/shl13obQY5yYvVPFSZMfgaz6Ft7v9f+ETeZuW2pIHFCmuKiOmXU8RQuTCWzxtdE 2GcQJYe4ycrpQMYZ70RjBlpJPGFOPhyr9hCsiTrzRyT+nMKanZdpnbC3P0i1WHEdn3mt 2ER1HsqSKB0OK5SwXP8LcGHeXIx1NbatQR6ItXEulgpB28yZSyqSVlhmANcoN3j6bNdP el+KEJ1RJpy+XpaVafdhwCC+Z4ucNeG1YB6EcrJsdIci1ILLMkBA/UXYXcKEaNDUftzf Fp+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=cL5/ltEAlnk8oTsCLP5IqL5HwrHgJtGohH3kpNdMAIM=; b=NWv8hHHNtmwOPjLSU+VpjYhoQ/BZa3OKHpRT/5zZ3+MtBj3pGRk3uhnSZbB/gsO0qE VqKbnMXhp+I+YS5Dq/CDfdqJtp7J/PqEbsdyd3e2pUBwiWIM2ZbHP//REx2o96IWnp6o ytUKBcdLazralrQH6aK26zidMA+fgMgZb488FYiNSuPmvrVAhV4G/1tl9vA0sQd+xf2d he9FC1nptHixQEexyLZiZ7c8qVk4BnxRJ0v/kpfRWn2jInBlq6NxrTaO/ZU0VtDcrNRU eqQL4ihnNiinciTFFfS3rizawQZDmGlYlbQBPv00XkZvjK8ezFfmU0Ge8BQ1Bm6KdjXN U6BA== X-Gm-Message-State: AEkoouuU44JaKuHgDgDhttcLjujlZDzCOUBPXwE+ixMY/lGHyomWzMvzYq66xwUttMX5cOSSFrJwX6lnQQKRIA== X-Received: by 10.28.9.194 with SMTP id 185mr14751344wmj.37.1470656075872; Mon, 08 Aug 2016 04:34:35 -0700 (PDT) MIME-Version: 1.0 Received: by 10.28.238.198 with HTTP; Mon, 8 Aug 2016 04:34:35 -0700 (PDT) In-Reply-To: References: From: Michael Zimmermann Date: Mon, 8 Aug 2016 13:34:35 +0200 Message-ID: To: Ard Biesheuvel Cc: edk2-devel-01 X-Content-Filtered-By: Mailman/MimeDel 2.1.21 Subject: Re: ARM PrePI Memory setup recommendations X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Aug 2016 11:34:37 -0000 Content-Type: text/plain; charset=UTF-8 While that can be a problem in general it's definitely not a problem in my case because the "MinimalMemorySizeNeeded" check is done after a ResourceHob is found which contains the PHIT Hob.(this is the part which broke). And since at that point I've already registered resourcehob's for all existing dram memory(usually between 512MB and 3GB) there's plenty of memory for bootservices to use. So I really think the only problem was that I had to take the stack size into account when calculating PcdSystemMemorySize. Thanks Michael On Mon, Aug 8, 2016 at 11:25 AM, Ard Biesheuvel wrote: > On 8 August 2016 at 11:18, Michael Zimmermann > wrote: > > I'm using the defaults from the (now deleted) 2ndstage sample dsc: > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|50 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|20 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiBootServicesCode|400 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiBootServicesData|20000 > > 20000 pages == 80 MB, so this value makes little sense if you have 32 > MB of memory. Dropping this to a sane value fixed some problems we saw > with QEMU when running with the default 128 MB > > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderCode|20 > > gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiLoaderData|0 > > > > I figured that the PrePi's stack setup may cause this, especially since > I'm > > using all the memory up to 0xFFFFFFFF+1 which causes the logic to ignore > the > > last page(I think). > > I'm now using PcdSystemMemorySize - PcdFdSize - EFI_PAGE_SIZE which > seems to > > work pretty good even though the "missing" memory was just like 900bytes > and > > not 4096. > > > > Thanks > > Michael > > > > On Mon, Aug 8, 2016 at 10:50 AM, Ard Biesheuvel < > ard.biesheuvel@linaro.org> > > wrote: > >> > >> On 6 August 2016 at 13:05, Michael Zimmermann > > >> wrote: > >> > Hi, > >> > > >> > I have some problems with my current setup and would like to ask you > for > >> > the > >> > best way to configure the following setup: > >> > - I'm using PrePi, I don't need relocations because the loading > address > >> > is > >> > fixed(0xfe000000) > >> > - The SystemMemory address range is unknown at compile time > >> > - the system-memory consists of multiple ranges with holes so > providing > >> > just > >> > a base and a size is not enough > >> > - There's no NOR, edk2 gets loaded into dram > >> > > >> > my current setup: > >> > PcdSystemMemoryBase = loading addr(0xfe000000) > >> > PcdSystemMemorySize = 32MB > >> > FD_BASE = PcdSystemMemoryBase > >> > FD_SIZE = 4MB > >> > in ArmPlatformGetVirtualMemoryMap I call BuildResourceDescriptorHob > for > >> > all > >> > DRAM memory ranges. > >> > In PrePiMain after MemoryPeim I call BuildMemoryAllocationHob for some > >> > reserved memory ranges > >> > > >> > After running into out-of-memory errors during PrePi when using bigger > >> > FV's > >> > I revisited everything and came across a PCD I somehow never looked at > >> > when > >> > setting up my platform a year ago: PcdSystemMemoryUefiRegionSize. > >> > this was hardcoded to 0x00e00000 which obviously caused allocation > >> > errors > >> > while extracting big FV's even though PcdSystemMemorySize was big > >> > enough. > >> > > >> > Since there already was this nice comment 'PcdSystemMemorySize - > >> > PcdFdSize' > >> > I did exactly that: 32MB-4MB = 28MB. > >> > Unfortunately after doing that MdeModulePkg/Core/Dxe/Gcd/Gcd.c starts > >> > bugging that he can't find a resource hob that contains the PHIT Hob. > >> > > >> > >> Could you check the > >> > >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiXXX|nnnn > >> > >> definitions in your .DSC? Fairly recently, the core code was modified > >> to use the sum of these definitions as an estimate of the minimally > >> required allocation size, and it broke some other platforms in a > >> similar way. > >> > >> > So now that my setup and problems are clear, the questions: > >> > - Is it ok to define ResourceHob's that cover the PcdSystemMemory > range? > >> > - what can I do to fix the PHIT Hob problem? should PcdFdSize + > >> > PcdSystemMemoryUefiRegionSize be smaller than PcdSystemMemorySize ? > >> > - I took a quick look at ArmVirt where they change the PcdSystemMemory > >> > range > >> > at runtime, but they use lots of custom code like their own > >> > MemoryPeiLib, > >> > and I think everything but Xen uses PEIM instead of PrePi. Also as I > >> > said I > >> > have more than one range so this would work anyway. > >> > > >> > >> ArmVirtQemu uses PEIM, since it runs from [emulated] NOR flash. > >> ArmVirtQemuKernel and ArmVirtXen use PrePi (the former is simply > >> ArmVirtQemu with the FD executing from DRAM, using the QEMU '-kernel' > >> option) > > > > >