mirror of
https://github.com/italicsjenga/vello.git
synced 2025-01-10 12:41:30 +11:00
d9d518b248
The compute shaders have a check for the succesful completion of their
preceding stage. However, consider a shader execution path like the
following:
void main()
if (mem_error != NO_ERROR) {
return;
}
...
malloc(...);
...
barrier();
...
}
and shader execution that fails to allocate memory, thereby setting
mem_error to ERR_MALLOC_FAILED in malloc before reaching the barrier. If
another shader execution then begins execution, its mem_eror check will
make it return early and not reach the barrier.
All GPU APIs require (dynamically) uniform control flow for barriers,
and the above case may lead to GPU hangs in practice.
Fix this issue by replacing the early exits with careful checks that
don't interrupt barrier control flow.
Unfortunately, it's harder to prove the soundness of the new checks, so
this change also clears dynamic memory ranges in MEM_DEBUG mode when
memory is exhausted. The result is that accessing memory after
exhaustion triggers an error.
Signed-off-by: Elias Naur <mail@eliasnaur.com>
|
||
|---|---|---|
| .. | ||
| annotated.h | ||
| backdrop.comp | ||
| backdrop.spv | ||
| binning.comp | ||
| binning.spv | ||
| bins.h | ||
| build.ninja | ||
| coarse.comp | ||
| coarse.spv | ||
| elements.comp | ||
| elements.spv | ||
| kernel4.comp | ||
| kernel4.spv | ||
| kernel4_idx.spv | ||
| mem.h | ||
| path_coarse.comp | ||
| path_coarse.spv | ||
| pathseg.h | ||
| ptcl.h | ||
| scene.h | ||
| setup.h | ||
| state.h | ||
| tile.h | ||
| tile_alloc.comp | ||
| tile_alloc.spv | ||