Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
From:   "Luck, Tony" <tony.luck@intel.com>
To:     "Sang, Oliver" <oliver.sang@intel.com>
CC:     "oe-lkp@lists.linux.dev" <oe-lkp@lists.linux.dev>,
        lkp <lkp@intel.com>,
        "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
        "Huang, Ying" <ying.huang@intel.com>,
        "Tang, Feng" <feng.tang@intel.com>,
        "Yin, Fengwei" <fengwei.yin@intel.com>,
        Borislav Petkov <bp@alien8.de>,
        Yazen Ghannam <yazen.ghannam@amd.com>,
        "Smita.KoralahalliChannabasappa@amd.com" 
        <Smita.KoralahalliChannabasappa@amd.com>,
        "dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
        "x86@kernel.org" <x86@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "patches@lists.linux.dev" <patches@lists.linux.dev>
Subject: RE: [PATCH v9 2/3] x86/mce: Add per-bank CMCI storm mitigation
Thread-Topic: [PATCH v9 2/3] x86/mce: Add per-bank CMCI storm mitigation
Thread-Index: AQHZ9vG7TU6w8G+TdkO4n2sYy/mmYrBEV9iAgABlLAA=
Date:   Wed, 11 Oct 2023 15:16:46 +0000
Message-ID: <SJ1PR11MB6083D191286779302821A7EAFCCCA@SJ1PR11MB6083.namprd11.prod.outlook.com>
References: <20231004183623.17067-3-tony.luck@intel.com>
 <202310111637.dee70328-oliver.sang@intel.com>
In-Reply-To: <202310111637.dee70328-oliver.sang@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?gSY+7jofN+nCplZBozQRBcd/pGSR/ZV5J74jccVfN+jKSVFfzftyXhf0Csst?=
 =?us-ascii?Q?TT8qBKcxEpsz+in7/w7yMZBslGAo18lBRik0mw8jP2SOoGv+zhQzgAncKIJU?=
 =?us-ascii?Q?uz0lFSrSFpWY62AXeBEZskkUx/csstfEOIivsyHkhctFckfNmm7vSMQTo1Sn?=
 =?us-ascii?Q?Ur5iXeUnroOY4J/8zec7RMpiR83dyjASr8krXa2rQ2BdSH8Kkn7eVKa//OLt?=
 =?us-ascii?Q?0uoeCervwZSGOXHUHbcWywoe7hE1Ho2EUWulyyVaqlTxqxYbA1f8TcPY3r18?=
 =?us-ascii?Q?q8BOOxLPxrlUGamwNBqnKm97Xl9+JCj4QDNVn5cTv9msFWE04arAoWBnjjdQ?=
 =?us-ascii?Q?JOozA+Zbm8uKvpH9J4Kivl6KgoqGI/lz+yzNtGUhzLLlAxph7ukkEseUaaQF?=
 =?us-ascii?Q?0eAQrpaKeB+/vNQAfcNZVNC9H9md0MDL41ELwfAXIcDmPjujfwb0D6al3+j2?=
 =?us-ascii?Q?QmkBhpTIsvq8Bhc40bi+oywJXOgJlHKVSuqESR6DQeDDvnyRQDtyCckk8rWJ?=
 =?us-ascii?Q?GiohdfWIR1dDLeRIcWXlNY5irtDda7jVUMstez+FSosgnlCOrUH98ykL7TVr?=
 =?us-ascii?Q?Gu2z8FSrktkisVenNUxacTXr5VRX3ncMHmw34BecGIdxc/NQ397wxku7cNFT?=
 =?us-ascii?Q?IoeyvWtMrEdbh1kyX+bL0QG6mqBRaabUEvk2KVl2LW7IZ9iZ6ato31SUkDY4?=
 =?us-ascii?Q?cYaJeTwjyGjxeAUXqXU47wbeWqAjEsmAqTW4eClWSiyIOlnyod7em5/TJzB8?=
 =?us-ascii?Q?6TBWenW+iMJDLhiPXqij0rxSA74FbYR3fahDgB+vhGuheHD9bvAy392djX5O?=
 =?us-ascii?Q?PZiFzOR2STbqbUshW5iuprBlRotzXJKQlD4cG/HpRwGKt08wU8dm3QNJfKVh?=
 =?us-ascii?Q?5sIMuqSXE0yCNqE7XZmgpHsKh32ORvdFGUCHrGa7+QEPaNv1aQby9kbepZKB?=
 =?us-ascii?Q?D1Rwf6dDCyb19RW3NFGcEZqB/GT6PMz/pY9r8r3VbYCUIKFW60Dr/hYn4bEi?=
 =?us-ascii?Q?UK9sjVFou3Nr+sLwvV2pYeOuWlNCTE5zYOt5d0FTC5zxkQfXoqang4QTxFh6?=
 =?us-ascii?Q?ZAnvTMsjGsuFy8lVC+GRvmdfIevmxQ7DWzjQNUSAKEgDS+kvd1xtzYpMIWrV?=
 =?us-ascii?Q?iDAZd4XiV4xi9cgsAwVunnlar+WO5cmtTBYir2sdGw3s8XE1WHZG4XZyl7z5?=
 =?us-ascii?Q?Y/xt7KIfLYjAANxH4XyfHw7w8+smx5GCcWyqCgobHmp1Lh83+DS/mbmG8mvC?=
 =?us-ascii?Q?j0McIXQ9fjvZPXFX+2lwIyg675nPYxhW5XnuuHKsFySl4RM2AJf0AkzNU8+X?=
 =?us-ascii?Q?QShaQoqkYDLvZKwKwjS0CbkqhkQPR+QFCxHrRxblI+P6U3kxihWPicR2nIdb?=
 =?us-ascii?Q?hGKSLJgjv1Pu+jYzrHb3v864GWjZioMPKpS8aP5NF6hYWzDdDg0uG1paKoeb?=
 =?us-ascii?Q?2hqFSS+cBDkXEEfb0eb5yxAjykOWE2UWyOpy6zoZEG+BV90AEwaNDWeNBmFN?=
 =?us-ascii?Q?E5Hlm1FoRVK0SntggQy/9aX2Gl5E0Qb3YWHYSDWbl8Pl6fNlsHmPYNUxQGWt?=
 =?us-ascii?Q?kmSJvj/OBRfBs1NtHCvy3HivDGFTfEw/8Zwbni2D?=
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: eb84bbf1-881e-4b08-cfcb-08dbca6d15c8
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Oct 2023 15:16:46.8492
 (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: 2ClaIeMZAXZordNALkxNyoRyXowHeYt5e/OF6hKphzjTCAi0ZT6FY8tN94kl6zpKUfKEiO+vjq+YejraFK4nmw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB4860
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=2.7 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RCVD_IN_SBL_CSS,SPF_HELO_NONE,SPF_PASS,
	URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Wed, 11 Oct 2023 08:17:08 -0700 (PDT)
X-Spam-Level: **

> kernel test robot noticed a -8.8% regression of stress-ng.clock.ops_per_s=
ec on:
>
>
> commit: 26bff7b04b829cccc6a97726d6398391a62e34ef ("[PATCH v9 2/3] x86/mce=
: Add per-bank CMCI storm mitigation")
> url: https://github.com/intel-lab-lkp/linux/commits/Tony-Luck/x86-mce-Rem=
ove-old-CMCI-storm-mitigation-code/20231005-024047
> patch link: https://lore.kernel.org/all/20231004183623.17067-3-tony.luck@=
intel.com/
> patch subject: [PATCH v9 2/3] x86/mce: Add per-bank CMCI storm mitigation
>
> testcase: stress-ng
> test machine: 64 threads 2 sockets Intel(R) Xeon(R) Gold 6346 CPU @ 3.10G=
Hz (Ice Lake) with 256G memory
> parameters:
>
>       nr_threads: 10%
>       disk: 1HDD
>       testtime: 60s
>       fs: ext4
>       class: os
>       test: clock
>       cpufreq_governor: performance
>
>
>
>
> If you fix the issue in a separate patch/commit (i.e. not just a new vers=
ion of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <oliver.sang@intel.com>
> | Closes: https://lore.kernel.org/oe-lkp/202310111637.dee70328-oliver.san=
g@intel.com

Is the test injecting massive numbers of corrected memory errors? The code =
in this patch
is only executed when handling CMCI interrupts, or polling machine check ba=
nks (at most
once per second).

I'm guessing this report is just because alignment of some hot path code ch=
anged.

-Tony