Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-08-06 11:24:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

Multiple interrupt status bits can be active at the same time, process them

Browse source 
all instead of just one and ignoring the rest.
On an active network card, this would eventually cause all transmit buffers
to be marked "in use", blocking all transmits. Sinds no TCP acks would go out,
it would block TCP receives too.

svn path=/trunk/; revision=20043
This commit is contained in:
Gé van Geldorp 2005-12-10 22:21:28 +00:00
parent 22da8ae308
commit c20dd5efa7
1 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
reactos/drivers/net/dd/pcnet/pcnet.c
View file

@ -87,11 +87,11 @@ MiniportHandleInterrupt(
if (Data & CSR0_CERR)
Adapter->Statistics.XmtCollisions++;
}
else if(Data & CSR0_IDON)
if(Data & CSR0_IDON)
{
DPRINT("IDON\n");
}
else if(Data & CSR0_RINT)
if(Data & CSR0_RINT)
{
DPRINT("receive interrupt\n");
@ -147,7 +147,7 @@ MiniportHandleInterrupt(
Adapter->Statistics.RcvGoodFrames++;
}
}
else if(Data & CSR0_TINT)
if(Data & CSR0_TINT)
{
PTRANSMIT_DESCRIPTOR Descriptor;
@ -201,10 +201,9 @@ MiniportHandleInterrupt(
}
NdisMSendResourcesAvailable(Adapter->MiniportAdapterHandle);
}
else
if(Data & ~(CSR0_ERR | CSR0_IDON | CSR0_RINT | CSR0_TINT))
{
DPRINT1("UNHANDLED INTERRUPT\n");
ASSERT(FALSE);
DPRINT("UNHANDLED INTERRUPT CSR0 0x%x\n", Data);
}
NdisRawReadPortUshort(Adapter->PortOffset + RDP, &Data);
@ -1081,7 +1080,7 @@ MiniportSend(
(Adapter->CurrentTransmitEndIndex == NUMBER_OF_BUFFERS - 1 &&
Adapter->CurrentTransmitStartIndex == 0))
{
DPRINT("No free space in circular buffer\n");
DPRINT1("No free space in circular buffer\n");
NdisDprReleaseSpinLock(&Adapter->Lock);
return NDIS_STATUS_RESOURCES;
}