From mboxrd@z Thu Jan 1 00:00:00 1970 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 CF02CC3DA78 for ; Tue, 17 Jan 2023 20:09:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231656AbjAQUIz (ORCPT ); Tue, 17 Jan 2023 15:08:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44078 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231419AbjAQUGt (ORCPT ); Tue, 17 Jan 2023 15:06:49 -0500 Received: from lobo.ruivo.org (lobo.ruivo.org [173.14.175.98]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C50A246D7E for ; Tue, 17 Jan 2023 10:58:41 -0800 (PST) Received: by lobo.ruivo.org (Postfix, from userid 1011) id 27509529DE; Tue, 17 Jan 2023 13:58:33 -0500 (EST) Received: from jake.ruivo.org (bob.qemu.ruivo [192.168.72.19]) by lobo.ruivo.org (Postfix) with ESMTPA id 03E6E52723; Tue, 17 Jan 2023 13:58:16 -0500 (EST) Received: by jake.ruivo.org (Postfix, from userid 1000) id E28FA220056; Tue, 17 Jan 2023 13:58:15 -0500 (EST) Date: Tue, 17 Jan 2023 13:58:15 -0500 From: Aristeu Rozanski To: "Luck, Tony" Cc: Borislav Petkov , "linux-edac@vger.kernel.org" , "aris@redhat.com" Subject: Re: [RFC PATCH] mce: prevent concurrent polling of MCE events Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.2.9 (2022-11-12) Precedence: bulk List-ID: X-Mailing-List: linux-edac@vger.kernel.org On Tue, Jan 17, 2023 at 06:54:02PM +0000, Luck, Tony wrote: > > Yes, but could change the patch to include mce_available() into the > > protection of the lock. It should cleared once machine_check_poll() clears > > the bank state, no? > > Which machines are showing this problem? Most modern systems support > CMCI. So I'm thinking that this case shows up because the sysadmin booted > with "mce=no_cmc;". In that case I don't think mce_available() check would > change anything. That is correct, ignore what I said. -- Aristeu