Re: [vsipl++] [patch] pwarp

[Jules Bergmann <jules@xxxxxxxxxxxxxxxx>]

Also, the location of the SPU kernel (in vsip)
> > > > makes it unsightly to depend on code from vsip_csl.
> > > Right. We should think a bit about how to decouple that. In particular,
> > > how to not make the task_manager.hpp file the central repository for all
> > > tasks / task types.
> > Yeah, I was thinking about that too.  For library code like this, could
> > we move the DEINE_TASK instantiation from task_manager.hpp to the file
> > using the task?
> I'm not sure, it will at least be quite fragile, as we want to generate
> unique task-ids so we can meaningfully compare tasks to determine
> whether to reuse the old one or not. 
Why can't we just test the spe_program_handle_t* for uniqueness?  Then 
we can decentralize the selection of the right spe_program_handle_t* to 
the PPU bridge functions, and task_manager will just have to remember 
the last task loaded to avoid the extra reload.
FWIW, that seems to work OK in our skunkworks ALF replacement.

				-- Jules

--
Jules Bergmann
CodeSourcery
jules@xxxxxxxxxxxxxxxx
(650) 331-3385 x705