diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2019-07-17 15:17:50 +0900 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2019-07-17 22:39:27 +0900 |
commit | 1bd9a468018dd318283c7c620729ddf1923f2024 (patch) | |
tree | a7e65fc6bc709a9aab1cbd74400bcb70354e324e /scripts/modules-check.sh | |
parent | 7e13191879d6d589cd987a8db3a70019251fc757 (diff) | |
download | lwn-1bd9a468018dd318283c7c620729ddf1923f2024.tar.gz lwn-1bd9a468018dd318283c7c620729ddf1923f2024.zip |
kbuild: get rid of kernel/ prefix from in-tree modules.{order,builtin}
Removing the 'kernel/' prefix will make our life easier because we can
simply do 'cat modules.order' to get all built modules with full paths.
Currently, we parse the first line of '*.mod' files in $(MODVERDIR).
Since we have duplicated functionality here, I plan to remove MODVERDIR
entirely.
In fact, modules.order is generated also for external modules in a
broken format. It adds the 'kernel/' prefix to the absolute path of
the module, like this:
kernel//path/to/your/external/module/foo.ko
This is fine for now since modules.order is not used for external
modules. However, I want to sanitize the format everywhere towards
the goal of removing MODVERDIR.
We cannot change the format of installed module.{order,builtin}.
So, 'make modules_install' will add the 'kernel/' prefix while copying
them to $(MODLIB)/.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'scripts/modules-check.sh')
-rwxr-xr-x | scripts/modules-check.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/modules-check.sh b/scripts/modules-check.sh index 39e8cb36ba19..f51f446707b8 100755 --- a/scripts/modules-check.sh +++ b/scripts/modules-check.sh @@ -9,7 +9,7 @@ check_same_name_modules() for m in $(sed 's:.*/::' modules.order | sort | uniq -d) do echo "warning: same module names found:" >&2 - sed -n "/\/$m/s:^kernel/: :p" modules.order >&2 + sed -n "/\/$m/s:^: :p" modules.order >&2 done } |