RE: [PATCH 2/8] EDAC/amd64: Support more than two controllers for chip selects handling

2019-06-14 Thread Ghannam, Yazen
> -Original Message- > From: linux-kernel-ow...@vger.kernel.org > On Behalf Of Borislav Petkov > Sent: Thursday, June 13, 2019 5:23 PM > To: Ghannam, Yazen > Cc: linux-e...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 2/8] EDAC/amd64:

Re: [PATCH 2/8] EDAC/amd64: Support more than two controllers for chip selects handling

2019-06-13 Thread Borislav Petkov
On Thu, Jun 13, 2019 at 08:58:16PM +, Ghannam, Yazen wrote: > The first patch is meant as a fix for existing systems, and this patch > is to add new functionality. > > I can merge them together if you think that's more appropriate. Is it fixing such a critical issue that it needs to be a separ

RE: [PATCH 2/8] EDAC/amd64: Support more than two controllers for chip selects handling

2019-06-13 Thread Ghannam, Yazen
> -Original Message- > From: Borislav Petkov > Sent: Thursday, June 13, 2019 9:17 AM > To: Ghannam, Yazen > Cc: linux-e...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 2/8] EDAC/amd64: Support more than two controllers for > chip selects hand

Re: [PATCH 2/8] EDAC/amd64: Support more than two controllers for chip selects handling

2019-06-13 Thread Borislav Petkov
On Fri, May 31, 2019 at 11:45:12PM +, Ghannam, Yazen wrote: > diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c > index 9fa2f205f05c..dd60cf5a3d96 100644 > --- a/drivers/edac/amd64_edac.c > +++ b/drivers/edac/amd64_edac.c > @@ -943,91 +943,101 @@ static void prep_chip_selects(s

[PATCH 2/8] EDAC/amd64: Support more than two controllers for chip selects handling

2019-05-31 Thread Ghannam, Yazen
From: Yazen Ghannam The struct chip_select array that's used for saving chip select bases and masks is fixed at length of two. There should be one struct chip_select for each controller, so this array should be increased to support systems that may have more than two controllers. Increase the si