recordmcount: Kernel style function signature formatting
[linux-2.6-block.git] / scripts / recordmcount.c
index 8387a9bc064a59f35211420ab28afb5aeec750ef..67f9c45b824f2bde7def2d5bacef99a2011e1571 100644 (file)
@@ -27,7 +27,6 @@
 #include <getopt.h>
 #include <elf.h>
 #include <fcntl.h>
-#include <setjmp.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -43,7 +42,6 @@ static int fd_map;    /* File descriptor for file being modified. */
 static int mmap_failed; /* Boolean flag. */
 static char gpfx;      /* prefix for global symbol name (sometimes '_') */
 static struct stat sb; /* Remember .st_size, etc. */
-static jmp_buf jmpenv; /* setjmp/longjmp per-file error escape */
 static const char *altmcount;  /* alternate mcount symbol name */
 static int warn_on_notrace_sect; /* warn when section has mcount not being recorded */
 static void *file_map; /* pointer of the mapped file */
@@ -53,16 +51,8 @@ static void *file_ptr;       /* current file pointer location */
 static void *file_append; /* added to the end of the file */
 static size_t file_append_size; /* how much is added to end of file */
 
-/* setjmp() return values */
-enum {
-       SJ_SETJMP = 0,  /* hardwired first return */
-       SJ_FAIL,
-       SJ_SUCCEED
-};
-
 /* Per-file resource cleanup when multiple files. */
-static void
-cleanup(void)
+static void cleanup(void)
 {
        if (!mmap_failed)
                munmap(file_map, sb.st_size);
@@ -75,24 +65,9 @@ cleanup(void)
        file_updated = 0;
 }
 
-static void __attribute__((noreturn))
-fail_file(void)
-{
-       cleanup();
-       longjmp(jmpenv, SJ_FAIL);
-}
-
-static void __attribute__((noreturn))
-succeed_file(void)
-{
-       cleanup();
-       longjmp(jmpenv, SJ_SUCCEED);
-}
+/* ulseek, uwrite, ...:  Check return value for errors. */
 
-/* ulseek, uread, ...:  Check return value for errors. */
-
-static off_t
-ulseek(int const fd, off_t const offset, int const whence)
+static off_t ulseek(off_t const offset, int const whence)
 {
        switch (whence) {
        case SEEK_SET:
@@ -107,24 +82,12 @@ ulseek(int const fd, off_t const offset, int const whence)
        }
        if (file_ptr < file_map) {
                fprintf(stderr, "lseek: seek before file\n");
-               fail_file();
+               return -1;
        }
        return file_ptr - file_map;
 }
 
-static size_t
-uread(int const fd, void *const buf, size_t const count)
-{
-       size_t const n = read(fd, buf, count);
-       if (n != count) {
-               perror("read");
-               fail_file();
-       }
-       return n;
-}
-
-static size_t
-uwrite(int const fd, void const *const buf, size_t const count)
+static ssize_t uwrite(void const *const buf, size_t const count)
 {
        size_t cnt = count;
        off_t idx = 0;
@@ -140,7 +103,8 @@ uwrite(int const fd, void const *const buf, size_t const count)
                }
                if (!file_append) {
                        perror("write");
-                       fail_file();
+                       cleanup();
+                       return -1;
                }
                if (file_ptr < file_end) {
                        cnt = file_end - file_ptr;
@@ -160,13 +124,13 @@ uwrite(int const fd, void const *const buf, size_t const count)
        return count;
 }
 
-static void *
-umalloc(size_t size)
+static void * umalloc(size_t size)
 {
        void *const addr = malloc(size);
        if (addr == 0) {
                fprintf(stderr, "malloc failed: %zu bytes\n", size);
-               fail_file();
+               cleanup();
+               return NULL;
        }
        return addr;
 }
@@ -194,8 +158,10 @@ static int make_nop_x86(void *map, size_t const offset)
                return -1;
 
        /* convert to nop */
-       ulseek(fd_map, offset - 1, SEEK_SET);
-       uwrite(fd_map, ideal_nop, 5);
+       if (ulseek(offset - 1, SEEK_SET) < 0)
+               return -1;
+       if (uwrite(ideal_nop, 5) < 0)
+               return -1;
        return 0;
 }
 
@@ -243,10 +209,12 @@ static int make_nop_arm(void *map, size_t const offset)
                return -1;
 
        /* Convert to nop */
-       ulseek(fd_map, off, SEEK_SET);
+       if (ulseek(off, SEEK_SET) < 0)
+               return -1;
 
        do {
-               uwrite(fd_map, ideal_nop, nop_size);
+               if (uwrite(ideal_nop, nop_size) < 0)
+                       return -1;
        } while (--cnt > 0);
 
        return 0;
@@ -263,8 +231,10 @@ static int make_nop_arm64(void *map, size_t const offset)
                return -1;
 
        /* Convert to nop */
-       ulseek(fd_map, offset, SEEK_SET);
-       uwrite(fd_map, ideal_nop, 4);
+       if (ulseek(offset, SEEK_SET) < 0)
+               return -1;
+       if (uwrite(ideal_nop, 4) < 0)
+               return -1;
        return 0;
 }
 
@@ -283,14 +253,23 @@ static int make_nop_arm64(void *map, size_t const offset)
  */
 static void *mmap_file(char const *fname)
 {
+       file_map = NULL;
+       sb.st_size = 0;
        fd_map = open(fname, O_RDONLY);
-       if (fd_map < 0 || fstat(fd_map, &sb) < 0) {
+       if (fd_map < 0) {
                perror(fname);
-               fail_file();
+               cleanup();
+               return NULL;
+       }
+       if (fstat(fd_map, &sb) < 0) {
+               perror(fname);
+               cleanup();
+               goto out;
        }
        if (!S_ISREG(sb.st_mode)) {
                fprintf(stderr, "not a regular file: %s\n", fname);
-               fail_file();
+               cleanup();
+               goto out;
        }
        file_map = mmap(0, sb.st_size, PROT_READ|PROT_WRITE, MAP_PRIVATE,
                        fd_map, 0);
@@ -298,8 +277,18 @@ static void *mmap_file(char const *fname)
        if (file_map == MAP_FAILED) {
                mmap_failed = 1;
                file_map = umalloc(sb.st_size);
-               uread(fd_map, file_map, sb.st_size);
+               if (!file_map) {
+                       perror(fname);
+                       goto out;
+               }
+               if (read(fd_map, file_map, sb.st_size) != sb.st_size) {
+                       perror(fname);
+                       free(file_map);
+                       file_map = NULL;
+                       goto out;
+               }
        }
+out:
        close(fd_map);
 
        file_end = file_map + sb.st_size;
@@ -307,13 +296,13 @@ static void *mmap_file(char const *fname)
        return file_map;
 }
 
-static void write_file(const char *fname)
+static int write_file(const char *fname)
 {
        char tmp_file[strlen(fname) + 4];
        size_t n;
 
        if (!file_updated)
-               return;
+               return 0;
 
        sprintf(tmp_file, "%s.rc", fname);
 
@@ -325,25 +314,32 @@ static void write_file(const char *fname)
        fd_map = open(tmp_file, O_WRONLY | O_TRUNC | O_CREAT, sb.st_mode);
        if (fd_map < 0) {
                perror(fname);
-               fail_file();
+               cleanup();
+               return -1;
        }
        n = write(fd_map, file_map, sb.st_size);
        if (n != sb.st_size) {
                perror("write");
-               fail_file();
+               cleanup();
+               close(fd_map);
+               return -1;
        }
        if (file_append_size) {
                n = write(fd_map, file_append, file_append_size);
                if (n != file_append_size) {
                        perror("write");
-                       fail_file();
+                       cleanup();
+                       close(fd_map);
+                       return -1;
                }
        }
        close(fd_map);
        if (rename(tmp_file, fname) < 0) {
                perror(fname);
-               fail_file();
+               cleanup();
+               return -1;
        }
+       return 0;
 }
 
 /* w8rev, w8nat, ...: Handle endianness. */
@@ -394,8 +390,7 @@ static uint32_t (*w)(uint32_t);
 static uint32_t (*w2)(uint16_t);
 
 /* Names of the sections that could contain calls to mcount. */
-static int
-is_mcounted_section_name(char const *const txtname)
+static int is_mcounted_section_name(char const *const txtname)
 {
        return strncmp(".text",          txtname, 5) == 0 ||
                strcmp(".init.text",     txtname) == 0 ||
@@ -405,10 +400,11 @@ is_mcounted_section_name(char const *const txtname)
                strcmp(".irqentry.text", txtname) == 0 ||
                strcmp(".softirqentry.text", txtname) == 0 ||
                strcmp(".kprobes.text", txtname) == 0 ||
-               strcmp(".cpuidle.text", txtname) == 0 ||
-               strcmp(".text.unlikely", txtname) == 0;
+               strcmp(".cpuidle.text", txtname) == 0;
 }
 
+static char const *already_has_rel_mcount = "success"; /* our work here is done! */
+
 /* 32 bit and 64 bit are very similar */
 #include "recordmcount.h"
 #define RECORD_MCOUNT_64
@@ -447,11 +443,14 @@ static void MIPS64_r_info(Elf64_Rel *const rp, unsigned sym, unsigned type)
        }).r_info;
 }
 
-static void
-do_file(char const *const fname)
+static int do_file(char const *const fname)
 {
        Elf32_Ehdr *const ehdr = mmap_file(fname);
        unsigned int reltype = 0;
+       int rc = -1;
+
+       if (!ehdr)
+               goto out;
 
        w = w4nat;
        w2 = w2nat;
@@ -461,8 +460,8 @@ do_file(char const *const fname)
        default:
                fprintf(stderr, "unrecognized ELF data encoding %d: %s\n",
                        ehdr->e_ident[EI_DATA], fname);
-               fail_file();
-               break;
+               cleanup();
+               goto out;
        case ELFDATA2LSB:
                if (*(unsigned char const *)&endian != 1) {
                        /* main() is big endian, file.o is little endian. */
@@ -494,7 +493,8 @@ do_file(char const *const fname)
        ||  w2(ehdr->e_type) != ET_REL
        ||  ehdr->e_ident[EI_VERSION] != EV_CURRENT) {
                fprintf(stderr, "unrecognized ET_REL file %s\n", fname);
-               fail_file();
+               cleanup();
+               goto out;
        }
 
        gpfx = 0;
@@ -502,8 +502,8 @@ do_file(char const *const fname)
        default:
                fprintf(stderr, "unrecognized e_machine %u %s\n",
                        w2(ehdr->e_machine), fname);
-               fail_file();
-               break;
+               cleanup();
+               goto out;
        case EM_386:
                reltype = R_386_32;
                rel_type_nop = R_386_NONE;
@@ -543,20 +543,22 @@ do_file(char const *const fname)
        default:
                fprintf(stderr, "unrecognized ELF class %d %s\n",
                        ehdr->e_ident[EI_CLASS], fname);
-               fail_file();
-               break;
+               cleanup();
+               goto out;
        case ELFCLASS32:
                if (w2(ehdr->e_ehsize) != sizeof(Elf32_Ehdr)
                ||  w2(ehdr->e_shentsize) != sizeof(Elf32_Shdr)) {
                        fprintf(stderr,
                                "unrecognized ET_REL file: %s\n", fname);
-                       fail_file();
+                       cleanup();
+                       goto out;
                }
                if (w2(ehdr->e_machine) == EM_MIPS) {
                        reltype = R_MIPS_32;
                        is_fake_mcount32 = MIPS32_is_fake_mcount;
                }
-               do32(ehdr, fname, reltype);
+               if (do32(ehdr, fname, reltype) < 0)
+                       goto out;
                break;
        case ELFCLASS64: {
                Elf64_Ehdr *const ghdr = (Elf64_Ehdr *)ehdr;
@@ -564,7 +566,8 @@ do_file(char const *const fname)
                ||  w2(ghdr->e_shentsize) != sizeof(Elf64_Shdr)) {
                        fprintf(stderr,
                                "unrecognized ET_REL file: %s\n", fname);
-                       fail_file();
+                       cleanup();
+                       goto out;
                }
                if (w2(ghdr->e_machine) == EM_S390) {
                        reltype = R_390_64;
@@ -576,17 +579,19 @@ do_file(char const *const fname)
                        Elf64_r_info = MIPS64_r_info;
                        is_fake_mcount64 = MIPS64_is_fake_mcount;
                }
-               do64(ghdr, fname, reltype);
+               if (do64(ghdr, fname, reltype) < 0)
+                       goto out;
                break;
        }
        }  /* end switch */
 
-       write_file(fname);
+       rc = write_file(fname);
+out:
        cleanup();
+       return rc;
 }
 
-int
-main(int argc, char *argv[])
+int main(int argc, char *argv[])
 {
        const char ftrace[] = "/ftrace.o";
        int ftrace_size = sizeof(ftrace) - 1;
@@ -613,7 +618,6 @@ main(int argc, char *argv[])
        /* Process each file in turn, allowing deep failure. */
        for (i = optind; i < argc; i++) {
                char *file = argv[i];
-               int const sjval = setjmp(jmpenv);
                int len;
 
                /*
@@ -626,28 +630,16 @@ main(int argc, char *argv[])
                    strcmp(file + (len - ftrace_size), ftrace) == 0)
                        continue;
 
-               switch (sjval) {
-               default:
-                       fprintf(stderr, "internal error: %s\n", file);
-                       exit(1);
-                       break;
-               case SJ_SETJMP:    /* normal sequence */
-                       /* Avoid problems if early cleanup() */
-                       fd_map = -1;
-                       mmap_failed = 1;
-                       file_map = NULL;
-                       file_ptr = NULL;
-                       file_updated = 0;
-                       do_file(file);
-                       break;
-               case SJ_FAIL:    /* error in do_file or below */
+               /* Avoid problems if early cleanup() */
+               fd_map = -1;
+               mmap_failed = 1;
+               file_map = NULL;
+               file_ptr = NULL;
+               file_updated = 0;
+               if (do_file(file)) {
                        fprintf(stderr, "%s: failed\n", file);
                        ++n_error;
-                       break;
-               case SJ_SUCCEED:    /* premature success */
-                       /* do nothing */
-                       break;
-               }  /* end switch */
+               }
        }
        return !!n_error;
 }