[lkml]   [2001]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
Subject[BUG/PATCH] iphase ATM driver (Oops - Div0)

please find attached a fix for a nasty bug in the iphase ATM NIC. If te max_sdu
exceeds the limit the internal structure ia_vcc is freed BUT the pointer in the
vcc struct is NOT been reset. This results in a Div0 error in ia_close
(ctimeout = 300000 / ia_vcc->pcr) - clearing the referencing pointer fixes the
I've also diabled the "Misaligned SKB" check since it works fine w/o it
AND re-aligning is to expensive if it works (I know, jumping to conclusions, but
it works fine on my boxes)...
Could you please - at least - apply the pointer fix?



Till Immanuel Patzschke mailto:
interNetwork AG Phone: +49-(0)611-1731-121
Bierstadter Str. 7 Fax: +49-(0)611-1731-31
D-65189 Wiesbaden Web:

diff -Naur linux-2.4.10/drivers/atm/iphase.c linux-2.4.10-new/drivers/atm/iphase.c
--- linux-2.4.10/drivers/atm/iphase.c Tue Sep 18 07:52:34 2001
+++ linux-2.4.10-new/drivers/atm/iphase.c Tue Nov 13 18:40:49 2001
@@ -1777,8 +1777,9 @@
memset((caddr_t)ia_vcc, 0, sizeof(*ia_vcc));
if (vcc->qos.txtp.max_sdu >
(iadev->tx_buf_sz - sizeof(struct cpcs_trailer))){
- printk("IA: SDU size over the configured SDU size %d\n",
- iadev->tx_buf_sz);
+ printk("IA: SDU size over (%d) the configured SDU size %d\n",
+ vcc->qos.txtp.max_sdu,iadev->tx_buf_sz);
+ INPH_IA_VCC(vcc) = NULL;
return -EINVAL;
@@ -2896,14 +2897,16 @@
return 0;
+#if 0
if ((u32)skb->data & 3) {
- printk("Misaligned SKB\n");
+ printk("Misaligned SKB (0x%p)\n",skb->data);
if (vcc->pop)
vcc->pop(vcc, skb);
return 0;
/* Get a descriptor number from our free descriptor queue
We get the descr number from the TCQ now, since I am using
the TCQ as a free buffer queue. Initially TCQ will be
 \ /
  Last update: 2005-03-22 13:13    [W:0.023 / U:0.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site