Page MenuHomeFreeBSD
Differential D36992

[PATCH 9/19] bhyve: add basl support for common table header
ClosedPublic
Actions

Authored by corvink on Oct 14 2022, 9:18 AM.
Tags
Referenced Files
Unknown Object (File)
Tue, Nov 4, 2:41 AM
Unknown Object (File)
Tue, Nov 4, 1:44 AM
Unknown Object (File)
Thu, Oct 30, 8:47 PM
Unknown Object (File)
Mon, Oct 27, 1:40 AM
Unknown Object (File)
Sun, Oct 26, 12:47 AM
Unknown Object (File)
Sat, Oct 11, 5:42 PM
Unknown Object (File)
Sat, Oct 11, 8:06 AM
Unknown Object (File)
Thu, Oct 9, 12:57 PM
View All Files
Subscribers
bcran
imp
rgrimes

Details

Reviewers
jhb
markj
rew
Group Reviewers
bhyve
Commits
rG818d66e29bcb: bhyve: add basl support for common table header
rG2fb0f352b977: bhyve: add basl support for common table header
Summary

Most ACPI tables are using the same header. Make it easy to create this
header by creating a function for it.

This is the 9. patch required to merge D36983

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 48355
Build 45241: arc lint + arc unit

Event Timeline

corvink created this revision.Oct 14 2022, 9:18 AM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptOct 14 2022, 9:18 AM
corvink requested review of this revision.Oct 14 2022, 9:18 AM
corvink mentioned this in D36983: bhyve: improve acpi table generation.Oct 14 2022, 9:35 AM
corvink retitled this revision from [acpi-table part 9] bhyve: add basl support for common table header to [PATCH 9/19] bhyve: add basl support for common table header.Oct 14 2022, 9:44 AM
corvink mentioned this in D36984: [PATCH 1/19] bhyve: add basic basl implementation.Oct 24 2022, 7:50 AM
markj accepted this revision as: markj.Oct 26 2022, 2:21 PM
markj added inline comments.
usr.sbin/bhyve/basl.c
470
This revision is now accepted and ready to land.Oct 26 2022, 2:21 PM
corvink updated this revision to Diff 112769.Nov 8 2022, 9:25 AM
  • minor style fix
This revision now requires review to proceed.Nov 8 2022, 9:25 AM
Harbormaster completed remote builds in B48227: Diff 112769.Nov 8 2022, 9:25 AM
corvink marked an inline comment as done.Nov 8 2022, 9:25 AM
jhb added a child revision: D36993: [PATCH 10/19] bhyve: build DSDT table by basl.Nov 10 2022, 6:36 PM
jhb added a parent revision: D36991: [PATCH 8/19] bhyve: add basl support for pointers.
jhb accepted this revision.Nov 10 2022, 8:34 PM
This revision is now accepted and ready to land.Nov 10 2022, 8:34 PM
corvink updated this revision to Diff 113123.Nov 15 2022, 9:45 AM
  • remove oem_id and oem_table_id parameter from basl_table_append_header
This revision now requires review to proceed.Nov 15 2022, 9:45 AM
Harbormaster completed remote builds in B48345: Diff 113123.Nov 15 2022, 9:45 AM
corvink updated this revision to Diff 113132.Nov 15 2022, 10:14 AM
  • inline defines
Harbormaster completed remote builds in B48354: Diff 113132.Nov 15 2022, 10:14 AM
corvink updated this revision to Diff 113133.Nov 15 2022, 10:31 AM
  • fix build
Harbormaster completed remote builds in B48355: Diff 113133.Nov 15 2022, 10:31 AM
jhb accepted this revision.Nov 15 2022, 6:14 PM
This revision is now accepted and ready to land.Nov 15 2022, 6:14 PM
Closed by commit rG2fb0f352b977: bhyve: add basl support for common table header (authored by corvink). · Explain WhyNov 16 2022, 11:48 AM
This revision was automatically updated to reflect the committed changes.
corvink added a commit: rG2fb0f352b977: bhyve: add basl support for common table header.
corvink added a commit: rG818d66e29bcb: bhyve: add basl support for common table header.Nov 30 2022, 10:09 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
3 lines
36 lines

Diff 113133

View Options

usr.sbin/bhyve/basl.h

Loading...
View Options

usr.sbin/bhyve/basl.c

Loading...