Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     "Zhuo, Qiuxu" <qiuxu.zhuo@intel.com>
CC:     Borislav Petkov <bp@alien8.de>, Aristeu Rozanski <aris@redhat.com>,
        "Mauro Carvalho Chehab" <mchehab@kernel.org>,
        "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Lim, Ee Wey" <ee.wey.lim@intel.com>
Subject: RE: [PATCH 1/1] EDAC/igen6: Fix the issue of no error events
Thread-Topic: [PATCH 1/1] EDAC/igen6: Fix the issue of no error events
Thread-Index: AQHZvs70AZA+3i1QEU28InVclO7Ajq/XfnFw
Date:   Wed, 2 Aug 2023 20:18:17 +0000
Message-ID: <SJ1PR11MB608356E4C19E9A3D7FE030D4FC0BA@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <20230725080427.23883-1-qiuxu.zhuo@intel.com>
In-Reply-To: <20230725080427.23883-1-qiuxu.zhuo@intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?mwRkD9CjFC3+wuZwsZR+wjZON/Es3bDE1f580gvQxjNCBawjbCC5xw2HeoNi?=
 =?us-ascii?Q?Qy8gMEsNMysAuGgDYHlo/mk3kCXCdj3QoTZ+5aGL7z/aECW3YzKhfCS2IxUu?=
 =?us-ascii?Q?QXBhGUj4g52vJEkXLHzDbVmw+zsL4H67ATnhbeOEdP62ekizZkhOqTxMOC7I?=
 =?us-ascii?Q?PQV4WBHi3VLa3AHy4wr0tKHeYThVETvEJFEmsusoOgwlaD3QPgqFyGoJEKfo?=
 =?us-ascii?Q?CQ3n3yHwy67nQ+0QKkYKL8Lrtj/8nSOQW+Uhuo+IuNSerHi4NBo3KSPB8BJu?=
 =?us-ascii?Q?oSFFt4Bc4Qre2TqyqymuA262MGwMCjDFRC19Mo4+RDwsQN5/Rn1HZnB8zohA?=
 =?us-ascii?Q?SqLdOqKl5oXNPO/deZh7IWrG+FjkxxYNSnSI3fTnWjQrF7Hr2QgrqG+wo0Jg?=
 =?us-ascii?Q?m0ZlPZip1pfmk5GJ2rhi3PE6Xf+oLelreKMYiZoqO8wnrx/vo+gdzRbOmwf4?=
 =?us-ascii?Q?RFEEMB5iSMk+9ApQ1lvDjtNtq2oFPSiov56r726nv4XFM1BxJ0TgfIJFua45?=
 =?us-ascii?Q?f6Eifu/YdQZvhdJQrGp4KhTF8/quSjIgfsdSpQi5ALO+16Mcuz0pe/h/Eauw?=
 =?us-ascii?Q?hWexuBX21xltj2cw/m8bT98Er7KS2GcAB84hnusmOANFvupAu9QXhsZu7oAG?=
 =?us-ascii?Q?exW0Z6LjeAmridrOY7lPgTcSmUpKJGQPnuSMJrtZ/04owSL6DEHJC/KDPCpC?=
 =?us-ascii?Q?JtAphsGGmxVDFB+aVhbQGpF/svPavyVvndhBIiwtOPTaPq34u81uQoPf5OEZ?=
 =?us-ascii?Q?nyjMHMUeehkzfQVtNri6qdyp9BJDsFVZ5Jw9xCSJsilKejQ1RhIAXDyiJhM/?=
 =?us-ascii?Q?h4k3zDRCxdG4/Qs5nKRNYmpv428+ZswyvxJMfiikXp1sV3xXzCJ3ucZuW/Vb?=
 =?us-ascii?Q?nGc3uOnVhbfYcFrfyEVJ/FWqHcokZpFGyT2c9hLxI+X96SYWFQiBySt8vdA+?=
 =?us-ascii?Q?iwxSW9ieP/H8Ng43fkd6qMALzgCAqxtlBVlw437pAwJVo8tZxw5M7NoCI4f/?=
 =?us-ascii?Q?sXCozOVFMG8tcJmtP7bVg6myx6IU2Cq/3syvRbeCXlLjcERzhwmZQxgICp7N?=
 =?us-ascii?Q?uEEcnft00uUyt2A4X2fszeyt0k6YgdewkbdbkmLQrp4VQItkhYIeeBR2VYXo?=
 =?us-ascii?Q?x3Cj4LxW4u7U8gYn9lBDMzdJPms2XvDrJ4jAm6R/CHsNaUFDvwWHDU04KY7K?=
 =?us-ascii?Q?DYUvrsxCOCAa0XGHd/nPkvHJxfsbPl0XNKJHZ8srAbYFrxHVlicmyRNAAVjN?=
 =?us-ascii?Q?dR8NZRBqzZ00qobZUkO9sxHrIQEAyUDCeJZBQIBC7RwvDIiHg0jeYjvVRVgV?=
 =?us-ascii?Q?mXK+52cdWy3aATylnNRJr7zneTaVGOoPgEizAZFPt19u4WOGgjfzOTNyfpxE?=
 =?us-ascii?Q?GPQwpNZVbugr+7plGENKiOZuyw0cYqLqMiWIlLr/zeWBv+eSqXBGmIcp/ph4?=
 =?us-ascii?Q?Y27wUmkCcrVS+Ul09d4SRYrro1zerpQcxGu2Ho5ShPHBiLN6Qx2WSbkbFsdO?=
 =?us-ascii?Q?4Yg1IptDmEy1heZhcYGcD2tJJI+M0hiSh32E29cg2bH/R8PZvYf+QeFiMhha?=
 =?us-ascii?Q?qJGyznEz9gRt/iMSwNQ=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ1PR11MB6083.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e15a3c63-e8d9-457c-e6c5-08db93959bb6
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Aug 2023 20:18:17.4395
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: g+3sf68s+zgFrTKRlZ1r3yhc6KNi+/rnCNhB2KVLahpQBKJnKk3wxRM01yR0jFa+/+XBb/Z+MW0KM8cftVTryw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7384
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Current igen6_edac checks for pending errors before the registration
> of the error handler. However, there is a possibility that the error
> occurs during the registration process, leading to unhandled pending
> errors and no future error events. This issue can be reproduced by
> repeatedly injecting errors during the loading of the igen6_edac.
>
> Fix this issue by moving the pending error handler after the registration
> of the error handler, ensuring that no pending errors are left unhandled.
>
> Fixes: 10590a9d4f23 ("EDAC/igen6: Add EDAC driver for Intel client SoCs u=
sing IBECC")
> Reported-by: Ee Wey Lim <ee.wey.lim@intel.com>
> Tested-by: Ee Wey Lim <ee.wey.lim@intel.com>
> Signed-off-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>

Applied to RAS tree for next merge window.

Thanks

-Tony