]> Joshua Wise's Git repositories - netwatch.git/commitdiff
Properly ref and deref old pbufs in 3c90x.c.
authorJoshua Wise <joshua@nyus.joshuawise.com>
Sat, 6 Dec 2008 11:11:50 +0000 (06:11 -0500)
committerJoshua Wise <joshua@nyus.joshuawise.com>
Sat, 6 Dec 2008 11:11:50 +0000 (06:11 -0500)
net/3c90x.c

index 30283a29c17de2f053033c06211d315a5ec13fa9..a1b319c6ffd723f12c50f1f85e66521441de8e4b 100644 (file)
@@ -478,10 +478,10 @@ static void
 a3c90x_transmit(struct pbuf *p)
 {
        unsigned char status;
 a3c90x_transmit(struct pbuf *p)
 {
        unsigned char status;
-       static unsigned int stillwaiting = 0;
+       static struct pbuf *oldpbuf = NULL;
        unsigned int n, len;
        
        unsigned int n, len;
        
-       if (stillwaiting)
+       if (oldpbuf)
        {
                while (!(inw(INF_3C90X.IOAddr + regCommandIntStatus_w) & INT_TXCOMPLETE) && oneshot_running())
                        ;
        {
                while (!(inw(INF_3C90X.IOAddr + regCommandIntStatus_w) & INT_TXCOMPLETE) && oneshot_running())
                        ;
@@ -492,7 +492,8 @@ a3c90x_transmit(struct pbuf *p)
                }
                status = inb(INF_3C90X.IOAddr + regTxStatus_b);
                outb(INF_3C90X.IOAddr + regTxStatus_b, 0x00);
                }
                status = inb(INF_3C90X.IOAddr + regTxStatus_b);
                outb(INF_3C90X.IOAddr + regTxStatus_b, 0x00);
-               stillwaiting = 0;
+               pbuf_free(oldpbuf);
+               oldpbuf = NULL;
        }
 
        _issue_command(INF_3C90X.IOAddr, cmdStallCtl, 2 /* Stall download */);
        }
 
        _issue_command(INF_3C90X.IOAddr, cmdStallCtl, 2 /* Stall download */);
@@ -501,11 +502,13 @@ a3c90x_transmit(struct pbuf *p)
        INF_3C90X.TransmitDPD.DnNextPtr = 0;
        len = 0;
        n = 0;
        INF_3C90X.TransmitDPD.DnNextPtr = 0;
        len = 0;
        n = 0;
+       oldpbuf = p;
        for (; p; p = p->next)
        {
                INF_3C90X.TransmitDPD.segments[n].addr = v2p(p->payload);
                INF_3C90X.TransmitDPD.segments[n].len = p->len | (p->next ? 0 : (1 << 31));
                len += p->len;
        for (; p; p = p->next)
        {
                INF_3C90X.TransmitDPD.segments[n].addr = v2p(p->payload);
                INF_3C90X.TransmitDPD.segments[n].len = p->len | (p->next ? 0 : (1 << 31));
                len += p->len;
+               pbuf_ref(p);
                n++;
        }
        /** set notification for transmission completion (bit 15) **/
                n++;
        }
        /** set notification for transmission completion (bit 15) **/
@@ -526,9 +529,6 @@ a3c90x_transmit(struct pbuf *p)
                return;
        }
 
                return;
        }
 
-       oneshot_start_ms(10);
-       stillwaiting = 1;
-               
 #if 0          
        /** successful completion (sans "interrupt Requested" bit) **/
        if ((status & 0xbf) == 0x80)
 #if 0          
        /** successful completion (sans "interrupt Requested" bit) **/
        if ((status & 0xbf) == 0x80)
This page took 0.02387 seconds and 4 git commands to generate.