[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <482085EF.1040005@garzik.org>
Date: Tue, 06 May 2008 12:23:11 -0400
From: Jeff Garzik <jeff@...zik.org>
To: akpm@...ux-foundation.org
CC: netdev@...r.kernel.org, bns_robson@...mail.com
Subject: Re: [patch 5/5] IPv6 to Intel EtherExpress 16 fails - multicast problems
akpm@...ux-foundation.org wrote:
> From: Bruce Robson <bns_robson@...mail.com>
>
> Taken from http://bugzilla.kernel.org/show_bug.cgi?id=10577
>
> I was unable to access a computer containing an Intel EtherExpress 16 network
> card using IPv6.
>
> I traced this to failure of neighbour discovery. When I used an "ip -6 neigh
> add" command, on the computer attempting access, to insert a binding between
> the IPv6 address of the computer with the Intel EtherExpress 16 network card
> and the card's ethernet address, I was able to access that computer using
> IPv6.
>
> Neighbour discovery requires working multicast. The driver sources file
> eexpress.c contains an approximately 30 line function eexp_setup_filter used
> when loading multicast addresses.
>
> I found 3 problems in this function
>
> 1) It wrote the number of multicast addresses to the card instead of the
> number of bytes in the multicast addresses.
>
> 2) When loading multiple multicast addresses it loaded the first one
> provided multiple times instead of loading each one once.
>
> 3) The setting of pointer 'data' from 'dmi->dmi_addr' occured before the
> test for the error situation of 'dmi' being NULL.
>
> Correcting these problems allows the computer with the Intel EtherExpress 16
> network card to found by IPv6 neighbour discovery.
>
> p.s. There is some information on the Intel EtherExpress 16 at
> http://www.intel.com/support/etherexpress/vintage/sb/cs-013500.htm
> Datasheet for the Intel 82586 ethernet controller used by the card
> http://www.datasheetcatalog.com/datasheets_pdf/8/2/5/8/82586.shtml
>
> Signed-off-by: Bruce Robson <bns_robson@...mail.com>
> Cc: Jeff Garzik <jeff@...zik.org>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
>
> drivers/net/eexpress.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
applied
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists