]> git.baikalelectronics.ru Git - kernel.git/commit
r8169: remove member irq_enabled from struct rtl8169_private
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 19 Aug 2020 11:03:13 +0000 (13:03 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Aug 2020 20:03:04 +0000 (13:03 -0700)
commit28a1d13ed27ef985f34aa82b9d1723f9b68ca033
tree927f80dcd965300a26eae62bda04e3e84fbf832d
parent328886489d4d9b4e597fc8266d8421f030cd0728
r8169: remove member irq_enabled from struct rtl8169_private

After making use of the gro_flush_timeout attribute I once got a
tx timeout due to an interrupt that wasn't handled. Seems using
irq_enabled can be racy, and it's not needed any longer anyway,
so remove it. I've never seen a report about such a race before,
therefore treat the change as an improvement.

There's just one small drawback: If a legacy PCI interrupt is used,
and if this interrupt is shared with a device with high interrupt
rate, then we may handle interrupts even if NAPI disabled them,
and we may see a certain performance decrease under high network load.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169_main.c