uml: xterm driver tidying

Major tidying of the xterm console driver:
	got rid of the tt-mode gdb support
	tidied up the includes
	fixed lots of style violations
	replaced os_* calls with glibc calls in xterm.c
	all printk calls now have a severity indicator
	the error paths of xterm_open are closer to being right

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>

1 files changed, 88 insertions, 85 deletions

diff --git a/arch/um/drivers/xterm.c b/arch/um/drivers/xterm.c
index fe238e03a300..35912846ed21 100644
--- a/arch/um/drivers/xterm.c
+++ b/arch/um/drivers/xterm.c
@@ -1,22 +1,20 @@
 /* 
- * Copyright (C) 2001, 2002 Jeff Dike (jdike@karaya.com)
+ * Copyright (C) 2001 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
-#include <stdio.h>
 #include <stdlib.h>
+#include <stdio.h>
 #include <unistd.h>
 #include <string.h>
 #include <errno.h>
 #include <termios.h>
-#include <signal.h>
-#include <sched.h>
-#include <sys/socket.h>
-#include "kern_util.h"
 #include "chan_user.h"
-#include "user.h"
 #include "os.h"
+#include "init.h"
+#include "user.h"
 #include "xterm.h"
+#include "kern_constants.h"
 
 struct xterm_chan {
         int pid;
@@ -25,25 +23,21 @@ struct xterm_chan {
         int device;
         int raw;
         struct termios tt;
-        unsigned long stack;
-        int direct_rcv;
 };
 
-/* Not static because it's called directly by the tt mode gdb code */
-void *xterm_init(char *str, int device, const struct chan_opts *opts)
+static void *xterm_init(char *str, int device, const struct chan_opts *opts)
 {
         struct xterm_chan *data;
 
         data = malloc(sizeof(*data));
-        if(data == NULL) return(NULL);
-        *data = ((struct xterm_chan) { .pid             = -1, 
+        if (data == NULL)
+                return NULL;
+        *data = ((struct xterm_chan) { .pid             = -1,
                                        .helper_pid      = -1,
-                                       .device          = device, 
+                                       .device          = device,
                                        .title           = opts->xterm_title,
-                                       .raw             = opts->raw,
-                                       .stack           = opts->tramp_stack,
-                                       .direct_rcv      = !opts->in_kernel } );
-        return(data);
+                                       .raw             = opts->raw } );
+        return data;
 }
 
 /* Only changed by xterm_setup, which is a setup */
@@ -57,16 +51,22 @@ static int __init xterm_setup(char *line, int *add)
         terminal_emulator = line;
 
         line = strchr(line, ',');
-        if(line == NULL) return(0);
+        if (line == NULL)
+                return 0;
+
         *line++ = '\0';
-        if(*line) title_switch = line;
+        if (*line)
+                title_switch = line;
 
         line = strchr(line, ',');
-        if(line == NULL) return(0);
+        if (line == NULL)
+                return 0;
+
         *line++ = '\0';
-        if(*line) exec_switch = line;
+        if (*line)
+                exec_switch = line;
 
-        return(0);
+        return 0;
 }
 
 __uml_setup("xterm=", xterm_setup,
@@ -82,114 +82,128 @@ __uml_setup("xterm=", xterm_setup,
 "    are 'xterm=gnome-terminal,-t,-x'.\n\n"
 );
 
-/* XXX This badly needs some cleaning up in the error paths
- * Not static because it's called directly by the tt mode gdb code
- */
-int xterm_open(int input, int output, int primary, void *d,
+static int xterm_open(int input, int output, int primary, void *d,
                       char **dev_out)
 {
         struct xterm_chan *data = d;
-        unsigned long stack;
         int pid, fd, new, err;
         char title[256], file[] = "/tmp/xterm-pipeXXXXXX";
-        char *argv[] = { terminal_emulator, title_switch, title, exec_switch, 
+        char *argv[] = { terminal_emulator, title_switch, title, exec_switch,
                          "/usr/lib/uml/port-helper", "-uml-socket",
                          file, NULL };
 
-        if(os_access(argv[4], OS_ACC_X_OK) < 0)
+        if (access(argv[4], X_OK) < 0)
                 argv[4] = "port-helper";
 
         /* Check that DISPLAY is set, this doesn't guarantee the xterm
          * will work but w/o it we can be pretty sure it won't. */
-        if (!getenv("DISPLAY")) {
-                printk("xterm_open: $DISPLAY not set.\n");
+        if (getenv("DISPLAY") == NULL) {
+                printk(UM_KERN_ERR "xterm_open: $DISPLAY not set.\n");
                 return -ENODEV;
         }
 
+        /*
+         * This business of getting a descriptor to a temp file,
+         * deleting the file and closing the descriptor is just to get
+         * a known-unused name for the Unix socket that we really
+         * want.
+         */
         fd = mkstemp(file);
-        if(fd < 0){
+        if (fd < 0) {
                 err = -errno;
-                printk("xterm_open : mkstemp failed, errno = %d\n", errno);
+                printk(UM_KERN_ERR "xterm_open : mkstemp failed, errno = %d\n",
+                       errno);
                 return err;
         }
 
-        if(unlink(file)){
+        if (unlink(file)) {
                 err = -errno;
-                printk("xterm_open : unlink failed, errno = %d\n", errno);
+                printk(UM_KERN_ERR "xterm_open : unlink failed, errno = %d\n",
+                       errno);
                 return err;
         }
-        os_close_file(fd);
+        close(fd);
 
         fd = os_create_unix_socket(file, sizeof(file), 1);
-        if(fd < 0){
-                printk("xterm_open : create_unix_socket failed, errno = %d\n", 
-                       -fd);
-                return(fd);
+        if (fd < 0) {
+                printk(UM_KERN_ERR "xterm_open : create_unix_socket failed, "
+                       "errno = %d\n", -fd);
+                return fd;
         }
 
         sprintf(title, data->title, data->device);
-        stack = data->stack;
-        pid = run_helper(NULL, NULL, argv, &stack);
-        if(pid < 0){
-                printk("xterm_open : run_helper failed, errno = %d\n", -pid);
-                return(pid);
+        pid = run_helper(NULL, NULL, argv, NULL);
+        if (pid < 0) {
+                err = pid;
+                printk(UM_KERN_ERR "xterm_open : run_helper failed, "
+                       "errno = %d\n", -err);
+                goto out_close1;
         }
 
-        if (data->direct_rcv) {
-                new = os_rcv_fd(fd, &data->helper_pid);
-        } else {
-                err = os_set_fd_block(fd, 0);
-                if(err < 0){
-                        printk("xterm_open : failed to set descriptor "
-                               "non-blocking, err = %d\n", -err);
-                        return(err);
-                }
-                new = xterm_fd(fd, &data->helper_pid);
+        err = os_set_fd_block(fd, 0);
+        if (err < 0) {
+                printk(UM_KERN_ERR "xterm_open : failed to set descriptor "
+                       "non-blocking, err = %d\n", -err);
+                goto out_kill;
         }
-        if(new < 0){
-                printk("xterm_open : os_rcv_fd failed, err = %d\n", -new);
-                goto out;
+
+        new = xterm_fd(fd, &data->helper_pid);
+        if (new < 0) {
+                err = new;
+                printk(UM_KERN_ERR "xterm_open : os_rcv_fd failed, err = %d\n",
+                       -err);
+                goto out_kill;
         }
 
         err = os_set_fd_block(new, 0);
         if (err) {
-                printk("xterm_open : failed to set xterm descriptor "
-                       "non-blocking, err = %d\n", -err);
-                goto out;
+                printk(UM_KERN_ERR "xterm_open : failed to set xterm "
+                       "descriptor non-blocking, err = %d\n", -err);
+                goto out_close2;
         }
 
         CATCH_EINTR(err = tcgetattr(new, &data->tt));
-        if(err){
+        if (err) {
                 new = err;
-                goto out;
+                goto out_close2;
         }
 
-        if(data->raw){
+        if (data->raw) {
                 err = raw(new);
-                if(err){
+                if (err) {
                         new = err;
-                        goto out;
+                        goto out_close2;
                 }
         }
 
+        unlink(file);
         data->pid = pid;
         *dev_out = NULL;
- out:
-        unlink(file);
-        return(new);
+
+        return new;
+
+ out_close2:
+        close(new);
+ out_kill:
+        os_kill_process(pid, 1);
+ out_close1:
+        close(fd);
+
+        return err;
 }
 
-/* Not static because it's called directly by the tt mode gdb code */
-void xterm_close(int fd, void *d)
+static void xterm_close(int fd, void *d)
 {
         struct xterm_chan *data = d;
         
-        if(data->pid != -1) 
+        if (data->pid != -1)
                 os_kill_process(data->pid, 1);
         data->pid = -1;
-        if(data->helper_pid != -1) 
+
+        if (data->helper_pid != -1)
                 os_kill_process(data->helper_pid, 0);
         data->helper_pid = -1;
+
         os_close_file(fd);
 }
 
@@ -210,14 +224,3 @@ const struct chan_ops xterm_ops = {
         .free           = xterm_free,
         .winch          = 1,
 };
-
-/*
- * 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:
- */