Blender Git Commit Log

Git Commits -> Revision 9582797

Revision 9582797 by Jeroen Bakker (master)
July 17, 2020, 11:47 (GMT)
Fix T77867: Link Duplicate Object crashes during batch creation

When using link duplicated objects it could happen that one object is
calculating the GPUBuffers and the second object is marking these
buffers invalid. This introduces threading issues.

This patch fixes this by combining the surface and surface per material
batches. Most likely the surface per material batches are used and when
requested you will most likely need the surface batch for the depth
tests and overlays.

During tests it slightly improves performance as batches aren't thrown
away without using it.

After this patch we can add a quick path for meshes with one material
and two materials.

Alternative approaches that have been checked:
- sync extraction per object: reduced performance to much (-15%)
({D8292})
- post checks: reduced the threading issues, but didn't solve it.
- separating preparation and execution of the extraction ({D8312})

Reviewed By: Cl�ment Foucault

Differential Revision: https://developer.blender.org/D8329

Commit Details:

Full Hash: 9582797d4b50a18040e96ae07aa8c7643cbcc25a
Parent Commit: bf2bb6d
Lines Changed: +37, -63

3 Modified Paths:

/source/blender/draw/intern/draw_cache_extract.h (+1, -2) (Diff)
/source/blender/draw/intern/draw_cache_extract_mesh.c (+5, -4) (Diff)
/source/blender/draw/intern/draw_cache_impl_mesh.c (+31, -57) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021