From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932555AbcHIV4o (ORCPT ); Tue, 9 Aug 2016 17:56:44 -0400 Received: from mail-by2nam01on0085.outbound.protection.outlook.com ([104.47.34.85]:48896 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932338AbcHIV4m (ORCPT ); Tue, 9 Aug 2016 17:56:42 -0400 Authentication-Results: spf=fail (sender IP is 192.88.158.2) smtp.mailfrom=nxp.com; vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=fail action=none header.from=nxp.com; From: York Sun To: CC: , , , , York Sun , Doug Thompson , , Subject: [Patch v4 1/9] driver/edac/mpc85xx_edac: Drop setting/clearing RFXE bit in HID1 Date: Tue, 9 Aug 2016 14:55:38 -0700 Message-ID: <1470779760-16483-2-git-send-email-york.sun@nxp.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1470779760-16483-1-git-send-email-york.sun@nxp.com> References: <1470779760-16483-1-git-send-email-york.sun@nxp.com> X-EOPAttributedMessage: 0 X-Matching-Connectors: 131152534005970243;(91ab9b29-cfa4-454e-5278-08d120cd25b8);() X-Forefront-Antispam-Report: CIP:192.88.158.2;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(7916002)(2980300002)(1110001)(1109001)(339900001)(189002)(199003)(54534003)(356003)(105606002)(8666005)(48376002)(2351001)(106466001)(87936001)(229853001)(5003940100001)(33646002)(104016004)(189998001)(97736004)(110136002)(77096005)(92566002)(2950100001)(586003)(305945005)(19580405001)(36756003)(50986999)(69596002)(50226002)(8676002)(7846002)(4326007)(2906002)(85426001)(47776003)(8936002)(86362001)(50466002)(575784001)(68736007)(76176999)(81156014)(81166006)(19580395003)(7059030);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR0301MB1199;H:az84smr01.freescale.net;FPR:;SPF:Fail;PTR:InfoDomainNonexistent;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11OLC012;1:c+YWHU88RA1OgNmDmkv0R9DLX/aRS51ZYB0s6StFP2cRalXPGECBqagSnLVLNT2ZCp4g4bQWJSxOwnxgiKS9OX72Dv68RlleCKxwVt95F0YaXY0YBBCq8g6SkF6tjWIKxnXzZ3vNLWcpnTD66iEf+Jxf5O5s/pfUvh+KF2GNwMp033AFQIWwD/DdSZGsO7oJJxvo1B/ghEXlgwzYbO7pYbqt4Vb6f2b5km5hoIk2fSjJRijKjGqBIhhqsJ1+Si5dqlJI+/sEBf8D9kTTo7cbkFAtUcvdlPujd5zUwLBvzvf/XD2DRJ7B3M3qA7hwJqxMYiIpIAnstaHfX5HczsJmjsZHB0Urwm5fnFok9vfDPwEbB3LmClzZnkUZe1bzHUflcYNGSqiCdDS54NZrXtSX5elsJ+6tDdf/r42kCkbtr3qxuYlM3piLixUC2fM4h2KXrCNVNn/mSWIBlQMfPh49eAJDkX7kPemucimQmyQRCIzU43Ul4gtp8hawAxt8McFr5hRNBl3ynowysEHHOBKRAwd8pNbPZmlFzTxpxQrmkp/8fEy8yb71R46gX5Cly/Z909zHOORFSee984YIkI5E6g== MIME-Version: 1.0 Content-Type: text/plain X-MS-Office365-Filtering-Correlation-Id: 09378aa3-0b56-43f3-13e6-08d3c0a00b46 X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1199;2:7Dtuj5pfioNxPaUf0k6tS2AYSKFkzf/+u3gssh1nXVORG5lWxhUZdLJIuk7V+Irn4jLxnhTwguvSyY6TZCdb050oDqgUbhBeds7LvJi0K6kBo0ZjZAPsfNH0Ee+Pp2pZk4Hai7mjA6DCcYraw/9LilbIfKhELaeSY6qZdZVYqUX7OF1dewhOeN+mohsTGzXm;3:NtVYhU7J9kRMn/T71eV4W8BzkRYXfq5SOxdE79fOY467244b12epku0+8NrLdifYchO3S4MfcSA04n6HGmY1XonyppOOfHGPa/j9kO38311JkQbLFV8qtiGiVH5U1s8gonZd6bQVzu0vMNTAHEqfA5ozK1SPHNZRNwtESqBCjw5bpGQ+H4G0uOV22+ZrMYzdUqlMStmg6naQRisCHw2PxfIQrNf4giEBvCQMe7PRJzI=;25:1n0gYqhFcoTg4jrgX5qM+ORddMUgkOW86RXekLetTQ6Qg5oGfsqlF2dcO1ezw+rC6h3cqFrax608bddkrdUl9yiJnBkQpnquq5oYqXXocFVn3+5bltUgW4SmZ2s7sQO9J7Ut7bjr6vgz+wiE2SwqzWGCQ41tyham8DlKf2scdEh2EkejD8J5blDTHs+bbckBN+CThZ5p/sQ3k6nXo4Ql/xiK8W5UGgM+A5AT0KJckfEu2mLefaZu2v4xCtmajD7STUmUvIuuZ0e7wTY5MGzZCcuKmfrDAh9jO/dtuhqzc69cUtCLcMa9xEKw2ufpDFvpo20cwhHPJKgoyJOko4y0Z7hFqZY7xB1BFwYlsoD0MP0BCEB9HJ9I2G+AHg8dKSajmi45llQ6sQseqoXJn3UzvbVqN8fiCPwKmBGOhyCnIAE= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0301MB1199; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1199;31:FyK6Kww2QyUZxhJY47gjrIurLo/XfBBgbosvQHgefE2gLXQgzlPYEIUV7ieNRlZtVO7yqISK3lyae1S1WClIiyNLsoNzH0MD67NNt7gn/Vv1KyaD/8ukl+3mS+58VTESoIFUexpDpbN6SX9hQH5ONlIZZfb3UsxquBInN1u+bsZUz3dZifbRNFY9AbbESqkRsAylnCKVi8x4XSRzYkJh4fRoXSmjGlhx3mUU8LzHYfI=;4:3zLoY8xfURmztNaAG+jNXld/YNF7Dzq5ExPbBl63dvK80mKbXlzBxfxTazkBUF4RrD9IKgnc2kf/4Q4S51+rlGgRIyemQh6Zeo0Irt3vTt9ghB5Hgra1AsmRhM/vZCxNXx5BWEQ2qy3OzM3iyZ2tXMu6qaxfhB45T4kokOl54G8QdXK2N8Gfq8Y6/ue3IM0jmkPAIGCJ9WK21quUvADjr3vccBFLLUhGLiCE2+cCwQdpW0LViNiWc63x9W1XOXFxX41KfmFQX42TCUSUgfr0lNuJjYyxYc0aP6ffa6I6uzsxdH2Nt207MFznr1n5OQzS7nSK65dvIQJWPhL5DI5LSsuBo4zF13oij8nqQMzjQAq0vCrumABzMjVL5HhIQN1dSlBNaLhLVSV8nnxquamstDGaJ2NMwjpnCL3l5RqNhQZhLYdaFIV+ztHCdimpoXPF4LohMKG0zbaUfsf5Suy7ifIQkqGb1xjBeuokbMCAEXno8j4LP1uNZJJgMK3jgNelJRrg8YDnrb87noR0pL6pEg== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(13023025)(13024025)(13015025)(13018025)(13017025)(5005006)(8121501046)(3002001)(10201501046)(6055026);SRVR:DM2PR0301MB1199;BCL:0;PCL:0;RULEID:(400006);SRVR:DM2PR0301MB1199; X-Forefront-PRVS: 0029F17A3F X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DM2PR0301MB1199;23:bSn22PFIJzOKhIzpPh0TEKJ5fqMnZgfH2kvIOj1?= =?us-ascii?Q?VQnP+O2DE+74C5mrTsxxY9tTUI+w80Wdyb3IOj67ncIuXTzdSdK5O8+qJ5h/?= =?us-ascii?Q?xIWdJ08bZOY+CORiaC/aHsWJwvShnWkY4sgsNR0uEKA93YdXF/SbhL25yH7I?= =?us-ascii?Q?ATg5d+WuAxK75ZgyKcjTBYrmaJnQWiMf5pVLK5Qa3i/AklYyLBIkU9J1/iyO?= =?us-ascii?Q?/SWowR5aXa2BzKtw0oWS5fBtJrS2mLtGFv18K3IABi6i39hbwNzxoVv/O5Th?= =?us-ascii?Q?7bZNmPyBrJPxBWjYUrctRkaWdsTXUBUvmfwunluzbCE4T1cvZpXFZ9lLDHle?= =?us-ascii?Q?OYauxYFxJlm1N/evBJiwajO6DUzroJdSvBGR4HLQPs91lb9gdqVdGY59uAMp?= =?us-ascii?Q?4aVzMRsnsbgAtR4Ea/rIFg1aONu6fnr2UXUxobfrYp21UjfhWwcpmdl5VoGu?= =?us-ascii?Q?munP8ktR/uW/4ONE/sUEP5hL6XSzHRH16apKfBdliVSmZK7swN00rjZKE1nL?= =?us-ascii?Q?1eBLVrATWgE0FzI5nYWATlAsV13wEFezgg6glJXQlDkIAjofoiVADwvJ9Az7?= =?us-ascii?Q?taNrhDO0xfJT84FAMkeqbabeDQfoWSsu9Yexf3ikTYkndhzeXNcn2hzxLsHG?= =?us-ascii?Q?KOVPoWZ5n7LgVtACprnfACJpWVhyNE+RKZebqxfFHTlvuC4Xdq/9y1KtEF2z?= =?us-ascii?Q?l66+Jqtan0p+ZAH6MAFQqzQnMKyrxfi5fQqs1Lu1s/mqfsastBa04eGImWlR?= =?us-ascii?Q?20izEfCilJeJ0VL/YzO/hLhXYTZcZx6cBvPfHg4aZhs9Aei6n6dGmhOnaWQS?= =?us-ascii?Q?Q0qLlcnL0qprl/3CnkMP0klJgLEaSWeZ7sBzTijnnPjagie3XPKvVppMOEZ6?= =?us-ascii?Q?FDCoUpkGgbIzBauEIGl2sVXvnOK5toXxIk/ud8NlUmIA/S0MJiptVSHmzS0s?= =?us-ascii?Q?gR6ibQhphRDFWT+6UtYTjUruefLDrN2+/bP7ajQ6BGsf5Fkx3Pgt6kmUknOf?= =?us-ascii?Q?AjgtuTX/S3plDyNa8QLrTK7r1+jttrWUnL3IFXKxJNav++tPsC2wvxK1SkYP?= =?us-ascii?Q?tMFwn54IZ7ssC+ykriiJtb/E0qqDgnr/BO+W5LX0KyO3UoZDypFqQZRN3yNJ?= =?us-ascii?Q?LI+hbJDHN3MgaRLAh6B9FG03DxXktnpU/5vCFZ+cGxdC/qz7ifAPcWVTgQgg?= =?us-ascii?Q?4aeRfBWmRmbw01o8=3D?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB1199;6:G1BkoBC6WpltdaUriyMEm231fiov3Untom4IagbPiEXDAUx2RbqnrGkrePHGkhFDs9hCBvYjS5Et6PXbKGvn51YTZAdroA7ZRFLDz1HuxSLx7Gp7yk2OSLRGZ9sPhXGJZwcGNKgnfhXNfw5bportLLvn6h/odct8NrwRg+VfK9kAiBkWMdk+mdO9J+LP/ygSqFN+4S0u4aB7rcE8Swh31p4DimOV4ibk1sR1j2FMCBPjSjz8w8oGp7t72Jw2ZQx4Rm+RcaPXZobf8E65dBS6SKIbNiNvxl9t6LPg+GZnd9I=;5:pmeOh/uAn+Iy6gkiZFoAjhmNRQoZAhKu4mvPFR0nSIQD64nh0UgnZ6z8hz7b8kIUjpYYreMD5sIHmWGim5LDBPgqGduffxU/n5zxKoI5CcWNi0lCo9oJ0XQu6sxpP4qRRr1HVN80IawFiw0GmnMzm1No0fwlRjxm8hFRFQ2d+qg=;24:xnpg7i76IVbqy7951s4t6f4q1e+6DsfvG4tK3/5jmgpxoxJYWXcRj9sH1iKrAifhefhhz6YakC2p/WKaAf34v7LS7e6AH9dhrhru5q3kRo4=;7:Is2ToNlG5rducc0ZhdcXhTljcTuJm9xiKoQCuAGIGh3c5Tqx380vFWyXV7OhuHtGUO44xijCXa4AidYLHcM9/Ueoegfqc0chJiDo3zmDwWRcbNsrEYPsaQIUdRsob8oawJN14maVp67kFqK7RO6981DpHt3xinTHSSr13udLtGFD8z76Hhli/cqcaxiDsxc+yb+JMK+B5vgUajdfPpKqDQAn+0M7RwpD8/goBnjA6FJBmVQeHfAvjtoGa9AHlo3R SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Aug 2016 21:56:40.4878 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e;Ip=[192.88.158.2];Helo=[az84smr01.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0301MB1199 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On e500v1, read fault exception enable (RFXE) controls whether assertion of core_fault_in causes a machine check interrupt. Assertion of core_fault_in can result from uncorrectable data error, such as an L2 multi-bit ECC error. It can also occur from a system error if logic on the integrated device signals a fault for nonfatal errors. RFXE bit is cleared out of reset, and should be left clear for normal operation. Assertion of core_fault_in does not cause a machine check. RFXE is set specifically for RIO (Rapid IO) and PCI for book E to catch the errors by machine check. With this bit set, EDAC driver can't get the interrupt in case of uncorrectable error. So this bit is cleared in favor of EDAC. However, the benefit of catching such uncorrectable error doesn't outweigh the other errors which may hang the system. Beside, e500v2 has different errors masked by RFXE, and e500mc doesn't support this bit. It is more reasonable to leave RFXE as is in EDAC driver, and leave the uncorrectable errors triggering machine check for e500v1. Signed-off-by: York Sun Suggested-by: Scott Wood --- Change log v4: Fix typo in commit message v3: Revise commit message v2: new patch in this set drivers/edac/mpc85xx_edac.c | 40 ---------------------------------------- 1 file changed, 40 deletions(-) diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c index ca63d0d..fdc3d9b 100644 --- a/drivers/edac/mpc85xx_edac.c +++ b/drivers/edac/mpc85xx_edac.c @@ -46,9 +46,6 @@ static u32 orig_pci_err_en; #endif static u32 orig_l2_err_disable; -#ifdef CONFIG_FSL_SOC_BOOKE -static u32 orig_hid1[2]; -#endif /************************ MC SYSFS parts ***********************************/ @@ -1225,14 +1222,6 @@ static struct platform_driver mpc85xx_mc_err_driver = { }, }; -#ifdef CONFIG_FSL_SOC_BOOKE -static void __init mpc85xx_mc_clear_rfxe(void *data) -{ - orig_hid1[smp_processor_id()] = mfspr(SPRN_HID1); - mtspr(SPRN_HID1, (orig_hid1[smp_processor_id()] & ~HID1_RFXE)); -} -#endif - static struct platform_driver * const drivers[] = { &mpc85xx_mc_err_driver, &mpc85xx_l2_err_driver, @@ -1263,42 +1252,13 @@ static int __init mpc85xx_mc_init(void) if (res) printk(KERN_WARNING EDAC_MOD_STR "drivers fail to register\n"); -#ifdef CONFIG_FSL_SOC_BOOKE - pvr = mfspr(SPRN_PVR); - - if ((PVR_VER(pvr) == PVR_VER_E500V1) || - (PVR_VER(pvr) == PVR_VER_E500V2)) { - /* - * need to clear HID1[RFXE] to disable machine check int - * so we can catch it - */ - if (edac_op_state == EDAC_OPSTATE_INT) - on_each_cpu(mpc85xx_mc_clear_rfxe, NULL, 0); - } -#endif - return 0; } module_init(mpc85xx_mc_init); -#ifdef CONFIG_FSL_SOC_BOOKE -static void __exit mpc85xx_mc_restore_hid1(void *data) -{ - mtspr(SPRN_HID1, orig_hid1[smp_processor_id()]); -} -#endif - static void __exit mpc85xx_mc_exit(void) { -#ifdef CONFIG_FSL_SOC_BOOKE - u32 pvr = mfspr(SPRN_PVR); - - if ((PVR_VER(pvr) == PVR_VER_E500V1) || - (PVR_VER(pvr) == PVR_VER_E500V2)) { - on_each_cpu(mpc85xx_mc_restore_hid1, NULL, 0); - } -#endif platform_unregister_drivers(drivers, ARRAY_SIZE(drivers)); } -- 2.7.4