[PATCH] paride fix: make timeouts unsigned long

From: Andries.Brouwer@cwi.nl
Date: Tue Apr 08 2003 - 04:12:05 EST


diff -u --recursive --new-file -X /linux/dontdiff a/drivers/block/paride/pg.c b/drivers/block/paride/pg.c
--- a/drivers/block/paride/pg.c Tue Mar 25 04:54:31 2003
+++ b/drivers/block/paride/pg.c Tue Mar 25 04:59:59 2003
@@ -233,14 +233,14 @@
         int busy; /* write done, read expected */
         int start; /* jiffies at command start */
         int dlen; /* transfer size requested */
- int timeout; /* timeout requested */
+ unsigned long timeout; /* timeout requested */
         int status; /* last sense key */
         int drive; /* drive */
         unsigned long access; /* count of active opens ... */
         int present; /* device present ? */
         char *bufptr;
         char name[PG_NAMELEN]; /* pg0, pg1, ... */
- };
+};
 
 struct pg_unit pg[PG_UNITS];
 
@@ -292,43 +292,47 @@
         schedule_timeout(cs);
 }
 
-static int pg_wait( int unit, int go, int stop, int tmo, char * msg )
-
-{ int j, r, e, s, p;
+static int pg_wait(int unit, int go, int stop, unsigned long tmo, char *msg)
+{
+ int j, r, e, s, p, to;
 
         PG.status = 0;
 
         j = 0;
- while ((((r=RR(1,6))&go)||(stop&&(!(r&stop))))&&(time_before(jiffies,tmo))) {
- if (j++ < PG_SPIN) udelay(PG_SPIN_DEL);
- else pg_sleep(1);
- }
-
- if ((r&(STAT_ERR&stop))||time_after_eq(jiffies, tmo)) {
- s = RR(0,7);
- e = RR(0,1);
- p = RR(0,2);
- if (verbose > 1)
- printk("%s: %s: stat=0x%x err=0x%x phase=%d%s\n",
- PG.name,msg,s,e,p,time_after_eq(jiffies, tmo)?" timeout":"");
-
-
- if (time_after_eq(jiffies, tmo)) e |= 0x100;
- PG.status = (e >> 4) & 0xff;
- return -1;
+ while ((((r=RR(1,6))&go) || (stop&&(!(r&stop))))
+ && time_before(jiffies,tmo)) {
+ if (j++ < PG_SPIN)
+ udelay(PG_SPIN_DEL);
+ else
+ pg_sleep(1);
+ }
+
+ to = time_after_eq(jiffies, tmo);
+
+ if ((r&(STAT_ERR&stop)) || to) {
+ s = RR(0,7);
+ e = RR(0,1);
+ p = RR(0,2);
+ if (verbose > 1)
+ printk("%s: %s: stat=0x%x err=0x%x phase=%d%s\n",
+ PG.name, msg, s, e, p, to ? " timeout" : "");
+ if (to)
+ e |= 0x100;
+ PG.status = (e >> 4) & 0xff;
+ return -1;
         }
         return 0;
 }
 
-static int pg_command( int unit, char * cmd, int dlen, int tmo )
-
-{ int k;
+static int pg_command(int unit, char *cmd, int dlen, unsigned long tmo)
+{
+ int k;
 
         pi_connect(PI);
 
         WR(0,6,DRIVE);
 
- if (pg_wait(unit,STAT_BUSY|STAT_DRQ,0,tmo,"before command")) {
+ if (pg_wait(unit, STAT_BUSY|STAT_DRQ, 0, tmo, "before command")) {
                 pi_disconnect(PI);
                 return -1;
         }
@@ -337,15 +341,15 @@
         WR(0,5,dlen / 256);
         WR(0,7,0xa0); /* ATAPI packet command */
 
- if (pg_wait(unit,STAT_BUSY,STAT_DRQ,tmo,"command DRQ")) {
+ if (pg_wait(unit, STAT_BUSY, STAT_DRQ, tmo, "command DRQ")) {
                 pi_disconnect(PI);
                 return -1;
         }
 
         if (RR(0,2) != 1) {
- printk("%s: command phase error\n",PG.name);
- pi_disconnect(PI);
- return -1;
+ printk("%s: command phase error\n",PG.name);
+ pi_disconnect(PI);
+ return -1;
         }
 
         pi_write_block(PI,cmd,12);
@@ -358,27 +362,30 @@
         return 0;
 }
 
-static int pg_completion( int unit, char * buf, int tmo)
-
-{ int r, d, n, p;
+static int pg_completion(int unit, char *buf, unsigned long tmo)
+{
+ int r, d, n, p;
 
- r = pg_wait(unit,STAT_BUSY,STAT_DRQ|STAT_READY|STAT_ERR,
- tmo,"completion");
+ r = pg_wait(unit, STAT_BUSY, STAT_DRQ|STAT_READY|STAT_ERR,
+ tmo, "completion");
 
         PG.dlen = 0;
 
         while (RR(0,7)&STAT_DRQ) {
- d = (RR(0,4)+256*RR(0,5));
- n = ((d+3)&0xfffc);
- p = RR(0,2)&3;
- if (p == 0) pi_write_block(PI,buf,n);
- if (p == 2) pi_read_block(PI,buf,n);
- if (verbose > 1) printk("%s: %s %d bytes\n",PG.name,
- p?"Read":"Write",n);
- PG.dlen += (1-p)*d;
- buf += d;
- r = pg_wait(unit,STAT_BUSY,STAT_DRQ|STAT_READY|STAT_ERR,
- tmo,"completion");
+ d = (RR(0,4)+256*RR(0,5));
+ n = ((d+3)&0xfffc);
+ p = RR(0,2)&3;
+ if (p == 0)
+ pi_write_block(PI,buf,n);
+ if (p == 2)
+ pi_read_block(PI,buf,n);
+ if (verbose > 1)
+ printk("%s: %s %d bytes\n", PG.name,
+ p?"Read":"Write", n);
+ PG.dlen += (1-p)*d;
+ buf += d;
+ r = pg_wait(unit, STAT_BUSY, STAT_DRQ|STAT_READY|STAT_ERR,
+ tmo, "completion");
         }
 
         pi_disconnect(PI);
diff -u --recursive --new-file -X /linux/dontdiff a/drivers/block/paride/pseudo.h b/drivers/block/paride/pseudo.h
--- a/drivers/block/paride/pseudo.h Fri Nov 22 22:40:27 2002
+++ b/drivers/block/paride/pseudo.h Tue Mar 18 14:50:10 2003
@@ -39,7 +39,7 @@
 
 static void (* ps_continuation)(void);
 static int (* ps_ready)(void);
-static int ps_timeout;
+static unsigned long ps_timeout;
 static int ps_tq_active = 0;
 static int ps_nice = 0;
 
@@ -70,7 +70,7 @@
         spin_unlock_irqrestore(&ps_spinlock,flags);
 }
 
-static void ps_tq_int( void *data )
+static void ps_tq_int(void *data)
 {
         void (*con)(void);
         unsigned long flags;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Apr 15 2003 - 22:00:14 EST