amd64: Remove sc(4) from MINIMAL
AcceptedPublic
Actions

Authored by manu on Jun 21 2022, 6:15 PM.

Details

Reviewers

emaste
imp

Summary

vt(4) supports bios and efi boot.
And since 0518832011 ("vt/vga: ignore ACPI_FADT_NO_VGA unless running virtualized")
was applied it will work everywhere.

MFC after: 1 month
MFC to: stable/13 only
Sponsored by: Beckhoff Automation GmbH & Co. KG

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 46052
Build 42941: arc lint + arc unit

Event Timeline

manu created this revision.Jun 21 2022, 6:15 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 21 2022, 6:15 PM

manu requested review of this revision.Jun 21 2022, 6:15 PM

Harbormaster completed remote builds in B46052: Diff 107216.Jun 21 2022, 6:15 PM

manu added a parent revision: D35538: amd64: Remove sc(4) from GENERIC.Jun 21 2022, 6:15 PM

manu added a reviewer: emaste.Jun 21 2022, 6:19 PM

emaste accepted this revision.Jun 21 2022, 6:48 PM

This revision is now accepted and ready to land.Jun 21 2022, 6:48 PM

My only objection is that I can NOT get textmode or very stable X on any of the NVIDIA
cards I use unless I build against sc. Does sc(4) use so much space that current kernels
become too big with it's presence? I vote against it's removal.

Removing the 'backup' console from MINIMAL is fine.
Not too many people are using it, and those that are can hack up a MINIMAL_MINUS_VT_PLUS_SC config file easily enough.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

conf/

MINIMAL

6 lines

Diff 107216

View Options

amd64: Remove sc(4) from MINIMALAcceptedPublicActions