linux-kernel - Re: [PATCH 1/2] firmware: Intel: Add Stratix10 ECC DBE SMC call

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:   Tue, 2 Apr 2019 08:34:18 -0500
From:   Thor Thayer <thor.thayer@...ux.intel.com>
To:     Borislav Petkov <bp@...en8.de>
Cc:     mchehab@...nel.org, james.morse@....com, atull@...nel.org,
        richard.gong@...el.com, gregkh@...uxfoundation.org,
        linux-edac@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] firmware: Intel: Add Stratix10 ECC DBE SMC call

On 4/2/19 3:55 AM, Borislav Petkov wrote:
> On Fri, Mar 29, 2019 at 09:43:58AM -0500, thor.thayer@...ux.intel.com wrote:
>> From: Thor Thayer <thor.thayer@...ux.intel.com>
>>
>> Reserve ECC Double Bit Error SMC call to alert U-Boot that
>> a DBE has occurred. Moving the call from local EDAC header
>> file to this common file.
> 
> Is there any particular reason this needs to be two patches or can I
> merge them into one?
> 

Yes, they can be merged into one. I split into 2 patches since the 1st 
required the firmware directory sign-off and the 2nd required your EDAC 
sign-off. I see your point, it could have been 1 patch.

Thanks for reviewing,

Thor