lkml.org 
[lkml]   [2019]   [Aug]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.2 41/56] r8169: dont use MSI before RTL8168d
    Date
    From: Heiner Kallweit <hkallweit1@gmail.com>

    [ Upstream commit 003bd5b4a7b4a94b501e3a1e2e7c9df6b2a94ed4 ]

    It was reported that after resuming from suspend network fails with
    error "do_IRQ: 3.38 No irq handler for vector", see [0]. Enabling WoL
    can work around the issue, but the only actual fix is to disable MSI.
    So let's mimic the behavior of the vendor driver and disable MSI on
    all chip versions before RTL8168d.

    [0] https://bugzilla.kernel.org/show_bug.cgi?id=204079

    Fixes: 6c6aa15fdea5 ("r8169: improve interrupt handling")
    Reported-by: Dušan Dragić <dragic.dusan@gmail.com>
    Tested-by: Dušan Dragić <dragic.dusan@gmail.com>
    Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/net/ethernet/realtek/r8169.c | 9 +++++++--
    1 file changed, 7 insertions(+), 2 deletions(-)

    --- a/drivers/net/ethernet/realtek/r8169.c
    +++ b/drivers/net/ethernet/realtek/r8169.c
    @@ -7050,13 +7050,18 @@ static int rtl_alloc_irq(struct rtl8169_
    {
    unsigned int flags;

    - if (tp->mac_version <= RTL_GIGA_MAC_VER_06) {
    + switch (tp->mac_version) {
    + case RTL_GIGA_MAC_VER_02 ... RTL_GIGA_MAC_VER_06:
    rtl_unlock_config_regs(tp);
    RTL_W8(tp, Config2, RTL_R8(tp, Config2) & ~MSIEnable);
    rtl_lock_config_regs(tp);
    + /* fall through */
    + case RTL_GIGA_MAC_VER_07 ... RTL_GIGA_MAC_VER_24:
    flags = PCI_IRQ_LEGACY;
    - } else {
    + break;
    + default:
    flags = PCI_IRQ_ALL_TYPES;
    + break;
    }

    return pci_alloc_irq_vectors(tp->pci_dev, 1, 1, flags);

    \
     
     \ /
      Last update: 2019-08-08 21:26    [W:4.102 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site