The eal_parse_sysfs_value helper both returns an error code and logs an
error level message when something goes wrong.
On the other hand, internal users of this helper either ignore this
error code (like when trying to find out some numa information from the
Linux sysfs, or discovering some optional feature), or add their own error
logging when reading the file actually matters.

Lower this helper log messages to debug level as it provides no useful
information to final DPDK users.

Signed-off-by: David Marchand <david.march...@redhat.com>
---
 lib/eal/unix/eal_filesystem.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/eal/unix/eal_filesystem.c b/lib/eal/unix/eal_filesystem.c
index afbab9368a..282df50347 100644
--- a/lib/eal/unix/eal_filesystem.c
+++ b/lib/eal/unix/eal_filesystem.c
@@ -84,20 +84,20 @@ int eal_parse_sysfs_value(const char *filename, unsigned 
long *val)
        char *end = NULL;
 
        if ((f = fopen(filename, "r")) == NULL) {
-               RTE_LOG(ERR, EAL, "%s(): cannot open sysfs value %s\n",
+               RTE_LOG(DEBUG, EAL, "%s(): cannot open sysfs value %s\n",
                        __func__, filename);
                return -1;
        }
 
        if (fgets(buf, sizeof(buf), f) == NULL) {
-               RTE_LOG(ERR, EAL, "%s(): cannot read sysfs value %s\n",
+               RTE_LOG(DEBUG, EAL, "%s(): cannot read sysfs value %s\n",
                        __func__, filename);
                fclose(f);
                return -1;
        }
        *val = strtoul(buf, &end, 0);
        if ((buf[0] == '\0') || (end == NULL) || (*end != '\n')) {
-               RTE_LOG(ERR, EAL, "%s(): cannot parse sysfs value %s\n",
+               RTE_LOG(DEBUG, EAL, "%s(): cannot parse sysfs value %s\n",
                                __func__, filename);
                fclose(f);
                return -1;
-- 
2.41.0

Reply via email to