| commit | 689ef4721a06dd724e447d8b8e2dc3622c866557 | |
| author | Igor Mammedov <imammedo@redhat.com> | |
| Fri, 24 Sep 2021 12:27:36 +0000 (24 08:27 -0400) | ||
| committer | Michael S. Tsirkin <mst@redhat.com> | |
| Tue, 5 Oct 2021 21:30:57 +0000 (5 17:30 -0400) | ||
| tree | 2d33a5e3e5ad8036d70214bbf14f12c4aa739677 | treesnapshot (tar.gz zip) |
| parent | 578bc7a06462fd71c66d1562c457b01da307a7b6 | commitdiff |
acpi: build_hmat: use acpi_table_begin()/acpi_table_end() instead of build_header()
it replaces error-prone pointer arithmetic for build_header() API,
with 2 calls to start and finish table creation,
which hides offsets magic from API user.
Also since acpi_table_begin() reserves space only for standard header
while previous acpi_data_push() reserved the header + 4 bytes field,
add 4 bytes 'Reserved' field into hmat_build_table_structs()
which didn have it.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Message-Id: <20210924122802.1455362-10-imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
it replaces error-prone pointer arithmetic for build_header() API,
with 2 calls to start and finish table creation,
which hides offsets magic from API user.
Also since acpi_table_begin() reserves space only for standard header
while previous acpi_data_push() reserved the header + 4 bytes field,
add 4 bytes 'Reserved' field into hmat_build_table_structs()
which didn have it.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Message-Id: <20210924122802.1455362-10-imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
| hw/acpi/hmat.c | diffblobblamehistory |