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

Fix lots of places that assume sizeof(int) == sizeof(void *). Most of

Browse Source 
these were cosmetic.  However, at least two of these changes fix bugs
where a (fake) pointer to a RX descriptor was added to the physical
memory start address with the cast in the wrong place resulting in
rxdesc[pmem_start] instead of (uintptr_t)rxdesc + pmem_start.
This commit is contained in:
jhb 2002-11-07 21:13:15 +00:00
parent 5948946955
commit 8a9582d147
1 changed files with 18 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
sys/dev/sr/if_sr.c
View File

@ -387,9 +387,9 @@ sr_attach(device_t device)
* Report Card configuration information before we start configuring
* each channel on the card...
*/
printf("src%d: %uK RAM (%d mempages) @ %08x-%08x, %u ports.\n",
printf("src%d: %uK RAM (%d mempages) @ %p-%p, %u ports.\n",
hc->cunit, hc->memsize / 1024, hc->mempages,
(u_int)hc->mem_start, (u_int)hc->mem_end, hc->numports);
hc->mem_start, hc->mem_end, hc->numports);
src_init(hc);
sr_init_sca(hc);
@ -967,8 +967,8 @@ srstart(struct sr_softc *sc)
* can record this in the structure and fire it off w/ the DMA
* processor of the serial chip...
*/
txdesc = (sca_descriptor *)blkp->txdesc;
blkp->txeda = (u_short)((u_int)&txdesc[i]);
txdesc = (sca_descriptor *)(uintptr_t)blkp->txdesc;
blkp->txeda = (u_short)((uintptr_t)&txdesc[i]);
sc->txb_inuse++; /* update inuse status */
sc->txb_new++; /* new traffic wuz added */
@ -1634,7 +1634,7 @@ sr_init_rx_dmac(struct sr_softc *sc)
* needed to construct a circular buffer...
*/
rxd = (sca_descriptor *)(hc->mem_start + (sc->rxdesc & hc->winmsk));
rxda_d = (u_int) hc->mem_start - (sc->rxdesc & ~hc->winmsk);
rxda_d = (uintptr_t) hc->mem_start - (sc->rxdesc & ~hc->winmsk);
for (rxbuf = sc->rxstart;
rxbuf < sc->rxend;
@ -1642,7 +1642,7 @@ sr_init_rx_dmac(struct sr_softc *sc)
/*
* construct the circular chain...
*/
rxda = (u_int) & rxd[1] - rxda_d + hc->mem_pstart;
rxda = (uintptr_t) &rxd[1] - rxda_d + hc->mem_pstart;
rxd->cp = (u_short)(rxda & 0xffff);
/*
@ -1682,10 +1682,10 @@ sr_init_rx_dmac(struct sr_softc *sc)
SRC_PUT16(hc->sca_base, dmac->cda, cda_v);
SRC_PUT8(hc->sca_base, dmac->sarb, sarb_v);
rxd = (sca_descriptor *)sc->rxstart;
rxd = (sca_descriptor *)(uintptr_t)sc->rxstart;
SRC_PUT16(hc->sca_base, dmac->eda,
(u_short)((u_int) & rxd[sc->rxmax - 1] & 0xffff));
(u_short)((uintptr_t) &rxd[sc->rxmax - 1] & 0xffff));
SRC_PUT8(hc->sca_base, dmac->dir, 0xF0);
@ -1726,13 +1726,13 @@ sr_init_tx_dmac(struct sr_softc *sc)
blkp = &sc->block[blk];
txd = (sca_descriptor *)(hc->mem_start
+ (blkp->txdesc & hc->winmsk));
txda_d = (u_int) hc->mem_start
txda_d = (uintptr_t) hc->mem_start
- (blkp->txdesc & ~hc->winmsk);
x = 0;
txbuf = blkp->txstart;
for (; txbuf < blkp->txend; txbuf += SR_BUF_SIZ, txd++) {
txda = (u_int) & txd[1] - txda_d + hc->mem_pstart;
txda = (uintptr_t) &txd[1] - txda_d + hc->mem_pstart;
txd->cp = (u_short)(txda & 0xffff);
txd->bp = (u_short)((txbuf + hc->mem_pstart)
@ -1748,7 +1748,7 @@ sr_init_tx_dmac(struct sr_softc *sc)
txd->cp = (u_short)((blkp->txdesc + hc->mem_pstart)
& 0xffff);
blkp->txtail = (u_int)txd - (u_int)hc->mem_start;
blkp->txtail = (uintptr_t)txd - (uintptr_t)hc->mem_start;
}
SRC_PUT8(hc->sca_base, dmac->dsr, 0); /* Disable DMA */
@ -1980,9 +1980,9 @@ sr_eat_packet(struct sr_softc *sc, int single)
loopcnt++;
if (loopcnt > sc->rxmax) {
printf("sr%d: eat pkt %d loop, cda %x, "
"rxdesc %x, stat %x.\n",
sc->unit, loopcnt, (u_int) cda, (u_int) rxdesc,
printf("sr%d: eat pkt %d loop, cda %p, "
"rxdesc %p, stat %x.\n",
sc->unit, loopcnt, cda, rxdesc,
rxdesc->stat);
break;
}
@ -2006,12 +2006,12 @@ sr_eat_packet(struct sr_softc *sc, int single)
/*
* Update the eda to the previous descriptor.
*/
rxdesc = (sca_descriptor *)sc->rxdesc;
rxdesc = (sca_descriptor *)(uintptr_t)sc->rxdesc;
rxdesc = &rxdesc[(sc->rxhind + sc->rxmax - 2) % sc->rxmax];
SRC_PUT16(hc->sca_base,
hc->sca->dmac[DMAC_RXCH(sc->scachan)].eda,
(u_short)((u_int)(rxdesc + hc->mem_pstart) & 0xffff));
(u_short)(((uintptr_t)rxdesc + hc->mem_pstart) & 0xffff));
}
/*
@ -2167,13 +2167,13 @@ sr_get_packets(struct sr_softc *sc)
i = (len + SR_BUF_SIZ - 1) / SR_BUF_SIZ;
sc->rxhind = (sc->rxhind + i) % sc->rxmax;
rxdesc = (sca_descriptor *)sc->rxdesc;
rxdesc = (sca_descriptor *)(uintptr_t)sc->rxdesc;
rxndx = (sc->rxhind + sc->rxmax - 2) % sc->rxmax;
rxdesc = &rxdesc[rxndx];
SRC_PUT16(hc->sca_base,
hc->sca->dmac[DMAC_RXCH(sc->scachan)].eda,
(u_short)((u_int)(rxdesc + hc->mem_pstart)
(u_short)(((uintptr_t)rxdesc + hc->mem_pstart)
& 0xffff));
} else {