crash when viewport rendering point density texture #50094

New Issue

Joel Meyer · 2016-11-22T22:45:25+01:00

Joel Meyer commented

2016-11-22 22:45:25 +01:00

System Information
Windows 10 x64, GTX 1070

Blender Version
Broken: 2.78a (e8299c8), also tested daily build bd8cbf5
Worked: (optional)

Short description of error
When using a point density texture node in a cycles material, blender crashes after changing the resolution value.

Exact steps for others to reproduce the error

Render engine: Cycles (GPU or CPU rendering)
any material (must be visible in the 3d viewport, i. e. not on a hidden layer), use nodes
add a texture->point density node (node editor or properties panel). It must affect the material (e. g. color input of default diffuse).
switch 3d viewport to rendered mode
change the "Resolution" value of the node. Blender crashes.

Bug seems to appear only when changing the value immediately after switching to rendered viewport and changing the "Resolution" value with click-and-drag (not clicking or typing in directly)

**System Information** Windows 10 x64, GTX 1070 **Blender Version** Broken: 2.78a (e8299c8), also tested daily build bd8cbf5 Worked: (optional) **Short description of error** When using a point density texture node in a cycles material, blender crashes after changing the resolution value. **Exact steps for others to reproduce the error** - Render engine: Cycles (GPU or CPU rendering) - any material (must be visible in the 3d viewport, i. e. not on a hidden layer), use nodes - add a texture->point density node (node editor or properties panel). It must affect the material (e. g. color input of default diffuse). - switch 3d viewport to rendered mode - change the "Resolution" value of the node. Blender crashes. Bug seems to appear only when changing the value immediately after switching to rendered viewport and changing the "Resolution" value with click-and-drag (not clicking or typing in directly)

Joel Meyer commented

2016-11-22 22:45:25 +01:00

Changed status to: 'Open'

Joel Meyer commented

2016-11-22 22:45:25 +01:00

Added subscriber: @sirjofri

Aaron Nichols commented

2016-11-27 15:13:38 +01:00

Added subscriber: @newbs

Aaron Nichols commented

2016-11-27 15:13:38 +01:00

Joel Meyer, Can you upload a blend file please? Thank you.

Ray molenkamp commented

2016-11-27 23:55:37 +01:00

Added subscriber: @LazyDodo

Ray molenkamp commented

2016-11-27 23:55:37 +01:00

This one was easy to repro, point_density.blend

just switch to rendered view, and click the up or down button on the resolution a few times (doesn't happen all the time but after a few tries blender crashes) probably a race condition somewhere.

This one was easy to repro, [point_density.blend](https://archive.blender.org/developer/F406854/point_density.blend) just switch to rendered view, and click the up or down button on the resolution a few times (doesn't happen all the time but after a few tries blender crashes) probably a race condition somewhere.

Ray molenkamp self-assigned this 2016-11-28 03:49:27 +01:00

Ray molenkamp commented

2016-11-28 03:49:27 +01:00

Added subscriber: @Sergey

Ray molenkamp commented

2016-11-28 03:49:27 +01:00

race condition, in between the time ImageManager allocates ram for the texture and the time it gets sampled the ui managed to get in a change to the resolution. causing things to overflow in point_density_sample_func. Not quite sure how to stop that from happening. @Sergey any hints/tips?

Sergey Sharybin commented

2016-11-28 13:27:03 +01:00

Not sure what would be best here, can in theory use similar technique as initial sync is doing: have a main thread locked while sampling point density (see WM_job_main_thread_lock_acquire). This is annoying tho, because it introduces interface locks which we hate and must avoid.

Probably better approach for until we've got proper data separation would be to re-set viewport render from RNA update callback for PD resolution. We had that for some properties already (can't find them atm tho). Nice point of research is ED_render_engine_area_exit() and see it's users and such.

Not sure what would be best here, can in theory use similar technique as initial sync is doing: have a main thread locked while sampling point density (see `WM_job_main_thread_lock_acquire`). This is annoying tho, because it introduces interface locks which we hate and must avoid. Probably better approach for until we've got proper data separation would be to re-set viewport render from RNA update callback for PD resolution. We had that for some properties already (can't find them atm tho). Nice point of research is `ED_render_engine_area_exit()` and see it's users and such.

Ray molenkamp commented

2016-11-28 17:02:43 +01:00

Ideally i'd like to extend builtin_image_float_pixels_cb and builtin_image_pixels_cb to include the buffer dimensions (passing a pointer to a buffer, without telling the callee what size it is, just seems bad faith to me, and inviting issues like these) however that in turn causes a change to rna_ShaderNodePointDensity_density_calc (to include the buffer dimensions there as well) but that might be a breaking api change, not sure if that's ok? (doesn't look like any python is calling it though, so it just be a c/c++ code change)

Sergey Sharybin commented

2016-11-28 17:06:01 +01:00

Not sure that will solve anything. You can still change PD settings while builtin_image_float_pixels_cb() is sampling the PD causing threading conflicts.

Not sure that will solve anything. You can still change PD settings while `builtin_image_float_pixels_cb()` is sampling the PD causing threading conflicts.

Ray molenkamp commented

2016-11-28 17:21:43 +01:00

It would, BlenderSession::builtin_image_info would query the PD settings once, to get the resolution, and beyond that the buffer dimensions would be passed down, so there's no need for rna_ShaderNodePointDensity_density_calc to query the PD Settings and get a potentially different answer.

blender-admin commented

2016-11-29 11:39:14 +01:00

This issue was referenced by 1ec5edcc96

This issue was referenced by 1ec5edcc9669f46b1f0fd29c765a326ea525e7f3

Sergey Sharybin commented

2016-11-29 11:44:03 +01:00

Changed status from 'Open' to: 'Resolved'

Sergey Sharybin closed this issue

2016-11-29 11:44:03 +01:00

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

crash when viewport rendering point density texture #50094