[Mesa-dev] [PATCH v2 19/26] dd/util: extract dd_get_debug_filename_and_mkdir

Nicolai Hähnle nhaehnle at gmail.com
Mon Nov 6 10:23:50 UTC 2017


From: Nicolai Hähnle <nicolai.haehnle at amd.com>

Reviewed-by: Marek Olšák <marek.olsak at amd.com>
---
 src/gallium/drivers/ddebug/dd_util.h | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/src/gallium/drivers/ddebug/dd_util.h b/src/gallium/drivers/ddebug/dd_util.h
index 4e1a945c57d..cfc0fb0ccce 100644
--- a/src/gallium/drivers/ddebug/dd_util.h
+++ b/src/gallium/drivers/ddebug/dd_util.h
@@ -33,49 +33,55 @@
 #include <unistd.h>
 #include <sys/stat.h>
 
 #include "c99_alloca.h"
 #include "os/os_process.h"
 #include "util/u_debug.h"
 
 /* name of the directory in home */
 #define DD_DIR "ddebug_dumps"
 
-static inline FILE *
-dd_get_debug_file(bool verbose)
+static inline void
+dd_get_debug_filename_and_mkdir(char *buf, size_t buflen, bool verbose)
 {
    static unsigned index;
-   char proc_name[128], dir[256], name[512];
-   FILE *f;
+   char proc_name[128], dir[256];
 
    if (!os_get_process_name(proc_name, sizeof(proc_name))) {
       fprintf(stderr, "dd: can't get the process name\n");
-      return NULL;
+      strcpy(proc_name, "unknown");
    }
 
    snprintf(dir, sizeof(dir), "%s/"DD_DIR, debug_get_option("HOME", "."));
 
-   if (mkdir(dir, 0774) && errno != EEXIST) {
+   if (mkdir(dir, 0774) && errno != EEXIST)
       fprintf(stderr, "dd: can't create a directory (%i)\n", errno);
-      return NULL;
-   }
 
-   snprintf(name, sizeof(name), "%s/%s_%u_%08u", dir, proc_name, getpid(), index++);
+   snprintf(buf, buflen, "%s/%s_%u_%08u", dir, proc_name, getpid(), index++);
+
+   if (verbose)
+      fprintf(stderr, "dd: dumping to file %s\n", buf);
+}
+
+static inline FILE *
+dd_get_debug_file(bool verbose)
+{
+   char name[512];
+   FILE *f;
+
+   dd_get_debug_filename_and_mkdir(name, sizeof(name), verbose);
    f = fopen(name, "w");
    if (!f) {
       fprintf(stderr, "dd: can't open file %s\n", name);
       return NULL;
    }
 
-   if (verbose)
-      fprintf(stderr, "dd: dumping to file %s\n", name);
-
    return f;
 }
 
 static inline void
 dd_parse_apitrace_marker(const char *string, int len, unsigned *call_number)
 {
    unsigned num;
    char *s;
 
    if (len <= 0)
-- 
2.11.0



More information about the mesa-dev mailing list