[Bf-blender-cvs] [d9d71698374] blender-v2.93-release: Fix kernel loading time accounted twice in render progress

Sergey Sharybin noreply at git.blender.org
Wed May 5 15:09:29 CEST 2021


Commit: d9d71698374101bad61b712c742183c94e5d8e6c
Author: Sergey Sharybin
Date:   Wed May 5 11:31:04 2021 +0200
Branches: blender-v2.93-release
https://developer.blender.org/rBd9d71698374101bad61b712c742183c94e5d8e6c

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 38e8d9145dc..9713741d043 100644
--- a/intern/cycles/render/scene.cpp
+++ b/intern/cycles/render/scene.cpp
@@ -566,9 +566,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