Bug 210242
| Summary: | [PATCH] [EFI] Simplify libefi delay() implementation | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Base System | Reporter: | rebecca+freebsd <rebecca+freebsd> | ||||
| Component: | misc | Assignee: | Rebecca Cran <bcran> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | Andrew, bcran, emaste, imp, kevans | ||||
| Priority: | --- | Keywords: | patch | ||||
| Version: | 10.3-RELEASE | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
CC imp@, You aware of any problems with BootServices->Stall() ? This was fixed by r339796 on 2018-10-26. |
Created attachment 171361 [details] Patch to simplify delay() implementation The `delay` implementation currently uses timers to delay execution, while there's a built-in `Stall` function available in the EFI_BOOT_SERVICES table. I'm not sure if `Stall` has problems on certain platforms that's being worked around through the use of timers, but it appears as though we can simplify the implementation.