[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates |
Date: |
Fri, 12 Jul 2024 13:51:04 +0100 |
User-agent: |
Mutt/2.2.12 (2023-09-09) |
On Fri, Jul 12, 2024 at 02:43:19PM +0200, Igor Mammedov wrote:
> On Mon, 8 Jul 2024 17:17:32 +0530
> Sunil V L <sunilvl@ventanamicro.com> wrote:
>
> > This series adds few updates to RISC-V ACPI namespace for virt platform.
> > Additionally, it has patches to enable ACPI table testing for RISC-V.
> >
> > 1) PCI Link devices need to be created outside the scope of the PCI root
> > complex to ensure correct probe ordering by the OS. This matches the
> > example given in ACPI spec as well.
> >
> > 2) Add PLIC and APLIC as platform devices as well to ensure probing
> > order as per BRS spec [1] requirement.
> >
> > 3) BRS spec requires RISC-V to use new ACPI ID for the generic UART. So,
> > update the HID of the UART.
> >
> > 4) Enabled ACPI tables tests for RISC-V which were originally part of
> > [2] but couldn't get merged due to updates required in the expected AML
> > files. I think combining those patches with this series makes it easier
> > to merge since expected AML files are updated.
> >
> > [1] - https://github.com/riscv-non-isa/riscv-brs
> > [2] - https://lists.gnu.org/archive/html/qemu-devel/2024-06/msg04734.html
>
> btw: CI is not happy about series, see:
> https://gitlab.com/imammedo/qemu/-/pipelines/1371119552
> also 'cross-i686-tci' job routinely timeouts on bios-tables-test
> but we still keep adding more tests to it.
> We should either bump timeout to account for slowness or
> disable bios-tables-test for that job.
Asumming the test is functionally correct, and not hanging, then bumping
the timeout is the right answer. You can do this in the meson.build
file
We should never disable tests only in CI, because non-CI users
are just as likely to hit timeouts.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH v2 6/9] tests/qtest/bios-tables-test.c: Remove the fall back path, (continued)
- [PATCH v2 6/9] tests/qtest/bios-tables-test.c: Remove the fall back path, Sunil V L, 2024/07/08
- [PATCH v2 7/9] tests/acpi: Add empty ACPI data files for RISC-V, Sunil V L, 2024/07/08
- [PATCH v2 3/9] tests/acpi: Allow DSDT acpi table changes for aarch64, Sunil V L, 2024/07/08
- [PATCH v2 5/9] tests/acpi: update expected DSDT blob for aarch64 and microvm, Sunil V L, 2024/07/08
- [PATCH v2 8/9] tests/qtest/bios-tables-test.c: Enable basic testing for RISC-V, Sunil V L, 2024/07/08
- [PATCH v2 9/9] tests/acpi: Add expected ACPI AML files for RISC-V, Sunil V L, 2024/07/08
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Igor Mammedov, 2024/07/12
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates,
Daniel P . Berrangé <=
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Igor Mammedov, 2024/07/12
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Michael S. Tsirkin, 2024/07/14
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Igor Mammedov, 2024/07/15
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Sunil V L, 2024/07/16
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Igor Mammedov, 2024/07/16
- Re: [PATCH v2 0/9] RISC-V: ACPI: Namespace updates, Igor Mammedov, 2024/07/16