[PATCH] efi_loader: move efi_var_seed.o to objtree

[Heinrich Schuchardt]

Am 23. Juni 2026 21:03:38 MESZ schrieb Jon Mason <jdmason at kudzu.us>:
>On Fri, Jun 19, 2026 at 12:46 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>>
>> Am 19. Juni 2026 17:58:57 MESZ schrieb Jon Mason <jon.mason at arm.com>:
>> >With commit bd3f9ee679b4d1456d0d3c261ab76788950e6096
>> >(kbuild: Bump the build system to 6.1), there is an issue when trying to
>> >build out of tree, where efi_var_seed.S wants ubootefi.var to be in the
>> >object tree, but lib/efi_loader/efi_var_seed.o wants ubootefi.var to be
>> >in the source tree.
>>
>> Ubootefi.var is an optional file with the EFI security database. Why would you ever put it into the build directory?
>
>I didn't know that was a limitation.
>
>Would you prefer that I make a change like:
>-$(obj)/efi_var_seed.o: $(srctree)/$(EFI_VAR_SEED_FILE)
>+$(obj)/efi_var_seed.o: $(srctree)/lib/efi_loader/$(EFI_VAR_SEED_FILE)

Hello Jon,

It is still not clear to me why you want to change this.

If you don't like the current default, why wouldn't you change CONFIG_EFI_VAR_SEED_FILE in your local .config file to match the storage location of your seed file?

Best regards

Heinrich


>
>This seems to work for my use case, and still keeps it in the src directory.
>
>Thanks,
>Jon
>
>>
>> Heinrich
>>
>> >
>> >If we change efi_var_seed.o to look in the objtree, then it should work
>> >in both local and out-of-tree.  However, it would require anything
>> >previously working to put ubootefi.vars to the objtree, but it's broken
>> >now anyway.
>> >
>> >Signed-off-by: Jon Mason <jon.mason at arm.com>
>> >---
>> > lib/efi_loader/Makefile | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> >diff --git a/lib/efi_loader/Makefile b/lib/efi_loader/Makefile
>> >index f490081f6542..9c836f6ee237 100644
>> >--- a/lib/efi_loader/Makefile
>> >+++ b/lib/efi_loader/Makefile
>> >@@ -74,7 +74,7 @@ obj-$(CONFIG_EFI_ECPT) += efi_conformance.o
>> > obj-$(CONFIG_EFI_DEBUG_SUPPORT) += efi_debug_support.o
>> >
>> > EFI_VAR_SEED_FILE := $(subst $\",,$(CONFIG_EFI_VAR_SEED_FILE))
>> >-$(obj)/efi_var_seed.o: $(srctree)/$(EFI_VAR_SEED_FILE)
>> >+$(obj)/efi_var_seed.o: $(objtree)/$(EFI_VAR_SEED_FILE)
>> >
>> > ifeq ($(CONFIG_EFI_CAPSULE_AUTHENTICATE),y)
>> > capsule_crt_path=($(subst $(quote),,$(CONFIG_EFI_CAPSULE_CRT_FILE)))
>>