Page MenuHomeFreeBSD
Differential D16753

Simplify the EFI delay() function by calling BS->Stall().
ClosedPublic
Actions

Authored by bcran on Aug 16 2018, 7:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 19, 6:21 PM
Unknown Object (File)
Sat, Jan 18, 10:16 PM
Unknown Object (File)
Sat, Jan 18, 5:28 PM
Unknown Object (File)
Fri, Jan 17, 10:23 PM
Unknown Object (File)
Thu, Dec 26, 6:44 PM
Unknown Object (File)
Dec 10 2024, 1:49 PM
Unknown Object (File)
Dec 8 2024, 10:25 AM
Unknown Object (File)
Dec 8 2024, 10:25 AM
View All 70 Files
Subscribers
None

Details

Reviewers
imp
kevans
manu
eadler
Commits
rS339796: Simplify the EFI delay() function by calling BS->Stall()
Test Plan

Tested on Qemu 2.12.0-rc3 with OVMF.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18865
Build 18522: arc lint + arc unit

Event Timeline

bcran created this revision.Aug 16 2018, 7:13 PM
Harbormaster completed remote builds in B18865: Diff 46794.Aug 16 2018, 7:13 PM
bcran added reviewers: kevans, manu.Aug 17 2018, 1:21 AM
bcran added a reviewer: eadler.
imp requested changes to this revision.Aug 17 2018, 2:30 AM
imp added inline comments.
stand/efi/libefi/delay.c
36

When was this function added to the UEFI standard? If it's too new, then you'll need to provide a fallback...

This revision now requires changes to proceed.Aug 17 2018, 2:30 AM
bcran added inline comments.Aug 17 2018, 2:42 AM
stand/efi/libefi/delay.c
36

It's in UEFI 2.0 from 2006, and EFI 1.10 from 2002.

imp accepted this revision.Aug 17 2018, 2:42 AM
This revision is now accepted and ready to land.Aug 17 2018, 2:42 AM
bcran added a comment.Sep 14 2018, 1:01 AM

I've now tested the change on an ARM64 Overdrive 1000, an EFI 1.10 MacBook Pro from 2007, and a ThinkPad T530.

manu accepted this revision.Sep 14 2018, 7:02 AM

I haven't test on u-boot but the function is defined (https://elixir.bootlin.com/u-boot/latest/source/lib/efi_loader/efi_boottime.c#L1938) so ok for me.

imp added a comment.Sep 14 2018, 9:07 PM

This year's award for going above and beyond in testing goes to...

manu added a comment.Sep 14 2018, 10:36 PM
In D16753#366156, @imp wrote:

This year's award for going above and beyond in testing goes to...

Tested with U-Boot 2018.07 (current in ports) and upcoming 2018.09 :)

Closed by commit rS339796: Simplify the EFI delay() function by calling BS->Stall() (authored by bcran). · Explain WhyOct 26 2018, 9:57 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
stand/
efi/
libefi/
12 lines

Diff 46794

View Options

stand/efi/libefi/delay.c

Loading...