build-sys: move remaining warning flags to configure

build-sys: remove duplicated warning flags

"822ce23 beef compile flags" added warning flags to configure, but some
of them were already present in autogen.sh. Remove the flags from
autogen.sh.

coding style: be consistent with blank lines

coding style: fix lines over 80 chars

Lines should not go over 80 chars with a few exceptions:
- headers
- function definitions with only 1 argument
- long strings, otherwise we break grep

This should go later in a coding-style file

Merge branch 'use-mmap'

Conflicts:
libkmod/libkmod.c

API-BREAK: kmod_new() takes a second parameter for configuration directory.

This is required by modprobe and also to help doing unit tests in future.

tools: add modprobe

try to mimic original module-init-tools' modprobe as much as possible,
but this exposed some missing features in libkmod, these are now
listed in TODO.

add lsmod, insmod and rmmod tools.

these tools are compatible with module-init-tools (except insmod does
not take data from stdin).

add missing newline to log messages.

be less verbose on initstate for unexistent modules.

the module may be unloaded, in this case -NOENT is returned but it
should not present an error message

beef compile flags.

Beef the compile flags by using the same set recommended by systemd,
it uses attribute.m4 from the xine project to check supported flags.

insmod: allows providing option to module.

always normalize user-given alias.

export module's options and commands.

This will be required to implement modprobe later. The implementation
follows "man modprobe.conf" and allows options to be specified for
alias as well, thus the need for kmod_resolve_alias_options().

Example mod-a.conf:

    options mod-a a=1 b=2
    options mod-a c=3
    alias mymod-a mod-a
    options mymod-a d=4

Results in:
    options mod-a a=1 b=2 c=3
    options mymod-a a=1 b=2 c=3 d=4

Install commands are being concatenated with ";", but manpage is not
clean about this behavior.

parse_depline may be called from libkmod.c

allow parse_depline to be called when already initialized as it may be
called from libkmod.c and at that point there is no way to check
mod->init.dep

fix string replace.

configure files with this would fail:

options wl x=1

were being handled as " =1".

kmod: Add documentation and format comments

update TODO

index: fix memleak for non-matchin aliases.

index-mm: allocate values inline into node, strings points to mmap.

For mmap mode, we can avoid allocating and copying strings from the
mmap'ed memory.

With that we have fixed length "struct index_mm_value" that can be
allocated inline with "struct index_mm_node".

index: cleanup header, move as much as possible to libkmod-index.c

index-mm: no need to allocate prefix.

index-mm: fix memory access.

uint32_t reads must be aligned, they're not then use memcpy().

read_alloc_chars_mm() and read_chars_mm() were wrong, normalize all
address calculation using single byte pointer "addr" that is
incremented by the amount read, this will avoid further errors.

test-lookup: allow loading resources for testing.

kmod_load_resources: use proper full path to binary files to be mmaped.

kmod_lookup_alias_from_alias_bin: debug message for mmap case.

kmod_search_moddep: reorder to avoid creating path when not required.

if we're using mmap'ed index, say so and avoid creating the path as it
won't be used.

fix snprintf usage.

snprintf() takes the full buffer size, including \0 and guarantees it
will be there.

kmod: Add documentation and format comments

Use alias/symbol index_mm if it's open

Pass enum around instead of filename

This will allow us to later use the index to search these indexes.

Use index_mm if it's open for moddep search

Do not allocate path for known places an close resource asap

This place is not supposed to exceed PATH_MAX. So, use snprintf instead
of asprintf.

Close the index before iterating the values.

Load and unload resources

This call will mmap all the index files and in future some of the work
done in ctx creation can be put here.

index: mm: Add flag to open call to populate buffer

Refactor index file handling

Put all names in a static vector and declare a enum containing the
number of indexes. This way it's easier to create vectors inside ctx
that depend on these files.

remove useless look checking for duplicates

index: avoid strlen() whenever possible.

reduce calls to realloc() if size did not change.

index: improve buffer management and reduce mallocs.

Grow buffer based on a step, avoiding hitting the system over and over
again.

Do not allocate the 'struct buffer' as in all cases the lifetime is
known and the pattern was allocate then free in every call site.

fix path handling at dependencies parsing.

paths come relative to dirname, make them absolute to avoid confusion
later.

NULL safety in public places, allows disable logging.

log: give log function its data instead of kmod_ctx.

This will be the most common use case for logging, also changed
log_stderr() to log_filep() with data being stderr to test it.

modname_normalize: fix const and buffer overflow.

"buf[NAME_MAX] = value" is invalid since it would access the byte
right after the array.

Also fix the const of modname, do not mess with it to avoid mistakes.

kmod_loaded: document exported function

kmod_list: document exported functions

kmod_module: Remove const from path

It's an ugly hack. This is an internal variable, we know we shouldn't
change it everywhere.

add kmod_module_get_filtered_blacklist()

This function will filter the given list against the known blacklist,
returning a new list with remaining modules with the reference
incremented.

Only search path in moddep if it's not already set

rmmod/insmod should be able to operate directly on files, not relying on
indexes and configuration.

The following test was not working and now it is:

$ # go to a dir != mod->dirname
$ cd /lib/modules/$(uname -r)/kernel
$ # try to insert module giving a relative path
$ insmod drivers/acpi/ac.ko

Fix abs path helper function

test-insmod: show the path libkmod is using

test-insmod: print name to test modname_normalize().

kmod_module: treat module creation by path with same names

If a module with the same name already exists, try to reference it if
paths are the same. Otherwise fail.

kmod_module: store absolute path when creating module from path

Add helper path_make_absolute_cwd()

Add helper path_is_absolute()

kmod_config: parse install and remove commands

kmod_config: parse module options

Add helper strchr_replace()

Use streq() when possible

Split function so we don't call basename() unnecessarily

test: add test for modules' hash

Maintain a pool of modules alive

Based on previous implementation by
Gustavo Sverzut Barbieri <barbieri@profusion.mobi>

kmod_config: optimize config files handling

 1) Allocate less by not sorting the result with qsort. Instead,
    insert the nodes in the correct order;
 2) Do not maintain the whole path in memory, but rely on openat()

kmod_list: remove nodes in order

kmod_list: add helper function to merge two lists

This helper function will append the second list in the first one, so
they become one single list.

kmod_list: add helper kmod_list_insert_before()

kmod_list: add helper kmod_list_insert_after()

test: add test to get dependencies of a module

test: add test to convert name to path

If we create a kmod_module from a name, the path returned is relative to
the module dirname, as passed during kmod_ctx creation. Note that if
kmod_ctx is created with kmod_new(NULL), the dir used is the one
returned by uname.

kmod_module: normalize module name

kmod_module: get path on demand

kmod_module: parse dependencies on demand

Split function to search moddep file

kmod_module: return a new list and increase ref of dependencies

kmod_module_get_dependency is renamed to kmod_module_get_dependencies
since it's returning a list. To match other APIs, now it returns a new
list that user must free with kmod_module_unref_list().

Fix missing parenthesis in macro

Add simple hash implementation

kmod_module: inline name and make it always available

kmod_alias: reduce from 3 mallocs to a single one.

Use readdir_r in kmod_module_get_sections()

readdir() is not thread-safe. Use readdir_r instead.

Use readdir_r in kmod_module_get_holders()

readdir() is not thread-safe. Use readdir_r instead.

Add padding to enum to make sure it's an int

Inline foreach macro for internal usage

Avoid calling _next() function because it's an exported function and
linker can not optimize it.

Thanks to "Gustavo Sverzut Barbieri <barbieri@profusion.mobi>" for
suggestion.

tests: release memory before error exits.

this makes it easier to valgrind the error cases as well.

no more kmod_loaded and kmod_loaded_module.

kmod_loaded_get_list() now returns a regular list of kmod_modules, use
kmod_module_get_module(), kmod_module_unref() and
kmod_module_unref_list() to operate on it.

kmod_module: extended information gathering.

provide means to get:
 * refcount
 * initstate
 * holders
 * sections

this can be used to individually query properties from modules,
similar to /proc/modules (kmod_loaded / kmod_loaded_module).

kmod_module_get_module: safety against NULL pointers

module refcount should start at 1.

the current way was having the referenced modules to be released given
the unref comparison checking for "> 0".

fix missing ")".

TODO: update file

License library as LGPL

The only part under GPL was the index, that is now re-licensed. Switch
back to LGPL.

index: change license to LGPL

I've got an explicit permit from index author to re-license it as LGPL.

index: update copyrignt

mmap implementation

index: mmap: add support for seaching with wildcards

Almost a clean copy & paste from the previous implementation.

index: mmap: add support for searching

Almost a clean copy & paste from the previous implementation.

index: mmap: add support for searching node

Almost a clean copy & paste from the previous implementation.

index: mmap: read child node

index: mmap: read root node

index: mmap: create and destroy node

Add memdup() helper