Page MenuHomeFreeBSD

vt/vga: ignore ACPI_FADT_NO_VGA unless running virtualized
ClosedPublic

Authored by royger on Feb 28 2022, 9:29 AM.
Tags
None
Referenced Files
F108696190: D34392.diff
Mon, Jan 27, 9:17 AM
F108592704: D34392.id103938.diff
Sun, Jan 26, 6:02 PM
F108592246: D34392.id103941.diff
Sun, Jan 26, 5:58 PM
Unknown Object (File)
Thu, Jan 23, 6:27 PM
Unknown Object (File)
Sat, Jan 11, 11:40 AM
Unknown Object (File)
Dec 9 2024, 2:14 AM
Unknown Object (File)
Nov 26 2024, 4:31 AM
Unknown Object (File)
Nov 25 2024, 2:47 PM
Subscribers

Details

Summary

There's too many broken hardware out there that wrongly has the
ACPI_FADT_NO_VGA bit set. Ignore it unless running as a virtualized
guest, as then the expectation would be that the hypervisor does
provide correct ACPI tables.

Sponsored by: Citrix Systems R&D
PR: 230172

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44591
Build 41479: arc lint + arc unit

Event Timeline

emaste added inline comments.
sys/dev/vt/hw/vga/vt_vga.c
1267

Some other cases in the tree seem to add (not actually necessary) parens

ignore = (vm_guest == VM_GUEST_NO);

to me this is a bit clearer.

This revision is now accepted and ready to land.Feb 28 2022, 4:06 PM

Also let's MFC this to include in 13.1

We should update vt.4 also

hw.vga.acpi_ignore_no_vga
        Set to 1 to force the usage of the VGA driver regardless of
        whether ACPI IAPC_BOOT_ARCH signals no VGA support.  Can be used
        to workaround firmware bugs in the ACPI tables.
This revision now requires review to proceed.Mar 17 2022, 11:54 AM

Please merge it to stable/12, too.

This revision is now accepted and ready to land.Mar 17 2022, 12:34 PM
0mp added a subscriber: 0mp.

The manual page change seems OK.

share/man/man4/vt.4
209 ↗(On Diff #103938)

Please bump the Dd of the manual page when committing.