[PATCH v2 0/3] virt: Report UART correctly in ACPI DBG2/SPCR

Peter Maydell posted 3 patches 1 year ago
Only 2 patches received!
There is a newer version of this series
hw/arm/virt-acpi-build.c  |   4 ++--
tests/data/acpi/virt/DBG2 | Bin 87 -> 87 bytes
tests/data/acpi/virt/SPCR | Bin 80 -> 80 bytes
3 files changed, 2 insertions(+), 2 deletions(-)
[PATCH v2 0/3] virt: Report UART correctly in ACPI DBG2/SPCR
Posted by Peter Maydell 1 year ago
This patchseries is Udo's patch, plus the necessary extra patches
that update the golden-reference files for bios-tables-test so that
'make check' continues to pass. I'm sending these to the list for
completeness, mostly: review is of course welcome but I plan to put
these into target-arm.next fairly soon.

Patch 2 has already been reviewed.

thanks
-- PMM

Peter Maydell (2):
  tests/qtest/bios-tables-test: Allow changes to virt SPCR and DBG2
  tests/qtest/bios-tables-test: Update virt SPCR and DBG2 golden
    references

Udo Steinberg (1):
  hw/arm/virt: Report correct register sizes in ACPI DBG2/SPCR tables.

 hw/arm/virt-acpi-build.c  |   4 ++--
 tests/data/acpi/virt/DBG2 | Bin 87 -> 87 bytes
 tests/data/acpi/virt/SPCR | Bin 80 -> 80 bytes
 3 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.34.1
Re: [PATCH v2 0/3] virt: Report UART correctly in ACPI DBG2/SPCR
Posted by Peter Maydell 1 year ago
On Tue, 31 Oct 2023 at 13:45, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> This patchseries is Udo's patch, plus the necessary extra patches
> that update the golden-reference files for bios-tables-test so that
> 'make check' continues to pass. I'm sending these to the list for
> completeness, mostly: review is of course welcome but I plan to put
> these into target-arm.next fairly soon.
>
> Patch 2 has already been reviewed.

This turns out to break the Avocado test
machine_aarch64_virt.py:Aarch64VirtMachine.test_alpine_virt_tcg_gic_max
(which boots Linux under UEFI) -- UEFI boots and hands off control to
Linux, which never produces any output, presumably because it's
getting confused by the new ACPI table.

I need to investigate further, but for the moment I won't put this
patchset in the target-arm pullreq I'm setting up.

thanks
-- PMM