Re: [PATCH v2 1/8] PE: Add LoongArch definitions

[WANG Xuerui]

On 2/23/22 21:46, Xiaotian Wu wrote:

Signed-off-by: Xiaotian Wu <wuxiaot...@loongson.cn>
Signed-off-by: Zhou Yang <zhouy...@loongson.cn>
---
  include/grub/efi/pe32.h | 36 ++++++++++++++++++++----------------
  1 file changed, 20 insertions(+), 16 deletions(-)

diff --git a/include/grub/efi/pe32.h b/include/grub/efi/pe32.h
index 0ed8781f0..de56edef6 100644
--- a/include/grub/efi/pe32.h
+++ b/include/grub/efi/pe32.h
@@ -77,6 +77,8 @@ struct grub_pe32_coff_header
  #define GRUB_PE32_MACHINE_X86_64              0x8664
  #define GRUB_PE32_MACHINE_ARMTHUMB_MIXED      0x01c2
  #define GRUB_PE32_MACHINE_ARM64                       0xAA64
+#define GRUB_PE32_MACHINE_LOONGARCH32          0x6232
+#define GRUB_PE32_MACHINE_LOONGARCH64          0x6264
  #define GRUB_PE32_MACHINE_RISCV32             0x5032
  #define GRUB_PE32_MACHINE_RISCV64             0x5064

-#define GRUB_PE32_REL_BASED_HIGH       1
-#define GRUB_PE32_REL_BASED_LOW                2
-#define GRUB_PE32_REL_BASED_HIGHLOW    3
-#define GRUB_PE32_REL_BASED_HIGHADJ    4
-#define GRUB_PE32_REL_BASED_MIPS_JMPADDR 5
-#define GRUB_PE32_REL_BASED_ARM_MOV32A  5
-#define GRUB_PE32_REL_BASED_RISCV_HI20 5
-#define GRUB_PE32_REL_BASED_SECTION    6
-#define GRUB_PE32_REL_BASED_REL                7
-#define GRUB_PE32_REL_BASED_ARM_MOV32T  7
-#define GRUB_PE32_REL_BASED_RISCV_LOW12I 7
-#define GRUB_PE32_REL_BASED_RISCV_LOW12S 8
-#define GRUB_PE32_REL_BASED_IA64_IMM64 9
-#define GRUB_PE32_REL_BASED_DIR64      10
-#define GRUB_PE32_REL_BASED_HIGH3ADJ   11
+#define GRUB_PE32_REL_BASED_ABSOLUTE           0
+#define GRUB_PE32_REL_BASED_HIGH               1
+#define GRUB_PE32_REL_BASED_LOW                        2
+#define GRUB_PE32_REL_BASED_HIGHLOW            3
+#define GRUB_PE32_REL_BASED_HIGHADJ            4
+#define GRUB_PE32_REL_BASED_MIPS_JMPADDR       5
+#define GRUB_PE32_REL_BASED_ARM_MOV32A         5
+#define GRUB_PE32_REL_BASED_RISCV_HI20         5
+#define GRUB_PE32_REL_BASED_SECTION            6
+#define GRUB_PE32_REL_BASED_REL                        7
+#define GRUB_PE32_REL_BASED_ARM_MOV32T         7
+#define GRUB_PE32_REL_BASED_RISCV_LOW12I       7
+#define GRUB_PE32_REL_BASED_RISCV_LOW12S       8
+#define GRUB_PE32_REL_BASED_LOONGARCH32_MARK_LA        8
+#define GRUB_PE32_REL_BASED_LOONGARCH64_MARK_LA        8
+#define GRUB_PE32_REL_BASED_IA64_IMM64         9
+#define GRUB_PE32_REL_BASED_DIR64              10
+#define GRUB_PE32_REL_BASED_HIGH3ADJ           11

  {

@@ -283,22 +285,24 @@ struct grub_pe32_fixup_block #define GRUB_PE32_FIXUP_ENTRY(type, offset) (((type) << 12) | (offset)) -#define GRUB_PE32_REL_BASED_ABSOLUTE 0 struct grub_pe32_symbol

Reviewed-by: WANG Xuerui <g...@xen0n.name>

[1]: https://docs.microsoft.com/en-us/windows/win32/debug/pe-format
[2]: https://github.com/MicrosoftDocs/win32/pull/1067

This is consistent with the updated PE format docs [1] (added with this Microsoft PR [2]), so:

OpenPGP_0xC01F7214BC931414.asc
Description: OpenPGP public key

OpenPGP_signature
Description: OpenPGP digital signature

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel