lkml.org 
[lkml]   [2020]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v18 09/11] PCI/DPC: Expose dpc_process_error(), dpc_reset_link() for use by EDR
    Date
    From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>

    If firmware controls DPC, it is generally responsible for managing the DPC
    capability and events, and the OS should not access the DPC capability.

    However, if firmware controls DPC and both the OS and the platform support
    Error Disconnect Recover (EDR) notifications, the OS EDR notify handler is
    responsible for recovery, and the notify handler may read/write the DPC
    capability until it clears the DPC Trigger Status bit. See [1], sec 4.5.1,
    table 4-6.

    Expose some DPC error handling functions so they can be used by the EDR
    notify handler.

    [1] Downstream Port Containment Related Enhancements ECN, Jan 28, 2019,
    affecting PCI Firmware Specification, Rev. 3.2
    https://members.pcisig.com/wg/PCI-SIG/document/12888
    Link: https://lore.kernel.org/r/ac8816d4d41d0894720660f9b51dbeac0842869d.1583286655.git.sathyanarayanan.kuppuswamy@linux.intel.com
    Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
    Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
    ---
    drivers/pci/pci.h | 2 ++
    drivers/pci/pcie/dpc.c | 12 +++++++++---
    2 files changed, 11 insertions(+), 3 deletions(-)

    diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h
    index 6d09bb22b73d..25265bf80a83 100644
    --- a/drivers/pci/pci.h
    +++ b/drivers/pci/pci.h
    @@ -449,6 +449,8 @@ void aer_print_error(struct pci_dev *dev, struct aer_err_info *info);
    void pci_save_dpc_state(struct pci_dev *dev);
    void pci_restore_dpc_state(struct pci_dev *dev);
    void pci_dpc_init(struct pci_dev *pdev);
    +void dpc_process_error(struct pci_dev *pdev);
    +pci_ers_result_t dpc_reset_link(struct pci_dev *pdev);
    #else
    static inline void pci_save_dpc_state(struct pci_dev *dev) {}
    static inline void pci_restore_dpc_state(struct pci_dev *dev) {}
    diff --git a/drivers/pci/pcie/dpc.c b/drivers/pci/pcie/dpc.c
    index 5870a0f154fc..e9087f5f32ec 100644
    --- a/drivers/pci/pcie/dpc.c
    +++ b/drivers/pci/pcie/dpc.c
    @@ -90,7 +90,7 @@ static int dpc_wait_rp_inactive(struct pci_dev *pdev)
    return 0;
    }

    -static pci_ers_result_t dpc_reset_link(struct pci_dev *pdev)
    +pci_ers_result_t dpc_reset_link(struct pci_dev *pdev)
    {
    u16 cap;

    @@ -201,9 +201,8 @@ static int dpc_get_aer_uncorrect_severity(struct pci_dev *dev,
    return 1;
    }

    -static irqreturn_t dpc_handler(int irq, void *context)
    +void dpc_process_error(struct pci_dev *pdev)
    {
    - struct pci_dev *pdev = context;
    u16 cap = pdev->dpc_cap, status, source, reason, ext_reason;
    struct aer_err_info info;

    @@ -233,6 +232,13 @@ static irqreturn_t dpc_handler(int irq, void *context)
    pci_cleanup_aer_uncorrect_error_status(pdev);
    pci_aer_clear_fatal_status(pdev);
    }
    +}
    +
    +static irqreturn_t dpc_handler(int irq, void *context)
    +{
    + struct pci_dev *pdev = context;
    +
    + dpc_process_error(pdev);

    /* We configure DPC so it only triggers on ERR_FATAL */
    pcie_do_recovery(pdev, pci_channel_io_frozen, dpc_reset_link);
    --
    2.17.1
    \
     
     \ /
      Last update: 2020-03-24 01:27    [W:2.766 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site