From: "Laszlo Ersek" <lersek@redhat.com>
To: Michael D Kinney <michael.d.kinney@intel.com>, devel@edk2.groups.io
Cc: Sean Brogan <sean.brogan@microsoft.com>,
Eric Dong <eric.dong@intel.com>, Ray Ni <ray.ni@intel.com>
Subject: Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
Date: Wed, 23 Oct 2019 00:57:25 +0200 [thread overview]
Message-ID: <a6ba80cb-cb86-2147-4f14-f39e9bdea5a6@redhat.com> (raw)
In-Reply-To: <20191022190659.26912-1-michael.d.kinney@intel.com>
On 10/22/19 21:06, Michael D Kinney wrote:
> From: Sean Brogan <sean.brogan@microsoft.com>
>
> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
>
> Update UefiCpuPkg.dsc to guarantee all libraries and
> modules are always built. Add the following components.
>
> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> * UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
>
> ResetVector.inf is a binary INF, so no source builds are
> triggered from adding this line. However, a build with
> this component does verify the contents of the INF file.
>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> ---
> UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc b/UefiCpuPkg/UefiCpuPkg.dsc
> index e7dfe30eda..d28cb5cccb 100644
> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> @@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf
> + UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptionHandlerLib.inf
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExceptionHandlerLib.inf
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf
> @@ -146,6 +147,7 @@ [Components.IA32, Components.X64]
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmCpuFeaturesLibStm.inf
> }
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> + UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>
> [BuildOptions]
> *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
>
Can we retire
UefiCpuPkg/ResetVector/Vtf0/Bin/
instead (in a separate patch)?
Because, according to
git log --stat master -- UefiCpuPkg/ResetVector/Vtf0/Bin/
the most recent commit in which we updated the *.raw files has been
f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for finding SEC image of
type TE.", 2011-09-28).
I doubt anyone consumes this (binary) module, since Jordan's commits
- 5a1f324d946c ("UefiCpuPkg: Support building VTF0 ResetVector during
the EDK II build", 2014-08-18)
- eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page tables in NASM code",
2014-08-18).
We've by now dropped all top-level *BinPkg package directories too.
If we remove the Bin subdir, then we should probably remove:
- Build.py
- Tools/
too.
For the other addition (CpuCommonFeaturesLib): if that gets its own
patch (after the removal of UefiCpuPkg/ResetVector/Vtf0/Bin/), then
please add
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
to it.
Thanks
Laszlo
next prev parent reply other threads:[~2019-10-22 22:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-22 19:06 [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc Michael D Kinney
2019-10-22 22:57 ` Laszlo Ersek [this message]
2019-10-22 23:13 ` Michael D Kinney
2019-10-22 23:27 ` Laszlo Ersek
2019-10-22 23:38 ` Michael D Kinney
2019-10-23 1:28 ` Dong, Eric
2019-10-23 15:15 ` Michael D Kinney
2019-10-23 16:06 ` Michael D Kinney
2019-10-23 16:08 ` Agyeman, Prince
2019-10-24 11:03 ` Laszlo Ersek
2019-10-24 0:06 ` [edk2-devel] " Dong, Eric
2019-10-24 0:21 ` Ni, Ray
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=a6ba80cb-cb86-2147-4f14-f39e9bdea5a6@redhat.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