eficom: Don't allow this for !HYPERV machines
ClosedPublic
Actions

Authored by imp on May 22 2023, 10:28 PM.

Details

Reviewers

manu
kevans
emaste
tsoome

Commits

rG42b0b7a926e3: stand/efi/eficom: Don't allow this for !HYPERV machines

Summary

If the machine isn't hyperv on amd64, then this driver fails the probe
and will do nothing further now, even if explicitly listed in a config.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 51619
Build 48510: arc lint + arc unit

Event Timeline

imp created this revision.May 22 2023, 10:28 PM

Herald added a reviewer: manu. · View Herald TranscriptMay 22 2023, 10:28 PM

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

imp requested review of this revision.May 22 2023, 10:28 PM

Harbormaster completed remote builds in B51619: Diff 122258.May 22 2023, 10:28 PM

imp added a parent revision: D40220: stand/efi/eficom: better handling of absent device.May 22 2023, 10:28 PM

imp added a child revision: D40222: stand/efi/eficom: Make aarch64 compat code probe correctly.

imp added reviewers: kevans, emaste, tsoome.May 22 2023, 10:31 PM

gallatin added a subscriber: gallatin.May 23 2023, 1:41 PM

kevans accepted this revision.May 24 2023, 7:47 PM

This revision is now accepted and ready to land.May 24 2023, 7:47 PM

Closed by commit rG42b0b7a926e3: stand/efi/eficom: Don't allow this for !HYPERV machines (authored by imp). · Explain WhyMay 25 2023, 4:34 AM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG42b0b7a926e3: stand/efi/eficom: Don't allow this for !HYPERV machines.

Revision Contents
Changeset List

Path

Size

stand/

efi/

libefi/

eficom.c

17 lines

Diff 122258

View Options

stand/efi/eficom: Don't allow this for !HYPERV machinesClosedPublicActions