linux-kernel - RE: [PATCH v3] EDAC/versal: Make the bits in error injection configurable

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] [thread-next>] [day] [month] [year] [list]

Date: Wed, 14 Feb 2024 05:33:25 +0000
From: "Datta, Shubhrajyoti" <shubhrajyoti.datta@....com>
To: Borislav Petkov <bp@...en8.de>
CC: "linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>, "git
 (AMD-Xilinx)" <git@....com>, "Potthuri, Sai Krishna"
	<sai.krishna.potthuri@....com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "tony.luck@...el.com" <tony.luck@...el.com>,
	"james.morse@....com" <james.morse@....com>, "mchehab@...nel.org"
	<mchehab@...nel.org>, "rric@...nel.org" <rric@...nel.org>
Subject: RE: [PATCH v3] EDAC/versal: Make the bits in error injection
 configurable

[AMD Official Use Only - General]

> -----Original Message-----
> From: Borislav Petkov <bp@...en8.de>
> Sent: Wednesday, February 14, 2024 12:05 AM
> To: Datta, Shubhrajyoti <shubhrajyoti.datta@....com>
> Cc: linux-edac@...r.kernel.org; git (AMD-Xilinx) <git@....com>; Potthuri,
> Sai Krishna <sai.krishna.potthuri@....com>; linux-kernel@...r.kernel.org;
> tony.luck@...el.com; james.morse@....com; mchehab@...nel.org;
> rric@...nel.org
> Subject: Re: [PATCH v3] EDAC/versal: Make the bits in error injection
> configurable
>
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
>
>
> On Thu, Feb 08, 2024 at 03:16:53PM +0530, Shubhrajyoti Datta wrote:
> > Currently the error injection for correctable and uncorrectable errors
> > bits are hardcoded. To make it configurable add separate sysfs entries
> > to set the bit positions for injecting CE and UE errors.
> > It injects single bit error for CE and two bits errors for UE.
> >
> > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@....com>
> > ---
>
> Did some touchups ontop, especially make inject_data_ce_store() verify the
> user input first and then do the injection, just like the ue_store counterpart
> does.
>
> Pls check whether injection still works ok.
>
Thanks tested it working fine .

Thanks ,
Shubhrajyoti