[PATCH] efi_loader: avoid superfluous variable store writes on unchanged data

Ilias Apalodimas ilias.apalodimas at linaro.org
Fri Mar 13 11:28:34 CET 2026


Thanks Michal


The code looks sane apart from the runtime part.

[...]

> +
> +                       if (!memcmp(old_data, data1, size1)) {

So this will probably blowup in Linux. We had similar issues with
memcpy and added efi_memcpy_runtime()

> +                               *changep = false;
> +                               return EFI_SUCCESS;
> +                       }
> +               }
> +       }
> +
[...]\

Other than that LGTM

Thanks
/Ilias


More information about the U-Boot mailing list