=Comment: #0=================================================
KEITH MANNTHEY <kmannth.com> - 
Problem description:

The dimm pairs are reversed when reporting errors on the LS21 with R2. The error
should have been fixed long ago but alas. 

These patches tried to make it in right before RC1 but things went wacko. There
are a few consmetic issues (like getting rid of the crappy copyright symbols but
the changes are very minor).  This is what MRG (pre lucas) has been using. 

This needs to be updated.
=Comment: #1=================================================
KEITH MANNTHEY <kmannth.com> - 

small change for ibm-prtm. 

=Comment: #2=================================================
KEITH MANNTHEY <kmannth.com> - 

Fix LS21 ordering issue and fix small issues. 

=Comment: #3=================================================
Darren V. Hart <dvhltc.com> - 
(In reply to comment #2)
>  [edit]
> Fix LS21 ordering issue and fix small issues. 
> 

Believe we have a logic bug here:

> #Check if on hs21xm.
> def is_hs21():
>-	#check for 5000P memory controller
>+	#check for 5000P memory controller and HS21 in dmidecode 
> 	if os.popen("lspci | grep \"5000P Chipset Memory\"").readlines() != []:
>-		return True
>+		if os.popen("dmidecode | grep HS21").readlines() != []:
>+			return True
> 	else:
> 		return False

This patches the "is_hs21()" call to look like this:

#Check if on hs21xm.
def is_hs21():
	#check for 5000P memory controller and HS21 in dmidecode 
	if os.popen("lspci | grep \"5000P Chipset Memory\"").readlines() != []:
		if os.popen("dmidecode | grep HS21").readlines() != []:
			return True
	else:
		return False

So if we have the 5000p but we aren't an HS21, we return None, not False.  I
think it should read like this instead (remove the else, could also make the
test a an "and" instead of a nested if):

def is_hs21():
	#check for 5000P memory controller and HS21 in dmidecode 
	if os.popen("lspci | grep \"5000P Chipset Memory\"").readlines() != []:
		if os.popen("dmidecode | grep HS21").readlines() != []:
			return True
	return False


=Comment: #4=================================================
John G. Stultz <johnstul.com> - 
So the patches have been slightly reworked and submitted by Darren. They will be
in the final -rc3 release, which has just been built. I'm marking this as fixed
awaiting test until Keith can verify the fixes included in -rc3 are correct and
nothing else is needed.
=Comment: #5=================================================
John G. Stultz <johnstul.com> - 
The fixes have been included into R2-rc3. I'm going to mark this as submitted.

Keith/Vernon: if  MRG has already picked this up, this can be marked accepted.
=Comment: #7=================================================
Vernon Mauery <vernux.com> - 
A more recent version of ibm-prtm code has been mailed to Clark and he has included it in MRG.
=Comment: #11=================================================
Sripathi Kodi <sripathik.com> - 
Note to RH: Keith has already sent the patch to Clark Williams in an e-mail and AFAIK the patch has
been accepted already. Clark requested that we still mirror this bug.