From patchwork Mon Jan 16 18:32:54 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexey Bogoslavsky X-Patchwork-Id: 13103580 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B598BC54EBE for ; Mon, 16 Jan 2023 18:41:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235128AbjAPSkz (ORCPT ); Mon, 16 Jan 2023 13:40:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47674 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233090AbjAPSkV (ORCPT ); Mon, 16 Jan 2023 13:40:21 -0500 Received: from esa4.hgst.iphmx.com (esa4.hgst.iphmx.com [216.71.154.42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8890922038 for ; Mon, 16 Jan 2023 10:32:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1673893979; x=1705429979; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=fv9XyMp/AzDOnPrcLZoRjYLWWhxZQDGHAGul877dgxs=; b=hrbZw/cVkU9hIlMffEH12H9Ur+2nlgI0aPi3tCdd/I1NCY0UoJ+gMhax CrauV7wO/ckrEWmTa7y9mNuQFfOCAvOAsumrcFNi8TPYR19Z7ToFqotU1 nviBoaQj+Zwx2Msl33MwnNpOSQ+g/Szcozm4B3/8UzHydw64U/mM+PXo0 ToB/UbWYZ9GRlSf/RM+YcNPPG11r8dHjv75dJjrPs+LEXMr8d/ov2Cutz FLFiQ7wbyGK/UiRbwrbY8ylpaNjfnMgOmMkTm4b/xMHsTUh+9QrUrZccG HNmAhBlmEehs1e/NkOxMhBtz+iljfX9M/yFbT5KslLvcqX/tZnkome2yr g==; X-IronPort-AV: E=Sophos;i="5.97,221,1669046400"; d="scan'208";a="219277022" Received: from mail-bn7nam10lp2104.outbound.protection.outlook.com (HELO NAM10-BN7-obe.outbound.protection.outlook.com) ([104.47.70.104]) by ob1.hgst.iphmx.com with ESMTP; 17 Jan 2023 02:32:58 +0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DsmnfVLCvWhtxnQ6DTxo8qiViIQLDFOM+7cGjqo/jHGwoB6iDDU4PPPXQGyfgzZ3GshSbkaIoAHHGZygPU8/ew8taUT0sEeV620BlimmpiMcWnd8qEKWeejcNUd6AiYGFyCVZuhVyO02dgTe/4KiZq8S4QDoxuO6GOiL0sMKJ9VL8TGlY9CLKVP4EsnmIzgrTOoFYVpDAxt2kzZaxlKSEJ2Boa0VI13zesnyJM/19w9PBzWS6annivG27d9zDpZzWtKHMVZ0MLC+pzapyUQOwPa34GujuYdlX6htGAfwD43tF4wDVAWJWkOwyI6e5ObeXN30bFOFrFNBIYobJgt/JA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QFbXiuwHqj4jUgszCwAPZHguCr/fadSARjDCR7e/SL0=; b=RNHNVixjhuyFUlLg6m+15P0RDEr6xJ07K00WowkpjwhbGsEl73EyF3lBt+ZL/jXyl/UglPN8tut0gxAH/VYvkgPF+0oxbQM6LQRYalaRb9bG9PUxMOOgEmNFGZerbj+X6NMm8SbYD05lIrIA0iIUH7Ewppn9YAnxlTug8eelNfbwSkax/vJVc2lYp9jIFr+TGIPooujY9ba0LDkPwv7fPLq01k4It4Bj3Nr7VseWGYIVRCSyyyWouJSkUA4bCfudAuU7ehb8ip/leVvWR0rGzp9qtR1V1oy0m+ptSe2SkmwoE6tjGBzX8RWnEmtpYcY2n6VEp7WvQpf1Q8exhRm6Mg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wdc.com; dmarc=pass action=none header.from=wdc.com; dkim=pass header.d=wdc.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sharedspace.onmicrosoft.com; s=selector2-sharedspace-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QFbXiuwHqj4jUgszCwAPZHguCr/fadSARjDCR7e/SL0=; b=J3NYCLNCnL1tWBGfdaTqf3F6KNSWy1T/hNVmR1y7fSHYqnvuUwCMeWDwA0vnmWLdFSPFxiEkg6wPE2ZB6jwJvMG+Xgl7TZkcNct88snV5cG1o7G7ZiNaZKLQ/8x1m1GM+4ByfVild4tbYGh6/8a/mirENhktWEDQvgGIIfgb4cc= Received: from DM6PR04MB6473.namprd04.prod.outlook.com (2603:10b6:5:1ef::23) by BN8PR04MB5828.namprd04.prod.outlook.com (2603:10b6:408:aa::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6002.13; Mon, 16 Jan 2023 18:32:55 +0000 Received: from DM6PR04MB6473.namprd04.prod.outlook.com ([fe80::27e5:bc1:9b25:d24c]) by DM6PR04MB6473.namprd04.prod.outlook.com ([fe80::27e5:bc1:9b25:d24c%7]) with mapi id 15.20.5986.023; Mon, 16 Jan 2023 18:32:55 +0000 From: Alexey Bogoslavsky To: "linux-pci@vger.kernel.org" CC: "bhelgaas@google.com" , "'hch@lst.de'" Subject: [PATCH 1/1] PCI/AER: Ignore correctable error reports for SN730 WD SSD Thread-Topic: [PATCH 1/1] PCI/AER: Ignore correctable error reports for SN730 WD SSD Thread-Index: AQHZKdjydF/vmoLqekqeVIf5E6S3mw== Date: Mon, 16 Jan 2023 18:32:54 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=wdc.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM6PR04MB6473:EE_|BN8PR04MB5828:EE_ x-ms-office365-filtering-correlation-id: 358f765e-0414-4339-cd77-08daf7f0156a wdcipoutbound: EOP-TRUE x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: wt0dsaCXFpzt5iuR9zunGjP8z9s7ibCJaHegcMycAM4la+mUNqI3K4NHDNFe3IF2uSz+yVrC0iNiJPzn0bvfYqM8VE1XDunfrIMWfjOO23DlquNCXRdmHW36V7rYfLv7avNqgQz6ZhMCcG/mEGVvugaFSUGhO5sZmgTCUWHZCwOWndgELE0IKTNba3azw/OSOx4Z6fGVoNh3cmQhd4IOpMnyvPvdUCwVzt/bfyPVuWzKcrQjqk8TFqww7fUi8zbfqbVonslkXS46vS2PH9hw3SoMNGeSbgn5OOfINlCLg2TBkxZfah41N/pwHdtunwpiYbP+HqprXKgmbAZnjgsA6/gqgUoJqzozB+32OXNfxnVu7NWVCigjnWPt/zorTtG4yJDGhwYEhKrd3E4e3M71Ir6ceGquFUYPRv4OwwT+mHrQn4wrY4Qb4ge9vtu3n2hOUe3nNF7BLprrUkJ0BTgB7gdyVM0/UTWvDtxHmgPb4dXHweIoOpbROgQbatcQzd3+CGkfXO4DRXsWnWGbYgIuTiiilcAmlxXBrcmsBbFr3Z6FuPWQJzJ0nBfU1ookQlEpeOGwMDP/UB2HaVatZI+AGeheuVfijULQdVYl5mr8b5IvpW/jRFBpbSftEM7a3y8bPeVDKgSytJksOa7zWpbw0UyYQr6CXgyTDL3KADVRHTQM4yB1xjdiVoZq3V2841KAJ7VCR2POFTOEMifyJ7IDWw== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR04MB6473.namprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(136003)(396003)(39860400002)(346002)(366004)(376002)(451199015)(33656002)(66556008)(76116006)(66946007)(66476007)(41300700001)(66446008)(186003)(26005)(9686003)(8676002)(64756008)(4326008)(6916009)(86362001)(5660300002)(83380400001)(52536014)(82960400001)(8936002)(55016003)(54906003)(478600001)(6506007)(7696005)(316002)(71200400001)(38100700002)(2906002)(38070700005)(122000001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: 7n9t1qfZheh6XcLLtPtfbmfUjzw33YK9jVlsJY70pZ2wYd3KCiPR92x0ZHEeJktYkOGu7KcdfrXvb/xqLOEbpXqp1sWSgS3ENWS5DRo4YrfvZIqA1DYgUaLXFDxhDVJimbgTJS2aGQlysnXlHy35Im3ngxrZSsqXg+CaLD1lW1O+e+0JO3gtf9DgdtCKwNfEdoKNGtKYYk2fGiKiG5F71e+OHPYiQo3ahdaFgKGoiNatI07UHmw6FgjTjucRxZ/7N8x4pVV2ZzsUbP6O4kajN57LzCMGnpYy3k/vj7y2/q1J4k6y2KIyjoYFboGeRhH2I7JQEnOWI87O9j6q7su8S4J/QRxDGTVmIeUawuHtt5REJtlWvPj7WxLRSylZcf7GLPZiOHSGeyLBjTC74nTCp+DPsbxMZa32lRRSR+I8cM8m32FAvfOFveViBXO+tiGq6vexKmkkJ/t60qvceS3yZXhL4OKzyVBE4HHHdY9ciTDY1LnhqXqm4f12zzq/VNz598KYfxX2YW72FXcuAxERa0AOfkJXj3OvWYlVbvh+2ZqPcg9I96VGDiV8s8h9Zh1U/UELq53dq4RHsm3bcum7ZxwW/A7N2h/5gUAraS1cgS13FZsC6eGrF2ku+8sxwKVu3MF9P8XblP/cqtT8XXPVFd9VFc8FA426OriXulATR/XmY0yVpNhSuqXsNPfmvSb3D3ikcZiVP97leuNH49UMOVmCTvIisv3xLvHbSUrHykiRXt2zXGNO6CvWzxFlSna0FunErTEwzMtvq47LDe3Rbn5XfrUsWKQKmcUYL6ZYwMoWvke4YHtJGJJqVZRIs+Bahpdf6Fv5Wqi2qo3YvTK5aBxxu8HyN6+OjhIUQ1R+9Y4SZqhvuPNbvkFqFScZMNo1sTWEihntO5Iwzu/vN30GxinqyRzn89B6VPpif0ClX7IqmN0CQbMH6nCxoKQnzvz66EANk3rToEXMNxNogHZO4dwnJFpyJvTSaorx5MEDkyE7CENhw++BFJvfkXE8ZLGdaRgInoVJpkXwR799Ah7kbdOzaIuD0stXWeuVTpRCjBNaH77c+TYyNyjLqUtzW4Z3/UbXhL9PW7OFwFRSC+HL3eVq5zm1JaXEMPiRVlWH/Bk/J16fv2IINTp49u1PJvB+S2uQiZlM4DSs5nJqc58AoXOhaNKQp8fcXCWEBH4OU0doFqxLAw52JWFLpax5hhSl8ALurSB86tKqNny03+E9UhNN1aCNVfHGduap1MVzqtf4rZOOgkT5+p9Scm29vUPsLWEhUwSHUBwd2VDFPWVybry+M3p2ti4wXUqqiG0VU4WgfGDA+v0p8gyYRg82OwWJuVr3jPJy0t/T/Kzl/N03VOXLOx6yHn/cnGBzN2GsmShusVFFODdO4mCYgw9qt14cUQndsk8nFC3ImjOKjsx6d5G5D9j4GMF0feloFvwXJ8Tvhuy9TEEQMIp7GxIt6vbYWxTdORwwe/c0Vvio5Q1/zagd2Cq4ivrAdunJMHHajxehq/nIGBDZ0svP6D9IZNl2ItJItxY9VUkGgC4o+6BXT5yWQ8AqwwanYdnAS3yEszDI1eD/K9UPGkSlY8VbNb8RKjXHb9/9bKwDvoXnqPfn7A== MIME-Version: 1.0 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 6jiFWjjZ07wrRx6bQEuaOrYcR5CtgM7IgvwVYjOWRZerZ4eNbbgxsZAvZ2MbLVCPlnYbOHp8sundzMS6toF4i6YfcNvXmyLgjSFdVCsSU3bAFRAXQqTXIoElu2jukZzouyp0VcJp7k4IzKkXmnpps0ZqSmiacCsKOnglS+FgXp9w9beEF0xvnIXo55EZWg+MV/U6cH05hVVzaPca8qll94L+bRE3m/NfVDDaoddUk+ZUjNNjW4KHOIJjZEZO0CclXUc29hUmJu/NDkVr3UsqnR5y7xWVZMMEzEcvzXyk4mPyC7LEI1fwA5YjBXW1ccGP0ZrgWXal1Fqs27hcmmBblY+EKnOmKWEzmorXah/gXtrsQ9UaI6aDE06peOHR6th7rX0HvvxXT1ziPM0ZXQCOFaPKh+qAQwulLYTA1h15eJpxZJoPC6ASdIWHBC7Uu5nMNSH8rOR81jnTe2ZaYqEnqFnH0T2HIYVGCvKbbA7CsJ457pNMMnPBobmYVJPAPzAHVrNq/4/wlS5WgFYdmCBUTQc/UgoxHxd5jePbWYKayUBlvQYIwpLdzwoiYf46MMY74yDZpziYhBt0+n+BpttewxqfCmnWEtebeGiaotReDZw+O0GzoUdDZ+6hPiUoTZONlUE5ni8OmHXsR4S3BX7PbK6VGwMBAV6ZPdL8/pzoZ5dtn72zZ6aEiGa3hGSR/tRC1Sj6iAQ2071ndGWLWXdl+Q9XLMxt52wCn/8yDLAH46bbnuXRdRsa+JGBmXyI0JyArccXhEk+a8vfQuA4GW6djJ/1bUQhjPyXLnX3CD/zNHWZEHBpXOpRPj3mK++mICBdbazZO+wzwltXl4gXk+obvQ== X-OriginatorOrg: wdc.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6473.namprd04.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 358f765e-0414-4339-cd77-08daf7f0156a X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Jan 2023 18:32:54.9254 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: 294oRrXqiD4nObi7sNnB2WtuHeUBOWFVR1fpHe59/psAZ5gdwf81IYBppvFM3lxyq2mTbwYNdAbMxozCaQ5WWC19qco5DefWUXLIoPQU7OE= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR04MB5828 Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org From: Alexey Bogoslavsky A bug was found in SN730 WD SSD that causes occasional false AER reporting of correctable errors. While functionally harmless, this causes error messages to appear in the system log (dmesg) which, in turn, causes problems in automated platform validation tests. Since the issue can not be fixed by FW, customers asked for correctable error reporting to be quirked out in the kernel for this particular device. The patch was manually verified. It was checked that correctable errors are still detected but ignored for the target device (SN730), and are both detected and reported for devices not affected by this quirk. Signed-off-by: Alexey Bogoslavsky mailto:alexey.bogoslavsky@wdc.com --- drivers/pci/pcie/aer.c | 3 +++ drivers/pci/quirks.c | 6 ++++++ include/linux/pci.h | 1 + include/linux/pci_ids.h | 4 ++++ 4 files changed, 14 insertions(+) diff --git a/drivers/pci/pcie/aer.c b/drivers/pci/pcie/aer.c index d7ee79d7b192..5cc24d28b76d 100644 --- a/drivers/pci/pcie/aer.c +++ b/drivers/pci/pcie/aer.c @@ -721,6 +721,9 @@ void aer_print_error(struct pci_dev *dev, struct aer_err_info *info) goto out; } + if ((info->severity == AER_CORRECTABLE) && dev->ignore_correctable_errors) + return; + layer = AER_GET_LAYER_ERROR(info->severity, info->status); agent = AER_GET_AGENT(info->severity, info->status); diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 285acc4aaccc..21e4972fb242 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -5911,6 +5911,12 @@ DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_INTEL, 0x1536, rom_bar_overlap_defect); DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_INTEL, 0x1537, rom_bar_overlap_defect); DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_INTEL, 0x1538, rom_bar_overlap_defect); +static void wd_ignore_correctable_errors(struct pci_dev *pdev) +{ + pdev->ignore_correctable_errors = true; +} +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_WESTERN_DIGITAL, PCI_DEVICE_ID_WESTERN_DIGITAL_SN730, wd_ignore_correctable_errors); + #ifdef CONFIG_PCIEASPM /* * Several Intel DG2 graphics devices advertise that they can only tolerate diff --git a/include/linux/pci.h b/include/linux/pci.h index adffd65e84b4..1e002be07255 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h @@ -463,6 +463,7 @@ struct pci_dev { unsigned int link_active_reporting:1;/* Device capable of reporting link active */ unsigned int no_vf_scan:1; /* Don't scan for VFs after IOV enablement */ unsigned int no_command_memory:1; /* No PCI_COMMAND_MEMORY */ + unsigned int ignore_correctable_errors:1; /* Ignore correctable errors reported */ unsigned int rom_bar_overlap:1; /* ROM BAR disable broken */ pci_dev_flags_t dev_flags; atomic_t enable_cnt; /* pci_enable_device has been called */ diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index b362d90eb9b0..b5a08a4cf047 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -2299,9 +2299,13 @@ #define PCI_VENDOR_ID_QUICKNET 0x15e2 #define PCI_DEVICE_ID_QUICKNET_XJ 0x0500 +#define PCI_VENDOR_ID_WESTERN_DIGITAL 0x15B7 +#define PCI_DEVICE_ID_WESTERN_DIGITAL_SN730 0x5006 + /* * ADDI-DATA GmbH communication cards mailto:info@addi-data.com */ + #define PCI_VENDOR_ID_ADDIDATA 0x15B8 #define PCI_DEVICE_ID_ADDIDATA_APCI7500 0x7000 #define PCI_DEVICE_ID_ADDIDATA_APCI7420 0x7001