linux-kernel - Re: [PATCH 0/3] EDAC/igen6: Add polling support and allow setting edac_op

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Zy6GLJwyZCuGwdni@agluck-desk3>
Date: Fri, 8 Nov 2024 13:44:12 -0800
From: Tony Luck <tony.luck@...el.com>
To: Orange Kao <orange@...en.io>
Cc: qiuxu.zhuo@...el.com, bp@...en8.de, james.morse@....com,
	orange@...sy.org, linux-edac@...r.kernel.org,
	linux-kernel@...r.kernel.org, mchehab@...nel.org, rric@...nel.org
Subject: Re: [PATCH 0/3] EDAC/igen6: Add polling support and allow setting
 edac_op_state

On Wed, Nov 06, 2024 at 11:35:44AM +0000, Orange Kao wrote:
> Thank you Qiuxu and Boris.
> 
> Here is the updated patch. I would like to propose that we keep the 
> edac_op_state as a module parameter. Because it would allow users (regardless of
> CPU SKU) to test different options on their machine without compiling their own
> kernel. I hope this could lower the entry barrier and make it easier for them to
> test IBECC.
> 
> Patch 1: Initialize edac_op_state according to the configuration data
> Patch 2: Add polling support

Applied patches 1 & 2 to RAS tree. Thanks

> Patch 3: Allow setting edac_op_state

As discussed on mailing list, not taking this one as there
is no real use case.

-Tony