d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't arbitrarily set SCSI_NOSLEEP. It is now handled correctly by the

Browse Source 
higher level scsi code.

Spls should never be conditionalized, so don't do so here.

Restructure the get_ccb routine so that we can't get into an infinite
loop if the ccbs are exhausted and we are are called with SCSI_NOSLEEP set.

Other driver maintainer's that based their ccb allocation routines on Julian's
code should look at these changes and implement them for their driver.

Submitted by: John Dyson
This commit is contained in:
gibbs 1995-04-23 21:58:35 +00:00
parent b79d699d4c
commit 8248293bc2
1 changed files with 12 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
sys/i386/isa/bt742a.c
View File

@ -12,7 +12,7 @@
* on the understanding that TFS is not responsible for the correct
* functioning of this software in any circumstances.
*
* $Id: bt742a.c,v 1.32 1995/04/12 20:47:38 wollman Exp $
* $Id: bt742a.c,v 1.33 1995/04/23 09:13:05 julian Exp $
*/
/*
@ -866,9 +866,8 @@ bt_free_ccb(unit, ccb, flags)
int flags;
{
struct bt_data *bt = btdata[unit];
unsigned int opri = 0;
unsigned int opri;
if (!(flags & SCSI_NOMASK))
opri = splbio();
ccb->next = bt->bt_ccb_free;
@ -882,7 +881,6 @@ bt_free_ccb(unit, ccb, flags)
wakeup((caddr_t)&bt->bt_ccb_free);
}
if (!(flags & SCSI_NOMASK))
splx(opri);
}
@ -898,13 +896,12 @@ bt_get_ccb(unit, flags)
int flags;
{
struct bt_data *bt = btdata[unit];
unsigned opri = 0;
unsigned opri;
struct bt_ccb *ccbp;
struct bt_mbx *wmbx; /* Mail Box pointer specified unit */
BT_MBO *wmbo; /* Out Mail Box pointer */
int hashnum;
if (!(flags & SCSI_NOMASK))
opri = splbio();
/*
* If we can and have to, sleep waiting for one to come free
@ -934,7 +931,9 @@ bt_get_ccb(unit, flags)
if (!(flags & SCSI_NOSLEEP)) {
tsleep((caddr_t)&bt->bt_ccb_free, PRIBIO,
"btccb", 0);
continue;
}
break;
}
}
if (ccbp) {
@ -943,7 +942,6 @@ bt_get_ccb(unit, flags)
ccbp->flags = CCB_ACTIVE;
}
gottit:
if (!(flags & SCSI_NOMASK))
splx(opri);
return (ccbp);
@ -976,14 +974,13 @@ BT_MBO *
bt_send_mbo(int unit, int flags, int cmd, struct bt_ccb *ccb)
{
struct bt_data *bt = btdata[unit];
unsigned opri = 0;
unsigned opri;
BT_MBO *wmbo; /* Mail Box Out pointer */
struct bt_mbx *wmbx; /* Mail Box pointer specified unit */
int i, wait;
wmbx = &bt->bt_mbx;
if (!(flags & SCSI_NOMASK))
opri = splbio();
/* Get the Target OUT mail Box pointer and move to Next */
@ -1022,7 +1019,6 @@ bt_send_mbo(int unit, int flags, int cmd, struct bt_ccb *ccb)
/* Send it! */
outb(BT_CMD_DATA_PORT, BT_START_SCSI);
if (!(flags & SCSI_NOMASK))
splx(opri);
return (wmbo);
@ -1437,8 +1433,6 @@ bt_scsi_cmd(xs)
* then we can't allow it to sleep
*/
flags = xs->flags;
if (xs->bp)
flags |= (SCSI_NOSLEEP); /* just to be sure */
if (flags & ITSDONE) {
printf("bt%d: Already done?\n", unit);
xs->flags &= ~ITSDONE;