Blender Git Commit Log

Git Commits -> Revision 0c5c22f

Revision 0c5c22f by Sergey Sharybin (cycles-x)
August 31, 2021, 13:16 (GMT)
Fix missing tiles with GPU render and OIDN denoiser in Cycles X

Was noticed when testing upcoming tiled rendering with small tiles
sizes which stresses data transfers more than single tile rendering.

What happens is OIDN denoiser uses `copy_to_device()` using default
queue, which is not guaranteed to be synchronized before next call to
`copy_from_device()` happening via queue.

What we really want in the OIDN denoiser is synchronous data transfer
which is easiest to be achieved by creating a queue and synchronizing
it.

The issue is reported by Alaska in D12309#320253

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

Commit Details:

Full Hash: 0c5c22f2ce4a1445a6745121387ad5abc9fbb45c
Parent Commit: aa81152
Lines Changed: +42, -10

4 Modified Paths:

/intern/cycles/device/cuda/device.cpp (+2, -0) (Diff)
/intern/cycles/device/device.h (+2, -0) (Diff)
/intern/cycles/integrator/denoiser_oidn.cpp (+38, -9) (Diff)
/intern/cycles/integrator/denoiser_oidn.h (+0, -1) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021