To: Joe Perches for checkpatch.pl input On Mon, Feb 17, 2025 at 8:00 PM Thomas Weißschuh <thomas.weissschuh@xxxxxxxxxxxxx> wrote: > > userprogs sometimes need access to UAPI headers. > This is currently not possible for Usermode Linux, as UM is only > a pseudo architecture built on top of a regular architecture and does > not have its own UAPI. > Instead use the UAPI headers from the underlying regular architecture. > > Signed-off-by: Thomas Weißschuh <thomas.weissschuh@xxxxxxxxxxxxx> > --- > Makefile | 5 ++++- > lib/Kconfig.debug | 1 - > 2 files changed, 4 insertions(+), 2 deletions(-) Can we add some checks to scripts/checkpatch.pl for inappropriate commit subject prefixes? I really want to see "kconfig:" used in the subject only for changes under scripts/kconfig/. I wonder if scripts/checkpatch.pl shows a warning if "kconfig:" is used in the subject but nothing under scripts/kconfig/ is changed. People incorrectly use "kconfig:" prefix for random "Kconfig" file changes. I am fed up with pointing out this silly stuff. Apparently, this patch is not related to Kconfig at all. > > diff --git a/Makefile b/Makefile > index b4c208ae4041c1f4e32c2a158322422ce7353d06..275185d2ff5d3dc99bd7982abd1de85af0b9621a 100644 > --- a/Makefile > +++ b/Makefile > @@ -1357,9 +1357,12 @@ hdr-inst := -f $(srctree)/scripts/Makefile.headersinst obj > > PHONY += headers > headers: $(version_h) scripts_unifdef uapi-asm-generic archheaders archscripts > - $(if $(filter um, $(SRCARCH)), $(error Headers not exportable for UML)) > +ifdef HEADER_ARCH > + $(Q)$(MAKE) -f $(srctree)/Makefile HEADER_ARCH= SRCARCH=$(HEADER_ARCH) headers > +else > $(Q)$(MAKE) $(hdr-inst)=include/uapi > $(Q)$(MAKE) $(hdr-inst)=arch/$(SRCARCH)/include/uapi > +endif > > ifdef CONFIG_HEADERS_INSTALL > prepare: headers > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug > index 1af972a92d06f6e3f7beec4bd086c00b412c83ac..60026c8388db82c0055ccd8b8ac8789d6b939b62 100644 > --- a/lib/Kconfig.debug > +++ b/lib/Kconfig.debug > @@ -473,7 +473,6 @@ config READABLE_ASM > > config HEADERS_INSTALL > bool "Install uapi headers to usr/include" > - depends on !UML > help > This option will install uapi headers (headers exported to user-space) > into the usr/include directory for use during the kernel build. > > -- > 2.48.1 > -- Best Regards Masahiro Yamada
