linux-kernel - Re: [PATCH] lib: rename TEST_MODULE to TEST

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 for Android: free password hash cracker in your pocket

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

Message-ID: <1411492210.30136.15.camel@nebuchadnezzar>
Date:	Tue, 23 Sep 2014 19:10:10 +0200
From:	Valentin Rothberg <valentinrothberg@...il.com>
To:	Randy Dunlap <rdunlap@...radead.org>
Cc:	akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
	pebolle@...cali.nl
Subject: Re: [PATCH] lib: rename TEST_MODULE to TEST_LKM

On mar., 2014-09-23 at 09:49 -0700, Randy Dunlap wrote:
> On 09/22/14 23:58, Valentin Rothberg wrote:
> > The "_MODULE" suffix is reserved for tristates compiled as
> > loadable kernel modules (LKM). The "TEST_MODULE" feature thereby
> 
> Is that documented anywhere?

Sadly this is not made explicit, but the Kconfig code documents it. The
following code (./scripts/kconfig/confdata.c) is used to generate the
autoconf.h header file during the build process. When a feature is
selected as a kernel module ('m'), it is suffixed with "_MODULE" to
indicate it. 

»   »   switch (*value) {
»   »   case 'n':
»   »   »   break;
»   »   case 'm':
»   »   »   suffix = "_MODULE";
»   »   »   /* fall through */

> Was this causing some kind of problem or error?  Please tell us what that was if so.

It causes problems for static code analysis, which assumes a consistent
use of the "_MODULE" suffix. 

Another possible change would be to rename the reference in the Makefile
to "TEST_MODULE_MODULE". Personally, I prefer my proposed patch.
> 
> 
> > violates this convention. The feature is used to compile the
> > lib/test_module.c kernel module.
> > 
> > This patch renames the feature and its reference in a Makefile
> > to "TEST_LKM", which still expresses the test of a LKM.
> > 
> > Signed-off-by: Valentin Rothberg <valentinrothberg@...il.com>
> > ---
> >  lib/Kconfig.debug | 2 +-
> >  lib/Makefile      | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> > index 07c2832..db91c97 100644
> > --- a/lib/Kconfig.debug
> > +++ b/lib/Kconfig.debug
> > @@ -1627,7 +1627,7 @@ config DMA_API_DEBUG
> >  
> >  	  If unsure, say N.
> >  
> > -config TEST_MODULE
> > +config TEST_LKM
> >  	tristate "Test module loading with 'hello world' module"
> >  	default n
> >  	depends on m
> > diff --git a/lib/Makefile b/lib/Makefile
> > index d6b4bc4..382437a 100644
> > --- a/lib/Makefile
> > +++ b/lib/Makefile
> > @@ -31,7 +31,7 @@ obj-y += string_helpers.o
> >  obj-$(CONFIG_TEST_STRING_HELPERS) += test-string_helpers.o
> >  obj-y += kstrtox.o
> >  obj-$(CONFIG_TEST_KSTRTOX) += test-kstrtox.o
> > -obj-$(CONFIG_TEST_MODULE) += test_module.o
> > +obj-$(CONFIG_TEST_LKM) += test_module.o
> >  obj-$(CONFIG_TEST_USER_COPY) += test_user_copy.o
> >  obj-$(CONFIG_TEST_BPF) += test_bpf.o
> >  obj-$(CONFIG_TEST_FIRMWARE) += test_firmware.o
> > 
> 
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/