Re: [PATCH v2] Fix AMD Northbridge-ID contiguity assumptions

2012-10-12 Thread Steffen Persvold
On 10/12/2012 11:33, Borislav Petkov wrote: On Thu, Oct 04, 2012 at 03:18:02PM +0200, Borislav Petkov wrote: On Wed, Oct 03, 2012 at 09:21:14PM +0800, Daniel J Blueman wrote: The AMD Northbridge initialisation code and EDAC assume the Northbridge IDs are contiguous, which no longer holds on fed

Re: [PATCH v2] Fix AMD Northbridge-ID contiguity assumptions

2012-10-12 Thread Borislav Petkov
On Fri, Oct 12, 2012 at 11:57:00AM +0200, Steffen Persvold wrote: > This patch looks very clean and should serve our purpose as well (I'll > double check with Daniel). Thanks. I'll run it today or next week to check whether it works as expected - will let you know if there are issues. > Regardin

Re: [PATCH v2] Fix AMD Northbridge-ID contiguity assumptions

2012-10-12 Thread Borislav Petkov
On Thu, Oct 04, 2012 at 03:18:02PM +0200, Borislav Petkov wrote: > On Wed, Oct 03, 2012 at 09:21:14PM +0800, Daniel J Blueman wrote: > > The AMD Northbridge initialisation code and EDAC assume the Northbridge IDs > > are contiguous, which no longer holds on federated systems with multiple > > Hyper

Re: [PATCH v2] Fix AMD Northbridge-ID contiguity assumptions

2012-10-04 Thread Borislav Petkov
On Wed, Oct 03, 2012 at 09:21:14PM +0800, Daniel J Blueman wrote: > The AMD Northbridge initialisation code and EDAC assume the Northbridge IDs > are contiguous, which no longer holds on federated systems with multiple > HyperTransport fabrics and multiple PCI domains. > > Address this assumption

[PATCH v2] Fix AMD Northbridge-ID contiguity assumptions

2012-10-03 Thread Daniel J Blueman
The AMD Northbridge initialisation code and EDAC assume the Northbridge IDs are contiguous, which no longer holds on federated systems with multiple HyperTransport fabrics and multiple PCI domains. Address this assumption by searching the Northbridge ID array, rather than directly indexing it, usi