OK, fair enough, you mean like :-
#if defined(__ia64__)
if (len > RX_COPY_THRESHOLD && tp->rx_offset == 2) {
#else
if (len > RX_COPY_THRESHOLD) {
#endif
David S. Miller wrote:
On Wed, 08 Oct 2003 12:52:57 -0500
John Partridge <johnip@xxxxxxx> wrote:
Not too much different
The problem is that your change is arch-dependant yet you make it
run on all platforms.
On x86 we don't want to do what your change is doing, the unaligned
accesses are cheap enough.
We need to abstract this, probably in the same way it is done in
the Tulip and other drivers which have similar issues.
--
John Partridge
Silicon Graphics Inc
Tel: 651-683-3428
Vnet: 233-3428
E-Mail: johnip@xxxxxxx
|