[Bf-blender-cvs] [bc081c76722] cycles-x: Fix kernel loading time accounted twice in render progress

Sergey Sharybin noreply at git.blender.org
Wed May 5 11:34:37 CEST 2021


Commit: bc081c76722738e301a730a39d739ffc42937dbe
Author: Sergey Sharybin
Date:   Wed May 5 11:31:04 2021 +0200
Branches: cycles-x
https://developer.blender.org/rBbc081c76722738e301a730a39d739ffc42937dbe

Fix kernel loading time accounted twice in render progress

The render session is keeping track of the scene update, which includes
kernel loading time.

This fixes negative render times reported when CUDA kernels are compiled
at runtime.

A bit fragile logic, can be re-implemented using some user-counted
scope utility classes, so that only outer-most time skip is applied.

===================================================================

M	intern/cycles/render/scene.cpp

===================================================================

diff --git a/intern/cycles/render/scene.cpp b/intern/cycles/render/scene.cpp
index 3a8b336479e..494e6a5329f 100644
--- a/intern/cycles/render/scene.cpp
+++ b/intern/cycles/render/scene.cpp
@@ -591,9 +591,6 @@ bool Scene::load_kernels(Progress &progress, bool lock_scene)
       return false;
     }
 
-    progress.add_skip_time(timer, false);
-    VLOG(1) << "Total time spent loading kernels: " << time_dt() - timer.get_start();
-
     kernels_loaded = true;
     loaded_kernel_features = requested_features;
     return true;



More information about the Bf-blender-cvs mailing list