Changes in / [f27ada7:25e963a] in mainline

Files:

• HelenOS.config (modified) (1 diff)
• uspace/app/taskdump/Makefile (modified) (1 diff)
• uspace/app/taskdump/elf_core.c (deleted)
• uspace/app/taskdump/include/elf.h (modified) (1 diff)
• uspace/app/taskdump/include/elf_core.h (deleted)
• uspace/app/taskdump/symtab.c (modified) (3 diffs)
• uspace/app/taskdump/taskdump.c (modified) (7 diffs)
• uspace/srv/taskmon/taskmon.c (modified) (3 diffs)

HelenOS.config

@@ -505 +505 @@
 ! [CONFIG_START_BD=y] CONFIG_MOUNT_DATA (n/y)
 
-% Write core files
-! CONFIG_WRITE_CORE_FILES (n/y)
+% Verbose task dumps
+! CONFIG_VERBOSE_DUMPS (n/y)

uspace/app/taskdump/Makefile

@@ -34 +34 @@
 
 SOURCES = \
-        elf_core.c \
         taskdump.c \
         symtab.c

uspace/app/taskdump/include/elf.h

@@ -148 +148 @@
 #define SHF_MASKPROC            0xf0000000
 
-/** Macros for decomposing elf_symbol.st_info into binging and type */
-#define ELF_ST_BIND(i)          ((i) >> 4)
-#define ELF_ST_TYPE(i)          ((i) & 0x0f)
-#define ELF_ST_INFO(b, t)       (((b) << 4) + ((t) & 0x0f))
-
 /**
  * Symbol binding

uspace/app/taskdump/symtab.c

@@ -206 +206 @@
         size_t i;
         char *sname;
-        unsigned stype;
 
         for (i = 0; i < st->sym_size / sizeof(elf_symbol_t); ++i) {
                 if (st->sym[i].st_name == 0)
-                        continue;
-
-                stype = ELF_ST_TYPE(st->sym[i].st_info);
-                if (stype != STT_OBJECT && stype != STT_FUNC)
                         continue;
 
@@ -245 +240 @@
         uintptr_t saddr, best_addr;
         char *sname, *best_name;
-        unsigned stype;
 
         best_name = NULL;
@@ -252 +246 @@
         for (i = 0; i < st->sym_size / sizeof(elf_symbol_t); ++i) {
                 if (st->sym[i].st_name == 0)
-                        continue;
-
-                stype = ELF_ST_TYPE(st->sym[i].st_info);
-                if (stype != STT_OBJECT && stype != STT_FUNC)
                         continue;
 

uspace/app/taskdump/taskdump.c

@@ -40 +40 @@
 #include <udebug.h>
 #include <task.h>
-#include <as.h>
+#include <kernel/mm/as.h>
 #include <sys/types.h>
 #include <sys/typefmt.h>
@@ -49 +49 @@
 
 #include <symtab.h>
-#include <elf_core.h>
 #include <stacktrace.h>
 
@@ -59 +58 @@
 static int phoneid;
 static task_id_t task_id;
-static bool write_core_file;
-static char *core_file_name;
+static bool dump_memory;
 static char *app_name;
 static symtab_t *app_symtab;
@@ -82 +80 @@
         int rc;
 
+        /*
+         * FIXME: The stdio module cannot currently detect whether we are
+         * writing to a console or file. This workaround make file output
+         * faster.
+         */
+        setvbuf(stdout, NULL, _IOFBF, 32768);
+
         printf("Task Dump Utility\n");
-        write_core_file = false;
+        dump_memory = false;
 
         if (parse_args(argc, argv) < 0)
@@ -167 +172 @@
                                         return -1;
                                 }
-                        } else if (arg[1] == 'c' && arg[2] == '\0') {
-                                write_core_file = true;
-
-                                --argc; ++argv;
-                                core_file_name = *argv;
+                        } else if (arg[1] == 'm' && arg[2] == '\0') {
+                                dump_memory = true;
                         } else {
                                 printf("Uknown option '%s'\n", arg[0]);
@@ -201 +203 @@
 static void print_syntax(void)
 {
-        printf("Syntax: taskdump [-c <core_file>] -t <task_id>\n");
-        printf("\t-c <core_file_id>\tName of core file to write.\n");
+        printf("Syntax: taskdump [-m] -t <task_id>\n");
+        printf("\t-m\tDump memory area contents.\n");
         printf("\t-t <task_id>\tWhich task to dump.\n");
 }
@@ -295 +297 @@
                     (ainfo_buf[i].flags & AS_AREA_CACHEABLE) ? 'C' : '-',
                     ainfo_buf[i].start_addr, ainfo_buf[i].size);
+
+                if (dump_memory) {
+                        putchar('\n');
+                        area_dump(&ainfo_buf[i]);
+                        putchar('\n');
+                }
         }
 
         putchar('\n');
-
-        if (write_core_file) {
-                printf("Writing core file '%s'\n", core_file_name);
-                rc = elf_core_save(core_file_name, ainfo_buf, n_areas, phoneid);
-                if (rc != EOK) {
-                        printf("Failed writing core file.\n");
-                        return EIO;
-                }
-        }
 
         free(ainfo_buf);

uspace/srv/taskmon/taskmon.c

@@ -49 +49 @@
 static void fault_event(ipc_callid_t callid, ipc_call_t *call)
 {
-        char *argv[6];
+        char *argv[11];
         char *fname;
         char *dump_fname;
@@ -66 +66 @@
         }
 
-        if (asprintf(&dump_fname, "/data/core" PRIuTASKID, taskid) < 0) {
+        if (asprintf(&dump_fname, "/scratch/d" PRIuTASKID ".txt", taskid) < 0) {
                 printf("Memory allocation failed.\n");
                 return;
@@ -73 +73 @@
         printf(NAME ": Task %" PRIuTASKID " fault in thread %p.\n", taskid, thread);
 
-#ifdef CONFIG_WRITE_CORE_FILES
-        argv[0] = "/app/taskdump";
-        argv[1] = "-c";
-        argv[2] = dump_fname;
-        argv[3] = "-t";
-        argv[4] = s_taskid;
-        argv[5] = NULL;
+#ifdef CONFIG_VERBOSE_DUMPS
+        argv[0] = "/app/redir";
+        argv[1] = "-i";
+        argv[2] = "/readme";
+        argv[3] = "-o";
+        argv[4] = dump_fname;
+        argv[5] = "--";
+        argv[6] = "/app/taskdump";
+        argv[7] = "-m";
+        argv[8] = "-t";
+        argv[9] = s_taskid;
+        argv[10] = NULL;
 #else
         argv[0] = "/app/taskdump";