Browse Source

ag71xx: Remove ___cacheline_aligned from ring structs.

Qualcomm's struct members and inner workings of their driver are all different.
While this might make sense for their driver, it seems to hurt here. In
iperf3, i've seen inconsistent results including a drop of 100mbps on an
Archer C7v4. This patch keeps the results high and relatively consistent.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
Rosen Penev 6 years ago
parent
commit
9414697f92

+ 2 - 2
target/linux/ar71xx/files/drivers/net/ethernet/atheros/ag71xx/ag71xx.h

@@ -159,8 +159,8 @@ struct ag71xx {
 	 * Critical data related to the per-packet data path are clustered
 	 * early in this structure to help improve the D-cache footprint.
 	 */
-	struct ag71xx_ring	rx_ring ____cacheline_aligned;
-	struct ag71xx_ring	tx_ring ____cacheline_aligned;
+	struct ag71xx_ring	rx_ring;
+	struct ag71xx_ring	tx_ring;
 
 	unsigned int            max_frame_len;
 	unsigned int            desc_pktlen_mask;