Crash on transforming UVs #38109

New Issue

Daniel Salazar · 2014-01-07T22:19:08+01:00

Daniel Salazar commented

2014-01-07 22:19:08 +01:00

System Information

openSUSE 12.3 64bits
NVIDIA 480GTX SC

Blender Version

Broken: Blender 2.69.7 e369a5c

Short description of error

Crash on transforming UVs

Exact steps for others to reproduce the error

Open CrashUVs.blend
Go to UV editor
Press S and move mouse a bit
If it doesn't crash right away press R and move mouse a bit

**System Information** openSUSE 12.3 64bits NVIDIA 480GTX SC **Blender Version** Broken: Blender 2.69.7 e369a5c **Short description of error** Crash on transforming UVs **Exact steps for others to reproduce the error** * Open [CrashUVs.blend](https://archive.blender.org/developer/F63522/CrashUVs.blend) * Go to UV editor * Press S and move mouse a bit * If it doesn't crash right away press R and move mouse a bit

Daniel Salazar commented

2014-01-07 22:19:08 +01:00

Changed status to: 'Open'

Daniel Salazar commented

2014-01-07 22:19:08 +01:00

Added subscriber: @zanqdo

Brecht Van Lommel commented

2014-01-08 23:55:29 +01:00

Added subscriber: @brecht

Brecht Van Lommel commented

2014-01-08 23:55:29 +01:00

I couldn't reproduce this crash, with latest Git or e369a5c on OS X.

I couldn't reproduce this crash, with latest Git or e369a5c on OS X.

Daniel Salazar commented

2014-01-08 23:57:24 +01:00

I was kinda expecting that. It sometimes takes some effort. Anyway gonna do a debug build

Daniel Salazar commented

2014-01-09 00:09:35 +01:00

Here's the backtrace: http://www.pasteall.org/48555

Brecht Van Lommel commented

2014-01-09 00:32:53 +01:00

Added subscriber: @Sergey

Brecht Van Lommel commented

2014-01-09 00:32:53 +01:00

Might be related to threaded updates, maybe @Sergey has a clue from this backtrace, I don't see an obvious cause. Valgrind or hellgrind might be able to find an error here as well.

Daniel Salazar commented

2014-01-09 00:33:56 +01:00

@brecht ok maybe you can help me get a valgrind bt on irc?

Brecht Van Lommel commented

2014-01-09 00:43:13 +01:00

You just run valgrind ./blender and it prints error messages. It may be that some tweaks are needed if it prints too many false positives, but I don't know the commands off hand, would need to boot into linux but then I can run valgrind myself anyway.

You just run `valgrind ./blender` and it prints error messages. It may be that some tweaks are needed if it prints too many false positives, but I don't know the commands off hand, would need to boot into linux but then I can run valgrind myself anyway.

Daniel Salazar commented

2014-01-09 01:20:14 +01:00

Valgrind is impossible here.. takes minutes to open startup file with hundreds of errors, and then It just freezes opening the file :(

Bastien Montagne commented

2014-01-15 14:01:57 +01:00

Added subscriber: @mont29

Bastien Montagne commented

2014-01-15 14:01:57 +01:00

Here is a session with asan (both asan output and some gdb investigations): tmp.txt. Looks like there is some weird stuff happening at alloc time (the origindex_edge is only two ints-length), so smells indeed threading issue…

Here is a session with asan (both asan output and some gdb investigations): [tmp.txt](https://archive.blender.org/developer/F68858/tmp.txt). Looks like there is some weird stuff happening at alloc time (the origindex_edge is only two ints-length), so smells indeed threading issue…

Sergey Sharybin self-assigned this 2014-01-15 14:08:50 +01:00

Sergey Sharybin commented

2014-01-15 14:08:50 +01:00

Added subscriber: @ideasman42

Sergey Sharybin commented

2014-01-15 14:08:50 +01:00

This is indeed a threading issue caused by some modifiers modifying original mesh's MVerts by assigning ME_VERT_TMP_TAG. Working on this atm, but would need to talk to @ideasman42 about way to go with this.

Sergey Sharybin commented

2014-01-15 14:13:54 +01:00

Correction, only solidify uses this tag, which is not so bad. Here's the quick fix for the issue: http://www.pasteall.org/48745/diff

It adds extra array alloction, but on the one hand it's not so bad perhaps since there're some more arrays being allocated there. Possible ways to improve:

Keep the same array allocation, but pack tags into bits. Would make allocation 8x smaller
Try somehow re-using old_vert_arr array by storing tag there as well using some magic constant value (-1 i.e.)

@ideasman42, your call.

Correction, only solidify uses this tag, which is not so bad. Here's the quick fix for the issue: http://www.pasteall.org/48745/diff It adds extra array alloction, but on the one hand it's not so bad perhaps since there're some more arrays being allocated there. Possible ways to improve: * Keep the same array allocation, but pack tags into bits. Would make allocation 8x smaller * Try somehow re-using `old_vert_arr` array by storing tag there as well using some magic constant value (-1 i.e.) @ideasman42, your call.

Sergey Sharybin commented

2014-01-16 12:46:55 +01:00

Changed status from 'Open' to: 'Duplicate'

Sergey Sharybin closed this issue

2014-01-16 12:46:55 +01:00

Sergey Sharybin commented

2014-01-16 12:46:55 +01:00

✘ Merged into #38116.

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 on transforming UVs #38109