linux-kernel - Re: Bad MAINTAINERS pattern in section 'ACPI'

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJZ5v0hh899QjTTbzRK7oWqSH6GccV_H0weUUYJdVw8sQgyONQ@mail.gmail.com>
Date:   Sat, 29 Sep 2018 15:18:38 +0200
From:   "Rafael J. Wysocki" <rafael@...nel.org>
To:     Bjorn Helgaas <helgaas@...nel.org>
Cc:     Joe Perches <joe@...ches.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "Rafael J. Wysocki" <rjw@...ysocki.net>,
        Len Brown <lenb@...nel.org>,
        ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Keith Busch <keith.busch@...el.com>,
        Tony Luck <tony.luck@...el.com>,
        Borislav Petkov <bp@...en8.de>,
        Linux PCI <linux-pci@...r.kernel.org>
Subject: Re: Bad MAINTAINERS pattern in section 'ACPI'

On Sat, Sep 29, 2018 at 1:06 AM Bjorn Helgaas <helgaas@...nel.org> wrote:
>
> [+cc Tony, Borislav (ACPI APEI reviewers), linux-pci]
>
> On Fri, Sep 28, 2018 at 02:50:53PM -0700, Joe Perches wrote:
> > Please fix this defect appropriately.
> >
> > linux-next MAINTAINERS section:
> >
> >       308     ACPI
> >       309     M:      "Rafael J. Wysocki" <rjw@...ysocki.net>
> >       310     M:      Len Brown <lenb@...nel.org>
> >       311     L:      linux-acpi@...r.kernel.org
> >       312     W:      https://01.org/linux-acpi
> >       313     Q:      https://patchwork.kernel.org/project/linux-acpi/list/
> >       314     T:      git git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
> >       315     B:      https://bugzilla.kernel.org
> >       316     S:      Supported
> >       317     F:      drivers/acpi/
> >       318     F:      drivers/pnp/pnpacpi/
> >       319     F:      include/linux/acpi.h
> >       320     F:      include/linux/fwnode.h
> >       321     F:      include/acpi/
> >       322     F:      Documentation/acpi/
> >       323     F:      Documentation/ABI/testing/sysfs-bus-acpi
> >       324     F:      Documentation/ABI/testing/configfs-acpi
> >       325     F:      drivers/pci/*acpi*
> >       326     F:      drivers/pci/*/*acpi*
> > -->   327     F:      drivers/pci/*/*/*acpi*
> >       328     F:      tools/power/acpi/
>
> My proposal to fix this:
>
> commit a99051c0d3c59fd259fd76a8bbd9837b76b509d9
> Author: Bjorn Helgaas <bhelgaas@...gle.com>
> Date:   Fri Sep 28 17:34:21 2018 -0500
>
>     MAINTAINERS: Remove obsolete drivers/pci pattern from ACPI section
>
>     Prior to 256a45937093 ("PCI/AER: Squash aerdrv_acpi.c into aerdrv.c"),
>     drivers/pci/pcie/aer/aerdrv_acpi.c contained code to parse the ACPI HEST
>     table.  That code now lives in drivers/pci/pcie/aer.c.
>
>     Remove the "F: drivers/pci/*/*/*acpi*" pattern because it matches nothing.
>
>     We could add a "F: drivers/pci/pcie/aer.c" pattern to the ACPI APEI
>     section, but that file sees a lot of changes, almost none of which are of
>     interest to the ACPI folks.
>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@...gle.com>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 700408b7bc53..9babd8a0406b 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -324,7 +324,6 @@ F:  Documentation/ABI/testing/sysfs-bus-acpi
>  F:     Documentation/ABI/testing/configfs-acpi
>  F:     drivers/pci/*acpi*
>  F:     drivers/pci/*/*acpi*
> -F:     drivers/pci/*/*/*acpi*
>  F:     tools/power/acpi/

Looks good to me.

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>