Could you separate this change to two patches for IPF and EBC? > -----Original Message----- > From: Feng, Bob C > Sent: Wednesday, January 9, 2019 2:45 PM > To: edk2-devel@lists.01.org > Cc: Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming <liming....@intel.com>; > Carsey, Jaben <jaben.car...@intel.com> > Subject: [Patch 3/3] BaseTools: Remove unused logic from C tools > > From: "Feng, Bob C" <bob.c.f...@intel.com> > > Remove EDK module and IA64 support from BaseTools C code. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Bob Feng <bob.c.f...@intel.com> > Cc: Liming Gao <liming....@intel.com> > Cc: Jaben Carsey <jaben.car...@intel.com> > --- > BaseTools/Source/C/Common/BasePeCoff.c | 14 +- > BaseTools/Source/C/Common/PeCoffLoaderEx.c | 162 +----------------- > BaseTools/Source/C/EfiRom/EfiRom.h | 3 +- > BaseTools/Source/C/GenFv/GenFvInternalLib.c | 72 +------- > BaseTools/Source/C/GenFv/GenFvInternalLib.h | 10 -- > BaseTools/Source/C/GenFw/Elf64Convert.c | 5 - > BaseTools/Source/C/GenFw/GenFw.c | 6 +- > BaseTools/Source/C/GenFw/elf_common.h | 85 --------- > .../C/Include/IndustryStandard/PeImage.h | 3 - > .../Source/C/Include/IndustryStandard/pci22.h | 1 - > BaseTools/Source/C/Makefiles/header.makefile | 1 - > BaseTools/Source/C/VolInfo/VolInfo.c | 2 +- > 12 files changed, 10 insertions(+), 354 deletions(-) > > diff --git a/BaseTools/Source/C/Common/BasePeCoff.c > b/BaseTools/Source/C/Common/BasePeCoff.c > index 1d89a300de..d5cefbd0b0 100644 > --- a/BaseTools/Source/C/Common/BasePeCoff.c > +++ b/BaseTools/Source/C/Common/BasePeCoff.c > @@ -54,17 +54,10 @@ PeCoffLoaderRelocateIa32Image ( > IN OUT CHAR8 *Fixup, > IN OUT CHAR8 **FixupData, > IN UINT64 Adjust > ); > > -RETURN_STATUS > -PeCoffLoaderRelocateIpfImage ( > - IN UINT16 *Reloc, > - IN OUT CHAR8 *Fixup, > - IN OUT CHAR8 **FixupData, > - IN UINT64 Adjust > - ); > > RETURN_STATUS > PeCoffLoaderRelocateArmImage ( > IN UINT16 **Reloc, > IN OUT CHAR8 *Fixup, > @@ -182,11 +175,10 @@ Returns: > } else { > ImageContext->Machine = TeHdr->Machine; > } > > if (ImageContext->Machine != EFI_IMAGE_MACHINE_IA32 && \ > - ImageContext->Machine != EFI_IMAGE_MACHINE_IA64 && \ > ImageContext->Machine != EFI_IMAGE_MACHINE_X64 && \ > ImageContext->Machine != EFI_IMAGE_MACHINE_ARMT && \ > ImageContext->Machine != EFI_IMAGE_MACHINE_EBC && \ > ImageContext->Machine != EFI_IMAGE_MACHINE_AARCH64) { > if (ImageContext->Machine == IMAGE_FILE_MACHINE_ARM) { > @@ -814,13 +806,10 @@ Returns: > Status = PeCoffLoaderRelocateIa32Image (Reloc, Fixup, &FixupData, > Adjust); > break; > case EFI_IMAGE_MACHINE_ARMT: > Status = PeCoffLoaderRelocateArmImage (&Reloc, Fixup, &FixupData, > Adjust); > break; > - case EFI_IMAGE_MACHINE_IA64: > - Status = PeCoffLoaderRelocateIpfImage (Reloc, Fixup, &FixupData, > Adjust); > - break; > default: > Status = RETURN_UNSUPPORTED; > break; > } > if (RETURN_ERROR (Status)) { > @@ -1317,13 +1306,12 @@ PeCoffLoaderGetPdbPointer ( > // Assume PE32 image with IA32 Machine field. > // > Magic = EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC; > break; > case EFI_IMAGE_MACHINE_X64: > - case EFI_IMAGE_MACHINE_IPF: > // > - // Assume PE32+ image with X64 or IPF Machine field > + // Assume PE32+ image with X64 Machine field > // > Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC; > break; > default: > // > diff --git a/BaseTools/Source/C/Common/PeCoffLoaderEx.c > b/BaseTools/Source/C/Common/PeCoffLoaderEx.c > index 2d9a2a8978..fa8c7e3d17 100644 > --- a/BaseTools/Source/C/Common/PeCoffLoaderEx.c > +++ b/BaseTools/Source/C/Common/PeCoffLoaderEx.c > @@ -1,7 +1,7 @@ > /** @file > -IA32, X64 and IPF Specific relocation fixups > +IA32 and X64 Specific relocation fixups > > Copyright (c) 2004 - 2018, Intel Corporation. All rights reserved.<BR> > Portions Copyright (c) 2011 - 2013, ARM Ltd. All rights reserved.<BR> > This program and the accompanying materials > are licensed and made available under the terms and conditions of the BSD > License > @@ -97,170 +97,10 @@ Returns: > --*/ > { > return RETURN_UNSUPPORTED; > } > > -RETURN_STATUS > -PeCoffLoaderRelocateIpfImage ( > - IN UINT16 *Reloc, > - IN OUT CHAR8 *Fixup, > - IN OUT CHAR8 **FixupData, > - IN UINT64 Adjust > - ) > -/*++ > - > -Routine Description: > - > - Performs an Itanium-based specific relocation fixup > - > -Arguments: > - > - Reloc - Pointer to the relocation record > - > - Fixup - Pointer to the address to fix up > - > - FixupData - Pointer to a buffer to log the fixups > - > - Adjust - The offset to adjust the fixup > - > -Returns: > - > - Status code > - > ---*/ > -{ > - UINT64 *F64; > - UINT64 FixupVal; > - > - switch ((*Reloc) >> 12) { > - > - case EFI_IMAGE_REL_BASED_IA64_IMM64: > - > - // > - // Align it to bundle address before fixing up the > - // 64-bit immediate value of the movl instruction. > - // > - > - Fixup = (CHAR8 *)((UINTN) Fixup & (UINTN) ~(15)); > - FixupVal = (UINT64)0; > - > - // > - // Extract the lower 32 bits of IMM64 from bundle > - // > - EXT_IMM64(FixupVal, > - (UINT32 *)Fixup + IMM64_IMM7B_INST_WORD_X, > - IMM64_IMM7B_SIZE_X, > - IMM64_IMM7B_INST_WORD_POS_X, > - IMM64_IMM7B_VAL_POS_X > - ); > - > - EXT_IMM64(FixupVal, > - (UINT32 *)Fixup + IMM64_IMM9D_INST_WORD_X, > - IMM64_IMM9D_SIZE_X, > - IMM64_IMM9D_INST_WORD_POS_X, > - IMM64_IMM9D_VAL_POS_X > - ); > - > - EXT_IMM64(FixupVal, > - (UINT32 *)Fixup + IMM64_IMM5C_INST_WORD_X, > - IMM64_IMM5C_SIZE_X, > - IMM64_IMM5C_INST_WORD_POS_X, > - IMM64_IMM5C_VAL_POS_X > - ); > - > - EXT_IMM64(FixupVal, > - (UINT32 *)Fixup + IMM64_IC_INST_WORD_X, > - IMM64_IC_SIZE_X, > - IMM64_IC_INST_WORD_POS_X, > - IMM64_IC_VAL_POS_X > - ); > - > - EXT_IMM64(FixupVal, > - (UINT32 *)Fixup + IMM64_IMM41a_INST_WORD_X, > - IMM64_IMM41a_SIZE_X, > - IMM64_IMM41a_INST_WORD_POS_X, > - IMM64_IMM41a_VAL_POS_X > - ); > - > - // > - // Update 64-bit address > - // > - FixupVal += Adjust; > - > - // > - // Insert IMM64 into bundle > - // > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM7B_INST_WORD_X), > - IMM64_IMM7B_SIZE_X, > - IMM64_IMM7B_INST_WORD_POS_X, > - IMM64_IMM7B_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM9D_INST_WORD_X), > - IMM64_IMM9D_SIZE_X, > - IMM64_IMM9D_INST_WORD_POS_X, > - IMM64_IMM9D_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM5C_INST_WORD_X), > - IMM64_IMM5C_SIZE_X, > - IMM64_IMM5C_INST_WORD_POS_X, > - IMM64_IMM5C_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IC_INST_WORD_X), > - IMM64_IC_SIZE_X, > - IMM64_IC_INST_WORD_POS_X, > - IMM64_IC_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM41a_INST_WORD_X), > - IMM64_IMM41a_SIZE_X, > - IMM64_IMM41a_INST_WORD_POS_X, > - IMM64_IMM41a_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM41b_INST_WORD_X), > - IMM64_IMM41b_SIZE_X, > - IMM64_IMM41b_INST_WORD_POS_X, > - IMM64_IMM41b_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_IMM41c_INST_WORD_X), > - IMM64_IMM41c_SIZE_X, > - IMM64_IMM41c_INST_WORD_POS_X, > - IMM64_IMM41c_VAL_POS_X > - ); > - > - INS_IMM64(FixupVal, > - ((UINT32 *)Fixup + IMM64_SIGN_INST_WORD_X), > - IMM64_SIGN_SIZE_X, > - IMM64_SIGN_INST_WORD_POS_X, > - IMM64_SIGN_VAL_POS_X > - ); > - > - F64 = (UINT64 *) Fixup; > - if (*FixupData != NULL) { > - *FixupData = ALIGN_POINTER(*FixupData, sizeof(UINT64)); > - *(UINT64 *)(*FixupData) = *F64; > - *FixupData = *FixupData + sizeof(UINT64); > - } > - break; > - > - default: > - return RETURN_UNSUPPORTED; > - } > - > - return RETURN_SUCCESS; > -} > > /** > Pass in a pointer to an ARM MOVT or MOVW immediate instruciton and > return the immediate data encoded in the instruction > > diff --git a/BaseTools/Source/C/EfiRom/EfiRom.h > b/BaseTools/Source/C/EfiRom/EfiRom.h > index ea179f5487..af78034727 100644 > --- a/BaseTools/Source/C/EfiRom/EfiRom.h > +++ b/BaseTools/Source/C/EfiRom/EfiRom.h > @@ -113,13 +113,12 @@ typedef struct { > // > // Machine Types > // > static STRING_LOOKUP mMachineTypes[] = { > { EFI_IMAGE_MACHINE_IA32, "IA32" }, > - { EFI_IMAGE_MACHINE_IA64, "IA64" }, > - { EFI_IMAGE_MACHINE_EBC, "EBC" }, > { EFI_IMAGE_MACHINE_X64, "X64" }, > + { EFI_IMAGE_MACHINE_EBC, "EBC" }, > { EFI_IMAGE_MACHINE_ARMT, "ARM" }, > { EFI_IMAGE_MACHINE_AARCH64, "AA64" }, > { 0, NULL } > }; > > diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c > b/BaseTools/Source/C/GenFv/GenFvInternalLib.c > index 32bbcce0a6..4d0339e1f9 100644 > --- a/BaseTools/Source/C/GenFv/GenFvInternalLib.c > +++ b/BaseTools/Source/C/GenFv/GenFvInternalLib.c > @@ -903,19 +903,11 @@ Returns: > } else { > fprintf (FvMapFile, "%s (Fixed Flash Address, ", KeyWord); > fprintf (FvMapFile, "BaseAddress=0x%010llx, ", (unsigned long long) > (ImageBaseAddress + Offset)); > } > > - if (FfsFile->Type != EFI_FV_FILETYPE_SECURITY_CORE && > pImageContext->Machine == EFI_IMAGE_MACHINE_IA64) { > - // > - // Process IPF PLABEL to get the real address after the image has been > rebased. > - // PLABEL structure is got by AddressOfEntryPoint offset to ImageBuffer > stored in pImageContext->Handle. > - // > - fprintf (FvMapFile, "EntryPoint=0x%010llx", (unsigned long long) > (*(UINT64 *)((UINTN) pImageContext->Handle + (UINTN) > AddressOfEntryPoint))); > - } else { > - fprintf (FvMapFile, "EntryPoint=0x%010llx", (unsigned long long) > (ImageBaseAddress + AddressOfEntryPoint)); > - } > + fprintf (FvMapFile, "EntryPoint=0x%010llx", (unsigned long long) > (ImageBaseAddress + AddressOfEntryPoint)); > fprintf (FvMapFile, ")\n"); > > fprintf (FvMapFile, "(GUID=%s", FileGuidName); > TextVirtualAddress = 0; > DataVirtualAddress = 0; > @@ -1539,22 +1531,19 @@ Returns: > UINT32 EntryPoint; > UINT32 BaseOfCode; > UINT16 MachineType; > EFI_PHYSICAL_ADDRESS PeiCorePhysicalAddress; > EFI_PHYSICAL_ADDRESS SecCorePhysicalAddress; > - EFI_PHYSICAL_ADDRESS *SecCoreEntryAddressPtr; > INT32 Ia32SecEntryOffset; > UINT32 *Ia32ResetAddressPtr; > UINT8 *BytePointer; > UINT8 *BytePointer2; > UINT16 *WordPointer; > UINT16 CheckSum; > UINT32 IpiVector; > UINTN Index; > EFI_FFS_FILE_STATE SavedState; > - UINT64 FitAddress; > - FIT_TABLE *FitTablePtr; > BOOLEAN Vtf0Detected; > UINT32 FfsHeaderSize; > UINT32 SecHeaderSize; > > // > @@ -1690,66 +1679,11 @@ Returns: > PeiCorePhysicalAddress += (UINTN) Pe32Section.Pe32Section + > SecHeaderSize - (UINTN) FvImage->FileImage; > PeiCorePhysicalAddress += EntryPoint; > DebugMsg (NULL, 0, 9, "PeiCore physical entry point address", "Address = > 0x%llX", (unsigned long long) PeiCorePhysicalAddress); > } > > - if (MachineType == EFI_IMAGE_MACHINE_IA64) { > - // > - // Update PEI_CORE address > - // > - // > - // Set the uncached attribute bit in the physical address > - // > - PeiCorePhysicalAddress |= 0x8000000000000000ULL; > - > - // > - // Check if address is aligned on a 16 byte boundary > - // > - if (PeiCorePhysicalAddress & 0xF) { > - Error (NULL, 0, 3000, "Invalid", > - "PEI_CORE entry point is not aligned on a 16 byte boundary, address > specified is %llXh.", > - (unsigned long long) PeiCorePhysicalAddress > - ); > - return EFI_ABORTED; > - } > - // > - // First Get the FIT table address > - // > - FitAddress = (*(UINT64 *) (FvImage->Eof - IPF_FIT_ADDRESS_OFFSET)) & > 0xFFFFFFFF; > - > - FitTablePtr = (FIT_TABLE *) (FvImage->FileImage + (FitAddress - > FvInfo->BaseAddress)); > - > - Status = UpdatePeiCoreEntryInFit (FitTablePtr, > PeiCorePhysicalAddress); > - > - if (!EFI_ERROR (Status)) { > - UpdateFitCheckSum (FitTablePtr); > - } > - > - // > - // Update SEC_CORE address > - // > - // > - // Set the uncached attribute bit in the physical address > - // > - SecCorePhysicalAddress |= 0x8000000000000000ULL; > - // > - // Check if address is aligned on a 16 byte boundary > - // > - if (SecCorePhysicalAddress & 0xF) { > - Error (NULL, 0, 3000, "Invalid", > - "SALE_ENTRY entry point is not aligned on a 16 byte boundary, > address specified is %llXh.", > - (unsigned long long) SecCorePhysicalAddress > - ); > - return EFI_ABORTED; > - } > - // > - // Update the address > - // > - SecCoreEntryAddressPtr = (EFI_PHYSICAL_ADDRESS *) ((UINTN) FvImage->Eof > - IPF_SALE_ENTRY_ADDRESS_OFFSET); > - *SecCoreEntryAddressPtr = SecCorePhysicalAddress; > - > - } else if (MachineType == EFI_IMAGE_MACHINE_IA32 || MachineType == > EFI_IMAGE_MACHINE_X64) { > +if (MachineType == EFI_IMAGE_MACHINE_IA32 || MachineType == > EFI_IMAGE_MACHINE_X64) { > if (PeiCorePhysicalAddress != 0) { > // > // Get the location to update > // > Ia32ResetAddressPtr = (UINT32 *) ((UINTN) FvImage->Eof - > IA32_PEI_CORE_ENTRY_OFFSET); > @@ -2434,11 +2368,11 @@ Returns: > } > > // > // Verify machine type is supported > // > - if ((*MachineType != EFI_IMAGE_MACHINE_IA32) && (*MachineType != > EFI_IMAGE_MACHINE_IA64) && (*MachineType != > EFI_IMAGE_MACHINE_X64) && (*MachineType != EFI_IMAGE_MACHINE_EBC) && > + if ((*MachineType != EFI_IMAGE_MACHINE_IA32) && (*MachineType != > EFI_IMAGE_MACHINE_X64) && (*MachineType != > EFI_IMAGE_MACHINE_EBC) && > (*MachineType != EFI_IMAGE_MACHINE_ARMT) && (*MachineType != > EFI_IMAGE_MACHINE_AARCH64)) { > Error (NULL, 0, 3000, "Invalid", "Unrecognized machine type in the PE32 > file."); > return EFI_UNSUPPORTED; > } > > diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.h > b/BaseTools/Source/C/GenFv/GenFvInternalLib.h > index 6cfa370645..4476b06a23 100644 > --- a/BaseTools/Source/C/GenFv/GenFvInternalLib.h > +++ b/BaseTools/Source/C/GenFv/GenFvInternalLib.h > @@ -171,20 +171,10 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, > EITHER EXPRESS OR IMPLIED. > // > // Defines to calculate the offset for IA32 SEC CORE entry point > // > #define IA32_SEC_CORE_ENTRY_OFFSET 0xD > > -// > -// Defines to calculate the FIT table > -// > -#define IPF_FIT_ADDRESS_OFFSET 0x20 > - > -// > -// Defines to calculate the offset for SALE_ENTRY > -// > -#define IPF_SALE_ENTRY_ADDRESS_OFFSET 0x18 > - > // > // Symbol file definitions, current max size if 512K > // > #define SYMBOL_FILE_SIZE 0x80000 > > diff --git a/BaseTools/Source/C/GenFw/Elf64Convert.c > b/BaseTools/Source/C/GenFw/Elf64Convert.c > index 469979c3b1..0a5237dd14 100644 > --- a/BaseTools/Source/C/GenFw/Elf64Convert.c > +++ b/BaseTools/Source/C/GenFw/Elf64Convert.c > @@ -484,11 +484,10 @@ ScanSections64 ( > // > mCoffOffset = sizeof(EFI_IMAGE_DOS_HEADER) + 0x40; > mNtHdrOffset = mCoffOffset; > switch (mEhdr->e_machine) { > case EM_X86_64: > - case EM_IA_64: > case EM_AARCH64: > mCoffOffset += sizeof (EFI_IMAGE_NT_HEADERS64); > break; > default: > VerboseMsg ("%s unknown e_machine type %hu. Assume X64", mInImageName, > mEhdr->e_machine); > @@ -691,14 +690,10 @@ ScanSections64 ( > switch (mEhdr->e_machine) { > case EM_X86_64: > NtHdr->Pe32Plus.FileHeader.Machine = EFI_IMAGE_MACHINE_X64; > NtHdr->Pe32Plus.OptionalHeader.Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC; > break; > - case EM_IA_64: > - NtHdr->Pe32Plus.FileHeader.Machine = EFI_IMAGE_MACHINE_IPF; > - NtHdr->Pe32Plus.OptionalHeader.Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC; > - break; > case EM_AARCH64: > NtHdr->Pe32Plus.FileHeader.Machine = EFI_IMAGE_MACHINE_AARCH64; > NtHdr->Pe32Plus.OptionalHeader.Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC; > break; > default: > diff --git a/BaseTools/Source/C/GenFw/GenFw.c > b/BaseTools/Source/C/GenFw/GenFw.c > index 7976bec60a..37278bbc68 100644 > --- a/BaseTools/Source/C/GenFw/GenFw.c > +++ b/BaseTools/Source/C/GenFw/GenFw.c > @@ -1012,11 +1012,11 @@ Returns: > free ((VOID *) MemoryImagePointer); > > // > // Update Image Base Address > // > - if ((ImgHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) && (ImgHdr->Pe32.FileHeader.Machine != > IMAGE_FILE_MACHINE_IA64)) { > + if ((ImgHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC)) { > ImgHdr->Pe32.OptionalHeader.ImageBase = (UINT32) NewPe32BaseAddress; > } else if (ImgHdr->Pe32Plus.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC) { > ImgHdr->Pe32Plus.OptionalHeader.ImageBase = NewPe32BaseAddress; > } else { > Error (NULL, 0, 3000, "Invalid", "unknown PE magic signature %X in PE32 > image %s", > @@ -2178,11 +2178,11 @@ Returns: > > // > // Set new base address into image > // > if (mOutImageType == FW_REBASE_IMAGE || mOutImageType == > FW_SET_ADDRESS_IMAGE) { > - if ((PeHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) && (PeHdr->Pe32.FileHeader.Machine != > IMAGE_FILE_MACHINE_IA64)) { > + if ((PeHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC)) { > if (NewBaseAddress >= 0x100000000ULL) { > Error (NULL, 0, 3000, "Invalid", "New base address is larger than 4G > for 32bit PE image"); > goto Finish; > } > } > @@ -2449,11 +2449,11 @@ Returns: > > // > // Zero the .pdata section for X64 machine and don't check the Debug > Directory is empty > // For Itaninum and X64 Image, remove .pdata section. > // > - if ((!KeepExceptionTableFlag && PeHdr->Pe32.FileHeader.Machine == > IMAGE_FILE_MACHINE_X64) || > PeHdr->Pe32.FileHeader.Machine == IMAGE_FILE_MACHINE_IA64) { > + if ((!KeepExceptionTableFlag && PeHdr->Pe32.FileHeader.Machine == > IMAGE_FILE_MACHINE_X64)) { > if (Optional64->NumberOfRvaAndSizes > > EFI_IMAGE_DIRECTORY_ENTRY_EXCEPTION && > > Optional64->DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_EXCEPTION].VirtualAddress > != 0 && > Optional64->DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_EXCEPTION].Size > != 0) { > SectionHeader = (EFI_IMAGE_SECTION_HEADER *) ((UINT8 *) > &(PeHdr->Pe32.OptionalHeader) + > PeHdr->Pe32.FileHeader.SizeOfOptionalHeader); > for (Index = 0; Index < PeHdr->Pe32.FileHeader.NumberOfSections; > Index++, SectionHeader++) { > diff --git a/BaseTools/Source/C/GenFw/elf_common.h > b/BaseTools/Source/C/GenFw/elf_common.h > index 03dec50cf3..74c253117a 100644 > --- a/BaseTools/Source/C/GenFw/elf_common.h > +++ b/BaseTools/Source/C/GenFw/elf_common.h > @@ -168,11 +168,10 @@ typedef struct { > #define EM_ARC 45 /* Argonaut RISC Core. */ > #define EM_H8_300 46 /* Hitachi H8/300. */ > #define EM_H8_300H 47 /* Hitachi H8/300H. */ > #define EM_H8S 48 /* Hitachi H8S. */ > #define EM_H8_500 49 /* Hitachi H8/500. */ > -#define EM_IA_64 50 /* Intel IA-64 Processor. */ > #define EM_MIPS_X 51 /* Stanford MIPS-X. */ > #define EM_COLDFIRE 52 /* Motorola ColdFire. */ > #define EM_68HC12 53 /* Motorola M68HC12. */ > #define EM_MMA 54 /* Fujitsu MMA. */ > #define EM_PCP 55 /* Siemens PCP. */ > @@ -773,94 +772,10 @@ typedef struct { > #define R_ARM_RREL32 252 > #define R_ARM_RABS32 253 > #define R_ARM_RPC24 254 > #define R_ARM_RBASE 255 > > - > - > -/* Name Value Field Calculation */ > -#define R_IA_64_NONE 0 /* None */ > -#define R_IA_64_IMM14 0x21 /* immediate14 S + A */ > -#define R_IA_64_IMM22 0x22 /* immediate22 S + A */ > -#define R_IA_64_IMM64 0x23 /* immediate64 S + A */ > -#define R_IA_64_DIR32MSB 0x24 /* word32 MSB S + A */ > -#define R_IA_64_DIR32LSB 0x25 /* word32 LSB S + A */ > -#define R_IA_64_DIR64MSB 0x26 /* word64 MSB S + A */ > -#define R_IA_64_DIR64LSB 0x27 /* word64 LSB S + A */ > -#define R_IA_64_GPREL22 0x2a /* immediate22 @gprel(S + A) */ > -#define R_IA_64_GPREL64I 0x2b /* immediate64 @gprel(S + A) */ > -#define R_IA_64_GPREL32MSB 0x2c /* word32 MSB @gprel(S + A) */ > -#define R_IA_64_GPREL32LSB 0x2d /* word32 LSB @gprel(S + A) */ > -#define R_IA_64_GPREL64MSB 0x2e /* word64 MSB @gprel(S + A) */ > -#define R_IA_64_GPREL64LSB 0x2f /* word64 LSB @gprel(S + A) */ > -#define R_IA_64_LTOFF22 0x32 /* immediate22 @ltoff(S + A) */ > -#define R_IA_64_LTOFF64I 0x33 /* immediate64 @ltoff(S + A) */ > -#define R_IA_64_PLTOFF22 0x3a /* immediate22 @pltoff(S + A) */ > -#define R_IA_64_PLTOFF64I 0x3b /* immediate64 @pltoff(S + A) */ > -#define R_IA_64_PLTOFF64MSB 0x3e /* word64 MSB @pltoff(S + A) */ > -#define R_IA_64_PLTOFF64LSB 0x3f /* word64 LSB @pltoff(S + A) */ > -#define R_IA_64_FPTR64I 0x43 /* immediate64 @fptr(S + A) */ > -#define R_IA_64_FPTR32MSB 0x44 /* word32 MSB @fptr(S + A) */ > -#define R_IA_64_FPTR32LSB 0x45 /* word32 LSB @fptr(S + A) */ > -#define R_IA_64_FPTR64MSB 0x46 /* word64 MSB @fptr(S + A) */ > -#define R_IA_64_FPTR64LSB 0x47 /* word64 LSB @fptr(S + A) */ > -#define R_IA_64_PCREL60B 0x48 /* immediate60 form1 S + A - P */ > -#define R_IA_64_PCREL21B 0x49 /* immediate21 form1 S + A - P */ > -#define R_IA_64_PCREL21M 0x4a /* immediate21 form2 S + A - P */ > -#define R_IA_64_PCREL21F 0x4b /* immediate21 form3 S + A - P */ > -#define R_IA_64_PCREL32MSB 0x4c /* word32 MSB S + A - P */ > -#define R_IA_64_PCREL32LSB 0x4d /* word32 LSB S + A - P */ > -#define R_IA_64_PCREL64MSB 0x4e /* word64 MSB S + A - P */ > -#define R_IA_64_PCREL64LSB 0x4f /* word64 LSB S + A - P */ > -#define R_IA_64_LTOFF_FPTR22 0x52 /* immediate22 @ltoff(@fptr(S + A)) */ > -#define R_IA_64_LTOFF_FPTR64I 0x53 /* immediate64 @ltoff(@fptr(S + A)) */ > -#define R_IA_64_LTOFF_FPTR32MSB 0x54 /* word32 MSB @ltoff(@fptr(S + A)) > */ > -#define R_IA_64_LTOFF_FPTR32LSB 0x55 /* word32 LSB @ltoff(@fptr(S + A)) > */ > -#define R_IA_64_LTOFF_FPTR64MSB 0x56 /* word64 MSB @ltoff(@fptr(S + A)) > */ > -#define R_IA_64_LTOFF_FPTR64LSB 0x57 /* word64 LSB @ltoff(@fptr(S + A)) > */ > -#define R_IA_64_SEGREL32MSB 0x5c /* word32 MSB @segrel(S + A) */ > -#define R_IA_64_SEGREL32LSB 0x5d /* word32 LSB @segrel(S + A) */ > -#define R_IA_64_SEGREL64MSB 0x5e /* word64 MSB @segrel(S + A) */ > -#define R_IA_64_SEGREL64LSB 0x5f /* word64 LSB @segrel(S + A) */ > -#define R_IA_64_SECREL32MSB 0x64 /* word32 MSB @secrel(S + A) */ > -#define R_IA_64_SECREL32LSB 0x65 /* word32 LSB @secrel(S + A) */ > -#define R_IA_64_SECREL64MSB 0x66 /* word64 MSB @secrel(S + A) */ > -#define R_IA_64_SECREL64LSB 0x67 /* word64 LSB @secrel(S + A) */ > -#define R_IA_64_REL32MSB 0x6c /* word32 MSB BD + A */ > -#define R_IA_64_REL32LSB 0x6d /* word32 LSB BD + A */ > -#define R_IA_64_REL64MSB 0x6e /* word64 MSB BD + A */ > -#define R_IA_64_REL64LSB 0x6f /* word64 LSB BD + A */ > -#define R_IA_64_LTV32MSB 0x74 /* word32 MSB S + A */ > -#define R_IA_64_LTV32LSB 0x75 /* word32 LSB S + A */ > -#define R_IA_64_LTV64MSB 0x76 /* word64 MSB S + A */ > -#define R_IA_64_LTV64LSB 0x77 /* word64 LSB S + A */ > -#define R_IA_64_PCREL21BI 0x79 /* immediate21 form1 S + A - P */ > -#define R_IA_64_PCREL22 0x7a /* immediate22 S + A - P */ > -#define R_IA_64_PCREL64I 0x7b /* immediate64 S + A - P */ > -#define R_IA_64_IPLTMSB 0x80 /* function descriptor MSB special */ > -#define R_IA_64_IPLTLSB 0x81 /* function descriptor LSB speciaal */ > -#define R_IA_64_SUB 0x85 /* immediate64 A - S */ > -#define R_IA_64_LTOFF22X 0x86 /* immediate22 special */ > -#define R_IA_64_LDXMOV 0x87 /* immediate22 special */ > -#define R_IA_64_TPREL14 0x91 /* imm14 @tprel(S + A) */ > -#define R_IA_64_TPREL22 0x92 /* imm22 @tprel(S + A) */ > -#define R_IA_64_TPREL64I 0x93 /* imm64 @tprel(S + A) */ > -#define R_IA_64_TPREL64MSB 0x96 /* word64 MSB @tprel(S + A) */ > -#define R_IA_64_TPREL64LSB 0x97 /* word64 LSB @tprel(S + A) */ > -#define R_IA_64_LTOFF_TPREL22 0x9a /* imm22 @ltoff(@tprel(S+A)) */ > -#define R_IA_64_DTPMOD64MSB 0xa6 /* word64 MSB @dtpmod(S + A) */ > -#define R_IA_64_DTPMOD64LSB 0xa7 /* word64 LSB @dtpmod(S + A) */ > -#define R_IA_64_LTOFF_DTPMOD22 0xaa /* imm22 @ltoff(@dtpmod(S+A)) */ > -#define R_IA_64_DTPREL14 0xb1 /* imm14 @dtprel(S + A) */ > -#define R_IA_64_DTPREL22 0xb2 /* imm22 @dtprel(S + A) */ > -#define R_IA_64_DTPREL64I 0xb3 /* imm64 @dtprel(S + A) */ > -#define R_IA_64_DTPREL32MSB 0xb4 /* word32 MSB @dtprel(S + A) */ > -#define R_IA_64_DTPREL32LSB 0xb5 /* word32 LSB @dtprel(S + A) */ > -#define R_IA_64_DTPREL64MSB 0xb6 /* word64 MSB @dtprel(S + A) */ > -#define R_IA_64_DTPREL64LSB 0xb7 /* word64 LSB @dtprel(S + A) */ > -#define R_IA_64_LTOFF_DTPREL22 0xba /* imm22 @ltoff(@dtprel(S+A)) */ > - > #define R_PPC_NONE 0 /* No relocation. */ > #define R_PPC_ADDR32 1 > #define R_PPC_ADDR24 2 > #define R_PPC_ADDR16 3 > #define R_PPC_ADDR16_LO 4 > diff --git a/BaseTools/Source/C/Include/IndustryStandard/PeImage.h > b/BaseTools/Source/C/Include/IndustryStandard/PeImage.h > index 6ce7fe527f..85b23ebbdd 100644 > --- a/BaseTools/Source/C/Include/IndustryStandard/PeImage.h > +++ b/BaseTools/Source/C/Include/IndustryStandard/PeImage.h > @@ -40,23 +40,20 @@ > > // > // PE32+ Machine type for EFI images > // > #define IMAGE_FILE_MACHINE_I386 0x014c > -#define IMAGE_FILE_MACHINE_IA64 0x0200 > #define IMAGE_FILE_MACHINE_EBC 0x0EBC > #define IMAGE_FILE_MACHINE_X64 0x8664 > #define IMAGE_FILE_MACHINE_ARM 0x01c0 // Thumb only > #define IMAGE_FILE_MACHINE_ARMT 0x01c2 // 32bit Mixed ARM and > Thumb/Thumb 2 Little Endian > #define IMAGE_FILE_MACHINE_ARM64 0xAA64 // 64bit ARM Architecture, > Little Endian > > // > // Support old names for backward compatible > // > #define EFI_IMAGE_MACHINE_IA32 IMAGE_FILE_MACHINE_I386 > -#define EFI_IMAGE_MACHINE_IA64 IMAGE_FILE_MACHINE_IA64 > -#define EFI_IMAGE_MACHINE_IPF IMAGE_FILE_MACHINE_IA64 > #define EFI_IMAGE_MACHINE_EBC IMAGE_FILE_MACHINE_EBC > #define EFI_IMAGE_MACHINE_X64 IMAGE_FILE_MACHINE_X64 > #define EFI_IMAGE_MACHINE_ARMT IMAGE_FILE_MACHINE_ARMT > #define EFI_IMAGE_MACHINE_AARCH64 IMAGE_FILE_MACHINE_ARM64 > > diff --git a/BaseTools/Source/C/Include/IndustryStandard/pci22.h > b/BaseTools/Source/C/Include/IndustryStandard/pci22.h > index d7c5f0c858..873ce69094 100644 > --- a/BaseTools/Source/C/Include/IndustryStandard/pci22.h > +++ b/BaseTools/Source/C/Include/IndustryStandard/pci22.h > @@ -178,11 +178,10 @@ typedef struct { > #define PCI_IF_16850 0x05 > #define PCI_IF_16950 0x06 > #define PCI_SUBCLASS_PARALLEL 0x01 > #define PCI_IF_PARALLEL_PORT 0x00 > #define PCI_IF_BI_DIR_PARALLEL_PORT 0x01 > -#define PCI_IF_ECP_PARALLEL_PORT 0x02 > #define PCI_IF_1284_CONTROLLER 0x03 > #define PCI_IF_1284_DEVICE 0xFE > #define PCI_SUBCLASS_MULTIPORT_SERIAL 0x02 > #define PCI_SUBCLASS_MODEM 0x03 > #define PCI_IF_GENERIC_MODEM 0x00 > diff --git a/BaseTools/Source/C/Makefiles/header.makefile > b/BaseTools/Source/C/Makefiles/header.makefile > index 7f283d6464..7828bbe4dc 100644 > --- a/BaseTools/Source/C/Makefiles/header.makefile > +++ b/BaseTools/Source/C/Makefiles/header.makefile > @@ -1,11 +1,10 @@ > ## @file > # > # The makefile can be invoked with > # HOST_ARCH = x86_64 or x64 for EM64T build > # HOST_ARCH = ia32 or IA32 for IA32 build > -# HOST_ARCH = ia64 or IA64 for IA64 build > # HOST_ARCH = Arm or ARM for ARM build > # > # Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR> > # This program and the accompanying materials > # are licensed and made available under the terms and conditions of the BSD > License > diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c > b/BaseTools/Source/C/VolInfo/VolInfo.c > index bf00af0566..89fe3a044d 100644 > --- a/BaseTools/Source/C/VolInfo/VolInfo.c > +++ b/BaseTools/Source/C/VolInfo/VolInfo.c > @@ -1571,11 +1571,11 @@ Returns: > free ((VOID *) MemoryImagePointer); > > // > // Update Image Base Address > // > - if ((ImgHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) && (ImgHdr->Pe32.FileHeader.Machine != > IMAGE_FILE_MACHINE_IA64)) { > + if ((ImgHdr->Pe32.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC)) { > ImgHdr->Pe32.OptionalHeader.ImageBase = (UINT32) NewPe32BaseAddress; > } else if (ImgHdr->Pe32Plus.OptionalHeader.Magic == > EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC) { > ImgHdr->Pe32Plus.OptionalHeader.ImageBase = NewPe32BaseAddress; > } else { > Error (NULL, 0, 3000, "Invalid", "unknown PE magic signature %X in PE32 > image %s", > -- > 2.19.1.windows.1
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel