Commit 70f5817d authored by Robert Hancock's avatar Robert Hancock Committed by David S. Miller
Browse files

net: axienet: limit minimum TX ring size 



The driver will not work properly if the TX ring size is set to below
MAX_SKB_FRAGS + 1 since it needs to hold at least one full maximally
fragmented packet in the TX ring. Limit setting the ring size to below
this value.

Fixes: 8b09ca82 ("net: axienet: Make RX/TX ring sizes configurable")
Signed-off-by: default avatarRobert Hancock <robert.hancock@calian.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 95978df6

drivers/net/ethernet/xilinx/xilinx_axienet_main.c

+3 −1
Original line number Diff line number Diff line
@@ -43,6 +43,7 @@ 
/* Descriptors defines for Tx and Rx DMA */

#define TX_BD_NUM_DEFAULT		64

#define RX_BD_NUM_DEFAULT		1024

#define TX_BD_NUM_MIN			(MAX_SKB_FRAGS + 1)

#define TX_BD_NUM_MAX			4096

#define RX_BD_NUM_MAX			4096
@@ -1370,7 +1371,8 @@ axienet_ethtools_set_ringparam(struct net_device *ndev, 
	if (ering->rx_pending > RX_BD_NUM_MAX ||

	    ering->rx_mini_pending ||

	    ering->rx_jumbo_pending ||

	    ering->rx_pending > TX_BD_NUM_MAX)

	    ering->tx_pending < TX_BD_NUM_MIN ||

	    ering->tx_pending > TX_BD_NUM_MAX)

		return -EINVAL;



	if (netif_running(ndev))