Please disregard the patch for "Cell BE SPU: daemon changes" that I
posted earlier today as there was an error in it.
This is a rework of Cell BE SPU patch 2, originally posted on April 20.
The main change here was to move the Cell SPU-specific daemon code into
its own source file. Since this code uses many of the same functions
and variables as opd_trans.c, I had to move some declarations, as well
as some static inline functions, into opd_trans.h so that both
opd_trans.c and the new file, opd_spu.c, could access them.
From: John Levon <levon@mo...> - 2007-05-10 14:44:22
On Wed, May 02, 2007 at 04:15:27PM -0500, Maynard Johnson wrote:
> +void code_spu_profiling(struct transient * trans)
> + /* Next value in buffer is the number of SPUs. */
> + unsigned long long num_spus = pop_buffer_value(trans);
> + /* Free the cache from previous run */
> + free(spu_context_cache);
> + spu_context_cache = xmalloc(sizeof(struct spu_context_info) * num_spus);
> + special_processor = process_spu_samples;
> + process_spu_samples(trans);
Unnecessary blank line. And elsewhere. Mixed up spaces and tabs in this
file and elsewhere.
> + /* First value in the buffer for an SPU context switch is
> + * the SPU number. For SPU profiling, 'cpu' = 'spu'. */
Please keep to a consistent formatting for comments (follow rest of
Looks fine otherwise