public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* PATCH for dl2k driver
@ 2001-07-10 18:24 Konstantin Volckov
  2001-07-10 19:17 ` Jeff Garzik
  0 siblings, 1 reply; 2+ messages in thread
From: Konstantin Volckov @ 2001-07-10 18:24 UTC (permalink / raw)
  To: linux-kernel; +Cc: Alan Cox, Edward Peng.

[-- Attachment #1: Type: text/plain, Size: 188 bytes --]

Hi!

There is a mistake in dl2k.c network driver added in 2.4.6-ac2.

Without this patch gcc-2.95.3 produces a broken symbol __ucmpdi2, while
gcc-2.96 works ok.

-- 
Good luck,
Konstantin

[-- Attachment #2: linux-2.4.6-ac2-dl2k.patch --]
[-- Type: application/octet-stream, Size: 681 bytes --]

--- linux/drivers/net/dl2k.c_old	Tue Jul 10 19:05:33 2001
+++ linux/drivers/net/dl2k.c	Tue Jul 10 22:11:21 2001
@@ -559,7 +559,7 @@
 			}
 		}
 		/* Free used tx skbuffs */
-		for (; np->cur_tx - np->old_tx > 0; np->old_tx++) {
+		for (; np->cur_tx > np->old_tx; np->old_tx++) {
 			int entry = np->old_tx % TX_RING_SIZE;
 			if (!(np->tx_ring[entry].status & TFDDone))
 				break;
@@ -710,7 +710,7 @@
 	}
 
 	/* Re-allocate skbuffs to fill the descriptor ring */
-	for (; np->cur_rx - np->old_rx > 0; np->old_rx++) {
+	for (; np->cur_rx > np->old_rx; np->old_rx++) {
 		struct sk_buff *skb;
 		entry = np->old_rx % RX_RING_SIZE;
 		/* Dropped packets don't need to re-allocate */

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: PATCH for dl2k driver
  2001-07-10 18:24 PATCH for dl2k driver Konstantin Volckov
@ 2001-07-10 19:17 ` Jeff Garzik
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2001-07-10 19:17 UTC (permalink / raw)
  To: Konstantin Volckov; +Cc: linux-kernel, Alan Cox, Edward Peng.

Konstantin Volckov wrote:
> There is a mistake in dl2k.c network driver added in 2.4.6-ac2.

I doubt you have hardware for it, so no big deal ;-)


> Without this patch gcc-2.95.3 produces a broken symbol __ucmpdi2, while
> gcc-2.96 works ok.

Alan please do not apply, there is a better update coming your way,
including other bug fixes as well.

-- 
Jeff Garzik      | A recent study has shown that too much soup
Building 1024    | can cause malaise in laboratory mice.
MandrakeSoft     |

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2001-07-10 19:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-07-10 18:24 PATCH for dl2k driver Konstantin Volckov
2001-07-10 19:17 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox