lkml.org 
[lkml]   [1999]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [linux-tr] SysKonnect driver broken in new kernels?
Date
From
Adam Fritzler wrote:
>
> Looks that the problem is actually in tr_type_trans() (net/802/tr.c). It
> got changed drastically between 2.1.107 (when the driver was put in) and
> now. Pasting the old tr_type_trans() into it and calling it seemingly
> makes it recieve just fine (it even tries to respond to pings now, but it
> fails...). For some reason the ibmtr driver wasn't affected by the
> change; I don't understand that driver's code enough to figure that out.

The ibmtr driver got changed together with the tr_type_trans rewrite.

> Now I can see that transmission is also broken. It sends something, just
> aparently not the right thing. I haven't had a chance to really look into
> it yet. I was happy for one evening to get a single direction going!

The IBMTR driver used to like packets laid out like this:

<header> <padding> <data>
0 0x20

which 1) assumes that headers can't be longer than 20 bytes (*), and
2) makes things problematic for DMA-capable cards that want one
packet-buffer (i.e. without scatter-gather).

This rewrite makes things lots easier for other token ring cards, but
you'll have to adjust to the new layout:

<header> <data>
0 ??

I hope this helps.

Best regards,

Roger.

(*) Which (according to older standards,) indeed used to be true.

--
** R.E.Wolff@BitWizard.nl ** http://www.BitWizard.nl/ ** +31-15-2137555 **
*-- BitWizard writes Linux device drivers for any device you may have! --*
------ Microsoft SELLS you Windows, Linux GIVES you the whole house ------


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:54    [W:0.040 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site