Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8cf5a8c5729116d0d60815393ec9f1c0f0bc2df7
Commit:     8cf5a8c5729116d0d60815393ec9f1c0f0bc2df7
Parent:     95e0791480af8347460d0cbe34a46eca7e77d0d0
Author:     Jiri Slaby <[EMAIL PROTECTED]>
AuthorDate: Thu Oct 18 03:06:25 2007 -0700
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Thu Oct 18 14:37:27 2007 -0700

    Char: rocket, switch sleep_on to completion
    
    rocket, switch sleep_on to completion
    
    - sleep_on is deprecated and racy, use completion instead
    - also check retval of interruptible function and return ERESTARTSYS
      eventually
    
    Signed-off-by: Jiri Slaby <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
---
 drivers/char/rocket.c     |   12 ++++++++----
 drivers/char/rocket_int.h |    7 +------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/drivers/char/rocket.c b/drivers/char/rocket.c
index 56cbba7..d790b08 100644
--- a/drivers/char/rocket.c
+++ b/drivers/char/rocket.c
@@ -84,6 +84,7 @@
 #include <linux/mutex.h>
 #include <linux/ioport.h>
 #include <linux/delay.h>
+#include <linux/completion.h>
 #include <linux/wait.h>
 #include <linux/pci.h>
 #include <asm/uaccess.h>
@@ -650,7 +651,7 @@ static void init_r_port(int board, int aiop, int chan, 
struct pci_dev *pci_dev)
        info->closing_wait = 3000;
        info->close_delay = 50;
        init_waitqueue_head(&info->open_wait);
-       init_waitqueue_head(&info->close_wait);
+       init_completion(&info->close_wait);
        info->flags &= ~ROCKET_MODE_MASK;
        switch (pc104[board][line]) {
        case 422:
@@ -878,7 +879,8 @@ static int block_til_ready(struct tty_struct *tty, struct 
file *filp,
        if (tty_hung_up_p(filp))
                return ((info->flags & ROCKET_HUP_NOTIFY) ? -EAGAIN : 
-ERESTARTSYS);
        if (info->flags & ROCKET_CLOSING) {
-               interruptible_sleep_on(&info->close_wait);
+               if (wait_for_completion_interruptible(&info->close_wait))
+                       return -ERESTARTSYS;
                return ((info->flags & ROCKET_HUP_NOTIFY) ? -EAGAIN : 
-ERESTARTSYS);
        }
 
@@ -983,8 +985,10 @@ static int rp_open(struct tty_struct *tty, struct file 
*filp)
                return -ENOMEM;
 
        if (info->flags & ROCKET_CLOSING) {
-               interruptible_sleep_on(&info->close_wait);
+               retval = wait_for_completion_interruptible(&info->close_wait);
                free_page(page);
+               if (retval)
+                       return retval;
                return ((info->flags & ROCKET_HUP_NOTIFY) ? -EAGAIN : 
-ERESTARTSYS);
        }
 
@@ -1176,7 +1180,7 @@ static void rp_close(struct tty_struct *tty, struct file 
*filp)
        }
        info->flags &= ~(ROCKET_INITIALIZED | ROCKET_CLOSING | 
ROCKET_NORMAL_ACTIVE);
        tty->closing = 0;
-       wake_up_interruptible(&info->close_wait);
+       complete_all(&info->close_wait);
        atomic_dec(&rp_num_ports_open);
 
 #ifdef ROCKET_DEBUG_OPEN
diff --git a/drivers/char/rocket_int.h b/drivers/char/rocket_int.h
index b4c53df..55b8f2d 100644
--- a/drivers/char/rocket_int.h
+++ b/drivers/char/rocket_int.h
@@ -1163,13 +1163,8 @@ struct r_port {
        int read_status_mask;
        int cps;
 
-#ifdef DECLARE_WAITQUEUE
        wait_queue_head_t open_wait;
-       wait_queue_head_t close_wait;
-#else
-       struct wait_queue *open_wait;
-       struct wait_queue *close_wait;
-#endif
+       struct completion close_wait;
        spinlock_t slock;
        struct mutex write_mtx;
 };
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to