[PATCH] efi_loader: efi_var_file: specify absolute path

[Caleb Connolly]

On 2/10/25 14:04, Heinrich Schuchardt wrote:
> On 10.02.25 14:10, Caleb Connolly wrote:
>> The EXT4 FS driver gets unhappy if you attempt to write a file without a
>> '/' prefix. Adjust the EFI_VAR_FILE_NAME string to be the full path to
>> the file.
> 
> Thank you for reporting the issue.
> 
> Our FS systems should all behave the same. U-Boot file paths are always
> absolute. We should identify the problem in the ext4 driver.
> 
> I thought I fixed this in
> 4ed2b1af4599 ("fs: ext4: all file paths are absolute")

hmm, ahh yeah that looks right. For some reason I thought this patch was 
still relevant but it seems not.
> 
> Best regards
> 
> Heinrich
> 
>>
>> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
>> ---
>>   include/efi_variable.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/efi_variable.h b/include/efi_variable.h
>> index 4065cf45ecaf..83fbca93b6bd 100644
>> --- a/include/efi_variable.h
>> +++ b/include/efi_variable.h
>> @@ -91,9 +91,9 @@ efi_status_t efi_query_variable_info_int(u32 
>> attributes,
>>                        u64 *maximum_variable_storage_size,
>>                        u64 *remaining_variable_storage_size,
>>                        u64 *maximum_variable_size);
>>
>> -#define EFI_VAR_FILE_NAME "ubootefi.var"
>> +#define EFI_VAR_FILE_NAME "/ubootefi.var"
>>
>>   #define EFI_VAR_BUF_SIZE CONFIG_EFI_VAR_BUF_SIZE
>>
>>   /*
> 

-- 
Caleb (they/them)