Fix unaligned memory access

Bug found on sparc64. Thanks to "Jan Engelhardt <jengelh@medozas.de>"
for providing access to such a machine.

do not (mis-)use the config file generator for non-autobuild stuff

"Similarly, you should not rely on AC_CONFIG_FILES to replace bindir
and friends in your shell scripts and other files; instead, let make
manage their replacement."

http://www.gnu.org/software/autoconf/manual/autoconf.html#Makefile-Substitutions

gitignore: ignore dist files

kmod 2

Change error message to reflect reality

Update doc to reflect function behavior

Testing for return of alias_normalize.

build-sys: do not use some compile flags by default

Removing comments saying flags are not implemented

Both removal and insertion flags are already implemented.

Move symbols introduced in libkmod2 to the right place

build-sys: remove remaining _SOURCES var

build-sys: default to "" as rootprefix

kmod_modprobe: fix description of -C flag

configure: disable static zlib build.

It was not being done correctly, disable until we find a way for
libtool to cooperate.

introduce --with-rootprefix=DIR

Configure the location of the rootfs directories and use it
to find (/usr)/lib/modules and (/usr)/lib/modprobe.d.

Use sorted configuration files in precedence order

We need to keep config files sorted and use them taking the precedence
order into account.

The following message was taken from module-init-tools commit doing a
similar thing:

Configuration files are parsed in alphabetic order, regardles of
what directory they reside in. Furthermore, if several files by
the same name exist in different directories only the one in the
directory with highest precedence is loaded.

The order of precedence is /run, /etc, /usr/lib, /lib.

The sad thing is that we are not using openat() anymore since each file
is in different directories. In future we might change the
implementation to open all DIRs and keep a reference
to them instead of the path. However we'd have to keep a separate list
with all the opened dirs so we can close them later (when all configs
are parsed).

build: use dist-xz

It's 2011 already.

gitignore: only ignore build system files in the top directory

libkmod: use C99 __func__ over GNU-specific __FUNCTION__

kmod_config: fix kcmd line parser with dots after =

Check if '=' appeared before the dot. In this case, it's not a valid
module option in kernel command line.

The command line that was failing is:

"root=/dev/sda3 ro pcie_aspm=force init=/sbin/bootchartd
initrd=../initramfs-linux.img BOOT_IMAGE=../vmlinuz-linux"

Removing warnings when compiling on i686 platform.

kmod_module_new_*: improve debugging.

kmod_modprobe: Fix regression when inserting module

Commit "e5e2a68 kmod_modprobe: properly handle install/remove commands"
introduced a regression that, while it worked for install/remove
commands, it ceased to work for normal module names. Move the check for
whether it's a install command or a module so both cases work.

Fix check for opened indexes

elf: fix regression with empty strings

Commit "b20dc17 Remove unneeded reference to last string" reverted the
fix in "47a0ef6 elf: do not output empty strings." and empty strings are
appearing again in kmod-modinfo.

With this commit we do a bit different and instead of keeping the
reference to last string we skip the '\0' inside the loop.

Cache the offset of crc

This way we don't need to check every time in loop. Indeed compiler
could already optimize out that checks, but this ways is cleaner and
shorter.

Make section size constant since it's always the same

Optimize kmod_elf_get_strings() by reducing calls to memcpy

Copy the entire string only once and then scan it, setting up the
pointers of the vector.

Remove unneeded reference to last string

Put blank lines where it's due

See CODING-STYLE for reference

Terminate strv with NULL

Reduce ident by continuing early

build-sys: remove unneeded _SOURCES vars

elf: skip leading symbol dot in modversion.

Mimics module-init-tools's dump_modversions() that calls skip_dot()
to skip leading dots. This is required for PPC64.

elf: fix typo that resulted in invalid uint reads for big-endian platforms.

kmod-modinfo: -p (-F parm) shows also parmtype in Debian expected formatting.

debian expected formatting is:

    name:description (type)

variants:

    name:description
    name: (type)

elf: do not output empty strings.

kmod-modinfo: fix typo and output format.

 * it's parm, not param (does not have the second 'a');
 * output format requires ':' after the name.

kmod-modinfo: add missing short options.

Thanks to falconindy for testing.

Introduce kmod-modinfo.

elf: fix minimum file size calculation.

there is only a single file header, multiple sections.

kmod-modprobe: improve --help output.

kmod-modprobe: implement --dump-modversions

ELF: initial support for modinfo and strip of modversions and vermagic.

Needs testing, but should work.

Fix debug message formatting

TODO: add idea about single file with all compressed modules.

kmod_modprobe: properly handle install/remove commands

Handle install/remove commands just like modprobe does. Test configure
file:

install installme echo "this is a install message"
remove  removeme echo "this is a remove message"

Tests:

$ ./tools/kmod-modprobe installme
this is a install message

$ ./tools/kmod-modprobe -r removeme
this is a remove message

$ ./tools/kmod-modprobe removeme
FATAL: Module removeme not found.

./tools/kmod-modprobe -r installme
FATAL: Module installme not found.

kmod_modprobe: fix handling of remove commands

The check for remove/install commands must be before the ignore_loaded
check because we will actually run something instead of
removing/inserting a module and the modname might not correspond to a
real module. Otherwise a fake module like "remove removeme echo 'bla'"
would not work.

This also keeps compatibility with modprobe.

Fix wrong name len when converting path to modname

Fix leak of kmod_module

compatibility: match commands using fnmatch() instead of strcmp().

module-init-tools modprobe.c use fnmatch() and not strcmp() to match
commands and softdeps, although the man page does not say so. Then use
the same function to provide compatibility.

kmod-modprobe: implement softdeps.

Implement soft dependencies in a way similar to module-init-tools
modprobe. Unlike regular dependencies they are allowed to fail
inserting or removing.

The rmmod version walks the lists in reverse order, also doing post
before and pre later.

introduce kmod_list_foreach_reverse().

walks the list in the reverse order.

Use tab instead of spaces

introduce kmod_list_last()

This gets the last element in the list, that is, the previous element
of the head.

fix kmod_list_prev().

kmod_list_prev() should return NULL if the current element is the
head, not if the previous element is the head. This was likely a copy
& paste error from kmod_list_next().

implement softdeps.

fix error handling path.

implement zlib module loading.

Lookup for commands in kmod_module_new_from_lookup()

Install and remove commands are now properly treated on lookup. Example
config file:

$ ./test/test-lookup installme
libkmod version 1
Alias: 'installme'
Modules matching:
installme
install commands: 'echo "this is a install message"'

$ ./test/test-lookup removeme
libkmod version 1
Alias: 'removeme'
Modules matching:
removeme
remove commands: 'echo "this is a remove message"'

kmod_module: do not find more than the first command

modprobe from module-init-tools does not use more than one
install/remove command, it just stops on the first one. Test
modprobe.conf:

install bla echo "this is a message"
install bla echo "this is a message"

$ modprobe bla
this is a message
$

Install and remove commands are already a legacy thing we need to carry,
but let's not extend it so people do not start doing crazy things.

With this patch we are breaking on the first element we find in the
configuration. May be we can add a warning later when parsing the config
that install commands are duplicated.

Open more file descriptors with O_CLOEXEC

Library must use O_CLOEXEC whenever it opens file descriptors

Add address of mailing list

tools: fix kmod-modprobe -R trying to insert module

TODO: add new tasks and notes to future development

trivial: fix typo causing an infinite loop

kmod 1

Add libkmod files to EXTRA_DIST

Remove module from hash when it's gone

Module was never being removed from hash table. Therefore, if we create
a module, unref it and create it again we will access freed memory.
Commit "53385cf Improve test of double references" introduced a new test
in test-mod-double-ref.c that previously to this commit was crashing and
now it's working fine.

Improve test of double references

Add safety NULL checks in exported functions

Fix changing hash key after module is inserted in hash

The hash key is not copied so we can't change the string from:
modname/modalias

to:

modname'\0'modalias

in order to setup mod->name and mod->alias.

Now what we do is:

1) if name is in the form 'modname/modalias', the final struct
   kmod_module will be:

   struct kmod_module {
           char *alias;------,
           char *name;-----, |
           char *hashkey;--|-|-,
   }                       | | |
   name <------------------' | |
   alias <-------------------' |
   hashkey <-------------------'

2) if name is in the simple form 'modname', then the final struct
   kmod_module will be:

   struct kmod_module {
           char *alias;------> NULL
           char *name;-----,
           char *hashkey;--|---,
   }                       |   |
   name <------------------*---'

Fix docs with wrong function names

Add comment in public header about flags not implemented

Rename symbol group

Remove flags that likely will not be used

TODO: rephrase some tasks

Add file with details regarding coding style

Fix format of log message

kmod_config: parse kernel command line for options and blacklist

Do not depend on AM_SILENT_RULES if it is not available.

kmod_module: use 'modname/aliasname' as key for hash

1 alias may correspond to more than 1 module. This would cause a
conflict in the hash table when inserting a module there and bad things
could happen.

Now we use 'modname/aliasname' as key, '/aliasname' part being optional.
Internally kmod_module_new_from_alias() will setup a 'modname/aliasname'
string and pass to kmod_module_new_from_name() that will treat the case
with a '/' in the name.

User might call kmod_module_new_from_name() without any slashes, so the
key my not contain it.

Use malloc + memset instead of calloc

TODO: update tasks

kmod_module: move function to the right section

kmod_module: fix log message upon module removal

kmod_module: add missing documentation

Simplify kmod_load_resources()

We have a static vector of indexes. There's no need to check if they
have a suffix or if they are absolute, we just already know in advance
and there are no plans to change it.

Log and break early if index is already open

Update documentation

Remove function kmod_resolve_alias_options()

Remove function kmod_resolve_alias_options since it's not needed
anymore. Test is using the following configuration file:

alias blablabla ac
options ac test=1
options blablabla test=2

Lookup test by module name:
$ ./test/test-lookup ac
libkmod version 1
Alias: 'ac'
Modules matching:
ac
options: 'test=1'

Lookup test by alias:
$ ./test/test-lookup blablabla
libkmod version 1
Alias: 'blablabla'
Modules matching:
ac
options: 'test=1 test=2'

kmod_module: make get_options() search for alias names too

Create module by alias name where appropriate

Add private function kmod_module_new_from_alias()

This function will create a new kmod_module() by calling
kmod_module_new_from_name(), but instead of given the module name, it
gives the alias. This way, the modules' hash will contain the alias
instead of the name. If module was created successfully, then it swap
the alias with the actual name.

The downside is that the structure is not shared anymore if we create a
kmod_module by alias and after by name. However, in modprobe's
configuration it's possible to have different options for different
aliases. In future we might want to create a way to share the common
part of the structure (then having only one instance as we had before).

kmod_module: use pointer instead of vector for its name

We still have name allocated just after the struct kmod_module, but
now we use a pointer instead of putting name as a vector in the end of
the structure.

The previous way has some problems, the worst is:
- it's not possible to swap the name with another value: this is
  the real problem that this patch is solving. Later patches
  will make name be swappable with alias, which is not possible
  if name is a vector.

Return early on lookup error

There's no reason to keep looking for options if alias didn't match.

Use alias_normalize() instead of modname_normalize()

When normalizing alias names (or if we don't know if it's an alias or
modname), use alias_normalize() instead of modname_normalize(). The
difference is that alias names can contain dashes withing brackets, and
those should not be changed to underscores.

Most of the places using underscores() function might be converted to
alias_normalize(), but this is not done now.

Add helper alias_normalize()

Fix check for NULL variable