Currently file ubootefi.var is only created if the user sets a non-volatile
EFI variable. If the file is missing, a warning is written.

With the change PlatformLang is always persisted. So the file will exist on
second boot.

Signed-off-by: Heinrich Schuchardt <[email protected]>
---
 lib/efi_loader/efi_variable.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/lib/efi_loader/efi_variable.c b/lib/efi_loader/efi_variable.c
index 503a33ed65..7c32adf6e5 100644
--- a/lib/efi_loader/efi_variable.c
+++ b/lib/efi_loader/efi_variable.c
@@ -334,9 +334,11 @@ efi_status_t efi_set_variable_int(const u16 *variable_name,
        else
                ret = EFI_SUCCESS;
 
-       /* Write non-volatile EFI variables to file */
-       if (attributes & EFI_VARIABLE_NON_VOLATILE &&
-           ret == EFI_SUCCESS && efi_obj_list_initialized == EFI_SUCCESS)
+       /*
+        * Write non-volatile EFI variables to file
+        * TODO: check if a value change has occured to avoid superfluous writes
+        */
+       if (attributes & EFI_VARIABLE_NON_VOLATILE)
                efi_var_to_file();
 
        return EFI_SUCCESS;
-- 
2.38.1

Reply via email to