Building options lead to broken BVH in Cycles #70685

New Issue

Dalai Felinto · 2019-10-09T22:25:07+02:00

Dalai Felinto commented

2019-10-09 22:25:07 +02:00

System Information
Operating system: Windows-10-10.0.18362 64 Bits
Graphics card: GeForce GTX 960M/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 431.70

Blender Version
Broken: version: 2.81 (sub 14), e846852862
Worked: ?

Short description of error
With my build options set to a minimum (WITH_CYCLES, WITH_BOOST, WITH_CYCLES_NATIVE_ONLY) it builds fine, but I end up with a non-functional Cycles.

Exact steps for others to reproduce the error
Viewport render preview leads to an assert on bvh_traversal.h::BVH_FUNCTION_NAME.

In that function I'm getting kernel_data.bvh.bvh_layout = BVH_LAYOUT_BVH4, but this case is ifdef'ed.

Note: If I go to debug menu, set debug to 256, and manually set BVH Layout to BVH2 things work well.

**System Information** Operating system: Windows-10-10.0.18362 64 Bits Graphics card: GeForce GTX 960M/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 431.70 **Blender Version** Broken: version: 2.81 (sub 14), e84685286274 Worked: ? **Short description of error** With my build options set to a minimum (WITH_CYCLES, WITH_BOOST, WITH_CYCLES_NATIVE_ONLY) it builds fine, but I end up with a non-functional Cycles. **Exact steps for others to reproduce the error** Viewport render preview leads to an assert on bvh_traversal.h::BVH_FUNCTION_NAME. In that function I'm getting kernel_data.bvh.bvh_layout = BVH_LAYOUT_BVH4, but this case is ifdef'ed. **Note:** If I go to debug menu, set debug to 256, and manually set BVH Layout to BVH2 things work well.

Dalai Felinto commented

2019-10-09 22:25:07 +02:00

Added subscriber: @dfelinto

Ray molenkamp commented

2019-10-09 22:31:42 +02:00

Added subscriber: @LazyDodo

Ray molenkamp commented

2019-10-09 22:31:42 +02:00

can't seem to repro with the standard developer profile options,could you share your cmakecache.txt ?

can't seem to repro with the standard developer profile options,could you share your `cmakecache.txt` ?

Dalai Felinto commented

2019-10-09 22:45:25 +02:00

CMakeCache.txt:
CMakeCache.txt

CMakeCache.txt: [CMakeCache.txt](https://archive.blender.org/developer/F7799842/CMakeCache.txt)

Ray molenkamp commented

2019-10-09 23:02:47 +02:00

I matched your build options as close as i could, still nothing.

blender --factory-startup
select cycles as the renderer
z , rendered ,in the viewport

should do it right?

I matched your build options as close as i could, still nothing. 1) blender --factory-startup 2) select cycles as the renderer 3) z , rendered ,in the viewport should do it right?

Dalai Felinto commented

2019-10-09 23:08:48 +02:00

Steps are perfect. It goes without saying that It should be a debug build, and asserts should break. But I think even if they don't stop it would print in the windows console.

Sergey Sharybin commented

2019-10-10 13:03:12 +02:00

Added subscriber: @Sergey

Sergey Sharybin commented

2019-10-10 13:03:12 +02:00

Changed status from 'Open' to: 'Archived'

Sergey Sharybin closed this issue

2019-10-10 13:03:12 +02:00

Sergey Sharybin self-assigned this 2019-10-10 13:03:12 +02:00

Sergey Sharybin commented

2019-10-10 13:03:13 +02:00

This means that get_bvh_layout_mask in device_cpu.cpp detected SSE2 support, but somehow __QBVH__ was not defined in the kernel (even though on 64bit platforms we always enabling __KERNEL_SSE2__).

However, this is a report about custom build, which we don't accept in the bug tracker. Bug tracker about issues reproducable with official builds. All the rest is to go via developers communications channels.

This means that `get_bvh_layout_mask` in `device_cpu.cpp` detected SSE2 support, but somehow `__QBVH__` was not defined in the kernel (even though on 64bit platforms we always enabling `__KERNEL_SSE2__`). However, this is a report about custom build, which we don't accept in the bug tracker. Bug tracker about issues reproducable with official builds. All the rest is to go via developers communications channels.

Ray molenkamp commented

2019-10-10 14:41:27 +02:00

I asked for the ticket on chat, WITH_CYCLES_NATIVE_ONLY is not an option that gets a whole lot of exercise on windows, so if there's a problem with it i'd like to look into it and resolve the issue, that being said i'm unable repro

I asked for the ticket on chat, `WITH_CYCLES_NATIVE_ONLY` is not an option that gets a whole lot of exercise on windows, so if there's a problem with it i'd like to look into it and resolve the issue, that being said i'm unable repro

blender-admin commented

2020-02-14 13:54:09 +01:00

This issue was referenced by blender/cycles@6624140709

This issue was referenced by blender/cycles@6624140709d20b2f3c5f70af655620c3de947383

blender-admin commented

2020-02-14 13:54:09 +01:00

This issue was referenced by 9339dc6dd1

This issue was referenced by 9339dc6dd1080730d1cf0e9a3f14d6139b2fda06

Brecht Van Lommel commented

2020-02-14 14:12:14 +01:00

Changed status from 'Archived' to: 'Resolved'

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Building options lead to broken BVH in Cycles #70685