diff options
45 files changed, 60 insertions, 405 deletions
diff --git a/Makefile.am b/Makefile.am index 6d21ad2..d37b56d 100644 --- a/Makefile.am +++ b/Makefile.am @@ -40,7 +40,7 @@ AM_LDFLAGS = $(OUR_LDFLAGS) # 6. If any interfaces have been removed or changed since the last public # release, then set age to 0. LIBKMOD_CURRENT=6 -LIBKMOD_REVISION=1 +LIBKMOD_REVISION=2 LIBKMOD_AGE=4 noinst_LTLIBRARIES = shared/libshared.la @@ -111,9 +111,19 @@ install-exec-hook: ln -sf $$so_img_rel_target_prefix$(rootlibdir)/$$so_img_name $(DESTDIR)$(libdir)/libkmod.so && \ mv $(DESTDIR)$(libdir)/libkmod.so.* $(DESTDIR)$(rootlibdir); \ fi +if BUILD_TOOLS + for tool in insmod lsmod rmmod depmod modprobe modinfo; do \ + $(LN_S) kmod $(DESTDIR)$(bindir)/$$tool; \ + done +endif uninstall-hook: rm -f $(DESTDIR)$(rootlibdir)/libkmod.so* +if BUILD_TOOLS + for tool in insmod lsmod rmmod depmod modprobe modinfo; do \ + rm -f $(DESTDIR)$(bindir)/$$tool; \ + done +endif if BUILD_TOOLS bin_PROGRAMS = tools/kmod @@ -129,12 +139,6 @@ tools_kmod_SOURCES = \ tools/depmod.c tools/log.h tools/log.c \ tools/static-nodes.c -if BUILD_EXPERIMENTAL -tools_kmod_SOURCES += \ - tools/insert.c \ - tools/remove.c -endif - tools_kmod_LDADD = \ shared/libshared.la \ libkmod/libkmod-internal.la @@ -260,11 +264,6 @@ TESTSUITE = \ testsuite/test-dependencies testsuite/test-depmod \ testsuite/test-list -if BUILD_EXPERIMENTAL -TESTSUITE += \ - testsuite/test-tools -endif - check_PROGRAMS = $(TESTSUITE) TESTS = $(TESTSUITE) @@ -307,11 +306,6 @@ testsuite_test_depmod_CPPFLAGS = $(TESTSUITE_CPPFLAGS) testsuite_test_list_LDADD = $(TESTSUITE_LDADD) testsuite_test_list_CPPFLAGS = $(TESTSUITE_CPPFLAGS) -if BUILD_EXPERIMENTAL -testsuite_test_tools_LDADD = $(TESTSUITE_LDADD) -testsuite_test_tools_CPPFLAGS = $(TESTSUITE_CPPFLAGS) -endif - testsuite-distclean: $(RM) -r $(ROOTFS) $(RM) testsuite/stamp-rootfs @@ -1,3 +1,41 @@ +kmod 32 +======= + +- Improvements + + - Use any hash algo known by kernel/openssl instead of keep needing + to update the mapping + + - Teach kmod to load modprobe.d/depmod.d configuration from ${prefix}/lib + and allow it to be overriden during build with --with-distconfdir=DIR + + - Make kernel modules directory configurable. This allows distro to + make kmod use only files from /usr regardless of having a compat + symlink in place. + + - Install kmod.pc containing the features selected at build time. + + - Install all tools and symlinks by default. Previously kmod relied on + distro packaging to set up the symlinks in place like modprobe, + depmod, lsmod, etc. Now those symlinks are created by kmod itself + and they are always placed in $bindir. + +- Bug Fixes + + - Fix warnings due to -Walloc-size + +- Others + + - Drop python bindings. Those were not update in ages and not compatible + with latest python releases. + + - Cleanup test infra, dropping what was not used anymore + + - Drop experimental tools `kmod insert` / `kmod remove`. Building those + was protected by a configure option never set by distros. They also + didn't gain enough traction to replace the older interfaces via + modprobe/insmod/rmmod. + kmod 31 ======= diff --git a/configure.ac b/configure.ac index bfabbaa..b651b5f 100644 --- a/configure.ac +++ b/configure.ac @@ -1,6 +1,6 @@ AC_PREREQ(2.64) AC_INIT([kmod], - [31], + [32], [linux-modules@vger.kernel.org], [kmod], [http://git.kernel.org/?p=utils/kernel/kmod/kmod.git]) @@ -92,7 +92,7 @@ AC_SUBST([rootlibdir], [$with_rootlibdir]) # Ideally this would be $prefix/lib/modules but default to /lib/modules for compatibility with earlier versions AC_ARG_WITH([module_directory], - AS_HELP_STRING([--with-module-directory=DIR], [directory in which to look for kernel modules - typically '/lib/modules' or '${prefix}/lib/modules']), + AS_HELP_STRING([--with-module-directory=DIR], [directory in which to look for kernel modules @<:@default=/lib/modules@:>@]), [], [with_module_directory=/lib/modules]) AC_SUBST([module_directory], [$with_module_directory]) @@ -175,15 +175,6 @@ AC_SUBST([bashcompletiondir], [$with_bashcompletiondir]) # --enable- ##################################################################### -AC_ARG_ENABLE([experimental], - AS_HELP_STRING([--enable-experimental], [enable experimental tools and features. Do not enable it unless you know what you are doing. @<:@default=disabled@:>@]), - [], enable_experimental=no) -AM_CONDITIONAL([BUILD_EXPERIMENTAL], [test "x$enable_experimental" = "xyes"]) -AS_IF([test "x$enable_experimental" = "xyes"], [ - AC_DEFINE(ENABLE_EXPERIMENTAL, [1], [Experimental features.]) -]) -CC_FEATURE_APPEND([with_features], [enable_experimental], [EXPERIMENTAL]) - AC_ARG_ENABLE([tools], AS_HELP_STRING([--disable-tools], [disable building tools that provide same functionality as module-init-tools @<:@default=enabled@:>@]), [], enable_tools=yes) @@ -336,7 +327,6 @@ AC_MSG_RESULT([ cflags: ${with_cflags} ${CFLAGS} ldflags: ${with_ldflags} ${LDFLAGS} - experimental features: ${enable_experimental} tools: ${enable_tools} logging: ${enable_logging} compression: zstd=${with_zstd} xz=${with_xz} zlib=${with_zlib} diff --git a/man/Makefile.am b/man/Makefile.am index f550091..d62ff21 100644 --- a/man/Makefile.am +++ b/man/Makefile.am @@ -13,10 +13,10 @@ dist_man_MANS = $(MAN5) $(MAN8) $(MAN_STUB) modules.dep.bin.5: modules.dep.5 endif -EXTRA_DIST = $(MAN5:%.5=%.xml) $(MAN8:%.8=%.xml) +EXTRA_DIST = $(MAN5:%.5=%.5.xml) $(MAN8:%.8=%.8.xml) CLEANFILES = $(dist_man_MANS) -%.5 %.8: %.xml +define generate_manpage $(AM_V_XSLT)if [ '$(distconfdir)' != '/lib' ] ; then \ sed -e 's|@DISTCONFDIR@|$(distconfdir)|g' $< ; \ else \ @@ -29,3 +29,10 @@ CLEANFILES = $(dist_man_MANS) --stringparam man.output.quietly 1 \ --param funcsynopsis.style "'ansi'" \ http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl - +endef + +%.5: %.5.xml + $(generate_manpage) + +%.8: %.8.xml + $(generate_manpage) diff --git a/man/depmod.xml b/man/depmod.8.xml index fce2a4a..fce2a4a 100644 --- a/man/depmod.xml +++ b/man/depmod.8.xml diff --git a/man/depmod.d.xml b/man/depmod.d.5.xml index b07e6a2..b07e6a2 100644 --- a/man/depmod.d.xml +++ b/man/depmod.d.5.xml diff --git a/man/insmod.xml b/man/insmod.8.xml index 3ebdccd..3ebdccd 100644 --- a/man/insmod.xml +++ b/man/insmod.8.xml diff --git a/man/kmod.xml b/man/kmod.8.xml index 0706ad5..0706ad5 100644 --- a/man/kmod.xml +++ b/man/kmod.8.xml diff --git a/man/lsmod.xml b/man/lsmod.8.xml index 588f228..588f228 100644 --- a/man/lsmod.xml +++ b/man/lsmod.8.xml diff --git a/man/modinfo.xml b/man/modinfo.8.xml index b6c4d60..b6c4d60 100644 --- a/man/modinfo.xml +++ b/man/modinfo.8.xml diff --git a/man/modprobe.xml b/man/modprobe.8.xml index 4d1fd59..4d1fd59 100644 --- a/man/modprobe.xml +++ b/man/modprobe.8.xml diff --git a/man/modprobe.d.xml b/man/modprobe.d.5.xml index 2bf6537..2bf6537 100644 --- a/man/modprobe.d.xml +++ b/man/modprobe.d.5.xml diff --git a/man/modules.dep.xml b/man/modules.dep.5.xml index 8ef6d8b..8ef6d8b 100644 --- a/man/modules.dep.xml +++ b/man/modules.dep.5.xml diff --git a/man/rmmod.xml b/man/rmmod.8.xml index 67bcbed..67bcbed 100644 --- a/man/rmmod.xml +++ b/man/rmmod.8.xml diff --git a/testsuite/.gitignore b/testsuite/.gitignore index 9d26b88..5465b1a 100644 --- a/testsuite/.gitignore +++ b/testsuite/.gitignore @@ -18,7 +18,6 @@ /test-modprobe /test-hash /test-list -/test-tools /rootfs /stamp-rootfs /test-scratchbuf.log @@ -53,5 +52,3 @@ /test-testsuite.trs /test-list.log /test-list.trs -/test-tools.log -/test-tools.trs diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias deleted file mode 100644 index ba76e18..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias +++ /dev/null @@ -1 +0,0 @@ -# Aliases extracted from modules themselves. diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias.bin b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.alias.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin deleted file mode 100644 index e69de29..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin.bin b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.builtin.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep deleted file mode 100644 index 5476653..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep +++ /dev/null @@ -1 +0,0 @@ -kernel/mod-simple.ko: diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep.bin b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep.bin Binary files differdeleted file mode 100644 index b09a854..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.dep.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.devname b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.devname deleted file mode 100644 index 58f6d6d..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.devname +++ /dev/null @@ -1 +0,0 @@ -# Device nodes to trigger on-demand module loading. diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.order b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.order deleted file mode 100644 index e69de29..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.order +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.softdep b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.softdep deleted file mode 100644 index 5554ccc..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.softdep +++ /dev/null @@ -1 +0,0 @@ -# Soft dependencies extracted from modules themselves. diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols deleted file mode 100644 index 618c345..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols +++ /dev/null @@ -1 +0,0 @@ -# Aliases for symbols, used by symbol_request(). diff --git a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols.bin b/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/insert/lib/modules/4.4.4/modules.symbols.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias deleted file mode 100644 index ba76e18..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias +++ /dev/null @@ -1 +0,0 @@ -# Aliases extracted from modules themselves. diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias.bin b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.alias.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin deleted file mode 100644 index e69de29..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin.bin b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.builtin.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep deleted file mode 100644 index 5476653..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep +++ /dev/null @@ -1 +0,0 @@ -kernel/mod-simple.ko: diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep.bin b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep.bin Binary files differdeleted file mode 100644 index b09a854..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.dep.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.devname b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.devname deleted file mode 100644 index 58f6d6d..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.devname +++ /dev/null @@ -1 +0,0 @@ -# Device nodes to trigger on-demand module loading. diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.order b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.order deleted file mode 100644 index e69de29..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.order +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.softdep b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.softdep deleted file mode 100644 index 5554ccc..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.softdep +++ /dev/null @@ -1 +0,0 @@ -# Soft dependencies extracted from modules themselves. diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols deleted file mode 100644 index 618c345..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols +++ /dev/null @@ -1 +0,0 @@ -# Aliases for symbols, used by symbol_request(). diff --git a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols.bin b/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols.bin Binary files differdeleted file mode 100644 index 7075435..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/lib/modules/4.4.4/modules.symbols.bin +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/holders/.gitignore b/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/holders/.gitignore deleted file mode 100644 index e69de29..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/holders/.gitignore +++ /dev/null diff --git a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/initstate b/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/initstate deleted file mode 100644 index e23fe64..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/initstate +++ /dev/null @@ -1 +0,0 @@ -live diff --git a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/refcnt b/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/refcnt deleted file mode 100644 index 573541a..0000000 --- a/testsuite/rootfs-pristine/test-tools/remove/sys/module/mod_simple/refcnt +++ /dev/null @@ -1 +0,0 @@ -0 diff --git a/testsuite/setup-rootfs.sh b/testsuite/setup-rootfs.sh index a780f93..5477c69 100755 --- a/testsuite/setup-rootfs.sh +++ b/testsuite/setup-rootfs.sh @@ -103,8 +103,6 @@ map=( ["test-modinfo/mod-simple-sha256.ko"]="mod-simple.ko" ["test-modinfo/mod-simple-pkcs7.ko"]="mod-simple.ko" ["test-modinfo/external/lib/modules/external/mod-simple.ko"]="mod-simple.ko" - ["test-tools/insert$MODULE_DIRECTORY/4.4.4/kernel/"]="mod-simple.ko" - ["test-tools/remove$MODULE_DIRECTORY/4.4.4/kernel/"]="mod-simple.ko" ) gzip_array=( diff --git a/testsuite/test-tools.c b/testsuite/test-tools.c deleted file mode 100644 index 4a9ee9b..0000000 --- a/testsuite/test-tools.c +++ /dev/null @@ -1,71 +0,0 @@ -/* - * Copyright (C) 2015 Intel Corporation. All rights reserved. - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 2.1 of the License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library; if not, see <http://www.gnu.org/licenses/>. - */ - -#include <errno.h> -#include <inttypes.h> -#include <stddef.h> -#include <stdio.h> -#include <stdlib.h> -#include <string.h> -#include <unistd.h> - -#include "testsuite.h" - -static noreturn int kmod_tool_insert(const struct test *t) -{ - const char *progname = ABS_TOP_BUILDDIR "/tools/kmod"; - const char *const args[] = { - progname, - "insert", "mod-simple", - NULL, - }; - - test_spawn_prog(progname, args); - exit(EXIT_FAILURE); -} -DEFINE_TEST(kmod_tool_insert, - .description = "check kmod insert", - .config = { - [TC_UNAME_R] = "4.4.4", - [TC_ROOTFS] = TESTSUITE_ROOTFS "test-tools/insert", - [TC_INIT_MODULE_RETCODES] = "", - }, - .modules_loaded = "mod-simple", - ); - -static noreturn int kmod_tool_remove(const struct test *t) -{ - const char *progname = ABS_TOP_BUILDDIR "/tools/kmod"; - const char *const args[] = { - progname, - "remove", "mod-simple", - NULL, - }; - - test_spawn_prog(progname, args); - exit(EXIT_FAILURE); -} -DEFINE_TEST(kmod_tool_remove, - .description = "check kmod remove", - .config = { - [TC_UNAME_R] = "4.4.4", - [TC_ROOTFS] = TESTSUITE_ROOTFS "test-tools/remove", - [TC_DELETE_MODULE_RETCODES] = "", - }, - ); - -TESTSUITE_MAIN(); diff --git a/tools/insert.c b/tools/insert.c deleted file mode 100644 index 0ebcef9..0000000 --- a/tools/insert.c +++ /dev/null @@ -1,128 +0,0 @@ -/* - * kmod-insert - insert a module into the kernel. - * - * Copyright (C) 2015 Intel Corporation. All rights reserved. - * Copyright (C) 2011-2013 ProFUSION embedded systems - * - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation, either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program. If not, see <http://www.gnu.org/licenses/>. - */ - -#include <errno.h> -#include <getopt.h> -#include <stdlib.h> -#include <string.h> - -#include <libkmod/libkmod.h> - -#include "kmod.h" - -static const char cmdopts_s[] = "h"; -static const struct option cmdopts[] = { - {"help", no_argument, 0, 'h'}, - { } -}; - -static void help(void) -{ - printf("Usage:\n" - "\t%s insert [options] module\n" - "Options:\n" - "\t-h, --help show this help\n", - program_invocation_short_name); -} - -static const char *mod_strerror(int err) -{ - switch (err) { - case KMOD_PROBE_APPLY_BLACKLIST: - return "Module is blacklisted"; - case -EEXIST: - return "Module already in kernel"; - case -ENOENT: - return "Unknown symbol in module or unknown parameter (see dmesg)"; - default: - return strerror(-err); - } -} - -static int do_insert(int argc, char *argv[]) -{ - struct kmod_ctx *ctx; - struct kmod_list *list = NULL, *l; - const char *name; - int err, r = EXIT_SUCCESS; - - for (;;) { - int c, idx = 0; - c = getopt_long(argc, argv, cmdopts_s, cmdopts, &idx); - if (c == -1) - break; - switch (c) { - case 'h': - help(); - return EXIT_SUCCESS; - default: - ERR("Unexpected getopt_long() value '%c'.\n", c); - return EXIT_FAILURE; - } - } - - if (optind >= argc) { - ERR("Missing module name\n"); - return EXIT_FAILURE; - } - - ctx = kmod_new(NULL, NULL); - if (!ctx) { - ERR("kmod_new() failed!\n"); - return EXIT_FAILURE; - } - - name = argv[optind]; - err = kmod_module_new_from_lookup(ctx, name, &list); - if (err < 0) { - ERR("Could not lookup module matching '%s': %s\n", name, strerror(-err)); - r = EXIT_FAILURE; - goto end; - } - - if (list == NULL) { - ERR("No module matches '%s'\n", name); - r = EXIT_FAILURE; - goto end; - } - - kmod_list_foreach(l, list) { - struct kmod_module *mod = kmod_module_get_module(l); - - err = kmod_module_probe_insert_module(mod, KMOD_PROBE_APPLY_BLACKLIST, NULL, NULL, NULL, NULL); - if (err != 0) { - r = EXIT_FAILURE; - ERR("Could not insert '%s': %s\n", kmod_module_get_name(mod), mod_strerror(err)); - } - - kmod_module_unref(mod); - } - - kmod_module_unref_list(list); -end: - kmod_unref(ctx); - return r; -} - -const struct kmod_cmd kmod_cmd_insert = { - .name = "insert", - .cmd = do_insert, - .help = "insert a module into the kernel", -}; diff --git a/tools/kmod.c b/tools/kmod.c index 55689c0..1015575 100644 --- a/tools/kmod.c +++ b/tools/kmod.c @@ -42,11 +42,6 @@ static const struct kmod_cmd *kmod_cmds[] = { &kmod_cmd_help, &kmod_cmd_list, &kmod_cmd_static_nodes, - -#ifdef ENABLE_EXPERIMENTAL - &kmod_cmd_insert, - &kmod_cmd_remove, -#endif }; static const struct kmod_cmd *kmod_compat_cmds[] = { diff --git a/tools/remove.c b/tools/remove.c deleted file mode 100644 index 387ef0e..0000000 --- a/tools/remove.c +++ /dev/null @@ -1,153 +0,0 @@ -/* - * kmod-remove - remove modules from the kernel. - * - * Copyright (C) 2015 Intel Corporation. All rights reserved. - * Copyright (C) 2011-2013 ProFUSION embedded systems - * - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation, either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program. If not, see <http://www.gnu.org/licenses/>. - */ - -#include <errno.h> -#include <getopt.h> -#include <stdlib.h> -#include <string.h> - -#include <libkmod/libkmod.h> - -#include "kmod.h" - -static const char cmdopts_s[] = "h"; -static const struct option cmdopts[] = { - {"help", no_argument, 0, 'h'}, - { } -}; - -static void help(void) -{ - printf("Usage:\n" - "\t%s remove [options] module\n" - "Options:\n" - "\t-h, --help show this help\n", - program_invocation_short_name); -} - -static int check_module_inuse(struct kmod_module *mod) { - struct kmod_list *holders; - int state, ret; - - state = kmod_module_get_initstate(mod); - - if (state == KMOD_MODULE_BUILTIN) { - ERR("Module %s is builtin.\n", kmod_module_get_name(mod)); - return -ENOENT; - } else if (state < 0) { - ERR("Module %s is not currently loaded\n", - kmod_module_get_name(mod)); - return -ENOENT; - } - - holders = kmod_module_get_holders(mod); - if (holders != NULL) { - struct kmod_list *itr; - - ERR("Module %s is in use by:", kmod_module_get_name(mod)); - - kmod_list_foreach(itr, holders) { - struct kmod_module *hm = kmod_module_get_module(itr); - fprintf(stderr, " %s", kmod_module_get_name(hm)); - kmod_module_unref(hm); - } - fputc('\n', stderr); - - kmod_module_unref_list(holders); - return -EBUSY; - } - - ret = kmod_module_get_refcnt(mod); - if (ret > 0) { - ERR("Module %s is in use\n", kmod_module_get_name(mod)); - return -EBUSY; - } else if (ret == -ENOENT) { - ERR("Module unloading is not supported\n"); - } - - return ret; -} - -static int do_remove(int argc, char *argv[]) -{ - struct kmod_ctx *ctx; - struct kmod_module *mod; - const char *name; - int err, r = EXIT_SUCCESS; - - for (;;) { - int c, idx =0; - c = getopt_long(argc, argv, cmdopts_s, cmdopts, &idx); - if (c == -1) - break; - switch (c) { - case 'h': - help(); - return EXIT_SUCCESS; - - default: - ERR("Unexpected getopt_long() value '%c'.\n", c); - return EXIT_FAILURE; - } - } - - if (optind >= argc) { - ERR("Missing module name\n"); - return EXIT_FAILURE; - } - - ctx = kmod_new(NULL, NULL); - if (!ctx) { - ERR("kmod_new() failed!\n"); - return EXIT_FAILURE; - } - - name = argv[optind]; - err = kmod_module_new_from_name(ctx, name, &mod); - if (err < 0) { - ERR("Could not remove module %s: %s\n", name, strerror(-err)); - goto end; - } - - err = check_module_inuse(mod); - if (err < 0) - goto unref; - - err = kmod_module_remove_module(mod, 0); - if (err < 0) - goto unref; - -unref: - kmod_module_unref(mod); - -end: - kmod_unref(ctx); - if (err < 0) { - r = EXIT_FAILURE; - ERR("Could not remove module %s: %s\n", name, strerror(-err)); - } - return r; -} - -const struct kmod_cmd kmod_cmd_remove = { - .name = "remove", - .cmd = do_remove, - .help = "remove module from kernel", -}; |