lkml.org 
[lkml]   [2000]   [Sep]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectDrivers needlessly setting TASK_RUNNING

The following drivers appear to set TASK_RUNNING needlessly.
Against test8pre1.

Sorry for the mass mailing !

thanks
john

--- drivers/net/wan/cosa.c Sat Jul 15 21:11:56 2000
+++ drivers/net/wan/cosa.c.new Fri Sep 1 04:13:44 2000
@@ -524,7 +524,6 @@
current->state = TASK_INTERRUPTIBLE;
cosa_putstatus(cosa, SR_TX_INT_ENA);
schedule_timeout(30);
- current->state = TASK_RUNNING;
irq = probe_irq_off(irqs);
/* Disable all IRQs from the card */
cosa_putstatus(cosa, 0);
@@ -824,14 +823,12 @@
if (signal_pending(current) && chan->rx_status == 0) {
chan->rx_status = 1;
remove_wait_queue(&chan->rxwaitq, &wait);
- current->state = TASK_RUNNING;
spin_unlock_irqrestore(&cosa->lock, flags);
up(&chan->rsem);
return -ERESTARTSYS;
}
}
remove_wait_queue(&chan->rxwaitq, &wait);
- current->state = TASK_RUNNING;
kbuf = chan->rxdata;
count = chan->rxsize;
spin_unlock_irqrestore(&cosa->lock, flags);
@@ -909,14 +906,12 @@
if (signal_pending(current) && chan->tx_status == 0) {
chan->tx_status = 1;
remove_wait_queue(&chan->txwaitq, &wait);
- current->state = TASK_RUNNING;
chan->tx_status = 1;
spin_unlock_irqrestore(&cosa->lock, flags);
return -ERESTARTSYS;
}
}
remove_wait_queue(&chan->txwaitq, &wait);
- current->state = TASK_RUNNING;
up(&chan->wsem);
spin_unlock_irqrestore(&cosa->lock, flags);
kfree(kbuf);
@@ -1538,7 +1533,6 @@
#ifdef MODULE
current->state = TASK_INTERRUPTIBLE;
schedule_timeout(HZ/2);
- current->state = TASK_RUNNING;
#else
udelay(5*100000);
#endif
--- drivers/char/serial.c Fri Sep 1 04:16:40 2000
+++ drivers/char/serial.c.new Fri Sep 1 04:17:00 2000
@@ -2902,7 +2902,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- set_current_state(TASK_RUNNING);
#ifdef SERIAL_DEBUG_RS_WAIT_UNTIL_SENT
printk("lsr = %d (jiff=%lu)...done\n", lsr, jiffies);
#endif
--- drivers/char/cyclades.c Sat Jul 15 21:11:46 2000
+++ drivers/char/cyclades.c.new Fri Sep 1 04:19:13 2000
@@ -2789,14 +2789,12 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
} else {
// Nothing to do!
}
/* Run one more char cycle */
current->state = TASK_INTERRUPTIBLE;
schedule_timeout(char_time * 5);
- current->state = TASK_RUNNING;
#ifdef CY_DEBUG_WAIT_UNTIL_SENT
printk("Clean (jiff=%lu)...done\n", jiffies);
#endif
--- drivers/char/rocket.c Tue Jul 11 19:33:45 2000
+++ drivers/char/rocket.c.new Fri Sep 1 04:20:48 2000
@@ -1650,7 +1650,6 @@
if (signal_pending(current))
break;
}
- current->state = TASK_RUNNING;
#ifdef ROCKET_DEBUG_WAIT_UNTIL_SENT
printk("txcnt = %d (jiff=%lu)...done\n", txcnt, jiffies);
#endif
--- drivers/char/istallion.c Wed Aug 2 20:38:26 2000
+++ drivers/char/istallion.c.new Fri Sep 1 04:21:41 2000
@@ -1532,7 +1532,6 @@
if (len > 0) {
current->state = TASK_INTERRUPTIBLE;
schedule_timeout(len);
- current->state = TASK_RUNNING;
}
}
--- drivers/char/sx.c Sat Aug 26 16:26:25 2000
+++ drivers/char/sx.c.new Fri Sep 1 04:22:49 2000
@@ -1561,7 +1561,6 @@
if (signal_pending (current))
break;
}
- current->state = TASK_RUNNING;
if (sx_read_channel_byte (port, hi_hstat) != HS_IDLE_CLOSED) {
if (sx_send_command (port, HS_FORCE_CLOSED, -1, HS_IDLE_CLOSED) != 1) {
printk (KERN_ERR
--- drivers/char/serial_21285.c Sat Aug 26 16:26:24 2000
+++ drivers/char/serial_21285.c.new Fri Sep 1 04:24:07 2000
@@ -256,7 +256,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
}

static int rs285_open(struct tty_struct *tty, struct file *filp)
--- drivers/char/dtlk.c Sat Jul 15 21:11:47 2000
+++ drivers/char/dtlk.c.new Fri Sep 1 04:24:48 2000
@@ -392,7 +392,6 @@
{
current->state = TASK_INTERRUPTIBLE;
schedule_timeout((ms * HZ + 1000 - HZ) / 1000);
- current->state = TASK_RUNNING;
}

static int dtlk_readable(void)
--- drivers/char/synclink.c Wed Aug 2 20:38:26 2000
+++ drivers/char/synclink.c.new Fri Sep 1 04:26:26 2000
@@ -3354,7 +3354,6 @@
}
}

- set_current_state(TASK_RUNNING);
exit:
if (debug_level >= DEBUG_LEVEL_INFO)
printk("%s(%d):mgsl_wait_until_sent(%s) exit\n",
--- drivers/char/drm/gamma_dma.c Wed Aug 2 20:38:25 2000
+++ drivers/char/drm/gamma_dma.c.new Fri Sep 1 04:28:20 2000
@@ -797,7 +797,6 @@
break;
}
}
- current->state = TASK_RUNNING;
remove_wait_queue(&dev->lock.lock_queue, &entry);
}

--- drivers/char/generic_serial.c Sat Aug 26 16:26:19 2000
+++ drivers/char/generic_serial.c.new Fri Sep 1 04:29:33 2000
@@ -428,7 +428,6 @@
}

gs_dprintk (GS_DEBUG_FLUSH, "charsleft = %d.\n", charsleft);
- set_current_state (TASK_RUNNING);

func_exit();
return rv;
--- drivers/char/rio/rio_linux.c Sat Aug 26 16:26:22 2000
+++ drivers/char/rio/rio_linux.c.new Fri Sep 1 04:30:10 2000
@@ -349,7 +349,6 @@
rio_dprintk (RIO_DEBUG_DELAY, "delaying %d jiffies\n", njiffies);
current->state = TASK_INTERRUPTIBLE;
schedule_timeout(njiffies);
- current->state = TASK_RUNNING;
func_exit();

if (signal_pending(current))
@@ -367,7 +366,6 @@
rio_dprintk (RIO_DEBUG_DELAY, "delaying %d jiffies (ni)\n", njiffies);
current->state = TASK_UNINTERRUPTIBLE;
schedule_timeout(njiffies);
- current->state = TASK_RUNNING;
func_exit();
return !RIO_FAIL;
}
--- drivers/char/amiserial.c Wed Aug 2 20:38:24 2000
+++ drivers/char/amiserial.c.new Fri Sep 1 04:32:42 2000
@@ -1676,7 +1676,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
#ifdef SERIAL_DEBUG_RS_WAIT_UNTIL_SENT
printk("lsr = %d (jiff=%lu)...done\n", lsr, jiffies);
#endif
--- drivers/sbus/char/sab82532.c Sat Aug 26 16:26:43 2000
+++ drivers/sbus/char/sab82532.c.new Fri Sep 1 04:33:47 2000
@@ -1722,7 +1722,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
#ifdef SERIAL_DEBUG_WAIT_UNTIL_SENT
printk("xmit_cnt = %d, alls = %d (jiff=%lu)...done\n", info->xmit_cnt, info->all_sent, jiffies);
#endif
--- drivers/sbus/char/su.c Mon Jul 10 03:08:10 2000
+++ drivers/sbus/char/su.c.new Fri Sep 1 04:34:51 2000
@@ -1851,7 +1851,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
#ifdef SERIAL_DEBUG_RS_WAIT_UNTIL_SENT
printk("lsr = %d (jiff=%lu)...done\n", lsr, jiffies);
#endif
--- drivers/macintosh/macserial.c Mon Jul 10 03:15:20 2000
+++ drivers/macintosh/macserial.c.new Fri Sep 1 04:36:04 2000
@@ -2020,7 +2020,6 @@
if (timeout && time_after(jiffies, orig_jiffies + timeout))
break;
}
- current->state = TASK_RUNNING;
}

/*
--- drivers/usb/audio.c Fri Aug 11 00:21:37 2000
+++ drivers/usb/audio.c.new Fri Sep 1 04:37:13 2000
@@ -625,7 +625,6 @@
notkilled = 0;
}
}
- set_current_state(TASK_RUNNING);
if (u->durb[0].urb.transfer_buffer)
kfree(u->durb[0].urb.transfer_buffer);
if (u->durb[1].urb.transfer_buffer)
@@ -1096,7 +1095,6 @@
notkilled = 0;
}
}
- set_current_state(TASK_RUNNING);
if (u->durb[0].urb.transfer_buffer)
kfree(u->durb[0].urb.transfer_buffer);
if (u->durb[1].urb.transfer_buffer)
--- drivers/usb/serial/digi_acceleport.c Sat Aug 26 16:26:56 2000
+++ drivers/usb/serial/digi_acceleport.c.new Fri Sep 1 04:38:08 2000
@@ -547,8 +547,6 @@

timeout = schedule_timeout(timeout);

- set_current_state( TASK_RUNNING );
-
remove_wait_queue( q, &wait );

return( timeout );
--- drivers/tc/zs.c Tue Jul 11 19:34:18 2000
+++ drivers/tc/zs.c.new Fri Sep 1 04:39:04 2000
@@ -1318,7 +1318,6 @@
if (timeout && ((orig_jiffies + timeout) < jiffies))
break;
}
- current->state = TASK_RUNNING;
}

/*
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:38    [W:0.776 / U:0.176 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site