Cycles: GPU Performance #87836

New Issue

Brecht Van Lommel · 2021-04-26T17:14:32+02:00

Brecht Van Lommel commented

2021-04-26 17:14:32 +02:00

Memory usage

Auto detect good integrator state size depending on GPU (hardcoded to 1 million now)
Reduce size of IntegratorState
- Don't store float4 as float3
- Don't allocate memory for unused features (volumes, sss, denoising, light passes)
- Dynamic volume stack size depending on scene contents (D12925)
- Overlap SSS parameters with other memory
- Compress some floats as half float
- Reduce bits for integers where possible
- Can diffuse/glossy/transmission bounces be limited to 255?
SoA
- Individual arrays for XYZ components of float3?
- Pack together 8bit/16bit values into 32bit?
Reduce size of ShaderData
- Compute some differentials on the fly?
- Read some data directly from ray/intersection?
- Don't copy matrix if motion blur is disabled
Reduce kernel local memory
- Dynamically allocate closures based on used shaders
- Dynamically allocate SVM stack based on used shaders
- Check if SVM stack is allocated multiple times
- Check on deduplicating ShaderData instances
- Check for other unknown sources of memory usage
- Use either shade_surface or shade_surface_raytrace for reserving memory

Kernel Size

Replace megakernel used for last few samples? Only helps about 10% with viewport render, barely with batch render. But makes OptiX runtime compilation slow.
Make svm_eval_nodes a templated function and specialize it for
- Background
- Lights
- Shadows
- Shader-raytracing
- Volumes
Avoid shader ray-tracing for AO pass (D12900)
Verify if more specialization is possible in svm_eval_nodes (seems not)
Deduplicate shader evaluation call in shade_background

Scheduling

Make shadow paths fully independent so they do not block main path kernels (D12889)
Accurately count number of available paths for scheduling additional work tiles
Compact shadow paths similar to main paths (D12944)
Consider adding scrambling distance in advanced options (D12318)
Compact path states for coherence
Tweak tile size for better coherence
Tweak work tiles or pixel order to improve coherence (many small tiles, Z-order, ..)
Try other shader sorting techniques (but will be more expensive than bucket sort)
- Take into account object ID
- Take into account random number for BSDF/BSSRDF selection?
Overlapped kernel execution
- Use multiple independent GPU queueus? (so far was found to be 15% slower)
- Use multiple GPU queues to schedule different kernels?
Optimize active index, sorting and prefix sum kernels
- Parallelize prefix_sum
- Build active/sorted index array for specific based on another array indicating active paths for all kernels, especially when number of paths is small
Try pushing (part of) integrator state in queues rather than persistent location, for more coherent memory access
- Check potential performance benefit by coalescing state
- Shadow paths
- Main path
Cancelling renders and updates can be slow due to the occupancy heuristic that schedules more samples. Find a way to reduce this problem.
Shader sorting and locality balancing (D15331)
- Find better heuristics for scenes with many shaders
- Use for GPU devices other than Metal
Shader sorting by node graph similarity, to find coherence if e.g. the graph only has different image textures

Display

For final render, let Cycles draw the image instead of copying pixels to Blender
- Render pass support

Render Algorithms

Use native OptiX curve primitive for thick hair
Transparent shadows: can we terminate OptiX rays earlier when enough hits are found? (D12524)
Transparent shadows: tune max hits for performance / memory usage
Detect constant transparent shadows for triangles and avoid recording intersection and evaluating shader entirely
Detect transparent shadows that are purely an image texture lookup and perform it in the hit kernel
For volume stack init, implement volume all intersection that writes directly into the stack

Tuning

Automatically increase integrator state size depending on available memory
Tweak kernel compilation parameters (num threads per block, max registers)
- Different parameters per kernel?

**Memory usage** - [x] Auto detect good integrator state size depending on GPU (hardcoded to 1 million now) - [ ] Reduce size of IntegratorState - [x] Don't store float4 as float3 - [x] Don't allocate memory for unused features (volumes, sss, denoising, light passes) - [x] Dynamic volume stack size depending on scene contents ([D12925](https://archive.blender.org/developer/D12925)) - [ ] Overlap SSS parameters with other memory - [ ] Compress some floats as half float - [ ] Reduce bits for integers where possible - [ ] Can diffuse/glossy/transmission bounces be limited to 255? - [ ] SoA - [ ] Individual arrays for XYZ components of float3? - [ ] Pack together 8bit/16bit values into 32bit? - [ ] Reduce size of ShaderData - [ ] Compute some differentials on the fly? - [ ] Read some data directly from ray/intersection? - [x] Don't copy matrix if motion blur is disabled - [ ] Reduce kernel local memory - [ ] Dynamically allocate closures based on used shaders - [ ] Dynamically allocate SVM stack based on used shaders - [ ] Check if SVM stack is allocated multiple times - [ ] Check on deduplicating ShaderData instances - [ ] Check for other unknown sources of memory usage - [x] Use either `shade_surface` or `shade_surface_raytrace` for reserving memory **Kernel Size** - [x] Replace megakernel used for last few samples? Only helps about 10% with viewport render, barely with batch render. But makes OptiX runtime compilation slow. - [x] Make `svm_eval_nodes` a templated function and specialize it for - [x] Background - [x] Lights - [x] Shadows - [x] Shader-raytracing - [x] Volumes - [x] Avoid shader ray-tracing for AO pass ([D12900](https://archive.blender.org/developer/D12900)) - [x] Verify if more specialization is possible in `svm_eval_nodes` (seems not) - [ ] Deduplicate shader evaluation call in `shade_background` **Scheduling** - [x] Make shadow paths fully independent so they do not block main path kernels ([D12889](https://archive.blender.org/developer/D12889)) - [x] Accurately count number of available paths for scheduling additional work tiles - [x] Compact shadow paths similar to main paths ([D12944](https://archive.blender.org/developer/D12944)) - [x] Consider adding scrambling distance in advanced options ([D12318](https://archive.blender.org/developer/D12318)) - [x] Compact path states for coherence - [ ] Tweak tile size for better coherence - [ ] Tweak work tiles or pixel order to improve coherence (many small tiles, Z-order, ..) - [ ] Try other shader sorting techniques (but will be more expensive than bucket sort) - [ ] Take into account object ID - [ ] Take into account random number for BSDF/BSSRDF selection? - [ ] Overlapped kernel execution - [ ] Use multiple independent GPU queueus? (so far was found to be 15% slower) - [ ] Use multiple GPU queues to schedule different kernels? - [ ] Optimize active index, sorting and prefix sum kernels - [ ] Parallelize prefix_sum - [ ] Build active/sorted index array for specific based on another array indicating active paths for all kernels, especially when number of paths is small - [ ] Try pushing (part of) integrator state in queues rather than persistent location, for more coherent memory access - [ ] Check potential performance benefit by coalescing state - [ ] Shadow paths - [ ] Main path - [ ] Cancelling renders and updates can be slow due to the occupancy heuristic that schedules more samples. Find a way to reduce this problem. - [ ] Shader sorting and locality balancing ([D15331](https://archive.blender.org/developer/D15331)) - [ ] Find better heuristics for scenes with many shaders - [x] Use for GPU devices other than Metal - [ ] Shader sorting by node graph similarity, to find coherence if e.g. the graph only has different image textures **Display** - [x] For final render, let Cycles draw the image instead of copying pixels to Blender - [x] Render pass support **Render Algorithms** - [x] Use native OptiX curve primitive for thick hair - [x] Transparent shadows: can we terminate OptiX rays earlier when enough hits are found? ([D12524](https://archive.blender.org/developer/D12524)) - [x] Transparent shadows: tune max hits for performance / memory usage - [ ] Detect constant transparent shadows for triangles and avoid recording intersection and evaluating shader entirely - [ ] Detect transparent shadows that are purely an image texture lookup and perform it in the hit kernel - [ ] For volume stack init, implement volume all intersection that writes directly into the stack **Tuning** - [ ] Automatically increase integrator state size depending on available memory - [ ] Tweak kernel compilation parameters (num threads per block, max registers) - [ ] Different parameters per kernel?

Brecht Van Lommel commented

2021-04-26 17:14:32 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Brecht Van Lommel commented

2021-04-26 17:14:32 +02:00

Added subscriber: @brecht

Steffen Dünner commented

2021-04-26 17:18:04 +02:00

Added subscriber: @SteffenD

elias.andersson92 commented

2021-04-26 18:22:35 +02:00

Added subscriber: @elias.andersson92

Diogo Valadares Reis dos Santos commented

2021-04-26 18:54:44 +02:00

Added subscriber: @Diogo_Valadares

FilipPolbratt commented

2021-04-26 22:43:06 +02:00

Added subscriber: @FilipPolbratt

Alaska commented

2021-04-26 23:45:18 +02:00

Added subscriber: @Alaska

Paul Larson commented

2021-04-27 01:07:11 +02:00

Added subscriber: @GeorgiaPacific

Ladislav Lacko commented

2021-04-27 11:18:50 +02:00

Added subscriber: @lacilaci

Riccardo Giovanetti commented

2021-04-27 15:30:34 +02:00

Added subscriber: @Harvester

Patrick Mours commented

2021-04-27 17:08:30 +02:00

Added subscriber: @pmoursnv

Léo Depoix commented

2021-04-27 17:44:20 +02:00

Added subscriber: @PiloeGAO

Ghost commented

2021-04-27 23:02:49 +02:00

Added subscriber: @dabuxian

Jorge García-Brazales commented

2021-04-27 23:29:16 +02:00

Added subscriber: @strangerman

Dylan Neill commented

2021-04-28 06:25:19 +02:00

Added subscriber: @semimetallic

Jason Clarke commented

2021-04-28 20:53:47 +02:00

Added subscriber: @JasonClarke

Kenzie commented

2021-04-28 22:37:51 +02:00

Added subscriber: @KenzieMac130

Nana Beniako commented

2021-04-29 06:19:01 +02:00

Added subscriber: @Roggii-4

Brecht Van Lommel commented

2021-04-30 19:32:53 +02:00

In a simple queue based system with fixed queue size for each kernel, memory usage increases as the number of kernels goes up, and much of the queue memory remains unused. However it does have the benefit that memory reads/writes may be faster due to coalescing.

Using more paths and associated memory can significantly improve performance though. The question is if there is an implementation that can get us both coalescing and little memory waste. Some brainstorming.

Filling Gaps

The current initialization of main paths works by filling gaps in the state array. Before init_from_camera is called, an array is constructed with unused path indices which is then filled in.

The same mechansim could be extended to the shadow queue, constructing an array of empty entries and using that in shade_surface, rather than forcing the shadow queue to be emptied before shade_surface can be executed.

This still leaves gaps until additional paths can be scheduled, and the fragmentation may cause incoherent paths to be grouped together.

Compaction

Rather than trying to write densely packed arrays, we could compact the kernel state to remove any empty gaps, using a dedicated kernel. This kernel causes additional memory reads and writes, which are hoped to pay off when multiple subsequent kernels can read memory more efficiently. It's not obvious that this is a win. In many cases there may only be 1 or 2 kernel executions before the next path iteration, and the total cost of memory access may increase.

We can do an experiment with a slow coalescing kernel before every kernel execution, to see how much kernel execution is impacted by non-coalesced memory access.

Ring Buffers

In simple cases where we ping-pong between two kernels like intersect_shadow and shade_shadow, an idea would be to share a single queue and fill consecutive empty items. With the idea of a ring buffer this can be made to work for two kernels.

In the single-threaded case this is straightforward, however synchronization to avoid overwriting items from the input queue is not so obvious with many threads. In practice we'd likely need to allocate additional memory based on the number of blocks that are executed in parallel, and that can be significant.

Chunks

An idea to avoid that would be to use a chunked allocator for queues, with the size of the chunks being equal to the block size of kernels.

When a kernel is about to write out queue items, it would allocate an additional chunk for the queue if the current chunk does not have enough space for all items. Queue items would then be written into 1 or 2 chunks. When executing a kernel memory reads would be coalesced, since each chunk matches the block size for that kernel and contains all the queue items in order. Sorting by shader would break coalescing for the shade_surface kernel though, and a separate queue per shader would likely waste too much memory.

Significant memory could still be unused, for two reasons:

For input queue items to not overwrite output queue items, a simple implementation may need to double the memory. Chunks could become available as soon as a block finishes executing. However many blocks are executed in parallel, so the practical memory savings are not obvious.
Up to block size x number of kernels would need to allocated extra to handle memory lost by incompletely filled chunks.

In a simple queue based system with fixed queue size for each kernel, memory usage increases as the number of kernels goes up, and much of the queue memory remains unused. However it does have the benefit that memory reads/writes may be faster due to coalescing. Using more paths and associated memory can significantly improve performance though. The question is if there is an implementation that can get us both coalescing and little memory waste. Some brainstorming. **Filling Gaps** The current initialization of main paths works by filling gaps in the state array. Before `init_from_camera` is called, an array is constructed with unused path indices which is then filled in. The same mechansim could be extended to the shadow queue, constructing an array of empty entries and using that in `shade_surface`, rather than forcing the shadow queue to be emptied before `shade_surface` can be executed. This still leaves gaps until additional paths can be scheduled, and the fragmentation may cause incoherent paths to be grouped together. **Compaction** Rather than trying to write densely packed arrays, we could compact the kernel state to remove any empty gaps, using a dedicated kernel. This kernel causes additional memory reads and writes, which are hoped to pay off when multiple subsequent kernels can read memory more efficiently. It's not obvious that this is a win. In many cases there may only be 1 or 2 kernel executions before the next path iteration, and the total cost of memory access may increase. We can do an experiment with a slow coalescing kernel before every kernel execution, to see how much kernel execution is impacted by non-coalesced memory access. **Ring Buffers** In simple cases where we ping-pong between two kernels like `intersect_shadow` and `shade_shadow`, an idea would be to share a single queue and fill consecutive empty items. With the idea of a ring buffer this can be made to work for two kernels. In the single-threaded case this is straightforward, however synchronization to avoid overwriting items from the input queue is not so obvious with many threads. In practice we'd likely need to allocate additional memory based on the number of blocks that are executed in parallel, and that can be significant. **Chunks** An idea to avoid that would be to use a chunked allocator for queues, with the size of the chunks being equal to the block size of kernels. When a kernel is about to write out queue items, it would allocate an additional chunk for the queue if the current chunk does not have enough space for all items. Queue items would then be written into 1 or 2 chunks. When executing a kernel memory reads would be coalesced, since each chunk matches the block size for that kernel and contains all the queue items in order. Sorting by shader would break coalescing for the `shade_surface` kernel though, and a separate queue per shader would likely waste too much memory. Significant memory could still be unused, for two reasons: * For input queue items to not overwrite output queue items, a simple implementation may need to double the memory. Chunks could become available as soon as a block finishes executing. However many blocks are executed in parallel, so the practical memory savings are not obvious. * Up to block size x number of kernels would need to allocated extra to handle memory lost by incompletely filled chunks.

Brecht Van Lommel commented

2021-04-30 20:22:15 +02:00

The megakernel was a clear win with CUDA when we added it to speed up the last 2% of paths. But benchmarking this with OptiX now it's a mixed bag:

                              no-megakernel                 cycles-x                      
bmw27.blend                   10.0372                       10.0984                       
classroom.blend               13.3341                       13.3774                       
pabellon.blend                7.34786                       7.3563                        
monster.blend                 8.51194                       9.12812                       
barbershop_interior.blend     9.55334                       9.77131                       
junkshop.blend                12.8547                       13.2737                       
pvt_flat.blend                14.8002                       13.0439

A possible explanation is that pvt_flat.blend uses adaptive sampling, where batch sizes are smaller and the megakernel helps more. Also this does not benchmark viewport rendering, where the megakernel helps more.

There's still room for optimization when we have few paths active, without using a megakernel. Given these numbers, that seems worth looking into.

The megakernel was a clear win with CUDA when we added it to speed up the last 2% of paths. But benchmarking this with OptiX now it's a mixed bag: ``` no-megakernel cycles-x bmw27.blend 10.0372 10.0984 classroom.blend 13.3341 13.3774 pabellon.blend 7.34786 7.3563 monster.blend 8.51194 9.12812 barbershop_interior.blend 9.55334 9.77131 junkshop.blend 12.8547 13.2737 pvt_flat.blend 14.8002 13.0439 ``` A possible explanation is that `pvt_flat.blend` uses adaptive sampling, where batch sizes are smaller and the megakernel helps more. Also this does not benchmark viewport rendering, where the megakernel helps more. There's still room for optimization when we have few paths active, without using a megakernel. Given these numbers, that seems worth looking into.

Manuel Albert commented

2021-05-02 23:16:44 +02:00

Added subscriber: @ManuelAlbert

Maged afra commented

2021-05-03 03:38:30 +02:00

Added subscriber: @Maged_Afra

Petter Lundh commented

2021-05-03 21:39:09 +02:00

Added subscriber: @PetterLundh

rsgnz commented

2021-05-05 10:03:48 +02:00

Added subscriber: @rsgnz

Alexey Adamitsky commented

2021-05-05 21:48:38 +02:00

Added subscriber: @AlexeyAdamitsky

Lasse Foster commented

2021-05-06 10:48:42 +02:00

Added subscriber: @LasseFoster-2

Effi commented

2021-05-07 12:49:49 +02:00

Added subscriber: @ephraimpauli

Effi commented

2021-05-07 12:52:55 +02:00

Removed subscriber: @ephraimpauli

Salai Vedha Viradhan commented

2021-05-07 19:54:58 +02:00

Added subscriber: @salaivv

Brecht Van Lommel commented

2021-05-07 21:53:47 +02:00

Current progress on trying to eliminate the megakernel is in P2111, still not where I want it to be.

Compacting the state array seems not all that helpful.

What I did notice while working on that is that in the pvt_flat scene, the number of active paths often drops to a low number but is not refilled quickly. Reducing the tile size to avoid that not only avoids the performance regression, but actually speedups the rendering. However this slows down other scenes.

                              compact+tilesize/4    tilesize/4     compact       no-megakernel      megakernel
bmw27.blend                   8.34171               8.07403        7.54844       7.5659             7.71559                             
classroom.blend               11.2431               10.9255        10.6584       10.6141            10.8755                             
pabellon.blend                5.91434               5.96131        5.70281       5.73752            5.87296                             
monster.blend                 6.65674               6.52214        6.94078       7.00113            8.08601                             
barbershop_interior.blend     8.43154               7.90963        8.20633       8.16851            8.75592                             
junkshop.blend                10.2858               10.4201        10.5334       10.5217            11.1836                             
pvt_flat.blend                10.116                10.2103        12.115        12.4971            11.0911

There must be something that can be done to get closer to the best of both.

Looking at the kernel execution times of bmw27, it's clear that optimizing init_from_camera for multiple work tiles would help, but it's only a part of the performance gap. There's something else going on here that is harder to pin down.

compact+tilesize/4
6.71538s: integrator_shade_surface integrator_sorted_paths_array prefix_sum
1.47519s: integrator_intersect_closest integrator_queued_paths_array
0.53159s: integrator_intersect_shadow integrator_queued_shadow_paths_array
0.38923s: integrator_shade_shadow integrator_queued_shadow_paths_array
0.32022s: integrator_init_from_camera integrator_terminated_paths_array
0.16891s: integrator_shade_background integrator_queued_paths_array

no-megakernel
6.16877s: integrator_shade_surface integrator_sorted_paths_array prefix_sum
1.17981s: integrator_intersect_closest integrator_queued_paths_array
0.41735s: integrator_shade_shadow integrator_queued_shadow_paths_array
0.36235s: integrator_intersect_shadow integrator_queued_shadow_paths_array
0.21522s: integrator_shade_background integrator_queued_paths_array
0.17480s: integrator_init_from_camera integrator_terminated_paths_array

Current progress on trying to eliminate the megakernel is in [P2111](https://archive.blender.org/developer/P2111.txt), still not where I want it to be. Compacting the state array seems not all that helpful. What I did notice while working on that is that in the `pvt_flat` scene, the number of active paths often drops to a low number but is not refilled quickly. Reducing the tile size to avoid that not only avoids the performance regression, but actually speedups the rendering. However this slows down other scenes. ``` compact+tilesize/4 tilesize/4 compact no-megakernel megakernel bmw27.blend 8.34171 8.07403 7.54844 7.5659 7.71559 classroom.blend 11.2431 10.9255 10.6584 10.6141 10.8755 pabellon.blend 5.91434 5.96131 5.70281 5.73752 5.87296 monster.blend 6.65674 6.52214 6.94078 7.00113 8.08601 barbershop_interior.blend 8.43154 7.90963 8.20633 8.16851 8.75592 junkshop.blend 10.2858 10.4201 10.5334 10.5217 11.1836 pvt_flat.blend 10.116 10.2103 12.115 12.4971 11.0911 ``` There must be something that can be done to get closer to the best of both. Looking at the kernel execution times of bmw27, it's clear that optimizing `init_from_camera` for multiple work tiles would help, but it's only a part of the performance gap. There's something else going on here that is harder to pin down. ``` compact+tilesize/4 6.71538s: integrator_shade_surface integrator_sorted_paths_array prefix_sum 1.47519s: integrator_intersect_closest integrator_queued_paths_array 0.53159s: integrator_intersect_shadow integrator_queued_shadow_paths_array 0.38923s: integrator_shade_shadow integrator_queued_shadow_paths_array 0.32022s: integrator_init_from_camera integrator_terminated_paths_array 0.16891s: integrator_shade_background integrator_queued_paths_array no-megakernel 6.16877s: integrator_shade_surface integrator_sorted_paths_array prefix_sum 1.17981s: integrator_intersect_closest integrator_queued_paths_array 0.41735s: integrator_shade_shadow integrator_queued_shadow_paths_array 0.36235s: integrator_intersect_shadow integrator_queued_shadow_paths_array 0.21522s: integrator_shade_background integrator_queued_paths_array 0.17480s: integrator_init_from_camera integrator_terminated_paths_array ```

Vladislav Macíček commented

2021-05-10 21:54:07 +02:00

Added subscriber: @Aeraglyx

Marcus Papathoma commented

2021-05-11 18:49:38 +02:00

Added subscriber: @machieb

Emi Martinez commented

2021-05-15 07:15:12 +02:00

Added subscriber: @Emi_Martinez

Sander Vonk commented

2021-05-15 22:27:23 +02:00

Added subscriber: @sander.c.vonk

Brecht Van Lommel commented

2021-06-29 16:27:42 +02:00

Note about differentials: PBRT-v4 is not even passing differentials along with rays, but simply computing them using the camera information. This gives incorrect results through reflections and refractions, but may be close enough in practice.

Salai Vedha Viradhan commented

2021-07-09 17:54:02 +02:00

Removed subscriber: @salaivv

Konstantins Visnevskis commented

2021-07-16 20:08:21 +02:00

Added subscriber: @KonstantinsVisnevskis

Milan Jaros commented

2021-07-23 17:39:40 +02:00

Added subscriber: @MilanJaros

Peter Baintner commented

2021-07-27 15:11:11 +02:00

Added subscriber: @PeterBaintner

Sam Morse-Brown commented

2021-07-28 12:51:41 +02:00

Added subscriber: @ParallelMayhem

Nik-9 commented

2021-08-01 18:21:45 +02:00

Added subscriber: @Nik-9

heini commented

2021-08-10 13:36:41 +02:00

Added subscriber: @heini

Catalin commented

2021-08-11 16:25:14 +02:00

Added subscriber: @cata_cg

Raimund Klink commented

2021-08-12 12:17:58 +02:00

Added subscriber: @Raimund58

Johannes Wilde commented

2021-08-28 07:59:40 +02:00

Added subscriber: @johannes.wilde

blender-admin commented

2021-09-20 17:59:20 +02:00

This issue was referenced by blender/cycles@5db8d93df3

This issue was referenced by blender/cycles@5db8d93df3800ed9e90651ca305611c0612e606d

blender-admin commented

2021-09-20 17:59:20 +02:00

This issue was referenced by 0803119725

This issue was referenced by 08031197250aeecbaca3803254e6f25b8c7b7b37

blender-admin commented

2021-09-29 14:53:47 +02:00

This issue was referenced by blender/cycles@579813f4f8

This issue was referenced by blender/cycles@579813f4f8ed6c067d14d02c1d69dd37b69f2c45

blender-admin commented

2021-09-29 14:53:47 +02:00

This issue was referenced by 4d4113adc2

This issue was referenced by 4d4113adc2623c50888b63eaca3a055d8cdf3045

Brecht Van Lommel commented

2021-10-19 20:56:52 +02:00

Trying to figure out which parts of shade_surface_raytrace kernel are using most local memory:

zero max closures:                -64%
zero SVM stack size:              -11%
remove svm_eval_nodes:            -32%
remove direct light + AO pass:    -8%
remove voronoi node:              -1%
remove all texture nodes:         -2%
remove all closure nodes:         -2%
remove bevel + AO nodes:          -14%
remove all nodes but one:         -20%

So roughly:

65% closures
15% bevel + AO nodes
10% SVM stack size
5% other nodes
5% other (including shader data)

Trying to figure out which parts of `shade_surface_raytrace` kernel are using most local memory: ``` zero max closures: -64% zero SVM stack size: -11% remove svm_eval_nodes: -32% remove direct light + AO pass: -8% remove voronoi node: -1% remove all texture nodes: -2% remove all closure nodes: -2% remove bevel + AO nodes: -14% remove all nodes but one: -20% ``` So roughly: * 65% closures * 15% bevel + AO nodes * 10% SVM stack size * 5% other nodes * 5% other (including shader data)

blender-admin commented

2021-10-20 14:21:01 +02:00

This issue was referenced by blender/cycles@6f7dd81db8

This issue was referenced by blender/cycles@6f7dd81db8f0bd09e033f34d03366711575e5c44

blender-admin commented

2021-10-20 14:21:01 +02:00

This issue was referenced by 001f548227

This issue was referenced by 001f548227c413a4fdbee275744ea8bea886081a

blender-admin commented

2021-10-20 14:47:04 +02:00

This issue was referenced by blender/cycles@9026179d2e

This issue was referenced by blender/cycles@9026179d2e43d0f5f7942d3b28776fb249d84117