r8169: disable interrupt source RxOverflow

Vendor driver calls this bit RxDescUnavail. All we do in the interrupt
handler in this case is scheduling NAPI. If we should be out of
RX descriptors, then NAPI is scheduled anyway. Therefore remove this
interrupt source. Tested on RTL8168h.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Sunil Goutham <sgoutham@marvell.com>
Link: https://lore.kernel.org/r/9b2054b2-0548-4f48-bf91-b646572093b4@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
index 7b9e04884575e06dffa94ea8bae6e4615801e0e5..5c4296daf8f72462b6e2f46a6ffa9c63f75d5827 100644 (file)
--- a/drivers/net/ethernet/realtek/r8169_main.c
+++ b/drivers/net/ethernet/realtek/r8169_main.c
@@ -5086,12 +5086,10 @@ static void rtl_set_irq_mask(struct rtl8169_private *tp)
        tp->irq_mask = RxOK | RxErr | TxOK | TxErr | LinkChg;
 
        if (tp->mac_version <= RTL_GIGA_MAC_VER_06)
-               tp->irq_mask |= SYSErr | RxOverflow | RxFIFOOver;
+               tp->irq_mask |= SYSErr | RxFIFOOver;
        else if (tp->mac_version == RTL_GIGA_MAC_VER_11)
                /* special workaround needed */
                tp->irq_mask |= RxFIFOOver;
-       else
-               tp->irq_mask |= RxOverflow;
 }
 
 static int rtl_alloc_irq(struct rtl8169_private *tp)