Lucas De Marchi [Mon, 30 Jan 2012 15:29:17 +0000 (13:29 -0200)]
Use %zu for size_t
Remove warning on 32 bits systems.
Lucas De Marchi [Sat, 28 Jan 2012 23:30:57 +0000 (21:30 -0200)]
testsuite: remove unneeded call to memset()
Pedro Pedruzzi [Sat, 28 Jan 2012 05:26:32 +0000 (03:26 -0200)]
libkmod: Fix handling of square brackets expressions in function alias_normalize
Pedro Pedruzzi [Sat, 28 Jan 2012 05:22:47 +0000 (03:22 -0200)]
testsuite: add test for function alias_normalize
Lucas De Marchi [Sat, 28 Jan 2012 01:56:46 +0000 (23:56 -0200)]
modprobe: print end of configuration files
Lucas De Marchi [Thu, 26 Jan 2012 21:26:35 +0000 (19:26 -0200)]
build-sys: extract rootfs for make check
Commit "d2c2b8b build-sys: autotoolify build of check libraries" removed
rootfs extraction. Add it back.
Lucas De Marchi [Thu, 26 Jan 2012 19:01:41 +0000 (17:01 -0200)]
testsuite: add GPL license
Lucas De Marchi [Thu, 26 Jan 2012 18:10:41 +0000 (16:10 -0200)]
testsuite: trap calls to delete_module() including simple test
Lucas De Marchi [Thu, 26 Jan 2012 14:18:23 +0000 (12:18 -0200)]
build-sys: autotoolify build of check libraries
Many thanks to Iván Briano (sachieru@gmail.com) for pointing out this
thread in libtool mailing list:
http://www.mail-archive.com/libtool@gnu.org/msg09627.html
Passing '-rpath /nowhere' in LDFLAGS we are able to create shared libs
that are not installed.
Lucas De Marchi [Thu, 26 Jan 2012 04:09:28 +0000 (02:09 -0200)]
testsuite: add trap to init_module() including simple test
Lucas De Marchi [Thu, 26 Jan 2012 03:21:17 +0000 (01:21 -0200)]
elf: privately export kmod_elf_get_section()
This function is used to find a certain section in ELF file. It will be
used to get the modname from an ELF file.
Lucas De Marchi [Wed, 25 Jan 2012 22:32:48 +0000 (20:32 -0200)]
testsuite: add test for modinfo
These ext4 modules were sent by Jon Master so we can test the result of
modinfo with modules generated for different architectures.
They are now added to testsuite and their output tested automatically.
Lucas De Marchi [Wed, 25 Jan 2012 22:25:46 +0000 (20:25 -0200)]
testsuite: remind users to build tools
Lucas De Marchi [Wed, 25 Jan 2012 21:48:31 +0000 (19:48 -0200)]
testsuite: add colors to test output
Lucas De Marchi [Wed, 25 Jan 2012 20:16:45 +0000 (18:16 -0200)]
testsuite: add simple test for list of loaded modules
Lucas De Marchi [Wed, 25 Jan 2012 19:46:52 +0000 (17:46 -0200)]
testsuite: match outputs of test with a known correct one
Tests may put the correct output in a file and tell testsuite to check
if it matches the output from the test running.
Testsuite compares the outputs while running the test: it creates a pipe
between parent and child; parent reads both stdout and stderr from child
and compares with the correct output.
Lucas De Marchi [Wed, 25 Jan 2012 18:01:17 +0000 (16:01 -0200)]
testsuite: separate child from parent
Lucas De Marchi [Wed, 25 Jan 2012 17:13:45 +0000 (15:13 -0200)]
testsuite: allow to run tests outside of top_buildir
In order to locate where the shared libs to be preloaded are we need to
reference them using abs_top_buildir. Otherwise we are limited to
running tests from there.
Lucas De Marchi [Wed, 25 Jan 2012 14:42:13 +0000 (12:42 -0200)]
testsuite: add trap to opendir() including tests
Lucas De Marchi [Wed, 25 Jan 2012 14:22:50 +0000 (12:22 -0200)]
testsuite: add trap to stat() and friends including tests
Add trap to stat(): we need to trap other functions too, depending on
stat.h, the function from glibc that is actually called may be stat64 or
__xstat() too.
Lucas De Marchi [Wed, 25 Jan 2012 13:36:28 +0000 (11:36 -0200)]
testsuite: add trap to open() including tests
Lucas De Marchi [Wed, 25 Jan 2012 04:44:45 +0000 (02:44 -0200)]
testsuite: add trap to fopen() including tests
Lucas De Marchi [Wed, 25 Jan 2012 01:35:18 +0000 (23:35 -0200)]
testsuite: fake kernel 4.0.20-kmod is out
Go get it while it's fresh :-). Test fake results of "uname -r" by
LD_PRELOAD'ing uname.so.
Lucas De Marchi [Wed, 25 Jan 2012 01:31:46 +0000 (23:31 -0200)]
testsuite: export environment with flags and LD_PRELOAD
A certain config can add flags and each flag may be associated with a
lib to LD_PRELOAD. It's now done for uname(2), which requires uname.so
in order to trap the calls.
Other trap will be added in later commits.
Lucas De Marchi [Wed, 25 Jan 2012 01:28:39 +0000 (23:28 -0200)]
testsuite: move oneshot to inside the test struct
Lucas De Marchi [Wed, 25 Jan 2012 00:04:46 +0000 (22:04 -0200)]
testsuite: trap calls to uname
Lucas De Marchi [Tue, 24 Jan 2012 23:53:27 +0000 (21:53 -0200)]
testsuite: test libkmod initialization
Lucas De Marchi [Tue, 24 Jan 2012 22:59:54 +0000 (20:59 -0200)]
testsuite: add skeleton
Lucas De Marchi [Thu, 26 Jan 2012 18:01:34 +0000 (16:01 -0200)]
modprobe: fix build 'cause of excessive number of arguments
Dave Reisner [Thu, 26 Jan 2012 16:36:35 +0000 (11:36 -0500)]
modprobe: remove code referring to -t, --type
This was only useful with the --list function, which isn't implemented.
Lucas De Marchi [Mon, 23 Jan 2012 14:29:38 +0000 (12:29 -0200)]
README: add link to packages place
A lot of people trying to get latest package from cgit is not good.
They should get the package from the released versions instead.
Lucas De Marchi [Sat, 21 Jan 2012 17:38:25 +0000 (15:38 -0200)]
build-sys: do not create symlinks by default
Distro packagers should create them instead. It's too much trouble to
create them in the build system and every distro wants a different path
for them.
Lucas De Marchi [Sat, 21 Jan 2012 04:45:06 +0000 (02:45 -0200)]
modprobe: kill operations depending on path
It was not on module-init-tools and it doesn't make much sense. It will
deal with dependencies, but looking at modules in the index. This might
not be the module we want if we are loading another from outside of the
tree.
Dealing with paths causes this bug (supposing there's a module names
squashfs):
# cd /
# touch squashfs
# modprobe squashfs
That is because it detects that squashfs exists as a file and it will
try to load it instead of the alias "squashfs".
If you need to load a module from a path, use insmod.
Thanks to Silvan Calarco <silvan.calarco@mambasoft.it> who reported the
bug and helped debugging it.
Lucas De Marchi [Fri, 20 Jan 2012 15:04:43 +0000 (13:04 -0200)]
TODO: system() should not be used inside a library
Lucas De Marchi [Wed, 18 Jan 2012 18:37:59 +0000 (16:37 -0200)]
build-sys: forcefully create links
If link already existed we would fail. Really not important for
packagers (the link should not exist), but for those who manually install it.
Lucas De Marchi [Wed, 18 Jan 2012 18:02:45 +0000 (16:02 -0200)]
build-sys: move modprobe to sbindir
Lucas De Marchi [Wed, 18 Jan 2012 18:01:58 +0000 (16:01 -0200)]
Add program to calculate the shortest relative path for symlinks
Lucas De Marchi [Tue, 17 Jan 2012 23:16:23 +0000 (21:16 -0200)]
libkmod-list: allow to append an empty list
Lucas De Marchi [Tue, 17 Jan 2012 21:33:32 +0000 (19:33 -0200)]
build-sys: create symlinks if we are installing tools
Thierry Vignaud [Tue, 17 Jan 2012 19:32:48 +0000 (17:32 -0200)]
WEXITSTATUS is defined in sys/wait.h
Fix compiling with dietlibc:
../libkmod/libkmod-module.c:858:2: warning: implicit declaration of function 'WEXITSTATUS' [-Wimplicit-function-declaration]
../libkmod/libkmod-module.c:858:2: warning: nested extern declaration of 'WEXITSTATUS' [-Wnested-externs]
(...)
kmod-modprobe.c:(.text.command_do+0x157): undefined reference to `WEXITSTATUS'
Lucas De Marchi [Tue, 17 Jan 2012 14:10:42 +0000 (12:10 -0200)]
Check if struct stat has mtim member
Not all libc's have a mtim member in struct stat (dietlibc doesn't).
Change ts_usec() to receive a struct stat as parameter and implement it
accordingly for both cases.
Lucas De Marchi [Tue, 17 Jan 2012 12:05:02 +0000 (10:05 -0200)]
modprobe: flush stdout before dumping indexes
Index dump doesn't use stdio.h function and instead call write()
directly on STDOUT_FILENO file descriptor. Therefore we need to flush
stdio buffers before calling it, to be sure the configuration dump will
appear before index's.
Lucas De Marchi [Tue, 17 Jan 2012 01:03:33 +0000 (23:03 -0200)]
Fix common misspelling with codespell
-------8<-------
SUMMARY:
ommitted 1
Miklos Vajna [Tue, 17 Jan 2012 00:48:11 +0000 (01:48 +0100)]
man: spelling fixes
Lucas De Marchi [Mon, 16 Jan 2012 18:53:04 +0000 (16:53 -0200)]
kmod 4
Lucas De Marchi [Mon, 16 Jan 2012 02:42:28 +0000 (00:42 -0200)]
build-sys: add release helpers
Lucas De Marchi [Mon, 16 Jan 2012 18:43:47 +0000 (16:43 -0200)]
Add doc to kmod_dump_index
Lucas De Marchi [Mon, 16 Jan 2012 18:00:35 +0000 (16:00 -0200)]
modprobe: dump indexes as well
Lucas De Marchi [Mon, 16 Jan 2012 17:56:17 +0000 (15:56 -0200)]
libkmod: dump index files
Provide a function to dump the index files to a certain fd. It could be
more optimized (particularly the functions to dump the index that were
copied and pasted from m-i-t), but it seems like the only user of it is
'modprobe -c', used for debugging purposes. So, keep it as is.
Lucas De Marchi [Mon, 16 Jan 2012 17:46:01 +0000 (15:46 -0200)]
libkmod-util: add helper function to write to fd
Lucas De Marchi [Mon, 16 Jan 2012 17:45:12 +0000 (15:45 -0200)]
libkmod-util: adhere to coding style
Lucas De Marchi [Mon, 16 Jan 2012 14:01:48 +0000 (12:01 -0200)]
libkmod: export enum kmod_index and rename members
Lucas De Marchi [Mon, 16 Jan 2012 12:43:34 +0000 (10:43 -0200)]
libkmod: store prefix of each index
Prefix is useful when dumping the index (to be added later).
Lucas De Marchi [Sat, 14 Jan 2012 14:09:34 +0000 (12:09 -0200)]
Update documentation with recent changes
Lucas De Marchi [Sat, 14 Jan 2012 04:46:58 +0000 (02:46 -0200)]
depmod: deal with relative root dir
module paths coming from libkmod are absolute. Make sure we store the
absolute dirname to compare later.
Lucas De Marchi [Sat, 14 Jan 2012 04:31:51 +0000 (02:31 -0200)]
kmod_new(): deal with relative paths for dirname
Lucas De Marchi [Sat, 14 Jan 2012 04:04:56 +0000 (02:04 -0200)]
depmod: fix idx calculation after postponed array creation
We need to set up each module's idx when the array is created. So,
postpone its initialization as well.
Lucas De Marchi [Sat, 14 Jan 2012 04:04:31 +0000 (02:04 -0200)]
Reduce scope of counter variable
Lucas De Marchi [Sat, 14 Jan 2012 04:03:21 +0000 (02:03 -0200)]
depmod: adhere to coding style
Lucas De Marchi [Fri, 13 Jan 2012 04:35:34 +0000 (02:35 -0200)]
modprobe: dump configuration
Lucas De Marchi [Fri, 13 Jan 2012 12:49:31 +0000 (10:49 -0200)]
config: let softdeps dump their data
Lucas De Marchi [Fri, 13 Jan 2012 04:24:18 +0000 (02:24 -0200)]
config: add exported iterator functions
Config iterators are useful to get each configuration list, remember its
type and how to get their key/value pair.
softdeps don't have the value yet, because they are stored as string
vectors.
Lucas De Marchi [Fri, 13 Jan 2012 03:14:46 +0000 (01:14 -0200)]
libkmod-private: allow to get aliases from config
Lucas De Marchi [Thu, 12 Jan 2012 20:31:10 +0000 (18:31 -0200)]
build-sys: add script to bootstrap and configure
Lucas De Marchi [Thu, 12 Jan 2012 20:23:32 +0000 (18:23 -0200)]
build-sys: rename autogen.sh to bootstrap and keep a symlink
Lucas De Marchi [Thu, 12 Jan 2012 20:14:32 +0000 (18:14 -0200)]
TODO: add code unification to list of tasks
Lucas De Marchi [Thu, 12 Jan 2012 19:14:30 +0000 (17:14 -0200)]
modprobe: abort on dependency loop that cannot be broken
Lucas De Marchi [Thu, 12 Jan 2012 17:36:54 +0000 (15:36 -0200)]
modprobe: fix leak on error path
Lucas De Marchi [Thu, 12 Jan 2012 17:28:19 +0000 (15:28 -0200)]
TODO: update tasks
Lucas De Marchi [Thu, 12 Jan 2012 17:23:51 +0000 (15:23 -0200)]
modprobe: rework module removal without tree traversing
Just like the module insertion, module removal is remade.
The dependencies line that comes from modules.dep already contains all
the dependencies necessary to remove that module. Therefore modprobe
doesn't have to do the recursion between the modules in order to remove
it. All we have to do is to remove in order:
For the module being removed:
----------------------------
1. softdeps (in reverse order)
2. deps (in reverse order)
3. module
4. postdeps (in reverse order)
For any of the dependencies:
----------------------------
1. softdeps (in reverse order)
2. module
3. softdeps (in reverse order)
Lucas De Marchi [Thu, 12 Jan 2012 16:45:11 +0000 (14:45 -0200)]
TODO: update differences with m-i-t
Lucas De Marchi [Thu, 12 Jan 2012 04:21:26 +0000 (02:21 -0200)]
modprobe: rework module insertion without tree traversing
The dependencies line that comes from modules.dep already contains all
the dependencies necessary to insert that module. Therefore modprobe
doesn't have to do the recursion between the modules in order to load a
module. All we have to do is to load in order:
For the module being loaded:
----------------------------
1. softdeps
2. deps
3. module
4. postdeps
For any of the dependencies:
----------------------------
1. softdeps
2. module
3. softdeps
Lucas De Marchi [Thu, 12 Jan 2012 16:20:10 +0000 (14:20 -0200)]
man: build modinfo man page
Lucas De Marchi [Thu, 12 Jan 2012 16:12:27 +0000 (14:12 -0200)]
man: build modprobe man page
Lucas De Marchi [Thu, 12 Jan 2012 16:02:27 +0000 (14:02 -0200)]
man: build rmmod man page
Lucas De Marchi [Thu, 12 Jan 2012 15:57:45 +0000 (13:57 -0200)]
man: build lsmod man page
Lucas De Marchi [Thu, 12 Jan 2012 15:53:55 +0000 (13:53 -0200)]
man: build insmod man page
Lucas De Marchi [Thu, 12 Jan 2012 15:47:21 +0000 (13:47 -0200)]
man: build depmod man page
Lucas De Marchi [Thu, 12 Jan 2012 15:34:09 +0000 (13:34 -0200)]
man: build modules.dep{,.bin} man pages
Lucas De Marchi [Thu, 12 Jan 2012 15:18:08 +0000 (13:18 -0200)]
man: build modprobe.d man page
Lucas De Marchi [Thu, 12 Jan 2012 13:48:21 +0000 (11:48 -0200)]
build-sys: build man pages
Lucas De Marchi [Thu, 12 Jan 2012 13:24:38 +0000 (11:24 -0200)]
man: convert depmod.d.sgml to xml and update it
Lucas De Marchi [Thu, 12 Jan 2012 13:52:58 +0000 (11:52 -0200)]
build-sys: fix clean rule removing tracked files
Lucas De Marchi [Thu, 12 Jan 2012 12:05:25 +0000 (10:05 -0200)]
rename doc dir to man
Robby Workman [Thu, 12 Jan 2012 06:00:27 +0000 (00:00 -0600)]
doc/modprobe.sgml: Various updates wrt inclusion with kmod
* TODO: document "--remove-dependencies" flag
Robby Workman [Thu, 12 Jan 2012 05:47:11 +0000 (23:47 -0600)]
doc/depmod.sgml: Remove references to legacy map files
Robby Workman [Thu, 12 Jan 2012 05:40:24 +0000 (23:40 -0600)]
doc/depmod.sgml: Remove refs to depmod.conf
Robby Workman [Thu, 12 Jan 2012 05:37:03 +0000 (23:37 -0600)]
doc/depmod.d.sgml: Various touchups; mostly s/depmod.conf/depmod.d/g
Robby Workman [Thu, 12 Jan 2012 06:06:27 +0000 (00:06 -0600)]
doc/modprobe.d.sgml: Various touchups
* change references to "kmod" instead of "module-init-tools"
* remove references to modprobe.conf
Robby Workman [Thu, 12 Jan 2012 06:04:57 +0000 (00:04 -0600)]
doc: Import sgml manpages from module-init-tools repo
Lucas De Marchi [Wed, 11 Jan 2012 23:48:08 +0000 (21:48 -0200)]
modprobe: show if module is in kernel
Now with './tools/modprobe --show-depends ahci' (ahci is builtin) we have the following
output:
$ ./tools/modprobe --show-depends ahci
builtin ahci
Just like modprobe from m-i-t. Previously we had:
$ ./tools/modprobe --show-depends ahci
FATAL: Module ahci not found.
Lucas De Marchi [Wed, 11 Jan 2012 23:22:21 +0000 (21:22 -0200)]
test: add check of module's state
Jan Alexander Steffens (heftig) [Wed, 11 Jan 2012 22:17:38 +0000 (23:17 +0100)]
depmod: Postpone creation of module array
Deleting modules (we have found replacements) invalidates the indices
because the array collapses removed elements, hitting the assertion.
Since we don't make use of the array until the sorting step, build it from
the modules_by_name hash instead.
Lucas De Marchi [Wed, 11 Jan 2012 20:29:55 +0000 (18:29 -0200)]
modprobe: break dependency loop by checking if module is loaded
modprobe doesn't have support for handling dependency loop. That happens
with poorly written softdeps that can introduce a loop. We must deal
with them like it's being done in libkmod.
However, we can break a dependency loop when the dependency was already
inserted. This commit fixes this issue, that happens in the following
scenario:
dependencies:
-------------
modA:
modB: modA
modC: modA
config:
softdep modA post: modB modC
This creates the following loop:
modA
inserted ok
handle post-soft-deps of modA -> modB modC
modB
handle dependencies of modB -> modA
modA is already inserted
handle post-soft-deps of modA -> modB modC
And so on and so forth.
Now we break the loop by checking if module is already inserted, before
handling it. Thus this gives us:
modA
inserted ok
handle post-soft-deps of modA -> modB modC
modB
handle dependencies of modB -> modA
modA is already inserted
inserted ok
modC
handle dependencies of modC -> modA
modA is already inserted
inserted ok
Lucas De Marchi [Wed, 11 Jan 2012 19:54:05 +0000 (17:54 -0200)]
autogen.sh: configure with 3-clicks + paste
Lucas De Marchi [Wed, 11 Jan 2012 17:38:50 +0000 (15:38 -0200)]
modprobe: adhere do coding style
Dave Reisner [Wed, 11 Jan 2012 00:28:30 +0000 (19:28 -0500)]
rmmod: behavior more like m-i-t's rmmod
In line with m-i-t's behavior, we should check to see if each module is:
- loaded
- has any holders
- has a 0 refcnt
Detecting any of these lets us provide a more useful message than the
kernel's EPERM response to delete_module(2).
Additionally, alter the main loop behavior to avoid exiting early on the
first error.
Dave Reisner [Tue, 10 Jan 2012 18:36:27 +0000 (13:36 -0500)]
modprobe: check for EPERM on insertion
Throw an appropriate error when an unprivileged user attempts to load a
module.
Lucas De Marchi [Wed, 11 Jan 2012 02:28:12 +0000 (00:28 -0200)]
config: deprecate 'include' and 'config' commands
Lucas De Marchi [Wed, 11 Jan 2012 02:08:14 +0000 (00:08 -0200)]
Fix error code returned on module removal