From: Joe Perches <joe@perches.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: "Marco Elver" <elver@google.com>,
"Andrey Konovalov" <andreyknvl@gmail.com>,
"Paul Heidekrüger" <paul.heidekrueger@tum.de>,
"Andrey Ryabinin" <ryabinin.a.a@gmail.com>,
"Alexander Potapenko" <glider@google.com>,
"Dmitry Vyukov" <dvyukov@google.com>,
"Vincenzo Frascino" <vincenzo.frascino@arm.com>,
kasan-dev@googlegroups.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] kasan: default to inline instrumentation
Date: Wed, 15 Nov 2023 14:48:38 -0800 [thread overview]
Message-ID: <f9f628a0685b948898a83e7946833b2f5c5a1e7f.camel@perches.com> (raw)
In-Reply-To: <20231115143410.e2c1ea567221d591b58ada1f@linux-foundation.org>
On Wed, 2023-11-15 at 14:34 -0800, Andrew Morton wrote:
> On Tue, 14 Nov 2023 21:38:50 -0800 Joe Perches <joe@perches.com> wrote:
>
> > > +LIBRARY CODE
> > > +M: Andrew Morton <akpm@linux-foundation.org>
> > > +L: linux-kernel@vger.kernel.org
> > > +S: Supported
> >
> > Dunno.
> >
> > There are a lot of already specifically maintained or
> > supported files in lib/
>
> That's OK. I'll get printed out along with the existing list of
> maintainers, if any.
>
> > Maybe be a reviewer?
>
> Would that alter the get_maintainer output in any way?
Not really. It would allow someone to avoid cc'ing reviewers
and not maintainers though.
Perhaps change the
S: Supported
to something like
S: Supported for the files otherwise not supported
> I suppose I could list each file individually, but I'm not sure what
> that would gain.
>
> btw, I see MAINTAINERS lists non-existent file[s] (lib/fw_table.c).
> Maybe someone has a script to check...
--self-test works
$ ./scripts/get_maintainer.pl --self-test=patterns
./MAINTAINERS:3653: warning: no file matches F: Documentation/devicetree/bindings/iio/imu/bosch,bma400.yaml
./MAINTAINERS:6126: warning: no file matches F: Documentation/devicetree/bindings/watchdog/da90??-wdt.txt
./MAINTAINERS:10342: warning: no file matches F: drivers/iio/light/gain-time-scale-helper.c
./MAINTAINERS:10343: warning: no file matches F: drivers/iio/light/gain-time-scale-helper.h
./MAINTAINERS:22062: warning: no file matches F: arch/arm/boot/dts/imx*mba*.dts*
./MAINTAINERS:22063: warning: no file matches F: arch/arm/boot/dts/imx*tqma*.dts*
./MAINTAINERS:22064: warning: no file matches F: arch/arm/boot/dts/mba*.dtsi
and: see commit a103f46633fdcddc2aaca506420f177e8803a2bd
$ git log --stat -1 a103f46633fdcddc2aaca506420f177e8803a2bd
commit a103f46633fdcddc2aaca506420f177e8803a2bd
Author: Dave Jiang <dave.jiang@intel.com>
Date: Thu Oct 12 11:53:54 2023 -0700
acpi: Move common tables helper functions to common lib
Some of the routines in ACPI driver/acpi/tables.c can be shared with
parsing CDAT. CDAT is a device-provided data structure that is formatted
similar to a platform provided ACPI table. CDAT is used by CXL and can
exist on platforms that do not use ACPI. Split out the common routine
from ACPI to accommodate platforms that do not support ACPI and move that
to /lib. The common routines can be built outside of ACPI if
FIRMWARE_TABLES is selected.
Link: https://lore.kernel.org/linux-cxl/CAJZ5v0jipbtTNnsA0-o5ozOk8ZgWnOg34m34a9pPenTyRLj=6A@mail.gmail.com/
Suggested-by: "Rafael J. Wysocki" <rafael@kernel.org>
Reviewed-by: Hanjun Guo <guohanjun@huawei.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Acked-by: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
Link: https://lore.kernel.org/r/169713683430.2205276.17899451119920103445.stgit@djiang5-mobl3
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
MAINTAINERS | 2 ++
drivers/acpi/Kconfig | 1 +
drivers/acpi/tables.c | 173 -------------------------------------------------------------------------------------------------------
include/linux/acpi.h | 42 +++++++------------------
include/linux/fw_table.h | 43 ++++++++++++++++++++++++++
lib/Kconfig | 3 ++
lib/Makefile | 2 ++
lib/fw_table.c | 189 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
8 files changed, 251 insertions(+), 204 deletions(-)
next prev parent reply other threads:[~2023-11-15 22:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-09 15:51 Paul Heidekrüger
2023-11-09 21:08 ` Andrey Konovalov
2023-11-14 11:00 ` Marco Elver
2023-11-14 23:11 ` Andrew Morton
2023-11-15 5:38 ` Joe Perches
2023-11-15 8:26 ` Marco Elver
2023-11-15 22:34 ` Andrew Morton
2023-11-15 22:48 ` Joe Perches [this message]
2023-11-15 22:51 ` Andrew Morton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f9f628a0685b948898a83e7946833b2f5c5a1e7f.camel@perches.com \
--to=joe@perches.com \
--cc=akpm@linux-foundation.org \
--cc=andreyknvl@gmail.com \
--cc=dvyukov@google.com \
--cc=elver@google.com \
--cc=glider@google.com \
--cc=kasan-dev@googlegroups.com \
--cc=linux-kernel@vger.kernel.org \
--cc=paul.heidekrueger@tum.de \
--cc=ryabinin.a.a@gmail.com \
--cc=vincenzo.frascino@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®