Previously, depmod would relegate failures of kmod_module_get_symbols()
to debug output, assuming the "error" was simply a lack of symbols.
Leave the ENOENT return to debug output, but report anything else as a
real error.
---
 tools/kmod-depmod.c |    7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/kmod-depmod.c b/tools/kmod-depmod.c
index e89dff6..bceb407 100644
--- a/tools/kmod-depmod.c
+++ b/tools/kmod-depmod.c
@@ -1542,8 +1542,11 @@ static int depmod_load_symbols(struct depmod *depmod)
                struct kmod_list *l, *list = NULL;
                int err = kmod_module_get_symbols(mod->kmod, &list);
                if (err < 0) {
-                       DBG("ignoring %s: no symbols: %s\n",
-                               mod->path, strerror(-err));
+                       if (err == -ENOENT)
+                               DBG("ignoring %s: no symbols\n", mod->path);
+                       else
+                               ERR("failed to load symbols from %s: %s\n",
+                                               mod->path, strerror(-err));
                        continue;
                }
                kmod_list_foreach(l, list) {
-- 
1.7.10.1

--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to