1. Patchew
  2. QEMU
  3. [PATCH v3 00/29] ACPI PCI Hotplug support on ARM
  4. View patch

[PATCH v3 27/29] tests/qtest/bios-tables-test: Prepare for addition of acpi pci hp tests

Eric Auger posted 29 patches 5 months ago
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <anisinha@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Shannon Zhao <shannon.zhaosl@gmail.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Richard Henderson <richard.henderson@linaro.org>
There is a newer version of this series
[PATCH v3 27/29] tests/qtest/bios-tables-test: Prepare for addition of acpi pci hp tests
Posted by Eric Auger 5 months ago 
From: Gustavo Romero <gustavo.romero@linaro.org>

Soon we will introduce a new test related to ACPI PCI hotplug that
will use a new reference blob:

tests/data/acpi/aarch64/virt/DSDT.acpipcihp

Signed-off-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
---
 tests/qtest/bios-tables-test-allowed-diff.h | 1 +
 tests/data/acpi/aarch64/virt/DSDT.acpipcihp | 0
 2 files changed, 1 insertion(+)
 create mode 100644 tests/data/acpi/aarch64/virt/DSDT.acpipcihp

diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..dc3ab24d05 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,2 @@
 /* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/aarch64/virt/DSDT.acpipcihp",
diff --git a/tests/data/acpi/aarch64/virt/DSDT.acpipcihp b/tests/data/acpi/aarch64/virt/DSDT.acpipcihp
new file mode 100644
index 0000000000..e69de29bb2
-- 
2.49.0
Re: [PATCH v3 27/29] tests/qtest/bios-tables-test: Prepare for addition of acpi pci hp tests
Posted by Jonathan Cameron via 4 months, 4 weeks ago 
On Mon, 16 Jun 2025 11:46:56 +0200
Eric Auger <eric.auger@redhat.com> wrote:

> From: Gustavo Romero <gustavo.romero@linaro.org>
> 
> Soon we will introduce a new test related to ACPI PCI hotplug that
> will use a new reference blob:
> 
> tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> 
> Signed-off-by: Eric Auger <eric.auger@redhat.com>
> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>

Unusual sign off chain.  Eric is signing off as handling Gustavo's patch?
I'd expect Eric's sign off to be second.  Looks good in the next patch fo
instance.  With that fixed.
 
Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>

> ---
>  tests/qtest/bios-tables-test-allowed-diff.h | 1 +
>  tests/data/acpi/aarch64/virt/DSDT.acpipcihp | 0
>  2 files changed, 1 insertion(+)
>  create mode 100644 tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> 
> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> index dfb8523c8b..dc3ab24d05 100644
> --- a/tests/qtest/bios-tables-test-allowed-diff.h
> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> @@ -1 +1,2 @@
>  /* List of comma-separated changed AML files to ignore */
> +"tests/data/acpi/aarch64/virt/DSDT.acpipcihp",
> diff --git a/tests/data/acpi/aarch64/virt/DSDT.acpipcihp b/tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> new file mode 100644
> index 0000000000..e69de29bb2
Re: [PATCH v3 27/29] tests/qtest/bios-tables-test: Prepare for addition of acpi pci hp tests
Posted by Eric Auger 4 months, 3 weeks ago 
Hi Jonathan,

On 6/20/25 12:31 PM, Jonathan Cameron wrote:
> On Mon, 16 Jun 2025 11:46:56 +0200
> Eric Auger <eric.auger@redhat.com> wrote:
>
>> From: Gustavo Romero <gustavo.romero@linaro.org>
>>
>> Soon we will introduce a new test related to ACPI PCI hotplug that
>> will use a new reference blob:
>>
>> tests/data/acpi/aarch64/virt/DSDT.acpipcihp
>>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
> Unusual sign off chain.  Eric is signing off as handling Gustavo's patch?
> I'd expect Eric's sign off to be second.  Looks good in the next patch fo
> instance.  With that fixed.
Sure. sorry, unintentional. I did not know the rule either ;-)

Eric
>  
> Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
>
>> ---
>>  tests/qtest/bios-tables-test-allowed-diff.h | 1 +
>>  tests/data/acpi/aarch64/virt/DSDT.acpipcihp | 0
>>  2 files changed, 1 insertion(+)
>>  create mode 100644 tests/data/acpi/aarch64/virt/DSDT.acpipcihp
>>
>> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
>> index dfb8523c8b..dc3ab24d05 100644
>> --- a/tests/qtest/bios-tables-test-allowed-diff.h
>> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
>> @@ -1 +1,2 @@
>>  /* List of comma-separated changed AML files to ignore */
>> +"tests/data/acpi/aarch64/virt/DSDT.acpipcihp",
>> diff --git a/tests/data/acpi/aarch64/virt/DSDT.acpipcihp b/tests/data/acpi/aarch64/virt/DSDT.acpipcihp
>> new file mode 100644
>> index 0000000000..e69de29bb2
Re: [PATCH v3 27/29] tests/qtest/bios-tables-test: Prepare for addition of acpi pci hp tests
Posted by Igor Mammedov 4 months, 4 weeks ago 
On Fri, 20 Jun 2025 11:31:38 +0100
Jonathan Cameron <Jonathan.Cameron@huawei.com> wrote:

> On Mon, 16 Jun 2025 11:46:56 +0200
> Eric Auger <eric.auger@redhat.com> wrote:
> 
> > From: Gustavo Romero <gustavo.romero@linaro.org>
> > 
> > Soon we will introduce a new test related to ACPI PCI hotplug that
> > will use a new reference blob:
> > 
> > tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> > 
> > Signed-off-by: Eric Auger <eric.auger@redhat.com>
> > Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>  
> 
> Unusual sign off chain.  Eric is signing off as handling Gustavo's patch?
> I'd expect Eric's sign off to be second.  Looks good in the next patch fo
> instance.  With that fixed.
>  
> Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>

Reviewed-by: Igor Mammedov <imammedo@redhat.com>

> 
> > ---
> >  tests/qtest/bios-tables-test-allowed-diff.h | 1 +
> >  tests/data/acpi/aarch64/virt/DSDT.acpipcihp | 0
> >  2 files changed, 1 insertion(+)
> >  create mode 100644 tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> > 
> > diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> > index dfb8523c8b..dc3ab24d05 100644
> > --- a/tests/qtest/bios-tables-test-allowed-diff.h
> > +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> > @@ -1 +1,2 @@
> >  /* List of comma-separated changed AML files to ignore */
> > +"tests/data/acpi/aarch64/virt/DSDT.acpipcihp",
> > diff --git a/tests/data/acpi/aarch64/virt/DSDT.acpipcihp b/tests/data/acpi/aarch64/virt/DSDT.acpipcihp
> > new file mode 100644
> > index 0000000000..e69de29bb2  
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.