From: Eric Dumazet <dada1@cosmosbay.com>
Date: Fri, 11 Jul 2008 23:19:39 +0200

quoted text
> On i386 for example, sizeof(struct netdev_queue) = 0x2c
> 
> Shouldnt we make sure each queue sits on its own cache line on CONFIG_SMP ?

Yes.

quoted text
> Also, dev->_tx and dev->num_tx_queues might share a cache line with
> dev->rx_queue, was it intended ?

Nope.

I've added the ____cacheline_aligned_in_smp annotations to my
tree, thanks!

@@ -463,7 +463,7 @@ struct netdev_queue {
 	struct Qdisc		*qdisc_sleeping;
 	struct list_head	qdisc_list;
 	struct netdev_queue	*next_sched;
-};
+} ____cacheline_aligned_in_smp;
 ...
@@ -641,7 +641,9 @@ struct net_device
 	unsigned char		broadcast[MAX_ADDR_LEN];	/* hw bcast add	*/
 
 	struct netdev_queue	rx_queue;
-	struct netdev_queue	tx_queue ____cacheline_aligned_in_smp;
+
+	struct netdev_queue	*_tx ____cacheline_aligned_in_smp;
+	unsigned int		num_tx_queues;
 	unsigned long		tx_queue_len;	/* Max frames per queue allowed */
 ...
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html