public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [Patch 3/3] BaseTools: Remove unused logic from C tools
@ 2019-01-09  6:44 BobCF
  2019-01-09  7:03 ` Gao, Liming
  0 siblings, 1 reply; 2+ messages in thread
From: BobCF @ 2019-01-09  6:44 UTC (permalink / raw)
  To: edk2-devel; +Cc: Feng, Bob C, Liming Gao, Jaben Carsey

From: "Feng, Bob C" <bob.c.feng@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.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Jaben Carsey <jaben.carsey@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



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Patch 3/3] BaseTools: Remove unused logic from C tools
  2019-01-09  6:44 [Patch 3/3] BaseTools: Remove unused logic from C tools BobCF
@ 2019-01-09  7:03 ` Gao, Liming
  0 siblings, 0 replies; 2+ messages in thread
From: Gao, Liming @ 2019-01-09  7:03 UTC (permalink / raw)
  To: Feng, Bob C, edk2-devel@lists.01.org; +Cc: Carsey, Jaben

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.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Carsey, Jaben <jaben.carsey@intel.com>
> Subject: [Patch 3/3] BaseTools: Remove unused logic from C tools
> 
> From: "Feng, Bob C" <bob.c.feng@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.feng@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Jaben Carsey <jaben.carsey@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



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-01-09  7:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-01-09  6:44 [Patch 3/3] BaseTools: Remove unused logic from C tools BobCF
2019-01-09  7:03 ` Gao, Liming

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox