Changeset 58898d1d in mainline for uspace/app/taskdump

Timestamp:

2017-03-24T20:31:54Z (9 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

8e9b2534

Parents:

c9e3692

Message:

Remove VFS_IN_SEEK from VFS

Location:

uspace/app/taskdump

Files:

• elf_core.c (modified) (13 diffs)
• symtab.c (modified) (5 diffs)

uspace/app/taskdump/elf_core.c

@@ -67 +67 @@
 
 static off64_t align_foff_up(off64_t, uintptr_t, size_t);
-static int align_pos(int, size_t);
-static int write_mem_area(int, as_area_info_t *, async_sess_t *);
+static int write_mem_area(int, aoff64_t *, as_area_info_t *, async_sess_t *);
 
 #define BUFFER_SIZE 0x1000
@@ -97 +96 @@
         elf_note_t note;
         size_t word_size;
+        aoff64_t pos = 0;
 
         int fd;
@@ -207 +207 @@
         }
 
-        rc = write(fd, &elf_hdr, sizeof(elf_hdr));
+        rc = write(fd, &pos, &elf_hdr, sizeof(elf_hdr));
         if (rc != sizeof(elf_hdr)) {
                 printf("Failed writing ELF header.\n");
@@ -215 +215 @@
 
         for (i = 0; i < n_ph; ++i) {
-                rc = write(fd, &p_hdr[i], sizeof(p_hdr[i]));
+                rc = write(fd, &pos, &p_hdr[i], sizeof(p_hdr[i]));
                 if (rc != sizeof(p_hdr[i])) {
                         printf("Failed writing program header.\n");
@@ -223 +223 @@
         }
 
-        if (lseek(fd, p_hdr[0].p_offset, SEEK_SET) == (off64_t) -1) {
-                printf("Failed writing memory data.\n");
-                free(p_hdr);
-                return EIO;
-        }
+        pos = p_hdr[0].p_offset;
 
         /*
@@ -236 +232 @@
         note.type = NT_PRSTATUS;
 
-        rc = write(fd, &note, sizeof(elf_note_t));
+        rc = write(fd, &pos, &note, sizeof(elf_note_t));
         if (rc != sizeof(elf_note_t)) {
                 printf("Failed writing note header.\n");
@@ -243 +239 @@
         }
 
-        rc = write(fd, "CORE", note.namesz);
+        rc = write(fd, &pos, "CORE", note.namesz);
         if (rc != (ssize_t) note.namesz) {
                 printf("Failed writing note header.\n");
@@ -250 +246 @@
         }
 
-        rc = align_pos(fd, word_size);
-        if (rc != EOK) {
-                printf("Failed writing note header.\n");
-                free(p_hdr);
-                return EIO;
-        }
-
-        rc = write(fd, &pr_status, sizeof(elf_prstatus_t));
+        pos = ALIGN_UP(pos, word_size);
+
+        rc = write(fd, &pos, &pr_status, sizeof(elf_prstatus_t));
         if (rc != sizeof(elf_prstatus_t)) {
                 printf("Failed writing register data.\n");
@@ -265 +256 @@
 
         for (i = 1; i < n_ph; ++i) {
-                if (lseek(fd, p_hdr[i].p_offset, SEEK_SET) == (off64_t) -1) {
-                        printf("Failed writing memory data.\n");
-                        free(p_hdr);
-                        return EIO;
-                }
-                if (write_mem_area(fd, &ainfo[i - 1], sess) != EOK) {
+                pos = p_hdr[i].p_offset;
+                if (write_mem_area(fd, &pos, &ainfo[i - 1], sess) != EOK) {
                         printf("Failed writing memory data.\n");
                         free(p_hdr);
@@ -297 +284 @@
  *
  * @param fd   File to write to.
+ * @param pos  Pointer to the position to write to.
  * @param area Memory area info structure.
  * @param sess Debugging session.
@@ -303 +291 @@
  *
  */
-static int write_mem_area(int fd, as_area_info_t *area, async_sess_t *sess)
+static int write_mem_area(int fd, aoff64_t *pos, as_area_info_t *area,
+    async_sess_t *sess)
 {
         size_t to_copy;
@@ -321 +310 @@
                 }
 
-                rc = write(fd, buffer, to_copy);
+                rc = write(fd, pos, buffer, to_copy);
                 if (rc != (ssize_t) to_copy) {
                         printf("Failed writing memory contents.\n");
@@ -334 +323 @@
 }
 
-static int align_pos(int fd, size_t align)
-{
-        off64_t cur_pos;
-        size_t rem, adv;
-
-        cur_pos = lseek(fd, 0, SEEK_CUR);
-        if (cur_pos < 0)
-                return -1;
-
-        rem = cur_pos % align;
-        adv = align - rem;
-
-        cur_pos = lseek(fd, adv, SEEK_CUR);
-        if (cur_pos < 0)
-                return -1;
-
-        return EOK;
-}
-
 /** @}
  */

uspace/app/taskdump/symtab.c

@@ -68 +68 @@
         char *shstrt, *sec_name;
         void *data;
+        aoff64_t pos = 0;
 
         int fd;
@@ -88 +89 @@
         }
 
-        rc = read(fd, &elf_hdr, sizeof(elf_header_t));
+        rc = read(fd, &pos, &elf_hdr, sizeof(elf_header_t));
         if (rc != sizeof(elf_header_t)) {
                 printf("failed reading elf header\n");
@@ -304 +305 @@
 {
         int rc;
-
-        rc = lseek(fd, elf_hdr->e_shoff + idx * sizeof(elf_section_header_t),
-            SEEK_SET);
-        if (rc == (off64_t) -1)
-                return EIO;
-
-        rc = read(fd, sec_hdr, sizeof(elf_section_header_t));
+        aoff64_t pos = elf_hdr->e_shoff + idx * sizeof(elf_section_header_t);
+
+        rc = read(fd, &pos, sec_hdr, sizeof(elf_section_header_t));
         if (rc != sizeof(elf_section_header_t))
                 return EIO;
@@ -331 +328 @@
 {
         ssize_t rc;
-        off64_t offs;
-
-        offs = lseek(fd, start, SEEK_SET);
-        if (offs == (off64_t) -1) {
-                printf("failed seeking chunk\n");
-                *ptr = NULL;
-                return EIO;
-        }
+        aoff64_t pos = start;
 
         *ptr = malloc(size);
@@ -346 +336 @@
         }
 
-        rc = read(fd, *ptr, size);
+        rc = read(fd, &pos, *ptr, size);
         if (rc != (ssize_t) size) {
                 printf("failed reading chunk\n");