netdev
[Top] [All Lists]

Re: [PATCH] forcedeth: Additional ethtool support

To: Manfred Spraul <manfred@xxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] forcedeth: Additional ethtool support
From: Francois Romieu <romieu@xxxxxxxxxxxxx>
Date: Mon, 11 Jul 2005 01:26:03 +0200
Cc: Jeff Garzik <jgarzik@xxxxxxxxx>, Netdev <netdev@xxxxxxxxxxx>, renaud.lienhart@xxxxxxx
In-reply-to: <42D16656.6000207@xxxxxxxxxxxxxxxx>
References: <42D101EC.6000608@xxxxxxxxxxxxxxxx> <20050710172833.GA1951@xxxxxxxxxxxxxxxxxxxxxxxxxx> <42D16656.6000207@xxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
Manfred Spraul <manfred@xxxxxxxxxxxxxxxx> :
[memcpy_fromio]
> Not for a nic without complete documentation: What if an arch uses 
> 64-bit reads to read two registers at the same time?

So far, no citizen of arch/ does. Afaik it would probably be a bad
idea on pci-x.

> that, for example IIRC natsemi explicitely mandates 32-bit reads.
> x86-64 doesn't, it uses 32-bit reads, but I don't like the idea of using 
> memcpy to read registers.

Ok.

--
Ueimor

<Prev in Thread] Current Thread [Next in Thread>