perf buildid-cache: Use pr_debug instead of verbose && pr_info
authorMasami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Fri, 27 Feb 2015 04:50:28 +0000 (13:50 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 27 Feb 2015 18:52:36 +0000 (15:52 -0300)
Use pr_debug instead of the combination of verbose and pr_info.

"if (verbose) pr_info(...)" is same as "pr_debug(...)", replace it.

Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Suggested-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Borislav Petkov <bp@suse.de>
Cc: Hemant Kumar <hemant@linux.vnet.ibm.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20150227045028.1999.93137.stgit@localhost.localdomain
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-buildid-cache.c

index 86f9d78195a4499cd127162be18420e55b349714..04466c4ff9d78a1b452e6a2873439997ea778b21 100644 (file)
@@ -196,9 +196,8 @@ static int build_id_cache__add_file(const char *filename)
        build_id__sprintf(build_id, sizeof(build_id), sbuild_id);
        err = build_id_cache__add_s(sbuild_id, filename,
                                    false, false);
-       if (verbose)
-               pr_info("Adding %s %s: %s\n", sbuild_id, filename,
-                       err ? "FAIL" : "Ok");
+       pr_debug("Adding %s %s: %s\n", sbuild_id, filename,
+                err ? "FAIL" : "Ok");
        return err;
 }
 
@@ -216,9 +215,8 @@ static int build_id_cache__remove_file(const char *filename)
 
        build_id__sprintf(build_id, sizeof(build_id), sbuild_id);
        err = build_id_cache__remove_s(sbuild_id);
-       if (verbose)
-               pr_info("Removing %s %s: %s\n", sbuild_id, filename,
-                       err ? "FAIL" : "Ok");
+       pr_debug("Removing %s %s: %s\n", sbuild_id, filename,
+                err ? "FAIL" : "Ok");
 
        return err;
 }
@@ -235,17 +233,15 @@ static int build_id_cache__purge_path(const char *pathname)
 
        strlist__for_each(pos, list) {
                err = build_id_cache__remove_s(pos->s);
-               if (verbose)
-                       pr_info("Removing %s %s: %s\n", pos->s, pathname,
-                               err ? "FAIL" : "Ok");
+               pr_debug("Removing %s %s: %s\n", pos->s, pathname,
+                        err ? "FAIL" : "Ok");
                if (err)
                        break;
        }
        strlist__delete(list);
 
 out:
-       if (verbose)
-               pr_info("Purging %s: %s\n", pathname, err ? "FAIL" : "Ok");
+       pr_debug("Purging %s: %s\n", pathname, err ? "FAIL" : "Ok");
 
        return err;
 }
@@ -296,9 +292,8 @@ static int build_id_cache__update_file(const char *filename)
        if (!err)
                err = build_id_cache__add_s(sbuild_id, filename, false, false);
 
-       if (verbose)
-               pr_info("Updating %s %s: %s\n", sbuild_id, filename,
-                       err ? "FAIL" : "Ok");
+       pr_debug("Updating %s %s: %s\n", sbuild_id, filename,
+                err ? "FAIL" : "Ok");
 
        return err;
 }