On 19.02.24 05:10, 'Earl Chew' via EFI Boot Guard wrote:
> Jan,
> 
>> Why do you do this twice? You already detect this case during configure
>> time.
> 
> The runtime check ensures that the argument to BS->Stall() never overflows.
> Having said that, perhaps this would be more fitting, and then forget about
> the check at configure time:
> 
> +    const UINTN max_delay = ~ (UINTN) 0 / 1000 / 1000;
> +    BS->Stall(1000 * 1000 *
> +        (ENV_BOOT_DELAY < max_delay ? ENV_BOOT_DELAY : max_delay));
> 

This is a build-time configurable, so let's check at build-time.

Jan

-- 
Siemens AG, Technology
Linux Expert Center

-- 
You received this message because you are subscribed to the Google Groups "EFI 
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to efibootguard-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/efibootguard-dev/a32ee05f-31ce-43ac-b5e6-990ee6d866ea%40siemens.com.

Reply via email to