I think it'd be better to create some macro like this:
#ifdef FORCE_SKB_ALIGNMENT
# define SKB_COPY_CHECK(len) len > RX_COPY_THRESHOLD && tp->rx_offset == 2
#else
# define SKB_COPY_CHECK(len) len > RX_COPY_THRESHOLD
#endif
Then replace the code in the if with the new macro. We can define
FORCE_SKB_ALIGNMENT in our build environment as can others if necessary.
otherwise, everyone does what they did before.
Steve
John Partridge wrote:
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.
--
Steve Modica
work: 651-683-3224
MTS-Technical Lead
"Give a man a fish, and he will eat for a day, hit him with a fish and
he leaves you alone" - me
|