public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Star Zeng <star.zeng@intel.com>
To: edk2-devel@lists.01.org
Cc: Star Zeng <star.zeng@intel.com>
Subject: [PATCH 0/7] Remove PcdPeiCoreMaxXXX PCDs
Date: Fri, 14 Dec 2018 18:28:35 +0800	[thread overview]
Message-ID: <1544783322-17436-1-git-send-email-star.zeng@intel.com> (raw)

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1405

Repo: git@github.com:lzeng14/edk2.git
Branch: RemovePcdPeiCoreMaxXXX

Background as below.

Problem:
As static configuration from the PCDs, the binary PeiCore (for example
in FSP binary with dispatch mode) could not predict how many FVs,
Files or PPIs for different platforms.

Burden:
Platform developers need configure the PCDs accordingly for different
platforms.

To solve the problem and remove the burden, we can update code to
remove the using of PcdPeiCoreMaxFvSupported, PcdPeiCoreMaxPeimPerFv
and PcdPeiCoreMaxPpiSupported by extending buffer dynamically for FV,
File and PPI management.

Test done:
Build code with VS2012, VS2015 and GCC49.
Boot Nt32, Ovmf (including 32, 3264 and 64 with no SMM) to UEFI SHELL.
Boot some internal platforms (including a platform with dispatch mode
FSP binary) to UEFI SHELL and Windows.

More test will be welcome and appreciated. :)

Patches to update edk2-platforms Repo will be sent out separately.

Star Zeng (7):
  MdeModulePkg PeiCore: Remove the using of PcdPeiCoreMaxPeimPerFv
  SecurityPkg Tcg(2)Pei: Remove the using of PcdPeiCoreMaxFvSupported
  MdeModulePkg PeiCore: Remove the using of PcdPeiCoreMaxFvSupported
  MdeModulePkg PeiCore: Remove the using of PcdPeiCoreMaxPpiSupported
  OvmfPkg: Remove PcdPeiCoreMaxXXX PCDs' statement
  Vlv2TbltDevicePkg: Remove PcdPeiCoreMaxXXX PCDs' statement
  MdeModulePkg: Remove PcdPeiCoreMaxXXX PCDs

 MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 207 +++++++++------
 MdeModulePkg/Core/Pei/FwVol/FwVol.c           |  67 ++++-
 MdeModulePkg/Core/Pei/PeiMain.h               |  91 +++++--
 MdeModulePkg/Core/Pei/PeiMain.inf             |   3 -
 MdeModulePkg/Core/Pei/PeiMain/PeiMain.c       |  94 +++----
 MdeModulePkg/Core/Pei/Ppi/Ppi.c               | 355 ++++++++++++++------------
 MdeModulePkg/MdeModulePkg.dec                 |  13 -
 MdeModulePkg/MdeModulePkg.uni                 |  12 -
 OvmfPkg/OvmfPkgIa32.dsc                       |   2 -
 OvmfPkg/OvmfPkgIa32X64.dsc                    |   2 -
 OvmfPkg/OvmfPkgX64.dsc                        |   2 -
 SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c             |  59 +++--
 SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.inf           |   1 -
 SecurityPkg/Tcg/TcgPei/TcgPei.c               |  59 +++--
 SecurityPkg/Tcg/TcgPei/TcgPei.inf             |   1 -
 Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc       |   2 -
 Vlv2TbltDevicePkg/PlatformPkgIA32.dsc         |   2 -
 Vlv2TbltDevicePkg/PlatformPkgX64.dsc          |   2 -
 18 files changed, 560 insertions(+), 414 deletions(-)

-- 
2.7.0.windows.1



             reply	other threads:[~2018-12-14 10:28 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-14 10:28 Star Zeng [this message]
2018-12-14 10:28 ` [PATCH 1/7] MdeModulePkg PeiCore: Remove the using of PcdPeiCoreMaxPeimPerFv Star Zeng
2018-12-17 23:20   ` Chiu, Chasel
2018-12-18  2:04   ` Wang, Jian J
2018-12-18  2:45     ` Zeng, Star
2018-12-14 10:28 ` [PATCH 2/7] SecurityPkg Tcg(2)Pei: Remove the using of PcdPeiCoreMaxFvSupported Star Zeng
2018-12-17  5:06   ` Zhang, Chao B
2018-12-14 10:28 ` [PATCH 3/7] MdeModulePkg PeiCore: " Star Zeng
2018-12-17 23:20   ` Chiu, Chasel
2018-12-18  2:10   ` Wang, Jian J
2018-12-14 10:28 ` [PATCH 4/7] MdeModulePkg PeiCore: Remove the using of PcdPeiCoreMaxPpiSupported Star Zeng
2018-12-17 23:20   ` Chiu, Chasel
2018-12-18  2:24   ` Wang, Jian J
2018-12-18  3:12     ` Zeng, Star
2018-12-14 10:28 ` [PATCH 5/7] OvmfPkg: Remove PcdPeiCoreMaxXXX PCDs' statement Star Zeng
2018-12-14 10:57   ` Ard Biesheuvel
2018-12-14 13:20   ` Laszlo Ersek
2018-12-14 10:28 ` [PATCH 6/7] Vlv2TbltDevicePkg: " Star Zeng
     [not found]   ` <A6DBF04CB9C4D045926EB50EBFFE5A2950865373@shsmsx102.ccr.corp.intel.com>
2018-12-18 11:00     ` Zeng, Star
     [not found]       ` <A6DBF04CB9C4D045926EB50EBFFE5A29508656F9@shsmsx102.ccr.corp.intel.com>
2018-12-19  1:23         ` Zeng, Star
     [not found]           ` <A6DBF04CB9C4D045926EB50EBFFE5A2950865882@shsmsx102.ccr.corp.intel.com>
2018-12-19  3:12             ` Sun, Zailiang
2018-12-14 10:28 ` [PATCH 7/7] MdeModulePkg: Remove PcdPeiCoreMaxXXX PCDs Star Zeng
2018-12-17 23:19   ` Chiu, Chasel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1544783322-17436-1-git-send-email-star.zeng@intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox