[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/3] hw/acpi: Provide stub version of acpi_ghes_record_errors
From: |
Peter Maydell |
Subject: |
Re: [PATCH 1/3] hw/acpi: Provide stub version of acpi_ghes_record_errors() |
Date: |
Thu, 17 Jun 2021 13:25:44 +0100 |
On Thu, 17 Jun 2021 at 11:28, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> On 03/06/21 20:52, Richard Henderson wrote:
> > On 6/3/21 10:12 AM, Peter Maydell wrote:
> >> +softmmu_ss.add(when: 'CONFIG_ACPI', if_false: files('acpi-stub.c',
> >> 'aml-build-stub.c', 'ghes-stub.c'))
> >> softmmu_ss.add_all(when: 'CONFIG_ACPI', if_true: acpi_ss)
> >> softmmu_ss.add(when: 'CONFIG_ALL', if_true: files('acpi-stub.c',
> >> 'aml-build-stub.c',
> >> - 'acpi-x86-stub.c',
> >> 'ipmi-stub.c'))
> >> + 'acpi-x86-stub.c',
> >> 'ipmi-stub.c', 'ghes-stub.c'))
> >
> > Gosh that last line is confusing. I see it's documented in
> > build-system.rst, but yeesh.
> >
> > Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
>
> Yeah, it's vestigial of the makefiles and I should remove it.
>
> That said, here:
>
> > +acpi_ss.add(when: 'CONFIG_ACPI_APEI', if_true: files('ghes.c'),
> > if_false:('ghes-stub.c'))
>
> There's a missing "files" after if_false.
Thanks for finding this bug -- I have requeued the patches
to target-arm.next with the "files" added.
-- PMM
- [PATCH 0/3] hw/acpi, arm: Provide and use acpi_ghes_present(), Peter Maydell, 2021/06/03
- [PATCH 2/3] hw/acpi: Provide function acpi_ghes_present(), Peter Maydell, 2021/06/03
- [PATCH 3/3] target/arm: Use acpi_ghes_present() to see if we report ACPI memory errors, Peter Maydell, 2021/06/03
- Re: [PATCH 0/3] hw/acpi, arm: Provide and use acpi_ghes_present(), Peter Maydell, 2021/06/15