This commit was manufactured by cvs2svn to create tag
[linux-2.6.git] / drivers / char / generic_serial.c
index d95ad27..5e59c0b 100644 (file)
@@ -26,6 +26,7 @@
 #include <linux/mm.h>
 #include <linux/generic_serial.h>
 #include <linux/interrupt.h>
+#include <linux/tty_flip.h>
 #include <linux/delay.h>
 #include <asm/semaphore.h>
 #include <asm/uaccess.h>
@@ -33,7 +34,6 @@
 #define DEBUG 
 
 static char *                  tmp_buf; 
-static DECLARE_MUTEX(tmp_buf_sem);
 
 static int gs_debug;
 
@@ -45,11 +45,11 @@ static int gs_debug;
 
 #define func_enter() gs_dprintk (GS_DEBUG_FLOW, "gs: enter %s\n", __FUNCTION__)
 #define func_exit()  gs_dprintk (GS_DEBUG_FLOW, "gs: exit  %s\n", __FUNCTION__)
-
-#ifdef NEW_WRITE_LOCKING
+#define NEW_WRITE_LOCKING 1
+#if NEW_WRITE_LOCKING
 #define DECL      /* Nothing */
-#define LOCKIT    down (& port->port_write_sem);
-#define RELEASEIT up (&port->port_write_sem);
+#define LOCKIT    mutex_lock(& port->port_write_mutex);
+#define RELEASEIT mutex_unlock(&port->port_write_mutex);
 #else
 #define DECL      unsigned long flags;
 #define LOCKIT    save_flags (flags);cli ()
@@ -124,14 +124,14 @@ int gs_write(struct tty_struct * tty,
        /* get exclusive "write" access to this port (problem 3) */
        /* This is not a spinlock because we can have a disk access (page 
                 fault) in copy_from_user */
-       down (& port->port_write_sem);
+       mutex_lock(& port->port_write_mutex);
 
        while (1) {
 
                c = count;
  
                /* This is safe because we "OWN" the "head". Noone else can 
-                  change the "head": we own the port_write_sem. */
+                  change the "head": we own the port_write_mutex. */
                /* Don't overrun the end of the buffer */
                t = SERIAL_XMIT_SIZE - port->xmit_head;
                if (t < c) c = t;
@@ -153,7 +153,7 @@ int gs_write(struct tty_struct * tty,
                count -= c;
                total += c;
        }
-       up (& port->port_write_sem);
+       mutex_unlock(& port->port_write_mutex);
 
        gs_dprintk (GS_DEBUG_WRITE, "write: interrupts are %s\n", 
                    (port->flags & GS_TX_INTEN)?"enabled": "disabled"); 
@@ -208,13 +208,13 @@ int gs_write(struct tty_struct * tty,
        if (!port || !port->xmit_buf || !tmp_buf)
                return -EIO;
 
-       save_flags(flags);
+       local_save_flags(flags);
        while (1) {
                cli();
                c = count;
 
                /* This is safe because we "OWN" the "head". Noone else can 
-                  change the "head": we own the port_write_sem. */
+                  change the "head": we own the port_write_mutex. */
                /* Don't overrun the end of the buffer */
                t = SERIAL_XMIT_SIZE - port->xmit_head;
                if (t < c) c = t;
@@ -227,14 +227,14 @@ int gs_write(struct tty_struct * tty,
 
                /* Can't copy more? break out! */
                if (c <= 0) {
-                       restore_flags(flags);
+                       local_restore_flags(flags);
                        break;
                }
                memcpy(port->xmit_buf + port->xmit_head, buf, c);
                port->xmit_head = ((port->xmit_head + c) &
                                   (SERIAL_XMIT_SIZE-1));
                port->xmit_cnt += c;
-               restore_flags(flags);
+               local_restore_flags(flags);
                buf += c;
                count -= c;
                total += c;
@@ -279,7 +279,7 @@ int gs_chars_in_buffer(struct tty_struct *tty)
 }
 
 
-int gs_real_chars_in_buffer(struct tty_struct *tty)
+static int gs_real_chars_in_buffer(struct tty_struct *tty)
 {
        struct gs_port *port;
        func_enter ();
@@ -295,7 +295,7 @@ int gs_real_chars_in_buffer(struct tty_struct *tty)
 }
 
 
-static int gs_wait_tx_flushed (void * ptr, int timeout) 
+static int gs_wait_tx_flushed (void * ptr, unsigned long timeout) 
 {
        struct gs_port *port = ptr;
        unsigned long end_jiffies;
@@ -380,9 +380,9 @@ void gs_flush_buffer(struct tty_struct *tty)
        if (!port) return;
 
        /* XXX Would the write semaphore do? */
-       save_flags(flags); cli();
+       spin_lock_irqsave (&port->driver_lock, flags);
        port->xmit_cnt = port->xmit_head = port->xmit_tail = 0;
-       restore_flags(flags);
+       spin_unlock_irqrestore (&port->driver_lock, flags);
 
        wake_up_interruptible(&tty->write_wait);
        tty_wakeup(tty);
@@ -457,7 +457,7 @@ void gs_start(struct tty_struct * tty)
 }
 
 
-void gs_shutdown_port (struct gs_port *port)
+static void gs_shutdown_port (struct gs_port *port)
 {
        unsigned long flags;
 
@@ -468,8 +468,7 @@ void gs_shutdown_port (struct gs_port *port)
        if (!(port->flags & ASYNC_INITIALIZED))
                return;
 
-       save_flags (flags);
-       cli ();
+       spin_lock_irqsave(&port->driver_lock, flags);
 
        if (port->xmit_buf) {
                free_page((unsigned long) port->xmit_buf);
@@ -482,7 +481,7 @@ void gs_shutdown_port (struct gs_port *port)
        port->rd->shutdown_port (port);
 
        port->flags &= ~ASYNC_INITIALIZED;
-       restore_flags (flags);
+       spin_unlock_irqrestore(&port->driver_lock, flags);
 
        func_exit();
 }
@@ -511,27 +510,6 @@ void gs_hangup(struct tty_struct *tty)
 }
 
 
-void gs_do_softint(void *private_)
-{
-       struct gs_port *port = private_;
-       struct tty_struct *tty;
-
-       func_enter ();
-
-       if (!port) return;
-
-       tty = port->tty;
-
-       if (!tty) return;
-
-       if (test_and_clear_bit(RS_EVENT_WRITE_WAKEUP, &port->event)) {
-               tty_wakeup(tty);
-               wake_up_interruptible(&tty->write_wait);
-       }
-       func_exit ();
-}
-
-
 int gs_block_til_ready(void *port_, struct file * filp)
 {
        struct gs_port *port = port_;
@@ -540,6 +518,7 @@ int gs_block_til_ready(void *port_, struct file * filp)
        int    do_clocal = 0;
        int    CD;
        struct tty_struct *tty;
+       unsigned long flags;
 
        func_enter ();
 
@@ -591,10 +570,11 @@ int gs_block_til_ready(void *port_, struct file * filp)
        add_wait_queue(&port->open_wait, &wait);
 
        gs_dprintk (GS_DEBUG_BTR, "after add waitq.\n"); 
-       cli();
-       if (!tty_hung_up_p(filp))
+       spin_lock_irqsave(&port->driver_lock, flags);
+       if (!tty_hung_up_p(filp)) {
                port->count--;
-       sti();
+       }
+       spin_unlock_irqrestore(&port->driver_lock, flags);
        port->blocked_open++;
        while (1) {
                CD = port->rd->get_CD (port);
@@ -623,8 +603,9 @@ int gs_block_til_ready(void *port_, struct file * filp)
                    port->blocked_open);
        set_current_state (TASK_RUNNING);
        remove_wait_queue(&port->open_wait, &wait);
-       if (!tty_hung_up_p(filp))
+       if (!tty_hung_up_p(filp)) {
                port->count++;
+       }
        port->blocked_open--;
        if (retval)
                return retval;
@@ -654,27 +635,29 @@ void gs_close(struct tty_struct * tty, struct file * filp)
                port->tty = tty;
        }
 
-       save_flags(flags); cli();
+       spin_lock_irqsave(&port->driver_lock, flags);
 
        if (tty_hung_up_p(filp)) {
-               restore_flags(flags);
-               port->rd->hungup (port);
+               spin_unlock_irqrestore(&port->driver_lock, flags);
+               if (port->rd->hungup)
+                       port->rd->hungup (port);
                func_exit ();
                return;
        }
 
        if ((tty->count == 1) && (port->count != 1)) {
-               printk(KERN_ERR "gs: gs_close: bad port count;"
-                      " tty->count is 1, port count is %d\n", port->count);
+               printk(KERN_ERR "gs: gs_close port %p: bad port count;"
+                      " tty->count is 1, port count is %d\n", port, port->count);
                port->count = 1;
        }
        if (--port->count < 0) {
-               printk(KERN_ERR "gs: gs_close: bad port count: %d\n", port->count);
+               printk(KERN_ERR "gs: gs_close port %p: bad port count: %d\n", port, port->count);
                port->count = 0;
        }
+
        if (port->count) {
-               gs_dprintk(GS_DEBUG_CLOSE, "gs_close: count: %d\n", port->count);
-               restore_flags(flags);
+               gs_dprintk(GS_DEBUG_CLOSE, "gs_close port %p: count: %d\n", port, port->count);
+               spin_unlock_irqrestore(&port->driver_lock, flags);
                func_exit ();
                return;
        }
@@ -696,16 +679,17 @@ void gs_close(struct tty_struct * tty, struct file * filp)
         */
 
        port->rd->disable_rx_interrupts (port);
+       spin_unlock_irqrestore(&port->driver_lock, flags);
 
        /* close has no way of returning "EINTR", so discard return value */
        if (port->closing_wait != ASYNC_CLOSING_WAIT_NONE)
-               gs_wait_tx_flushed (port, port->closing_wait); 
+               gs_wait_tx_flushed (port, port->closing_wait);
 
        port->flags &= ~GS_ACTIVE;
 
        if (tty->driver->flush_buffer)
                tty->driver->flush_buffer(tty);
-               
+
        tty_ldisc_flush(tty);
        tty->closing = 0;
 
@@ -716,14 +700,15 @@ void gs_close(struct tty_struct * tty, struct file * filp)
 
        if (port->blocked_open) {
                if (port->close_delay) {
+                       spin_unlock_irqrestore(&port->driver_lock, flags);
                        msleep_interruptible(jiffies_to_msecs(port->close_delay));
+                       spin_lock_irqsave(&port->driver_lock, flags);
                }
                wake_up_interruptible(&port->open_wait);
        }
        port->flags &= ~(ASYNC_NORMAL_ACTIVE|ASYNC_CLOSING | ASYNC_INITIALIZED);
        wake_up_interruptible(&port->close_wait);
 
-       restore_flags(flags);
        func_exit ();
 }
 
@@ -748,6 +733,12 @@ void gs_set_termios (struct tty_struct * tty,
        port = tty->driver_data;
 
        if (!port) return;
+       if (!port->tty) {
+               /* This seems to happen when this is called after gs_close. */
+               gs_dprintk (GS_DEBUG_TERMIOS, "gs: Odd: port->tty is NULL\n");
+               port->tty = tty;
+       }
+
 
        tiosp = tty->termios;
 
@@ -842,7 +833,7 @@ void gs_set_termios (struct tty_struct * tty,
 
        if (!(old_termios->c_cflag & CLOCAL) &&
            (tty->termios->c_cflag & CLOCAL))
-               wake_up_interruptible(&info->open_wait);
+               wake_up_interruptible(&port->gs.open_wait);
 #endif
 
        func_exit();
@@ -857,56 +848,56 @@ int gs_init_port(struct gs_port *port)
        unsigned long flags;
        unsigned long page;
 
-       save_flags (flags);
-       if (!tmp_buf) {
-               page = get_zeroed_page(GFP_KERNEL);
+       func_enter ();
 
-               cli (); /* Don't expect this to make a difference. */ 
+        if (!tmp_buf) {
+               page = get_zeroed_page(GFP_KERNEL);
+               spin_lock_irqsave (&port->driver_lock, flags); /* Don't expect this to make a difference. */
                if (tmp_buf)
                        free_page(page);
                else
                        tmp_buf = (unsigned char *) page;
-               restore_flags (flags);
-
+               spin_unlock_irqrestore (&port->driver_lock, flags);
                if (!tmp_buf) {
+                       func_exit ();
                        return -ENOMEM;
                }
        }
 
-       if (port->flags & ASYNC_INITIALIZED)
+       if (port->flags & ASYNC_INITIALIZED) {
+               func_exit ();
                return 0;
-
+       }
        if (!port->xmit_buf) {
                /* We may sleep in get_zeroed_page() */
                unsigned long tmp;
 
                tmp = get_zeroed_page(GFP_KERNEL);
-
-               /* Spinlock? */
-               cli ();
+               spin_lock_irqsave (&port->driver_lock, flags);
                if (port->xmit_buf) 
                        free_page (tmp);
                else
                        port->xmit_buf = (unsigned char *) tmp;
-               restore_flags (flags);
-
-               if (!port->xmit_buf)
+               spin_unlock_irqrestore(&port->driver_lock, flags);
+               if (!port->xmit_buf) {
+                       func_exit ();
                        return -ENOMEM;
+               }
        }
 
-       cli();
-
+       spin_lock_irqsave (&port->driver_lock, flags);
        if (port->tty) 
                clear_bit(TTY_IO_ERROR, &port->tty->flags);
-
+       mutex_init(&port->port_write_mutex);
        port->xmit_cnt = port->xmit_head = port->xmit_tail = 0;
-
+       spin_unlock_irqrestore(&port->driver_lock, flags);
        gs_set_termios(port->tty, NULL);
-
+       spin_lock_irqsave (&port->driver_lock, flags);
        port->flags |= ASYNC_INITIALIZED;
        port->flags &= ~GS_TX_INTEN;
 
-       restore_flags(flags);
+       spin_unlock_irqrestore(&port->driver_lock, flags);
+       func_exit ();
        return 0;
 }
 
@@ -977,13 +968,15 @@ int gs_getserial(struct gs_port *port, struct serial_struct __user *sp)
 
 void gs_got_break(struct gs_port *port)
 {
+       func_enter ();
+
+       tty_insert_flip_char(port->tty, 0, TTY_BREAK);
+       tty_schedule_flip(port->tty);
        if (port->flags & ASYNC_SAK) {
                do_SAK (port->tty);
        }
-       *(port->tty->flip.flag_buf_ptr) = TTY_BREAK;
-       port->tty->flip.flag_buf_ptr++;
-       port->tty->flip.char_buf_ptr++;
-       port->tty->flip.count++;
+
+       func_exit ();
 }
 
 
@@ -996,7 +989,6 @@ EXPORT_SYMBOL(gs_flush_chars);
 EXPORT_SYMBOL(gs_stop);
 EXPORT_SYMBOL(gs_start);
 EXPORT_SYMBOL(gs_hangup);
-EXPORT_SYMBOL(gs_do_softint);
 EXPORT_SYMBOL(gs_block_til_ready);
 EXPORT_SYMBOL(gs_close);
 EXPORT_SYMBOL(gs_set_termios);