Re: [PATCH v22 9/9] MAINTAINERS: Add ACPI/HEST/GHES entries

[Philippe Mathieu-Daudé]

Hi Peter,

On 1/16/20 5:46 PM, Peter Maydell wrote:
> On Wed, 8 Jan 2020 at 11:32, Dongjiu Geng <address@hidden> wrote:
> > I and Xiang are willing to review the APEI-related patches and
> > volunteer as the reviewers for the HEST/GHES part.
> >
> > Signed-off-by: Dongjiu Geng <address@hidden>
> > Signed-off-by: Xiang Zheng <address@hidden>
> > ---
> >   MAINTAINERS | 9 +++++++++
> >   1 file changed, 9 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 387879a..5af70a5 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1423,6 +1423,15 @@ F: tests/bios-tables-test.c
> >   F: tests/acpi-utils.[hc]
> >   F: tests/data/acpi/
> >
> > +ACPI/HEST/GHES
> > +R: Dongjiu Geng <address@hidden>
> > +R: Xiang Zheng <address@hidden>
> > +L: address@hidden
> > +S: Maintained
> > +F: hw/acpi/ghes.c
> > +F: include/hw/acpi/ghes.h
> > +F: docs/specs/acpi_hest_ghes.rst
> > +
> >   ppc4xx
> >   M: David Gibson <address@hidden>
> >   L: address@hidden
> > --
>
> Michael, Igor: since this new MAINTAINERS section is
> moving files out of the 'ACPI/SMBIOS' section that you're
> currently responsible for, do you want to provide an
> acked-by: that you think this division of files makes sense?

The files are not 'moved out', Michael and Igor are still the 
maintainers of the supported ACPI/SMBIOS subsystem:

ACPI/SMBIOS
M: Michael S. Tsirkin <address@hidden>
M: Igor Mammedov <address@hidden>
S: Supported
F: include/hw/acpi/*
F: hw/acpi/*

Dongjiu and Xiang only add themselves as reviewers to get notified on 
changes on these specific files. The more eyes the better :)

The docs/specs/acpi_hest_ghes.rst document has no maintainer, as these 
others too:

- docs/specs/acpi_cpu_hotplug.txt
- docs/specs/acpi_hw_reduced_hotplug.rst
- docs/specs/acpi_mem_hotplug.txt
- docs/specs/acpi_nvdimm.txt

The only ACPI file reported as maintained in docs/specs/ is 
acpi_pci_hotplug.txt, from this entry:

PCI
M: Michael S. Tsirkin <address@hidden>
M: Marcel Apfelbaum <address@hidden>
S: Supported
F: docs/specs/*pci*

FWIW:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>