r8169: align number of tx descriptors with vendor driver
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 1 Nov 2020 22:23:52 +0000 (23:23 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 4 Nov 2020 01:34:11 +0000 (17:34 -0800)
Lowest number of tx descriptors used in the vendor drivers is 256 in
r8169. r8101/r8168/r8125 use 1024 what seems to be the hw limit. Stay
on the safe side and go with 256, same as number of rx descriptors.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Link: https://lore.kernel.org/r/a52a6de4-f792-5038-ae2f-240d3b7860eb@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/realtek/r8169_main.c

index 319399a..e08c514 100644 (file)
@@ -67,7 +67,7 @@
 
 #define R8169_REGS_SIZE                256
 #define R8169_RX_BUF_SIZE      (SZ_16K - 1)
-#define NUM_TX_DESC    64      /* Number of Tx descriptor registers */
+#define NUM_TX_DESC    256     /* Number of Tx descriptor registers */
 #define NUM_RX_DESC    256U    /* Number of Rx descriptor registers */
 #define R8169_TX_RING_BYTES    (NUM_TX_DESC * sizeof(struct TxDesc))
 #define R8169_RX_RING_BYTES    (NUM_RX_DESC * sizeof(struct RxDesc))