]> git.ipfire.org Git - thirdparty/kmod.git/commitdiff
testsuite: remove overly verbose logging
authorEmil Velikov <emil.l.velikov@gmail.com>
Sat, 18 Jul 2026 14:20:56 +0000 (15:20 +0100)
committerLucas De Marchi <ldemarchi@kernel.org>
Tue, 28 Jul 2026 14:11:03 +0000 (09:11 -0500)
The delete_module.c (LD_PRELOAD module) prints the result of the
TC_DELETE_MODULE_RETCODES parsing. At the same time the init_module.c
(and it's variant) don't bother.

Glancing throughout, no other test uses LOG so let's drop the logging
allowing us to garbage collect (hide really) the potentially conflicting
macros.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Link: https://github.com/kmod-project/kmod/pull/450
Signed-off-by: Lucas De Marchi <ldemarchi@kernel.org>
testsuite/delete_module.c

index 023f58a148976d42e44f36ff473b0e8ff8f2ad1e..46fffb91f6cff071a33e008e4c2ebdeb0d41ad1c 100644 (file)
@@ -108,7 +108,6 @@ static struct mod *find_module(struct mod *_modules, const char *modname)
 static void init_retcodes(void)
 {
        const char *s;
-       struct mod *mod;
 
        if (!need_init)
                return;
@@ -121,11 +120,6 @@ static void init_retcodes(void)
        }
 
        parse_retcodes(&modules, s);
-
-       for (mod = modules; mod != NULL; mod = mod->next) {
-               LOG("Added module to test delete_module:\n");
-               LOG("\tname=%s ret=%d errcode=%d\n", mod->name, mod->ret, mod->errcode);
-       }
 }
 
 static int remove_directory(const char *path)