From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 27 Oct 2008 19:26:02 +0100 From: Robert Richter To: Carl Love Subject: Re: [Cbe-oss-dev] [PATCH] Cell OProfile: Incorrect local array size in activate spu profiling function Message-ID: <20081027182602.GE9785@erda.amd.com> References: <1224874049.20229.5.camel@carll-linux-desktop> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: <1224874049.20229.5.camel@carll-linux-desktop> Cc: linux-kernel , linuxppc-dev@ozlabs.org, oprofile-list@lists.sourceforge.net, Ingo Molnar , cel@linux.vnet.ibm.com, cbe-oss-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 24.10.08 11:47:29, Carl Love wrote: > > The size of the pm_signal_local array should be equal to the > number of SPUs being configured in the call. Currently, the > array is of size 4 (NR_PHYS_CTRS) but being indexed by a for > loop from 0 to 7 (NUM_SPUS_PER_NODE). > > Signed-off-by: Carl Love I applied your patch to oprofile/oprofile-for-tip. Thanks Carl. -Robert -- Advanced Micro Devices, Inc. Operating System Research Center email: robert.richter@amd.com