UML - style fixes in arch/um/os-Linux
PATCH 3/3 UML - style fixes in arch/um/os-Linux
[ 2.6.25 material ]
Style fixes in arch/um/os-Linux/irq.c and arch/um/os-Linux/sigio.c:
Updated copyrights
trimmed includes
added severity indicators to printks
CodingStyle fixes
turned an bunch of panics into printks
call some libc functions directly instead of going through the
os_* wrappers
--- arch/um/os-Linux/irq.c | 27 ++--- arch/um/os-Linux/sigio.c | 241 ++++++++++++++++++++++++++--------------------- 2 files changed, 150 insertions(+), 118 deletions(-)
arch/um/os-Linux/irq.c +13 -14
@@ -1,22 +1,19 @@/* -* Copyright (C) 2000, 2001, 2002 Jeff Dike ([email protected]) +* Copyright (C) 2000 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com) * Licensed under the GPL */ #include <stdlib.h> -#include <unistd.h> #include <errno.h> +#include <poll.h> #include <signal.h> #include <string.h> -#include <sys/poll.h> -#include <sys/types.h> -#include <sys/time.h> -#include "user.h" -#include "process.h" -#include "sigio.h" #include "irq_user.h" +#include "kern_constants.h" #include "os.h" +#include "process.h" #include "um_malloc.h" +#include "user.h" /* * Locked by irq_lock in arch/um/kernel/irq.c. Changed by os_create_pollfd
@@ -35,7 +32,7 @@ int os_waiting_for_events(struct irq_fdif (n <0 ) { err = -errno;if (errno !=EINTR ) -printk ("sigio_handler: os_waiting_for_events:" +printk (UM_KERN_ERR "os_waiting_for_events:" " poll returned %d, errno = %d\n" , n, errno);return err; }
@@ -94,24 +91,26 @@ void os_free_irq_by_cb(int (*test)(strucstruct irq_fd *old_fd = *prev;if ((pollfds[i].fd !=-1 ) && (pollfds[i].fd != (*prev)->fd)) { -printk ("os_free_irq_by_cb - mismatch between " -"active_fds and pollfds, fd %d vs %d\n" , +printk (UM_KERN_ERR "os_free_irq_by_cb - " +"mismatch between active_fds and " +"pollfds, fd %d vs %d\n" , (*prev)->fd, pollfds[i].fd); goto out; } pollfds_num--; -/* This moves the *whole* array after pollfds[i] +/* +* This moves the *whole* array after pollfds[i] * (though it doesn't spot as such)! */ memmove (&pollfds[i], &pollfds[i +1 ], (pollfds_num - i) *sizeof (pollfds[0 ])); -if (*last_irq_ptr2 == &old_fd->next) +if (*last_irq_ptr2 == &old_fd->next) *last_irq_ptr2 = prev; *prev = (*prev)->next; -if (old_fd->type ==IRQ_WRITE ) +if (old_fd->type ==IRQ_WRITE )ignore_sigio_fd (old_fd->fd);kfree (old_fd);continue ;
arch/um/os-Linux/sigio.c +137 -104
451 lines not on this page · raw message
-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________
User-mode-linux-devel mailing list [email protected]
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel