From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AIpwx4/5laPeM6/vqeQfg54XXcHPDwyI+xMdpZVhce/MJ//MfG1ksowsX9yNN/txT+E+/Zp6wk9I ARC-Seal: i=1; a=rsa-sha256; t=1523284938; cv=none; d=google.com; s=arc-20160816; b=bF3dUzjAO3DjV7USW98Bk0a7YtL7WM6w6q7zvrR1rgt236Tosrh3KlcTqQFofehaUI CFXDKTuTke//oO7PLvPnt0f1PycM14fy7ssN9Mtsgr3fEMJkPiTnJilZAjI+x+IK3OAI 2RrOEzuA/QKXZze4MIrJxgZVK6pP5B52Zkg7cf8Z0rCiC6iSXl37lih0PI9sg5sMrUgb 9/mOCIQce01e/L3xk+X4fzD1PbMqIpiVqmBKZVnzEAQO1vqQQFFcx76w0Wx1ZPUKtVAC +ehOSZPmxckh27Pho/dfLOgN1ouOJ/k9PfU94vexp4/Di0PVzj+jb8yC+yDkcCviXi5v 6I6Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=message-id:date:subject:cc:to:from:arc-authentication-results; bh=NZxf5uBA/LcRQYkOpifrN4y5zrH5mjurmzlJ/pRwKdk=; b=jP0Cm3Fgez8eT0FP8G7K5/XqPX0mt6QMxeq4GQn0g3Qywl0yT4VDsbID0lSW0ejIEZ Qh/kpDq1lh1C/oJvAea+/f0fZaJLUiy7VIYjvADVBZfUHxndsZ2JAcYfWo8yWz+qoUOH eWYAaK4BzYUs7uKrT/4ImgmArV+uWh3DnGJUnaTRg9LM8nt8mJaOHfi1XZDP+WQ1cCZI T9mDPZ1azQfL9NDNz1tvGnkZViW4janx+pnT3ieKRcnHVIN27wKkqIBZYIpEaC5ELqDY LOJ90ymgXgx+mrVAB/Tlk4oAfstfzarp6ew+vd11EMyixVAGrtpclNUmzTBPP8M/HdOZ tRPw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of poza@qualcomm.com designates 199.106.114.251 as permitted sender) smtp.mailfrom=poza@qualcomm.com Authentication-Results: mx.google.com; spf=pass (google.com: domain of poza@qualcomm.com designates 199.106.114.251 as permitted sender) smtp.mailfrom=poza@qualcomm.com X-IronPort-AV: E=Sophos;i="5.48,427,1517904000"; d="scan'208";a="430754336" X-IronPort-AV: E=McAfee;i="5900,7806,8857"; a="134613312" From: Oza Pawandeep To: Bjorn Helgaas , Philippe Ombredanne , Thomas Gleixner , Greg Kroah-Hartman , Kate Stewart , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Dongdong Liu , Keith Busch , Wei Zhang , Sinan Kaya , Timur Tabi Cc: Oza Pawandeep Subject: [PATCH v13 0/6] Address error and recovery for AER and DPC Date: Mon, 9 Apr 2018 10:41:48 -0400 Message-Id: <1523284914-2037-1-git-send-email-poza@codeaurora.org> X-Mailer: git-send-email 2.7.4 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1597280027023020186?= X-GMAIL-MSGID: =?utf-8?q?1597280027023020186?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: This patch set brings in error handling support for DPC The current implementation of AER and error message broadcasting to the EP driver is tightly coupled and limited to AER service driver. It is important to factor out broadcasting and other link handling callbacks. So that not only when AER gets triggered, but also when DPC get triggered (for e.g. ERR_FATAL), callbacks are handled appropriately. DPC should behave identical to AER as far as error handling is concerned. DPC should remove the devices and not to do recovery for hotplug enabled system. Changes since v13: Bjorn's and Keith's Comments addressed. > Made DPC and AER error handling identical > hanldled cases for hotplug enabled system differently. Changes since v11: Bjorn's comments addressed. > rename pcie-err.c to err.c > removed EXPORT_SYMBOL > made generic find_serivce function in port driver. > removed mutex patch as no need to have mutex in pcie_do_recovery > brough in DPC_FATAL in aer.h > so now all the error codes (AER and DPC) are unified in aer.h Changes since v10: Christoph Hellwig's, David Laight's and Randy Dunlap's comments addressed. > renamed pci_do_recovery to pcie_do_recovery > removed inner braces in conditional statements. > restrctured the code in pci_wait_for_link > EXPORT_SYMBOL_GPL Changes since v9: Sinan's comments addressed. > bool active = true; unnecessary variable removed. Changes since v8: Fixed Kbuild errors. Changes since v7: Rebased the code on pci master > https://kernel.googlesource.com/pub/scm/linux/kernel/git/helgaas/pci Changes since v6: Sinan's and Stefan's comments implemented. > reordered patch 6 and 7 > cleaned up Changes since v5: Sinan's and Keith's comments incorporated. > made separate patch for mutex > unified error repotting codes into driver/pci/pci.h > got rid of wait link active/inactive and made generic function in driver/pci/pci.c Changes since v4: Bjorn's comments incorporated. > Renamed only do_recovery. > moved the things more locally to drivers/pci/pci.h Changes since v3: Bjorn's comments incorporated. > Made separate patch renaming generic pci_err.c > Introduce pci_err.h to contain all the error types and recovery > removed all the dependencies on pci.h Changes since v2: Based on feedback from Keith: " When DPC is triggered due to receipt of an uncorrectable error Message, the Requester ID from the Message is recorded in the DPC Error Source ID register and that Message is discarded and not forwarded Upstream. " Removed the patch where AER checks if DPC service is active Changes since v1: Kbuild errors fixed: > pci_find_dpc_dev made static > ras_event.h updated > pci_find_aer_service call with CONFIG check > pci_find_dpc_service call with CONFIG check Oza Pawandeep (6): PCI/AER: Rename error recovery to generic PCI naming PCI/AER: Factor out error reporting from AER PCI/PORTDRV: Implement generic find service PCI/DPC: Unify and plumb error handling into DPC PCI: Unify wait for link active into generic PCI PCI/DPC: Do not do recovery for hotplug enabled system drivers/pci/hotplug/pciehp_hpc.c | 20 +-- drivers/pci/pci.c | 29 ++++ drivers/pci/pci.h | 5 + drivers/pci/pcie/Makefile | 2 +- drivers/pci/pcie/aer/aerdrv.h | 30 ---- drivers/pci/pcie/aer/aerdrv_core.c | 317 +--------------------------------- drivers/pci/pcie/err.c | 340 +++++++++++++++++++++++++++++++++++++ drivers/pci/pcie/pcie-dpc.c | 107 +++++++++--- drivers/pci/pcie/portdrv.h | 2 + drivers/pci/pcie/portdrv_core.c | 45 +++++ include/linux/aer.h | 2 + 11 files changed, 509 insertions(+), 390 deletions(-) create mode 100644 drivers/pci/pcie/err.c -- 2.7.4