diff mbox series

[1/2] mkimage: Simplify header size logic

Message ID 20181128143107.37583-2-agraf@suse.de
State Superseded
Headers show
Series arm64: Support HP Envy X2 | expand

Commit Message

Alexander Graf Nov. 28, 2018, 2:31 p.m. UTC
For EFI images, we always have the following layout:

  [PE header]
  [padding]
  [first section (which also is the entry point)]

Currently there are 2 places where we define how big header+padding are:
in the .vaddr_offset member of our target image definition struct as well
as in code in grub_install_generate_image().

Remove the latter, so that we only have a single place to modify if we
need to change the padding.

Signed-off-by: Alexander Graf <agraf@suse.de>

---
 util/mkimage.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

-- 
2.19.0


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

Patch

diff --git a/util/mkimage.c b/util/mkimage.c
index b2f43fea6..05333a76a 100644
--- a/util/mkimage.c
+++ b/util/mkimage.c
@@ -1206,10 +1206,7 @@  grub_install_generate_image (const char *dir, const char *prefix,
 	int header_size;
 	int reloc_addr;
 
-	if (image_target->voidp_sizeof == 4)
-	  header_size = EFI32_HEADER_SIZE;
-	else
-	  header_size = EFI64_HEADER_SIZE;
+	header_size = image_target->vaddr_offset;
 
 	reloc_addr = ALIGN_UP (header_size + core_size,
 			       image_target->section_align);