trace: Simplify find_debugfs()
authorNamhyung Kim <namhyung@gmail.com>
Tue, 7 Nov 2017 15:31:34 +0000 (00:31 +0900)
committerStefan Hajnoczi <stefanha@redhat.com>
Mon, 18 Dec 2017 14:37:36 +0000 (14:37 +0000)
The return vale of find_debugfs() is 1 if it could find a mount point of
debugfs.  It can be saved in the while loop instead of checking it again.

Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
trace/ftrace.c

index 7de104debaf0f3e655634c173820dc3263f70bca..bfa38e71f0273abf700735d9939fd34067ec6e46 100644 (file)
@@ -19,6 +19,7 @@ static int find_debugfs(char *debugfs)
 {
     char type[100];
     FILE *fp;
+    int ret = 0;
 
     fp = fopen("/proc/mounts", "r");
     if (fp == NULL) {
@@ -28,15 +29,13 @@ static int find_debugfs(char *debugfs)
     while (fscanf(fp, "%*s %" STR(PATH_MAX) "s %99s %*s %*d %*d\n",
                   debugfs, type) == 2) {
         if (strcmp(type, "debugfs") == 0) {
+            ret = 1;
             break;
         }
     }
     fclose(fp);
 
-    if (strcmp(type, "debugfs") != 0) {
-        return 0;
-    }
-    return 1;
+    return ret;
 }
 
 bool ftrace_init(void)