lkml.org 
[lkml]   [2020]   [May]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 00/10] EDAC/mc/ghes: Fixes, cleanup and reworks
Boris,

On 22.04.20 13:58:04, Robert Richter wrote:
> This series contains edac fixes and a significant cleanup and rework
> of the ghes driver:
>
> * fixes and updates for edac_mc (patches #1, #2),
>
> * removed smbios_handle from struct dimm_info (patch #4),
>
> * fix of DIMM label in error reports (patch #5),
>
> * general ghes_edac cleanup and rework (patches #3, #6-#10).
>
> First 2 patches for edac_mc are individual patches, the remaining
> patches do not depend on them.
>
> Tested on a Marvell/Cavium ThunderX2 Sabre (dual socket) system.
>
> v1:
> * https://lore.kernel.org/patchwork/cover/1205901/
>
> v2:
> * reordered patches to have fixes and struct changes first, code
> refactoring patches last,
> * dropped v1 patches #9 to #11 (multiple conrollers) to handle them
> in a separate series,
> * rewrote patch to remove smbios_handle (based on v1 #9): EDAC/ghes:
> Move smbios_handle from struct dimm_info to ghes private data,
> * added lockdep_assert_held() checkers,
> * renamed struct ghes_dimm_fill to struct dimm_fill,
> * renamed local variable dimms to dimm_list to avoid conflict with
> the global variable,
> * removed dimm list for "fake" controller,
> * fixed return code check to use (rc < 0),
> * added: EDAC/mc: Fix usage of snprintf() and dimm location setup

thanks for review.

I have addressed all of your review comments if not otherwise noted.
Please take a look at my replies to you. I am a bit unsure on how to
proceed with 08/10. I have sent you a detailed explanation and hope we
can find a solution soon. I could send a v3 then.

Thanks,

-Robert

\
 
 \ /
  Last update: 2020-05-06 10:54    [W:0.275 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site