Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

32bit vst2 : certain modules cause crashes #80

Open
offthesky opened this issue Dec 10, 2021 · 33 comments
Open

32bit vst2 : certain modules cause crashes #80

offthesky opened this issue Dec 10, 2021 · 33 comments
Projects

Comments

@offthesky
Copy link

offthesky commented Dec 10, 2021

haven't tested them all yet but here are ones that consistently crash when trying to load them:
audible: liquid filter, eq filter
amalgamated: d'acchord
befaco: even vco
e series: cloud generator
mind meld: none of these work
vortico: vco, wt vco, lfo, wt lfo, vcf, sometimes adsr (or, it will let you load adsr and then the project will crash upon reopen. which is the case for several others)
Valley, all their oscillators and feline

seems polyphony is a common denominator.

of course no issues in win x64 (am testing this out in 32bit since i teach classes for kids in a place that still has 32bit pcs)

@falkTX
Copy link
Contributor

falkTX commented Dec 10, 2021

that is unusual, need to clarify some points:

  • do these 32bit PCs have a CPU capable of SSE2 instructions?
  • do you also get crashes if running the 32bit plugin on a 64bit PC? (preferably the same where you test the 64bit plugin)
  • what Windows version are the 32bit PCs running?

@offthesky
Copy link
Author

offthesky commented Dec 10, 2021

do these 32bit PCs have a CPU capable of SSE2 instructions

from wikichip.org ::
Atom x5-Z8350 Supported x86 Extensions & Processor Features

MMX MMX Extension
SSE Streaming SIMD Extensions
SSE2 Streaming SIMD Extensions 2
SSE3 Streaming SIMD Extensions 3
SSSE3 Supplemental SSE3
SSE4.1 Streaming SIMD Extensions 4.1
SSE4.2 Streaming SIMD Extensions 4.2
AES AES Encryption Instructions

x86-64 64-bit x86
NX No-eXecute
BPT Burst Performance Technology
EIST Enhanced SpeedStep Technology
VT-x VT-x (Virtualization)

do you also get crashes if running the 32bit plugin on a 64bit PC? (preferably the same where you test the 64bit plugin)

yes, the same modules mentioned above seem to crash the 32bit plugin when loaded up in reaper on my x64 win10 laptop.

what Windows version are the 32bit PCs running?

Windows 10 Home 32bit. my x64 laptop is running win 10 pro

@falkTX
Copy link
Contributor

falkTX commented Dec 10, 2021

ok so it is a problem with the builds, not with the PCs.

The 32bit build uses an old compiler, which could be a reason for such crashes.
Sadly the Ubuntu 20.04 repos seem broken at the moment for it.

Updating the compiler should fix the issue, I will see what can be done.

@offthesky
Copy link
Author

thank you much, you rock!

@falkTX
Copy link
Contributor

falkTX commented Dec 11, 2021

Please try one of the latest builds and let me know if they work now

@offthesky
Copy link
Author

offthesky commented Dec 11, 2021

just tried with 47350ab and unfortunately still getting the same crashes.

side note: when unpacking the latest build in windows, getting duplicate filenames with the new extratone modules (ty for adding those btw, the students will love these since all they want to do is make beats):
extratone 2 knobs same file name 3

@dromer
Copy link
Collaborator

dromer commented Dec 11, 2021

These are not duplicate filenames, but it seems windows case-insensitivity is being a PITA?
Is there an actual issue you are seeing with these?

@falkTX
Copy link
Contributor

falkTX commented Dec 11, 2021

This breaks also macOS too since it uses case-sensitive filenames by default.
It makes extracting resources showing question about replacing files, and what files end up actually on the filesystem is undefined.
Reported the issue upstream EaterOfSheep/Extratone#5

@offthesky
Copy link
Author

just got a chance to try 32bit f998ee2 and am, unfortunately, still getting the crashes with a lot of modules

@falkTX
Copy link
Contributor

falkTX commented Dec 15, 2021

I can confirm the crash.
The Befaco VCO one triggers

Unhandled page fault on read access to FFFFFFFF at address 0143CC4B

Likely a null pointer de-reference.

@offthesky
Copy link
Author

not sure if it would be useful but i have a fairly exhaustive "to avoid" list going now of most all the crashing modules. if for any reason that might be helpful here, let me know.

@dromer
Copy link
Collaborator

dromer commented Dec 16, 2021

Which host are you using? How is the LV2 version for you?

@falkTX
Copy link
Contributor

falkTX commented Dec 16, 2021

host doesnt really matter. I can trigger the issues with a compiled standalone.

@falkTX falkTX added this to Needs triage in Bug fixes via automation Jan 20, 2022
@falkTX
Copy link
Contributor

falkTX commented Feb 14, 2022

This seems to have improved considerably. I was able to load audible, mindmeld and valleyaudio stuff on a win32 build.
But some still cause the host to trigger an exception/crash warning dialog.
FLStudio was still able to continue with the audio and Cardinal running if we ignore just close the dialog, so not all bad..

Also noticed the knobs do not move, an issue we had previously due to NaNs being converted to 0.

So the first Cardinal release will be out without this fixed, but at least there is some ideas on things to investigate.

@offthesky
Copy link
Author

offthesky commented Feb 14, 2022

i'm still using an older 32bit (pre vortico/as modules removal), and tbh most of the modules still work great. am super grateful to have vcv pro 2 in any capacity running via 32bit! not sure if it's useful, i made a list of the following that cause crashes (sadly, a lot of the good vcos/vcfs are in this list) - however all other modules outside this list afaik work great. will try out cardinal again when it's fully released and update this list::

*crashes when clicking on the module in the browser
**you can load up and play with, but the daw will crash when reloading project w these modules
***you can load up and play with but will crash if you send a more extreme signal (i.e. white noise) into the modulation cv ports and turn up their modulation knobs (also will most likely crash on reloading the daw project)

amalgamated:
d'achord*

autinn **

audible:
eq**
filter**
dual dynamic gate**

befaco:
even vco*
spring reverb*
sampling modular*
rampage**
slew limiter**

bidoo:
tiare*
limonade*
acne*
zinc*

bog:
vcf**
lvcf**

chowdsp:
chow fdn**
chow chorus*
weiner filter**

e-series
cloud generator*

hetrick:
(all the polyphonic ones)
bitshift*
contrast*
delta compander*
exponent*
flip pan*
mid/side*
min/max*
shaper**
2 to 4*
xy-polar*

mindmeld
mixmaster*
mixmasterjr*
eqmaster**
bassmaster*
bassmasterjr*
meld*

shapemaster works for the most part but crashes when using the crossover/lp/hp filters

valley
dexter*
terraform*

zzc:
fn-3**

@offthesky
Copy link
Author

p.s. all the knobs seem to work fine for me afaik... again, am running an older build so not sure about the latest builds.

@falkTX
Copy link
Contributor

falkTX commented Feb 14, 2022

It is already tagged and released.
I am just doing the pushing to kxstudio repositories and writing the usual announcements.

Such list for the old builds is not that useful, since many core things have changed.
We need to retest.

@offthesky
Copy link
Author

gotcha that makes total sense will certainly re test when i can

@offthesky
Copy link
Author

had a chance to check out the ones on my list in reaper with the win32-22.02 build and everything on that list is still valid/causing crashes. removed the befaco/vortico ones. added autinn as they are causing crashes on project reload sometimes.

@falkTX
Copy link
Contributor

falkTX commented Feb 14, 2022

ah damn, I was hoping things would be better now :(
will revisit this a bit later

@falkTX
Copy link
Contributor

falkTX commented Mar 15, 2022

Good news, I was able to fix most of these issues. Seems the crashes were related to memory alignment, where 64bit OSes do it automatically as needed, but for 32bit ones we need to force it.
Assuming the latest build succeeds, please try it when you have the chance.

From the ones on your list that cause crashes on load I only now have ValleyAudio stuff still crashing, the other ones load and seem to work fine.

@offthesky
Copy link
Author

awesome! tyty will check this latest build out today when i can :)

@offthesky
Copy link
Author

had a chance to go through the list (as well as the new additions since that list was created)... everything seems to work now except two chow dsp ones on the list: chorus - still crashes on load, and the weiner filter still crashes when you reload a project with it in there. really loving having access to all these new filters/oscillators now, so ty for those fixes!!!

falkTX added a commit that referenced this issue Jun 27, 2022
Related to #80

Signed-off-by: falkTX <falktx@falktx.com>
@falkTX
Copy link
Contributor

falkTX commented Jun 27, 2022

FYI Fundamental wavetable modules (WTVCO and WTLFO) were crashing on win32, fixed on latest builds, will be part of 22.06

and btw, Fundamental has 2 new modules, should we try a reskin for those?

@offthesky
Copy link
Author

FYI Fundamental wavetable modules (WTVCO and WTLFO) were crashing on win32, fixed on latest builds, will be part of 22.06

and btw, Fundamental has 2 new modules, should we try a reskin for those?

awesome for the fix and yes, will whip up some svgs for those very soon

@offthesky
Copy link
Author

super stoked about these new mindmeld and stoermelder modules! bravo!! noticing however that the latest 32bit vst2 (build #1309) is crashing right when i try to load it in both reaper and or ableton 32bit

@falkTX
Copy link
Contributor

falkTX commented Dec 2, 2022

I have a suspicion of what is going on. only vst2 crashes, right? both vst3 and clap work under reaper, right?
if you can confirm this, then I will know what it is.

@offthesky
Copy link
Author

offthesky commented Dec 2, 2022

i can load the vst3, lv2 (reaper only).. but was still getting some crashes when connecting the bog llfo > bog vcf to the audio out.

with the clap plugin, i get this when trying to load (i didn't run the installer, guessing that's the problem there)
clap

@offthesky
Copy link
Author

offthesky commented Dec 2, 2022

also, just rebooted and very first thing loaded reaper then the vst3 and received a crash (although didn't crash when i loaded the vst3 after opening reaper a 2nd time)

@offthesky
Copy link
Author

offthesky commented Dec 7, 2022

i'm also noticing that the 32bit vst2 cardinal midi host module -midi out- isn't working in 22.10 or 22.11 (and one of the latest stoermelder builds) but seems to work okay in 22.09. not sure if it's related to any of the vst2 issues above. if not i can make a new ticket if you think that best.

@dromer
Copy link
Collaborator

dromer commented Sep 17, 2023

@offthesky do you mind retrying some of your breaking patches on the latest release?

@offthesky
Copy link
Author

@offthesky do you mind retrying some of your breaking patches on the latest release?

i dont see any mention in the changelist for 32b fixes w the 23.09 rls. once falktx mentions in this thread he's tried to fix something, i'll gladly retest all the diff plugin types when i have time. but until then, holding off.

if you want to try 23.09 in reaper 32b, please do- whenever i've tested in the past i just try using the built in demo patches. they either crash on load or when reloading the saved project. often the default start patch (host midi>vcv vco>vcv adsr>vcv vca>host audio) crashes right when loading the plugin. many of the stock vcv ones crash on load.

thankfully for now, the (mostly) stable 22.09 32b still work very well

@dromer
Copy link
Collaborator

dromer commented Sep 17, 2023

It's more that it has been quite a long time since you tested and I think the builds have been adjusted in the mean time (no more LTO on release builds, for instance). Still worth to confirm that things are broken for you.

I do not have any 32bit windows systems that I could test this on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Bug fixes
Needs triage
Development

No branches or pull requests

3 participants