[vsipl++] [patch] pwarp
Jules Bergmann
jules at codesourcery.com
Thu Dec 13 18:20:22 UTC 2007
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 at codesourcery.com
(650) 331-3385 x705
More information about the vsipl++
mailing list