Differential D43714

loader: For EFI, if we don't have ConOut, try ConIn
ClosedPublic
Actions

Authored by imp on Feb 2 2024, 9:29 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 27, 11:02 AM

	Unknown Object (File)
	Thu, Sep 24, 9:15 AM

	Unknown Object (File)
	Wed, Sep 23, 6:33 AM

	Unknown Object (File)
	Sun, Sep 20, 12:50 PM

	Unknown Object (File)
	Sep 6 2026, 9:33 AM

	Unknown Object (File)
	Sep 1 2026, 10:03 PM

	Unknown Object (File)
	Sep 1 2026, 3:38 PM

	Unknown Object (File)
	Aug 31 2026, 12:44 PM

Subscribers

Details

Reviewers

manu
kevans
tsoome
dab

Commits

rG9d2f548bbebd: loader: For EFI, if we don't have ConOut, try ConIn
rG20a6f4779ac6: loader: For EFI, if we don't have ConOut, try ConIn
rG064fa628ce37: loader: For EFI, if we don't have ConOut, try ConIn

Summary

Try ConIn if we don't have a ConOut variable. ConIn will contain HID
devices and/or serial devices. We currently just search for serial
devices and will use them instead of video with the current code. While
ConIn w/o ConOut is fairly common on laptops, is kinda rare on servers.
Some refinement may be needed in the future if servers come to
light. This is also minimal to allow possible integration into 13.3
release.

MFC After: 1 week
Sponsored by: Netflix

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

imp created this revision.Feb 2 2024, 9:29 PM

Herald added a reviewer: manu. · View Herald TranscriptFeb 2 2024, 9:29 PM

Herald added subscribers: dab, bcran. · View Herald Transcript

imp requested review of this revision.Feb 2 2024, 9:29 PM

Harbormaster completed remote builds in B55765: Diff 133766.Feb 2 2024, 9:29 PM

imp added reviewers: kevans, tsoome.Feb 2 2024, 9:30 PM

tsoome accepted this revision.Feb 3 2024, 6:28 AM

This revision is now accepted and ready to land.Feb 3 2024, 6:28 AM

dab accepted this revision.Feb 3 2024, 11:49 PM

Closed by commit rG064fa628ce37: loader: For EFI, if we don't have ConOut, try ConIn (authored by imp). · Explain WhyFeb 4 2024, 9:12 PM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG064fa628ce37: loader: For EFI, if we don't have ConOut, try ConIn.

imp added a commit: rG20a6f4779ac6: loader: For EFI, if we don't have ConOut, try ConIn.Feb 7 2024, 12:13 AM

imp added a commit: rG9d2f548bbebd: loader: For EFI, if we don't have ConOut, try ConIn.

Revision Contents
Changeset List

Path

Size

stand/

efi/

loader/

7 lines

Diff 133852

stand/efi/loader/main.c

Loading...