Commit: 8069faa5a4b8167283ae4c9f9c4b6628e59e2246
Author: Kavitha Sampath
Date:   Tue May 5 18:48:11 2015 +0530
Branches: cycles_kernel_split
https://developer.blender.org/rB8069faa5a4b8167283ae4c9f9c4b6628e59e2246

Cycles kenrel split : move use_split_kernel to DeviceInfo class

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

M       intern/cycles/device/device.h
M       intern/cycles/device/device_multi.cpp
M       intern/cycles/device/device_opencl.cpp
M       intern/cycles/render/session.cpp

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

diff --git a/intern/cycles/device/device.h b/intern/cycles/device/device.h
index af4fa60..3242a3b 100644
--- a/intern/cycles/device/device.h
+++ b/intern/cycles/device/device.h
@@ -56,6 +56,7 @@ public:
        bool advanced_shading;
        bool pack_images;
        bool extended_images; /* flag for GPU and Multi device */
+       bool use_split_kernel; /* Denotes if the device is going to run cycles 
using split-kernel */
        vector<DeviceInfo> multi_devices;
 
        DeviceInfo()
@@ -67,6 +68,7 @@ public:
                advanced_shading = true;
                pack_images = false;
                extended_images = false;
+               use_split_kernel = false;
        }
 };
 
@@ -79,7 +81,7 @@ struct DeviceDrawParams {
 
 class Device {
 protected:
-       Device(DeviceInfo& info_, Stats &stats_, bool background) : 
background(background), info(info_), stats(stats_) { use_split_kernel = false; }
+       Device(DeviceInfo& info_, Stats &stats_, bool background) : 
background(background), info(info_), stats(stats_) {}
 
        bool background;
        string error_msg;
@@ -98,8 +100,6 @@ public:
        /* variables/functions used exclusively for split kernel */
        /* Maximum closure count */
        int clos_max;
-       /* Denotes if the device is going to run cycles using split-kernel */
-       bool use_split_kernel;
        /* Get all closure nodes associated with the scene */
        set<int> closure_nodes;
        /* Return background */
diff --git a/intern/cycles/device/device_multi.cpp 
b/intern/cycles/device/device_multi.cpp
index 2cd3e09..e97df87 100644
--- a/intern/cycles/device/device_multi.cpp
+++ b/intern/cycles/device/device_multi.cpp
@@ -54,14 +54,14 @@ public:
 
                foreach(DeviceInfo& subinfo, info.multi_devices) {
                        device = Device::create(subinfo, stats, background);
-                       atleast_one_split_kernel_dev |= 
device->use_split_kernel;
+                       atleast_one_split_kernel_dev |= 
device->info.use_split_kernel;
                        devices.push_back(SubDevice(device));
                }
 
                /* Set use_split_kernel of multi device to true, even
                 * if one of the sub devices uses split kernel
                 */
-               this->use_split_kernel = atleast_one_split_kernel_dev;
+               this->info.use_split_kernel = atleast_one_split_kernel_dev;
 
 #ifdef WITH_NETWORK
                /* try to add network devices */
diff --git a/intern/cycles/device/device_opencl.cpp 
b/intern/cycles/device/device_opencl.cpp
index 7513ad6..8be44ca 100644
--- a/intern/cycles/device/device_opencl.cpp
+++ b/intern/cycles/device/device_opencl.cpp
@@ -1498,7 +1498,7 @@ public:
                : OpenCLDeviceBase(info, stats, background_)
        {
 
-               use_split_kernel = true;
+               this->info.use_split_kernel = true;
                background = background_;
 
                /* Initialize kernels */
diff --git a/intern/cycles/render/session.cpp b/intern/cycles/render/session.cpp
index 7a4c903..6c559a7 100644
--- a/intern/cycles/render/session.cpp
+++ b/intern/cycles/render/session.cpp
@@ -271,7 +271,7 @@ void Session::run_gpu()
                        /* update status and timing */
                        update_status_time();
 
-                       if (device->use_split_kernel) {
+                       if (device->info.use_split_kernel) {
                                /* OpenCL split - load kernels */
                                load_kernels();
                        }
@@ -560,7 +560,7 @@ void Session::run_cpu()
                        /* update status and timing */
                        update_status_time();
 
-                       if (device->use_split_kernel) {
+                       if (device->info.use_split_kernel) {
                                /* OpenCL split - load kernels */
                                load_kernels();
                        }
@@ -613,7 +613,7 @@ void Session::load_kernels()
 {
        thread_scoped_lock scene_lock(scene->mutex);
 
-       if (!kernels_loaded || (device->use_split_kernel && 
!device->get_background())) {
+       if (!kernels_loaded || (device->info.use_split_kernel && 
!device->get_background())) {
                /* for split kernel, in case if interactive rendering, we
                 * we need to check kernel-reload before doing path trace
                 */
@@ -662,7 +662,7 @@ static int getClosureCount(Scene *scene)
 
 void Session::run()
 {
-       if (device->use_split_kernel) {
+       if (device->info.use_split_kernel) {
                device->clos_max = getClosureCount(scene);
        }
 
@@ -671,7 +671,7 @@ void Session::run()
         * closures that will be used in rendering, which is not known at this 
point; Hence we
         * defer OpenCL split kernel load_kernels() to after device_update
         */
-       if (!device->use_split_kernel) {
+       if (!device->info.use_split_kernel) {
                load_kernels();
        }
 
@@ -840,7 +840,7 @@ void Session::update_status_time(bool show_pause, bool 
show_done)
 
                substatus = string_printf("Path Tracing Tile %d/%d", tile, 
num_tiles);
 
-               if(((is_gpu && !is_multidevice) || (is_cpu && num_tiles == 1)) 
&& !device->use_split_kernel) {
+               if(((is_gpu && !is_multidevice) || (is_cpu && num_tiles == 1)) 
&& !device->info.use_split_kernel) {
                        /* When using split-kernel (OpenCL) each thread in a 
tile will be working on a different
                         * sample. Can't display sample number when device uses 
split-kernel
                         */

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to