uml: style fixes in arch/um/os-Linux
authorJeff Dike <jdike@addtoit.com>
Fri, 8 Feb 2008 12:22:08 +0000 (04:22 -0800)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Fri, 8 Feb 2008 17:22:42 +0000 (09:22 -0800)
Style changes under arch/um/os-Linux:
include trimming
CodingStyle fixes
some printks needed severity indicators

make_tempfile turns out not to be used outside of mem.c, so it is now static.
Its declaration in tempfile.h is no longer needed, and tempfile.h itself is no
longer needed.

create_tmp_file was also made static.

checkpatch moans about an EXPORT_SYMBOL in user_syms.c which is part of a
macro definition - this is copying a bit of kernel infrastructure into the
libc side of UML because the kernel headers can't be included there.

Signed-off-by: Jeff Dike <jdike@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
14 files changed:
arch/um/include/tempfile.h [deleted file]
arch/um/os-Linux/aio.c
arch/um/os-Linux/drivers/ethertap_kern.c
arch/um/os-Linux/drivers/tuntap_kern.c
arch/um/os-Linux/include/file.h
arch/um/os-Linux/mem.c
arch/um/os-Linux/process.c
arch/um/os-Linux/signal.c
arch/um/os-Linux/skas/process.c
arch/um/os-Linux/sys-i386/registers.c
arch/um/os-Linux/sys-x86_64/registers.c
arch/um/os-Linux/uaccess.c
arch/um/os-Linux/user_syms.c
arch/um/os-Linux/util.c

diff --git a/arch/um/include/tempfile.h b/arch/um/include/tempfile.h
deleted file mode 100644 (file)
index d441eac..0000000
+++ /dev/null
@@ -1,11 +0,0 @@
-/* 
- * Copyright (C) 2000, 2001, 2002 Jeff Dike (jdike@karaya.com)
- * Licensed under the GPL
- */
-
-#ifndef __TEMPFILE_H__
-#define __TEMPFILE_H__
-
-extern int make_tempfile(const char *template, char **tempname, int do_unlink);
-
-#endif
index b8d8c9ca8d4afa48d3374520615fc5fdf8c2d8df..57e3d46c989cf0ba4ec4546d6e613e969537be3e 100644 (file)
@@ -142,7 +142,7 @@ static int do_not_aio(struct aio_thread_req *req)
        if (actual != req->offset)
                return -errno;
 
-       switch(req->type) {
+       switch (req->type) {
        case AIO_READ:
                n = read(req->io_fd, req->buf, req->len);
                break;
index 04f11b9f1ac09dbb6a882a7a1e0f270225bc13a7..046a131f61046260c293e12111f30a53266084fd 100644 (file)
@@ -6,7 +6,7 @@
  * Licensed under the GPL.
  */
 
-#include "linux/init.h"
+#include <linux/init.h>
 #include <linux/netdevice.h>
 #include "etap.h"
 #include "net_kern.h"
@@ -30,10 +30,10 @@ static void etap_init(struct net_device *dev, void *data)
        epri->control_fd = -1;
        epri->dev = dev;
 
-       printk("ethertap backend - %s", epri->dev_name);
+       printk(KERN_INFO "ethertap backend - %s", epri->dev_name);
        if (epri->gate_addr != NULL)
-               printk(", IP = %s", epri->gate_addr);
-       printk("\n");
+               printk(KERN_CONT ", IP = %s", epri->gate_addr);
+       printk(KERN_CONT "\n");
 }
 
 static int etap_read(int fd, struct sk_buff *skb, struct uml_net_private *lp)
index 9d384807b0775db5a5c31a7d8ea73c743562da35..6b9e33d5de20688d639ace827c57013e76b970ff 100644 (file)
@@ -29,10 +29,10 @@ static void tuntap_init(struct net_device *dev, void *data)
        tpri->fd = -1;
        tpri->dev = dev;
 
-       printk("TUN/TAP backend - ");
+       printk(KERN_INFO "TUN/TAP backend - ");
        if (tpri->gate_addr != NULL)
-               printk("IP = %s", tpri->gate_addr);
-       printk("\n");
+               printk(KERN_CONT "IP = %s", tpri->gate_addr);
+       printk(KERN_CONT "\n");
 }
 
 static int tuntap_read(int fd, struct sk_buff *skb, struct uml_net_private *lp)
index d82711efacfadf2a5a50e27fcd2af31a61c3e6a3..fe71be24bd59d68aaf8e62a080eb59250bbd646a 100644 (file)
@@ -1,5 +1,5 @@
 /* 
- * Copyright (C) 2002 Jeff Dike (jdike@karaya.com)
+ * Copyright (C) 2002 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
@@ -9,14 +9,3 @@
 #define DEV_NULL "/dev/null"
 
 #endif
-
-/*
- * Overrides for Emacs so that we follow Linus's tabbing style.
- * Emacs will notice this stuff at the end of the file and automatically
- * adjust the settings for this buffer only.  This must remain at the end
- * of the file.
- * ---------------------------------------------------------------------------
- * Local variables:
- * c-file-style: "linux"
- * End:
- */
index eedc2d88ef8a5a3d6b00e87e88a8a05e7c77ab89..38742c21def52499788ad6175674d1b6db0162fa 100644 (file)
@@ -1,22 +1,21 @@
+/*
+ * Copyright (C) 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
+ * Licensed under the GPL
+ */
+
 #include <stdio.h>
-#include <stdlib.h>
 #include <stddef.h>
-#include <stdarg.h>
+#include <stdlib.h>
 #include <unistd.h>
 #include <errno.h>
-#include <string.h>
 #include <fcntl.h>
-#include <sys/types.h>
+#include <string.h>
 #include <sys/mman.h>
-#include <sys/statfs.h>
-#include "user.h"
-#include "mem_user.h"
+#include <sys/param.h>
 #include "init.h"
-#include "os.h"
-#include "tempfile.h"
 #include "kern_constants.h"
-
-#include <sys/param.h>
+#include "os.h"
+#include "user.h"
 
 /* Modified by which_tmpdir, which is called during early boot */
 static char *default_tmpdir = "/tmp";
@@ -33,18 +32,19 @@ static void __init find_tempdir(void)
        int i;
        char *dir = NULL;
 
-       if(tempdir != NULL) /* We've already been called */
+       if (tempdir != NULL)
+               /* We've already been called */
                return;
-       for(i = 0; dirs[i]; i++){
+       for (i = 0; dirs[i]; i++) {
                dir = getenv(dirs[i]);
-               if((dir != NULL) && (*dir != '\0'))
+               if ((dir != NULL) && (*dir != '\0'))
                        break;
        }
-       if((dir == NULL) || (*dir == '\0'))
+       if ((dir == NULL) || (*dir == '\0'))
                dir = default_tmpdir;
 
        tempdir = malloc(strlen(dir) + 2);
-       if(tempdir == NULL){
+       if (tempdir == NULL) {
                fprintf(stderr, "Failed to malloc tempdir, "
                        "errno = %d\n", errno);
                return;
@@ -53,7 +53,8 @@ static void __init find_tempdir(void)
        strcat(tempdir, "/");
 }
 
-/* This will return 1, with the first character in buf being the
+/*
+ * This will return 1, with the first character in buf being the
  * character following the next instance of c in the file.  This will
  * read the file as needed.  If there's an error, -errno is returned;
  * if the end of the file is reached, 0 is returned.
@@ -64,11 +65,11 @@ static int next(int fd, char *buf, size_t size, char c)
        size_t len;
        char *ptr;
 
-       while((ptr = strchr(buf, c)) == NULL){
+       while ((ptr = strchr(buf, c)) == NULL) {
                n = read(fd, buf, size - 1);
-               if(n == 0)
+               if (n == 0)
                        return 0;
-               else if(n < 0)
+               else if (n < 0)
                        return -errno;
 
                buf[n] = '\0';
@@ -78,11 +79,12 @@ static int next(int fd, char *buf, size_t size, char c)
        len = strlen(ptr);
        memmove(buf, ptr, len + 1);
 
-       /* Refill the buffer so that if there's a partial string that we care
+       /*
+        * Refill the buffer so that if there's a partial string that we care
         * about, it will be completed, and we can recognize it.
         */
        n = read(fd, &buf[len], size - len - 1);
-       if(n < 0)
+       if (n < 0)
                return -errno;
 
        buf[len + n] = '\0';
@@ -92,7 +94,8 @@ static int next(int fd, char *buf, size_t size, char c)
 /* which_tmpdir is called only during early boot */
 static int checked_tmpdir = 0;
 
-/* Look for a tmpfs mounted at /dev/shm.  I couldn't find a cleaner
+/*
+ * Look for a tmpfs mounted at /dev/shm.  I couldn't find a cleaner
  * way to do this than to parse /proc/mounts.  statfs will return the
  * same filesystem magic number and fs id for both /dev and /dev/shm
  * when they are both tmpfs, so you can't tell if they are different
@@ -107,7 +110,7 @@ static void which_tmpdir(void)
        int fd, found;
        char buf[128] = { '\0' };
 
-       if(checked_tmpdir)
+       if (checked_tmpdir)
                return;
 
        checked_tmpdir = 1;
@@ -115,28 +118,28 @@ static void which_tmpdir(void)
        printf("Checking for tmpfs mount on /dev/shm...");
 
        fd = open("/proc/mounts", O_RDONLY);
-       if(fd < 0){
+       if (fd < 0) {
                printf("failed to open /proc/mounts, errno = %d\n", errno);
                return;
        }
 
-       while(1){
+       while (1) {
                found = next(fd, buf, ARRAY_SIZE(buf), ' ');
-               if(found != 1)
+               if (found != 1)
                        break;
 
-               if(!strncmp(buf, "/dev/shm", strlen("/dev/shm")))
+               if (!strncmp(buf, "/dev/shm", strlen("/dev/shm")))
                        goto found;
 
                found = next(fd, buf, ARRAY_SIZE(buf), '\n');
-               if(found != 1)
+               if (found != 1)
                        break;
        }
 
 err:
-       if(found == 0)
+       if (found == 0)
                printf("nothing mounted on /dev/shm\n");
-       else if(found < 0)
+       else if (found < 0)
                printf("read returned errno %d\n", -found);
 
 out:
@@ -146,10 +149,10 @@ out:
 
 found:
        found = next(fd, buf, ARRAY_SIZE(buf), ' ');
-       if(found != 1)
+       if (found != 1)
                goto err;
 
-       if(strncmp(buf, "tmpfs", strlen("tmpfs"))){
+       if (strncmp(buf, "tmpfs", strlen("tmpfs"))) {
                printf("not tmpfs\n");
                goto out;
        }
@@ -164,8 +167,8 @@ found:
  * (file: kernel/tt/ptproxy/proxy.c, proc: start_debugger).
  * So it isn't 'static' yet.
  */
-int __init make_tempfile(const char *template, char **out_tempname,
-                        int do_unlink)
+static int __init make_tempfile(const char *template, char **out_tempname,
+                               int do_unlink)
 {
        char *tempname;
        int fd;
@@ -182,16 +185,16 @@ int __init make_tempfile(const char *template, char **out_tempname,
                tempname[0] = '\0';
        strncat(tempname, template, MAXPATHLEN-1-strlen(tempname));
        fd = mkstemp(tempname);
-       if(fd < 0){
+       if (fd < 0) {
                fprintf(stderr, "open - cannot create %s: %s\n", tempname,
                        strerror(errno));
                goto out;
        }
-       if(do_unlink && (unlink(tempname) < 0)){
+       if (do_unlink && (unlink(tempname) < 0)) {
                perror("unlink");
                goto out;
        }
-       if(out_tempname){
+       if (out_tempname) {
                *out_tempname = tempname;
        } else {
                free(tempname);
@@ -204,27 +207,23 @@ out:
 
 #define TEMPNAME_TEMPLATE "vm_file-XXXXXX"
 
-/*
- * This proc is used in start_up.c
- * So it isn't 'static'.
- */
-int __init create_tmp_file(unsigned long long len)
+static int __init create_tmp_file(unsigned long long len)
 {
        int fd, err;
        char zero;
 
        fd = make_tempfile(TEMPNAME_TEMPLATE, NULL, 1);
-       if(fd < 0) {
+       if (fd < 0)
                exit(1);
-       }
 
        err = fchmod(fd, 0777);
-       if(err < 0){
+       if (err < 0) {
                perror("fchmod");
                exit(1);
        }
 
-       /* Seek to len - 1 because writing a character there will
+       /*
+        * Seek to len - 1 because writing a character there will
         * increase the file size by one byte, to the desired length.
         */
        if (lseek64(fd, len - 1, SEEK_SET) < 0) {
@@ -235,7 +234,7 @@ int __init create_tmp_file(unsigned long long len)
        zero = 0;
 
        err = write(fd, &zero, 1);
-       if(err != 1){
+       if (err != 1) {
                perror("write");
                exit(1);
        }
@@ -250,7 +249,7 @@ int __init create_mem_file(unsigned long long len)
        fd = create_tmp_file(len);
 
        err = os_set_exec_close(fd);
-       if(err < 0){
+       if (err < 0) {
                errno = -err;
                perror("exec_close");
        }
@@ -267,11 +266,11 @@ void __init check_tmpexec(void)
                    PROT_READ | PROT_WRITE | PROT_EXEC, MAP_PRIVATE, fd, 0);
        printf("Checking PROT_EXEC mmap in %s...",tempdir);
        fflush(stdout);
-       if(addr == MAP_FAILED){
+       if (addr == MAP_FAILED) {
                err = errno;
                perror("failed");
                close(fd);
-               if(err == EPERM)
+               if (err == EPERM)
                        printf("%s must be not mounted noexec\n",tempdir);
                exit(1);
        }
index abf6beae3df1bec14f60943754e54ba9e4d32f16..e0477c3ee89448f72320b01a493a97cb280ecad6 100644 (file)
@@ -224,7 +224,7 @@ int __init can_drop_memory(void)
                goto out_unmap;
        }
 
-       printk("OK\n");
+       printk(UM_KERN_CONT "OK\n");
        ok = 1;
 
 out_unmap:
index 0fb0cc8d47577c34d842a25a75f2e477dfc6d2c3..3f1694b134cb2606efa628deb4291c5813a549c0 100644 (file)
@@ -237,7 +237,7 @@ void unblock_signals(void)
         * interrupts may have arrived and we need to re-enable them and
         * recheck signals_pending.
         */
-       while(1) {
+       while (1) {
                /*
                 * Save and reset save_pending after enabling signals.  This
                 * way, signals_pending won't be changed while we're reading it.
index d36c89c24a45c5df05a652f88dc26901577bb19b..b14829469faefdf99a2511beede3ea6515c21927 100644 (file)
@@ -341,7 +341,7 @@ void userspace(struct uml_pt_regs *regs)
        int local_using_sysemu;
 
        if (getitimer(ITIMER_VIRTUAL, &timer))
-               printk("Failed to get itimer, errno = %d\n", errno);
+               printk(UM_KERN_ERR "Failed to get itimer, errno = %d\n", errno);
        nsecs = timer.it_value.tv_sec * UM_NSEC_PER_SEC +
                timer.it_value.tv_usec * UM_NSEC_PER_USEC;
        nsecs += os_nsecs();
@@ -388,7 +388,7 @@ void userspace(struct uml_pt_regs *regs)
 
                if (WIFSTOPPED(status)) {
                        int sig = WSTOPSIG(status);
-                       switch(sig) {
+                       switch (sig) {
                        case SIGSEGV:
                                if (PTRACE_FULL_FAULTINFO ||
                                    !ptrace_faultinfo) {
@@ -641,7 +641,7 @@ int start_idle_thread(void *stack, jmp_buf *switch_buf)
         * after returning to the jumper.
         */
        n = setjmp(initial_jmpbuf);
-       switch(n) {
+       switch (n) {
        case INIT_JMP_NEW_THREAD:
                (*switch_buf)[0].JB_IP = (unsigned long) new_thread_handler;
                (*switch_buf)[0].JB_SP = (unsigned long) stack +
index d1997ca76e5cbd182a6ce606b609516cfcdbbcc6..f74d853a0ee02372af46d4ab4ae244242a19ea4f 100644 (file)
@@ -62,10 +62,10 @@ void arch_init_registers(int pid)
        int err;
 
        err = ptrace(PTRACE_GETFPXREGS, pid, 0, fpx_regs);
-       if(!err)
+       if (!err)
                return;
 
-       if(errno != EIO)
+       if (errno != EIO)
                panic("check_ptrace : PTRACE_GETFPXREGS failed, errno = %d",
                      errno);
 
index 9bfa789992dec6ce7c23f532aa9de69bbfc143f3..a375853337a703d7764dded31549725cf17376e8 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2006-2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
+ * Copyright (C) 2006 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
@@ -7,31 +7,36 @@
 #include <sys/ptrace.h>
 #define __FRAME_OFFSETS
 #include <asm/ptrace.h>
+#include "kern_constants.h"
 #include "longjmp.h"
 #include "user.h"
 
 int save_fp_registers(int pid, unsigned long *fp_regs)
 {
-       if(ptrace(PTRACE_GETFPREGS, pid, 0, fp_regs) < 0)
+       if (ptrace(PTRACE_GETFPREGS, pid, 0, fp_regs) < 0)
                return -errno;
        return 0;
 }
 
 int restore_fp_registers(int pid, unsigned long *fp_regs)
 {
-       if(ptrace(PTRACE_SETFPREGS, pid, 0, fp_regs) < 0)
+       if (ptrace(PTRACE_SETFPREGS, pid, 0, fp_regs) < 0)
                return -errno;
        return 0;
 }
 
 unsigned long get_thread_reg(int reg, jmp_buf *buf)
 {
-       switch(reg){
-       case RIP: return buf[0]->__rip;
-       case RSP: return buf[0]->__rsp;
-       case RBP: return buf[0]->__rbp;
+       switch (reg) {
+       case RIP:
+               return buf[0]->__rip;
+       case RSP:
+               return buf[0]->__rsp;
+       case RBP:
+               return buf[0]->__rbp;
        default:
-               printk("get_thread_regs - unknown register %d\n", reg);
+               printk(UM_KERN_ERR "get_thread_regs - unknown register %d\n",
+                      reg);
                return 0;
        }
 }
index 8d27b6d1df91c58b69685fce86c2f4ae4ea9134e..087ed74ffca55081ab7052d51d74103f34d3c7f5 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Copyright (C) 2001 Chris Emerson (cemerson@chiark.greenend.org.uk)
- * Copyright (C) 2001, 2002 Jeff Dike (jdike@karaya.com)
+ * Copyright (C) 2001 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
@@ -16,7 +16,7 @@ unsigned long __do_user_copy(void *to, const void *from, int n,
 
        jmp_buf jbuf;
        *fault_catcher = &jbuf;
-       if(UML_SETJMP(&jbuf) == 0){
+       if (UML_SETJMP(&jbuf) == 0) {
                (*op)(to, from, n);
                ret = 0;
                *faulted_out = 0;
index 4c37b1b1d0b588d7a9c822a69d976a6a2ed26644..74f49bb9b125ed8728de851780b489895f2f4ac2 100644 (file)
@@ -34,8 +34,8 @@ EXPORT_SYMBOL(printf);
  * good; so the versions of these symbols will always match
  */
 #define EXPORT_SYMBOL_PROTO(sym)       \
-       int sym(void);                  \
-       EXPORT_SYMBOL(sym);
+       int sym(void);                  \
+       EXPORT_SYMBOL(sym);
 
 extern void readdir64(void) __attribute__((weak));
 EXPORT_SYMBOL(readdir64);
index a6f31d476993cd0314c8c6adbf9dad0c7c796f15..6ea77979531c95e1bf005eb0cc3e4a87fa6c913d 100644 (file)
@@ -1,39 +1,24 @@
 /*
- * Copyright (C) 2000, 2001, 2002 Jeff Dike (jdike@karaya.com)
+ * Copyright (C) 2000 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
 #include <stdio.h>
 #include <stdlib.h>
-#include <unistd.h>
-#include <limits.h>
-#include <sys/mman.h>
-#include <sys/stat.h>
-#include <sys/utsname.h>
-#include <sys/param.h>
-#include <sys/time.h>
-#include "asm/types.h"
-#include <ctype.h>
-#include <signal.h>
-#include <wait.h>
 #include <errno.h>
-#include <stdarg.h>
-#include <sched.h>
-#include <termios.h>
+#include <signal.h>
 #include <string.h>
-#include "kern_util.h"
-#include "user.h"
-#include "mem_user.h"
-#include "init.h"
-#include "ptrace_user.h"
-#include "uml-config.h"
-#include "os.h"
-#include "longjmp.h"
+#include <termios.h>
+#include <wait.h>
+#include <sys/mman.h>
+#include <sys/utsname.h>
 #include "kern_constants.h"
+#include "os.h"
+#include "user.h"
 
 void stack_protections(unsigned long address)
 {
-       if(mprotect((void *) address, UM_THREAD_SIZE,
+       if (mprotect((void *) address, UM_THREAD_SIZE,
                    PROT_READ | PROT_WRITE | PROT_EXEC) < 0)
                panic("protecting stack failed, errno = %d", errno);
 }
@@ -44,17 +29,19 @@ int raw(int fd)
        int err;
 
        CATCH_EINTR(err = tcgetattr(fd, &tt));
-       if(err < 0)
+       if (err < 0)
                return -errno;
 
        cfmakeraw(&tt);
 
        CATCH_EINTR(err = tcsetattr(fd, TCSADRAIN, &tt));
-       if(err < 0)
+       if (err < 0)
                return -errno;
 
-       /* XXX tcsetattr could have applied only some changes
-        * (and cfmakeraw() is a set of changes) */
+       /*
+        * XXX tcsetattr could have applied only some changes
+        * (and cfmakeraw() is a set of changes)
+        */
        return 0;
 }