qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 16/16] tests/qtest/bios-tables-test: Add expected ACPI dat


From: Sunil V L
Subject: Re: [PATCH v4 16/16] tests/qtest/bios-tables-test: Add expected ACPI data files for RISC-V
Date: Wed, 3 Jul 2024 15:52:55 +0530

On Tue, Jul 02, 2024 at 03:02:36PM +0100, Jonathan Cameron wrote:
> On Mon, 1 Jul 2024 17:03:43 -0400
> "Michael S. Tsirkin" <mst@redhat.com> wrote:
> 
> > On Thu, Jun 27, 2024 at 02:18:03PM +0200, Igor Mammedov wrote:
> > > On Tue, 25 Jun 2024 20:38:39 +0530
> > > Sunil V L <sunilvl@ventanamicro.com> wrote:
> > >   
> > > > As per the step 5 in the process documented in bios-tables-test.c,
> > > > generate the expected ACPI AML data files for RISC-V using the
> > > > rebuild-expected-aml.sh script and update the
> > > > bios-tables-test-allowed-diff.h.
> > > > 
> > > > These are all new files being added for the first time. Hence, iASL diff
> > > > output is not added.
> > > > 
> > > > Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
> > > > Acked-by: Alistair Francis <alistair.francis@wdc.com>
> > > > Acked-by: Igor Mammedov <imammedo@redhat.com>  
> > > 
> > > Michael,
> > > can it go via risc-v tree or
> > > do you plan to merge it via your tree?  
> > 
> > given patch 1 is merged, I took the rest.
> 
> Looks like your CI runs are catching this as well but
> RHCT here is failing.  I rebased the GI/GP set on top of this
> and ignored that failure by skipping riscv64 tests.
> 
> Jonathan
> 
Hi Jonathan, Michael,

Looks like a recent RISC-V PR updated the rva22s64 ISA string affecting
the RHCT I had in my series. I see that Michael dropped those 3 RISC-V
patches from the PR. So, let me update the expected RHCT AML file in a
new series. I will also include Igor's feedback to remove fallback path
in that series.

Thanks,
Sunil

Thanks,
Sunil



reply via email to

[Prev in Thread] Current Thread [Next in Thread]