Jeff Garzik wrote:

Eric Dumazet wrote:

quoted

On Tuesday 29 August 2006 18:45, Kok, Auke wrote:

quoted

     /* TX */
-    struct ixgb_desc_ring tx_ring;
+    struct ixgb_desc_ring tx_ring ____cacheline_aligned;
     unsigned long timeo_start;

On UP, this would only enlarge the size of structure and might consume 
more cache lines...

 I guess ____cacheline_aligned_in_smp is preferred here.

(see include/linux/netdevice.h)

changed in our tree to ____cacheline_aligned_in_smp.

Thanks,

Auke