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)
commit9e89d71911b6b312b06b654caa7c0c95197fecb1
tree927f80dcd965300a26eae62bda04e3e84fbf832d
parent52dbe8465e5b85263a4c28a48bc902e8c50b2f1b
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