lkml.org 
[lkml]   [2015]   [Sep]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.2 049/134] powerpc/eeh: Probe after unbalanced kref check
    Date
    4.2-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Daniel Axtens <dja@axtens.net>

    commit e642d11bdbfe8eb10116ab3959a2b5d75efda832 upstream.

    In the complete hotplug case, EEH PEs are supposed to be released
    and set to NULL. Normally, this is done by eeh_remove_device(),
    which is called from pcibios_release_device().

    However, if something is holding a kref to the device, it will not
    be released, and the PE will remain. eeh_add_device_late() has
    a check for this which will explictly destroy the PE in this case.

    This check in eeh_add_device_late() occurs after a call to
    eeh_ops->probe(). On PowerNV, probe is a pointer to pnv_eeh_probe(),
    which will exit without probing if there is an existing PE.

    This means that on PowerNV, devices with outstanding krefs will not
    be rediscovered by EEH correctly after a complete hotplug. This is
    affecting CXL (CAPI) devices in the field.

    Put the probe after the kref check so that the PE is destroyed
    and affected devices are correctly rediscovered by EEH.

    Fixes: d91dafc02f42 ("powerpc/eeh: Delay probing EEH device during hotplug")
    Cc: Gavin Shan <gwshan@linux.vnet.ibm.com>
    Signed-off-by: Daniel Axtens <dja@axtens.net>
    Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com>
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    arch/powerpc/kernel/eeh.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    --- a/arch/powerpc/kernel/eeh.c
    +++ b/arch/powerpc/kernel/eeh.c
    @@ -1116,9 +1116,6 @@ void eeh_add_device_late(struct pci_dev
    return;
    }

    - if (eeh_has_flag(EEH_PROBE_MODE_DEV))
    - eeh_ops->probe(pdn, NULL);
    -
    /*
    * The EEH cache might not be removed correctly because of
    * unbalanced kref to the device during unplug time, which
    @@ -1142,6 +1139,9 @@ void eeh_add_device_late(struct pci_dev
    dev->dev.archdata.edev = NULL;
    }

    + if (eeh_has_flag(EEH_PROBE_MODE_DEV))
    + eeh_ops->probe(pdn, NULL);
    +
    edev->pdev = dev;
    dev->dev.archdata.edev = edev;




    \
     
     \ /
      Last update: 2015-09-27 00:41    [W:4.188 / U:1.144 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site