On 09/23/14 10:10, Valentin Rothberg wrote: > 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.
Sure, your patch is fine, but we need to know *why* the patch is needed. Thanks. >> >> >>> 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 <valentinrothb...@gmail.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 -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/