From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>, <zhiguang.liu@intel.com>
Cc: "'Rebecca Cran'" <rebecca@bsdio.com>,
"'Bob Feng'" <bob.c.feng@intel.com>,
"'Yuwei Chen'" <yuwei.chen@intel.com>
Subject: 回复: [edk2-devel] [PATCH 1/4] BaseTools: Remove logic to create AP waking vector in GenFv
Date: Mon, 10 Jul 2023 19:38:56 +0800 [thread overview]
Message-ID: <013f01d9b323$1cd37e30$567a7a90$@byosoft.com.cn> (raw)
In-Reply-To: <20230710031706.1329-2-zhiguang.liu@intel.com>
Zhiguang:
If there is no such usage any more, I agree to remove this logic.
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Zhiguang Liu
> 发送时间: 2023年7月10日 11:17
> 收件人: devel@edk2.groups.io
> 抄送: Zhiguang Liu <zhiguang.liu@intel.com>; Rebecca Cran
> <rebecca@bsdio.com>; Liming Gao <gaoliming@byosoft.com.cn>; Bob Feng
> <bob.c.feng@intel.com>; Yuwei Chen <yuwei.chen@intel.com>
> 主题: [edk2-devel] [PATCH 1/4] BaseTools: Remove logic to create AP waking
> vector in GenFv
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4494
>
> Today for SEC core(not VTF-0), GenFv finds free 4K aligned space in
> FV for AP waking vector and JMP to 4G-30h in the waking vector.
> There is no usage of this today. Remove the logic to avoid confusing
> and save spaces in reset vector.
>
> Cc: Rebecca Cran <rebecca@bsdio.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Yuwei Chen <yuwei.chen@intel.com>
> Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
> BaseTools/Source/C/GenFv/GenFvInternalLib.c | 199 --------------------
> 1 file changed, 199 deletions(-)
>
> diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c
> b/BaseTools/Source/C/GenFv/GenFvInternalLib.c
> index f466324d61..29c3363a50 100644
> --- a/BaseTools/Source/C/GenFv/GenFvInternalLib.c
> +++ b/BaseTools/Source/C/GenFv/GenFvInternalLib.c
> @@ -116,63 +116,6 @@ CHAR8 *mFvbAlignmentName[] = {
> EFI_FVB2_ALIGNMENT_2G_STRING
> };
>
> -//
> -// This data array will be located at the base of the Firmware Volume
Header
> (FVH)
> -// in the boot block. It must not exceed 14 bytes of code. The last 2
bytes
> -// will be used to keep the FVH checksum consistent.
> -// This code will be run in response to a startup IPI for HT-enabled
systems.
> -//
> -#define SIZEOF_STARTUP_DATA_ARRAY 0x10
> -
> -UINT8
> m128kRecoveryStartupApDataArray[SIZEOF_STARTUP_DATA_ARRAY] = {
> - //
> - // EA D0 FF 00 F0 ; far jmp F000:FFD0
> - // 0, 0, 0, 0, 0, 0, 0, 0, 0, ; Reserved bytes
> - // 0, 0 ; Checksum Padding
> - //
> - 0xEA,
> - 0xD0,
> - 0xFF,
> - 0x0,
> - 0xF0,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00
> -};
> -
> -UINT8
> m64kRecoveryStartupApDataArray[SIZEOF_STARTUP_DATA_ARRAY] = {
> - //
> - // EB CE ; jmp short ($-0x30)
> - // ; (from offset 0x0 to offset 0xFFD0)
> - // 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ; Reserved bytes
> - // 0, 0 ; Checksum Padding
> - //
> - 0xEB,
> - 0xCE,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00,
> - 0x00
> -};
> -
> FV_INFO mFvDataInfo;
> CAP_INFO mCapDataInfo;
> BOOLEAN mIsLargeFfs = FALSE;
> @@ -1568,12 +1511,6 @@ Returns:
> EFI_PHYSICAL_ADDRESS SecCorePhysicalAddress;
> INT32 Ia32SecEntryOffset;
> UINT32 *Ia32ResetAddressPtr;
> - UINT8 *BytePointer;
> - UINT8 *BytePointer2;
> - UINT16 *WordPointer;
> - UINT16 CheckSum;
> - UINT32 IpiVector;
> - UINTN Index;
> EFI_FFS_FILE_STATE SavedState;
> BOOLEAN Vtf0Detected;
> UINT32 FfsHeaderSize;
> @@ -1745,65 +1682,6 @@ if (MachineType == IMAGE_FILE_MACHINE_I386
> || MachineType == IMAGE_FILE_MACHINE_
> Ia32ResetAddressPtr = (UINT32 *) ((UINTN) FvImage->Eof - 4);
> *Ia32ResetAddressPtr = (UINT32) (FvInfo->BaseAddress);
> DebugMsg (NULL, 0, 9, "update BFV base address in the top FV image",
> "BFV base address = 0x%llX.", (unsigned long long) FvInfo->BaseAddress);
> -
> - //
> - // Update the Startup AP in the FVH header block ZeroVector region.
> - //
> - BytePointer = (UINT8 *) ((UINTN) FvImage->FileImage);
> - if (FvInfo->Size <= 0x10000) {
> - BytePointer2 = m64kRecoveryStartupApDataArray;
> - } else if (FvInfo->Size <= 0x20000) {
> - BytePointer2 = m128kRecoveryStartupApDataArray;
> - } else {
> - BytePointer2 = m128kRecoveryStartupApDataArray;
> - //
> - // Find the position to place Ap reset vector, the offset
> - // between the position and the end of Fvrecovery.fv file
> - // should not exceed 128kB to prevent Ap reset vector from
> - // outside legacy E and F segment
> - //
> - Status = FindApResetVectorPosition (FvImage, &BytePointer);
> - if (EFI_ERROR (Status)) {
> - Error (NULL, 0, 3000, "Invalid", "FV image does not have enough
> space to place AP reset vector. The FV image needs to reserve at least 4KB
of
> unused space.");
> - return EFI_ABORTED;
> - }
> - }
> -
> - for (Index = 0; Index < SIZEOF_STARTUP_DATA_ARRAY; Index++) {
> - BytePointer[Index] = BytePointer2[Index];
> - }
> - //
> - // Calculate the checksum
> - //
> - CheckSum = 0x0000;
> - WordPointer = (UINT16 *) (BytePointer);
> - for (Index = 0; Index < SIZEOF_STARTUP_DATA_ARRAY / 2; Index++) {
> - CheckSum = (UINT16) (CheckSum + ((UINT16) *WordPointer));
> - WordPointer++;
> - }
> - //
> - // Update the checksum field
> - //
> - WordPointer = (UINT16 *) (BytePointer +
> SIZEOF_STARTUP_DATA_ARRAY - 2);
> - *WordPointer = (UINT16) (0x10000 - (UINT32) CheckSum);
> -
> - //
> - // IpiVector at the 4k aligned address in the top 2 blocks in the PEI
FV.
> - //
> - IpiVector = (UINT32) (FV_IMAGES_TOP_ADDRESS - ((UINTN)
> FvImage->Eof - (UINTN) BytePointer));
> - DebugMsg (NULL, 0, 9, "Startup AP Vector address", "IpiVector at
0x%X",
> (unsigned) IpiVector);
> - if ((IpiVector & 0xFFF) != 0) {
> - Error (NULL, 0, 3000, "Invalid", "Startup AP Vector address are not
4K
> aligned, because the FV size is not 4K aligned");
> - return EFI_ABORTED;
> - }
> - IpiVector = IpiVector >> 12;
> - IpiVector = IpiVector & 0xFF;
> -
> - //
> - // Write IPI Vector at Offset FvrecoveryFileSize - 8
> - //
> - Ia32ResetAddressPtr = (UINT32 *) ((UINTN) FvImage->Eof - 8);
> - *Ia32ResetAddressPtr = IpiVector;
> } else if (MachineType == IMAGE_FILE_MACHINE_ARMTHUMB_MIXED) {
> //
> // Since the ARM reset vector is in the FV Header you really don't
need
> a
> @@ -4190,83 +4068,6 @@ Returns:
> return EFI_SUCCESS;
> }
>
> -EFI_STATUS
> -FindApResetVectorPosition (
> - IN MEMORY_FILE *FvImage,
> - OUT UINT8 **Pointer
> - )
> -/*++
> -
> -Routine Description:
> -
> - Find the position in this FvImage to place Ap reset vector.
> -
> -Arguments:
> -
> - FvImage Memory file for the FV memory image.
> - Pointer Pointer to pointer to position.
> -
> -Returns:
> -
> - EFI_NOT_FOUND - No satisfied position is found.
> - EFI_SUCCESS - The suitable position is return.
> -
> ---*/
> -{
> - EFI_FFS_FILE_HEADER *PadFile;
> - UINT32 Index;
> - EFI_STATUS Status;
> - UINT8 *FixPoint;
> - UINT32 FileLength;
> -
> - for (Index = 1; ;Index ++) {
> - //
> - // Find Pad File to add ApResetVector info
> - //
> - Status = GetFileByType (EFI_FV_FILETYPE_FFS_PAD, Index, &PadFile);
> - if (EFI_ERROR (Status) || (PadFile == NULL)) {
> - //
> - // No Pad file to be found.
> - //
> - break;
> - }
> - //
> - // Get Pad file size.
> - //
> - FileLength = GetFfsFileLength(PadFile);
> - FileLength = (FileLength + EFI_FFS_FILE_HEADER_ALIGNMENT - 1) &
> ~(EFI_FFS_FILE_HEADER_ALIGNMENT - 1);
> - //
> - // FixPoint must be align on 0x1000 relative to FvImage Header
> - //
> - FixPoint = (UINT8*) PadFile + GetFfsHeaderLength(PadFile);
> - FixPoint = FixPoint + 0x1000 - (((UINTN) FixPoint - (UINTN)
> FvImage->FileImage) & 0xFFF);
> - //
> - // FixPoint be larger at the last place of one fv image.
> - //
> - while (((UINTN) FixPoint + SIZEOF_STARTUP_DATA_ARRAY - (UINTN)
> PadFile) <= FileLength) {
> - FixPoint += 0x1000;
> - }
> - FixPoint -= 0x1000;
> -
> - if ((UINTN) FixPoint < ((UINTN) PadFile +
GetFfsHeaderLength(PadFile)))
> {
> - //
> - // No alignment FixPoint in this Pad File.
> - //
> - continue;
> - }
> -
> - if ((UINTN) FvImage->Eof - (UINTN)FixPoint <= 0x20000) {
> - //
> - // Find the position to place ApResetVector
> - //
> - *Pointer = FixPoint;
> - return EFI_SUCCESS;
> - }
> - }
> -
> - return EFI_NOT_FOUND;
> -}
> -
> EFI_STATUS
> ParseCapInf (
> IN MEMORY_FILE *InfFile,
> --
> 2.31.1.windows.1
>
>
>
>
>
next prev parent reply other threads:[~2023-07-10 11:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-10 3:17 [PATCH 0/4] Remove AP waking vector in Reset Vector Zhiguang Liu
2023-07-10 3:17 ` [PATCH 1/4] BaseTools: Remove logic to create AP waking vector in GenFv Zhiguang Liu
2023-07-10 11:38 ` gaoliming [this message]
2023-07-10 3:17 ` [PATCH 2/4] UefiCpuPkg/SecCore: Remove AP waking Vector logic in SecCore Zhiguang Liu
2023-07-10 3:17 ` [PATCH 3/4] OvmfPkg: Remove applicationProcessorEntryPoint Zhiguang Liu
2023-07-24 10:17 ` [edk2-devel] " Anthony PERARD via groups.io
2023-07-10 3:17 ` [PATCH 4/4] UefiCpuPk/ResetVector: Remove AP waking vector from ResetVector Zhiguang Liu
2023-07-10 3:35 ` [edk2-devel] [PATCH 0/4] Remove AP waking vector in Reset Vector Ni, Ray
[not found] ` <1770634CE62FBB6D.7145@groups.io>
2023-07-24 2:47 ` [edk2-devel] [PATCH 3/4] OvmfPkg: Remove applicationProcessorEntryPoint Zhiguang Liu
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='013f01d9b323$1cd37e30$567a7a90$@byosoft.com.cn' \
--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