Page MenuHomeFreeBSD
Differential D46282

bhyve: Make boot ROM handling more consistent
ClosedPublic
Actions

Authored by markj on Tue, Aug 13, 5:20 PM.
Tags
None
Referenced Files
F93235955: D46282.id142048.diff
Sun, Sep 8, 8:47 AM
F93208939: D46282.id.diff
Sun, Sep 8, 3:51 AM
Unknown Object (File)
Sun, Sep 8, 2:14 AM
Unknown Object (File)
Sat, Sep 7, 8:10 PM
Unknown Object (File)
Sun, Sep 1, 2:57 PM
Unknown Object (File)
Sun, Sep 1, 2:56 PM
Unknown Object (File)
Thu, Aug 29, 6:05 AM
Unknown Object (File)
Mon, Aug 19, 1:59 PM
View All 9 Files
Subscribers
bcran
imp
rgrimes

Details

Reviewers
corvink
jrtc27
jhb
Group Reviewers
bhyve
Commits
rG43caa2e805c2: bhyve: Make boot ROM handling more consistent
Summary
  • On amd64, deprecate lpc.bootrom and lpc.bootvars. Use top-level config variables instead.
  • Introduce a generic predicate which can be used to determine whether the guest has a boot ROM.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Tue, Aug 13, 5:20 PM
Herald added a reviewer: bhyve. · View Herald TranscriptTue, Aug 13, 5:20 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
markj requested review of this revision.Tue, Aug 13, 5:20 PM
Harbormaster completed remote builds in B59004: Diff 142048.Tue, Aug 13, 5:20 PM
markj added a child revision: D45049: bhyve: Do not enable PCI BAR decoding if a boot ROM is present.Tue, Aug 13, 5:20 PM
corvink accepted this revision.Wed, Aug 14, 6:16 AM
This revision is now accepted and ready to land.Wed, Aug 14, 6:16 AM
jhb accepted this revision.Mon, Aug 19, 1:38 PM
jhb added inline comments.
usr.sbin/bhyve/bhyve_config.5
126

Is the detail about the memory location true on arm64?

markj marked an inline comment as done.Mon, Aug 19, 1:49 PM
markj added inline comments.
usr.sbin/bhyve/bhyve_config.5
126

No, it's wrong on arm64. I'll reword it to something more generic.

Closed by commit rG43caa2e805c2: bhyve: Make boot ROM handling more consistent (authored by markj). · Explain WhyMon, Aug 19, 1:59 PM
This revision was automatically updated to reflect the committed changes.
markj marked an inline comment as done.
markj added a commit: rG43caa2e805c2: bhyve: Make boot ROM handling more consistent.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
amd64/
20 lines
3 lines
3 lines
1 line
19 lines
24 lines
7 lines
3 lines
17 lines

Diff 142158

View Options

usr.sbin/bhyve/amd64/bhyverun_machdep.c

Loading...
View Options

usr.sbin/bhyve/amd64/ioapic.c

Loading...
View Options

usr.sbin/bhyve/amd64/pci_irq.c

Loading...
View Options

usr.sbin/bhyve/amd64/pci_lpc.h

Loading...
View Options

usr.sbin/bhyve/amd64/pci_lpc.c

Loading...
View Options

usr.sbin/bhyve/bhyve_config.5

Loading...
View Options

usr.sbin/bhyve/bhyverun.c

Loading...
View Options

usr.sbin/bhyve/bootrom.h

Loading...
View Options

usr.sbin/bhyve/bootrom.c

Loading...