From patchwork Mon Jun 22 12:05:25 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shiju Jose X-Patchwork-Id: 191318 Delivered-To: patch@linaro.org Received: by 2002:a54:3249:0:0:0:0:0 with SMTP id g9csp134279ecs; Mon, 22 Jun 2020 05:07:38 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzSS3llj2etbPvADgXMiEoe5qAKv5TBbkNsK7CV1gLkiI4RWn2zoa51bK/Muj96BvIuyzpB X-Received: by 2002:a17:906:7802:: with SMTP id u2mr15852428ejm.478.1592827658198; Mon, 22 Jun 2020 05:07:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1592827658; cv=none; d=google.com; s=arc-20160816; b=nbKj671i6CHY8F15ElX1MnA4ncg35Kp7f/7rdHVGsr332pM4m8cnOIa18ySs8vuYS+ IDUb5EvGxncByQHgAzVF1q4Hu/1By6k3NF0k6pmsfpYUJlCPeDZiYK/sFy6KIDY61ioX 9lrrMf/TwbmSaNJyF3wlgSTCJUnvmLPGYJhXze6WffZrH+yecIB/oeOQYX5tYvzBZgvb MiyYiYf90Wx2C2xZ/zoLR8Tc90EXvea3i0uI0cjPstL7+WdUlvElFEYv5ikwa9lOT2hm PMvo4n803u2J3SNqxaR1l95QndTVIW/2koWLsuhKWDNLzaMRB6VQz66v9JKta9UrbeZ5 pzGg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=Ks13dKA2vAldJaldZlWMjAu0qFEkCN1+HiE5WTTQTnI=; b=HR4baD26O9IIKfY1+OP4kSo4g5cRaFZQIgM3E3eLLN3NvEBQVLzxHEbYJ8nt+oyTCq rZQbEEiDAO3vXnYjSDQQcMQxt6M4t14lWP0CAwk0DLA1nCAMBloh5kw2Vjs+DlfiUaVF VUaAtB5vwkJujtFnoNbv5T/b7Vos8ZK2OlTNfHfanz+G1sTSgJeMtCzRGrTqEEa9vf5J B76c9iQHdNk2lIHFgMULBy37LLmGNadzMA4CQW/3ldUMn2VCL9btrfILXTSx1+I4daoQ MY/OQnz5vY9qU+b3aCF/Xwo9dhyxFCAs5tSJAQ4qn/6122S7ZqYszgVjao3ZPMRwlAku GBqg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-acpi-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-acpi-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ce7si1639854edb.4.2020.06.22.05.07.38; Mon, 22 Jun 2020 05:07:38 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-acpi-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-acpi-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-acpi-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728098AbgFVMHh (ORCPT + 7 others); Mon, 22 Jun 2020 08:07:37 -0400 Received: from lhrrgout.huawei.com ([185.176.76.210]:2352 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727864AbgFVMHg (ORCPT ); Mon, 22 Jun 2020 08:07:36 -0400 Received: from lhreml715-chm.china.huawei.com (unknown [172.18.7.107]) by Forcepoint Email with ESMTP id D5E64B3F874B0C45841F; Mon, 22 Jun 2020 13:07:34 +0100 (IST) Received: from DESKTOP-6T4S3DQ.china.huawei.com (10.47.81.228) by lhreml715-chm.china.huawei.com (10.201.108.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1913.5; Mon, 22 Jun 2020 13:07:34 +0100 From: Shiju Jose To: , , , , , , , , , , , , , CC: , , , Subject: [PATCH v11 0/2] ACPI / APEI: Add support to notify the vendor specific HW errors Date: Mon, 22 Jun 2020 13:05:25 +0100 Message-ID: <20200622120527.690-1-shiju.jose@huawei.com> X-Mailer: git-send-email 2.26.0.windows.1 MIME-Version: 1.0 X-Originating-IP: [10.47.81.228] X-ClientProxiedBy: lhreml706-chm.china.huawei.com (10.201.108.55) To lhreml715-chm.china.huawei.com (10.201.108.66) X-CFilter-Loop: Reflected Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org CPER records describing a firmware-first error are identified by GUID. The ghes driver currently logs, but ignores any unknown CPER records. This prevents describing errors that can't be represented by a standard entry, that would otherwise allow a driver to recover from an error. The UEFI spec calls these 'Non-standard Section Body' (N.2.3 of version 2.8). patch set 1. add a notifier chain for these non-standard/vendor-records in the ghes driver. 2. add a driver to handle HiSilicon hip PCIe controller's errors. Changes: V11: 1. Following modifications made by James Morse in the APEI patch for the vendor error record. - Removed kfifo and ghes_gdata_pool. Expanded commit message. Note: Kept the Signed-off-by: James Morse as he is given because I am not sure the right format. 2. Changes in the HIP PCIe error handler driver for the comments by Andy Shevchenko. V10: 1. Changes for Bjorn's comments on HIP PCIe error handler driver and APEI patch. 2. Changes in the HIP PCIe error handler driver for the feedbacks by Andy Shevchenko. V9: 1. Fixed 2 improvements suggested by the kbuild test robot. 1.1 Change ghes_gdata_pool_init() as static function. 1.2. Removed using buffer to store the error data for logging in the hisi_pcie_handle_error() V8: 1. Removed reporting the standard errors through the interface because of the conflict with the recent patches in the memory error handling path. 2. Fix comments by Dan Carpenter. V7: 1. Add changes in the APEI driver suggested by Borislav Petkov, for queuing up all the non-fatal HW errors to the work queue and notify the registered kernel drivers from the bottom half using blocking notifier, common interface for both standard and vendor-spcific errors. 2. Fix for further feedbacks in v5 HIP PCIe error handler driver by Bjorn Helgaas. V6: 1. Fix few changes in the patch subject line suggested by Bjorn Helgaas. V5: 1. Fix comments from James Morse. 1.1 Changed the notification method to use the atomic_notifier_chain. 1.2 Add the error handled status for the user space. V4: 1. Fix for the following smatch warning in the PCIe error driver, reported by kbuild test robot: warn: should '((((1))) << (9 + i))' be a 64 bit type? if (err->val_bits & BIT(HISI_PCIE_LOCAL_VALID_ERR_MISC + i)) ^^^ This should be BIT_ULL() because it goes up to 9 + 32. V3: 1. Fix the comments from Bjorn Helgaas. V2: 1. Changes in the HiSilicon PCIe controller's error handling driver for the comments from Bjorn Helgaas. 2. Changes in the APEI interface to support reporting the vendor error for module with multiple devices, but use the same section type. In the error handler will use socket id/sub module id etc to distinguish the device. V1: 1. Fix comments from James Morse. 2. add driver to handle HiSilicon hip08 PCIe controller's errors, which is an application of the above interface. Shiju Jose (1): ACPI / APEI: Add a notifier chain for unknown (vendor) CPER records Yicong Yang (1): PCI: hip: Add handling of HiSilicon HIP PCIe controller errors drivers/acpi/apei/ghes.c | 63 +++++ drivers/pci/controller/Kconfig | 8 + drivers/pci/controller/Makefile | 1 + drivers/pci/controller/pcie-hisi-error.c | 327 +++++++++++++++++++++++ include/acpi/ghes.h | 27 ++ 5 files changed, 426 insertions(+) create mode 100644 drivers/pci/controller/pcie-hisi-error.c -- 2.17.1