From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: To: Benjamin Herrenschmidt From: Michael Neuling Subject: Re: [PATCH] Fix pSeries identification in prom_init.c In-reply-to: <1147672282.21291.92.camel@localhost.localdomain> References: <1147671963.21291.91.camel@localhost.localdomain> <1147672282.21291.92.camel@localhost.localdomain> Date: Mon, 15 May 2006 11:17:20 -0500 Sender: mikey@ozlabs.org Message-Id: <20060515161810.E633C679EB@ozlabs.org> Cc: linuxppc-dev list , Paul Mackerras , segher@gate.crashing.org Reply-To: Michael Neuling List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , > > The OF trampoline code prom_init.c still needs to identify IBM > > pSeries (PAPR) machines in order to run some platform specific code > > on them like instanciating the TCE tables. The code doing that > > detection was changed recently in 2.6.17 early stages but was done > > slightly incorrectly. It should be testing for an exact match of > > "chrp" and it currently tests for anything that begins with > > "chrp". That means it will incorrectly match with platforms using > > Maple-like device-trees and have open firmware. This fixes it by > > using strcmp instead of strncmp to match what the actual platform > > detection code does. > > Michael, I noticed you changed strcmp to strncmp, any reason why you > did that ? To be safe if we are returned a non terminated string. I'd not realised the case you've mentioned. How much we should trust firmware? With strcpy, should we explicitly terminate the string first (I removed one of these originally)? Patch below, compiled not run. - The OF trampoline code prom_init.c still needs to identify IBM pSeries (PAPR) machines in order to run some platform specific code on them like instantiating the TCE tables. The code doing that detection was changed recently in 2.6.17 early stages but was done slightly incorrectly. It should be testing for an exact match of "chrp" and it currently tests for anything that begins with "chrp". That means it will incorrectly match with platforms using Maple-like device-trees and have open firmware. This fixes it by using strcmp instead of strncmp to match what the actual platform detection code does. Signed-off-by: Benjamin Herrenschmidt Signed-off-by: Michael Neuling --- arch/powerpc/kernel/prom_init.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletion(-) Index: linux-2.6-powerpc/arch/powerpc/kernel/prom_init.c =================================================================== --- linux-2.6-powerpc.orig/arch/powerpc/kernel/prom_init.c +++ linux-2.6-powerpc/arch/powerpc/kernel/prom_init.c @@ -1636,7 +1636,8 @@ static int __init prom_find_machine_type compat, sizeof(compat)-1); if (len <= 0) return PLATFORM_GENERIC; - if (strncmp(compat, RELOC("chrp"), 4)) + compat[len] = 0; + if (strcmp(compat, RELOC("chrp"))) return PLATFORM_GENERIC; /* Default to pSeries. We need to know if we are running LPAR */