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
F110440549: D34392.diff
Tue, Feb 18, 12:26 PM
Unknown Object (File)
Tue, Jan 28, 4:13 AM
Unknown Object (File)
Tue, Jan 28, 4:12 AM
Unknown Object (File)
Mon, Jan 27, 9:17 AM
Unknown Object (File)
Sun, Jan 26, 6:02 PM
Unknown Object (File)
Sun, Jan 26, 5:58 PM
Unknown Object (File)
Thu, Jan 23, 6:27 PM
Unknown Object (File)
Jan 11 2025, 11:40 AM
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.