On 28 December 2017 at 08:20, Andrew Jones <[email protected]> wrote: > On Thu, Dec 28, 2017 at 09:14:40AM +0800, Shannon Zhao wrote: >> From: Zhaoshenglong <[email protected]> >> >> acpi_data_push uses g_array_set_size to resize the memory size. If there >> is no enough contiguous memory, the address will be changed. If we use >> the old value, it will assert. >> qemu-kvm: hw/acpi/bios-linker-loader.c:214: bios_linker_loader_add_checksum: >> Assertion `start_offset < file->blob->len' failed.` >> >> This issue only happens in building SRAT table now but here we unify the >> pattern for other tables as well to avoid possible issues in the future. >> >> Signed-off-by: Zhaoshenglong <[email protected]> >> --- >> v3: unify the pattern for MCFG >> v2: update the commit message according to Andrew's comments >> --- >> hw/arm/virt-acpi-build.c | 18 +++++++++++------- >> 1 file changed, 11 insertions(+), 7 deletions(-) >> > > Reviewed-by: Andrew Jones <[email protected]>
Thanks; applied to target-arm.next. -- PMM
