diff options
author | Corvin Köhne <corvink@FreeBSD.org> | 2022-11-16 08:15:12 +0000 |
---|---|---|
committer | Corvin Köhne <corvink@FreeBSD.org> | 2022-12-05 07:39:33 +0000 |
commit | fa0b9dec693116a1c2366d0a39c7d3ea39514667 (patch) | |
tree | ed8954338cff17598b45e828c042929cc2a4b038 | |
parent | dd298db84a83b51639cc400295b013f98b4b4a7d (diff) | |
download | src-fa0b9dec693116a1c2366d0a39c7d3ea39514667.tar.gz src-fa0b9dec693116a1c2366d0a39c7d3ea39514667.zip |
bhyve: make basl_table_add_* functions public
The code will be more readable if we use struct definitions from ACPI-CA
to build ACPI tables. We can fill out the struct and append it to the
basl_table by using basl_table_append_bytes. After that, we have to
declare which checksums, length and pointers should be patched by basl.
That's done by the add_* functions.
Reviewed by: jhb, markj
Approved by: manu (mentor)
MFC after: 2 weeks
Sponsored by: Beckhoff Automation GmbH & Co. KG
Differential Revision: https://reviews.freebsd.org/D37405
(cherry picked from commit 7263419f387db306340ed39fc058580872b64f3b)
-rw-r--r-- | usr.sbin/bhyve/basl.c | 14 | ||||
-rw-r--r-- | usr.sbin/bhyve/basl.h | 7 |
2 files changed, 18 insertions, 3 deletions
diff --git a/usr.sbin/bhyve/basl.c b/usr.sbin/bhyve/basl.c index e8724b857381..7accc7088a9d 100644 --- a/usr.sbin/bhyve/basl.c +++ b/usr.sbin/bhyve/basl.c @@ -323,12 +323,14 @@ basl_init(void) return (0); } -static int +int basl_table_add_checksum(struct basl_table *const table, const uint32_t off, const uint32_t start, const uint32_t len) { struct basl_table_checksum *checksum; + assert(table != NULL); + checksum = calloc(1, sizeof(struct basl_table_checksum)); if (checksum == NULL) { warnx("%s: failed to allocate checksum", __func__); @@ -344,12 +346,15 @@ basl_table_add_checksum(struct basl_table *const table, const uint32_t off, return (0); } -static int +int basl_table_add_length(struct basl_table *const table, const uint32_t off, const uint8_t size) { struct basl_table_length *length; + assert(table != NULL); + assert(size == 4 || size == 8); + length = calloc(1, sizeof(struct basl_table_length)); if (length == NULL) { warnx("%s: failed to allocate length", __func__); @@ -364,13 +369,16 @@ basl_table_add_length(struct basl_table *const table, const uint32_t off, return (0); } -static int +int basl_table_add_pointer(struct basl_table *const table, const uint8_t src_signature[ACPI_NAMESEG_SIZE], const uint32_t off, const uint8_t size) { struct basl_table_pointer *pointer; + assert(table != NULL); + assert(size == 4 || size == 8); + pointer = calloc(1, sizeof(struct basl_table_pointer)); if (pointer == NULL) { warnx("%s: failed to allocate pointer", __func__); diff --git a/usr.sbin/bhyve/basl.h b/usr.sbin/bhyve/basl.h index aab4a16a63f5..5c6c5123d887 100644 --- a/usr.sbin/bhyve/basl.h +++ b/usr.sbin/bhyve/basl.h @@ -39,6 +39,13 @@ struct basl_table; int basl_finish(void); int basl_init(void); +int basl_table_add_checksum(struct basl_table *const table, const uint32_t off, + const uint32_t start, const uint32_t len); +int basl_table_add_length(struct basl_table *const table, const uint32_t off, + const uint8_t size); +int basl_table_add_pointer(struct basl_table *const table, + const uint8_t src_signature[ACPI_NAMESEG_SIZE], const uint32_t off, + const uint8_t size); int basl_table_append_bytes(struct basl_table *table, const void *bytes, uint32_t len); int basl_table_append_checksum(struct basl_table *table, uint32_t start, |