diff options
| author | Jan Setje-Eilers <jan.setjeeilers@oracle.com> | 2021-02-20 16:18:16 -0800 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2021-02-22 11:21:11 -0500 |
| commit | b9ca0895a0679d4d3223685d37f0600b82eae49c (patch) | |
| tree | 907d27326080fc22f3df95c654a41d750d65598a | |
| parent | 2b829d22cc26d032d8e778080f3294e99567cfc5 (diff) | |
| download | efi-boot-shim-b9ca0895a0679d4d3223685d37f0600b82eae49c.tar.gz efi-boot-shim-b9ca0895a0679d4d3223685d37f0600b82eae49c.zip | |
Drop --set-section-alignment from Makefile since linker ALIGN(4096)
already enforces the alignment, clarify that objcopy only needs to
do the alignment in the SBAT spec.
| -rw-r--r-- | Makefile | 2 | ||||
| -rw-r--r-- | SBAT.md | 5 |
2 files changed, 6 insertions, 1 deletions
@@ -95,7 +95,7 @@ VENDOR_SBATS := $(foreach x,$(wildcard data/sbat.*.csv),$(notdir $(x))) sbat_data.o : | $(SBATPATH) $(VENDOR_SBATS) sbat_data.o : /dev/null $(CC) $(CFLAGS) -x c -c -o $@ $< - $(OBJCOPY) --set-section-alignment '.sbat=512' --add-section .sbat=$(SBATPATH) $@ + $(OBJCOPY) --add-section .sbat=$(SBATPATH) $@ $(foreach vs,$(VENDOR_SBATS),$(call add-vendor-sbat,$(vs),$@)) $(SHIMNAME) : $(SHIMSONAME) @@ -336,6 +336,11 @@ objcopy --set-section-alignment '.sbat=512' --add-section .sbat=sbat.csv foo.efi ``` +Older versions of objcopy(1) do not support --set-section-alignment +which is required to force the correct alignment expected from a PE +file. As long as there is another step, later in the build process, +such as an linker invocation that forces alignment, objcopy(1) does +not need to align an intermediate file. #### UEFI SBAT Variable content |