From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTP id 64177DDDF7 for ; Sat, 11 Aug 2007 04:07:06 +1000 (EST) In-Reply-To: <20070809232700.GY25995@austin.ibm.com> References: <20070809232700.GY25995@austin.ibm.com> Mime-Version: 1.0 (Apple Message framework v623) Content-Type: text/plain; charset=US-ASCII; format=flowed Message-Id: <769ccd8091fc725571c5a695a70af4b9@kernel.crashing.org> From: Segher Boessenkool Subject: Re: [PATCH 1/2] pseries: device node status can be "ok" or "okay" Date: Fri, 10 Aug 2007 20:06:34 +0200 To: linas@austin.ibm.com (Linas Vepstas) Cc: ppc-dev , Paul Mackerras List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , > It seems that some versions of firmware will report a device > node status as the string "okay". As we are not expecting this > string, the device node will be ignored by the EEH subsystem. > Which means EEH will not be enabled. > > When EEH is not enabled, PCI errors will be converted into > Machine Check exceptions, and we'll have a very unhappy system. > - if (status && strcmp(status, "ok") != 0) > + if (status && strncmp(status, "ok", 2) != 0) > return NULL; /* ignore devices with bad status */ Shouldn't you check for the two literal strings, instead of only matching the common prefix? Seems safer. Segher