Page MenuHomeFreeBSD

bhyve: add helper to append a basl table without a header
ClosedPublic

Authored by corvink on Nov 16 2022, 12:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 19, 11:43 AM
Unknown Object (File)
Tue, Mar 19, 11:43 AM
Unknown Object (File)
Tue, Mar 19, 11:43 AM
Unknown Object (File)
Sun, Mar 17, 8:16 PM
Unknown Object (File)
Jan 1 2024, 7:43 AM
Unknown Object (File)
Jan 1 2024, 7:43 AM
Unknown Object (File)
Jan 1 2024, 7:43 AM
Unknown Object (File)
Dec 29 2023, 2:44 AM
Subscribers

Details

Summary

The common style for build an ACPI table will be:

  1. basl_table_create
  2. basl_table_append_header
  3. setup an ACPI_TABLE_* struct
  4. basl_table_append_bytes (without header)

Add a helper for the last step.

Diff Detail

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

Event Timeline

markj added inline comments.
usr.sbin/bhyve/basl.c
443

Assert len >= sizeof(ACPI_TABLE_HEADER)?

447

Missing parens around the return value.

usr.sbin/bhyve/basl.h
56
This revision is now accepted and ready to land.Nov 16 2022, 2:25 PM

Agree with @markj's feedback, but like the idea of a helper function for this.