Cell OProfile: Incorrect local array size in activate spu profiling function
authorCarl Love <cel@us.ibm.com>
Fri, 24 Oct 2008 18:47:29 +0000 (11:47 -0700)
committerRobert Richter <robert.richter@amd.com>
Mon, 27 Oct 2008 18:15:40 +0000 (19:15 +0100)
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 <carll@us.ibm.com>
Signed-off-by: Robert Richter <robert.richter@amd.com>
arch/powerpc/oprofile/op_model_cell.c

index 35141a8bc3d94691acdfd2b4833b9bf386f45c60..6b2d974880b696483172d9e87bd8f8d941b9c26a 100644 (file)
@@ -830,7 +830,7 @@ static int calculate_lfsr(int n)
 static int pm_rtas_activate_spu_profiling(u32 node)
 {
        int ret, i;
-       struct pm_signal pm_signal_local[NR_PHYS_CTRS];
+       struct pm_signal pm_signal_local[NUM_SPUS_PER_NODE];
 
        /*
         * Set up the rtas call to configure the debug bus to