| To: | "David S. Miller" <davem@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [ANN] removal of certain net drivers coming soon: eepro100, xircom_tulip_cb, iph5526 |
| From: | Russell King <rmk+lkml@xxxxxxxxxxxxxxxx> |
| Date: | Fri, 28 Jan 2005 00:14:30 +0000 |
| Cc: | jgarzik@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, greg@xxxxxxxxx, akpm@xxxxxxxx |
| In-reply-to: | <20050127153114.72be03e2.davem@davemloft.net>; from davem@davemloft.net on Thu, Jan 27, 2005 at 03:31:14PM -0800 |
| Mail-followup-to: | "David S. Miller" <davem@xxxxxxxxxxxxx>, jgarzik@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, greg@xxxxxxxxx, akpm@xxxxxxxx |
| References: | <41F952F4.7040804@pobox.com> <20050127225725.F3036@flint.arm.linux.org.uk> <20050127153114.72be03e2.davem@davemloft.net> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.2.5.1i |
On Thu, Jan 27, 2005 at 03:31:14PM -0800, David S. Miller wrote: > Therefore the only missing sync would be of the new RX descriptor > when linking things in like that, ie. at the end of e100_rx_alloc_skb() > if rx->prev->skb is non-NULL. And that is inherently unsafe, since the chip may be modifying the RFD while the CPU is accessing it. Adding extra sync calls does not fix it because as far as I can see, there's nothing to tell the chip "don't write to this RFD because any changes the CPU is making right now will overwrite the chips own changes." The fact of the matter is that eepro100.c works on ARM, e100.c doesn't. There's a message from me back on 30th June 2004 at about 10:30 BST on this very list which generated almost no interest from anyone... -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: 2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/ 2.6 Serial core |
| Previous by Date: | Re: on the wire behaviour of TSO on/off is supposed to be the same yes?, Rick Jones |
|---|---|
| Next by Date: | Re: Memory leak in 2.6.11-rc1?, Russell King |
| Previous by Thread: | Re: [ANN] removal of certain net drivers coming soon: eepro100, xircom_tulip_cb, iph5526, David S. Miller |
| Next by Thread: | Re: [ANN] removal of certain net drivers coming soon: eepro100, xircom_tulip_cb, iph5526, Lennert Buytenhek |
| Indexes: | [Date] [Thread] [Top] [All Lists] |