From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: chasel.chiu@intel.com) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by groups.io with SMTP; Fri, 06 Sep 2019 00:01:29 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Sep 2019 00:01:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,472,1559545200"; d="scan'208";a="383168786" Received: from cchiu4-mobl1.gar.corp.intel.com ([10.5.215.135]) by fmsmga005.fm.intel.com with ESMTP; 06 Sep 2019 00:01:27 -0700 From: "Chiu, Chasel" To: devel@edk2.groups.io Cc: Michael Kubacki , Nate DeSimone , Liming Gao Subject: [edk2-platforms: PATCH] KabylakeOpenBoardPkg: Missed 64KB Temporary RAM in Dispatch mode. Date: Fri, 6 Sep 2019 15:01:10 +0800 Message-Id: <20190906070110.18204-1-chasel.chiu@intel.com> X-Mailer: git-send-email 2.13.3.windows.1 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2159 FSP Dispatch mode will share same stack and heap with bootloader so we should not reserve separate temporary ram for FSP. Set PcdFspTemporaryRamSize to 0 in Dispatch mode. Test: internal platform can boot with FSP Dispatch mode with 64KB temporary RAM back. Cc: Michael Kubacki Cc: Nate DeSimone Cc: Liming Gao Signed-off-by: Chasel Chiu --- Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc index 55ae9f47ac..2009bb225a 100644 --- a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc +++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkgPcd.dsc @@ -69,10 +69,10 @@ gSiPkgTokenSpaceGuid.PcdPeiTemporaryRamStackSize|0x20000 !else # - # FSP Dispatch mode will share the same stack with boot loader, - # here temporary ram size is used by FSP heap and can be smaller + # FSP Dispatch mode will share the same stack and heap with boot loader, + # no separate temporary ram required by FSP. # - gIntelFsp2PkgTokenSpaceGuid.PcdFspTemporaryRamSize | 0x00010000 + gIntelFsp2PkgTokenSpaceGuid.PcdFspTemporaryRamSize | 0 # # In FSP Dispatch mode boot loader stack size must be big enough for executing -- 2.13.3.windows.1