bbc93d7413
Catch all exceptions during initialization
2021-05-01 18:48:46 +02:00
5c2da36ecd
Use a shorter delay for the first watchdog check
2021-05-01 18:47:45 +02:00
4ff538c7e1
Let group hosts shut down on their own if unused
...
We would never try to shut the group host down if nothing ever tried to
connect to it. This could happen when the native host gets killed after
initializing the yabridge plugin but before it gets the chance to
request the group host process to host a plugin.
2021-05-01 18:38:18 +02:00
a91c1a3733
Reword the watchdog changelog entry
2021-05-01 18:12:42 +02:00
1065095ee2
Add a changelog entry for the watchdog
2021-05-01 18:08:05 +02:00
56afefeb53
Terminate all threads when initialization fails
...
With this the watchdog can correctly terminate VST2 bridges that get
stuck during initialization.
2021-05-01 17:58:46 +02:00
bda9a0b75f
Catch exceptions in failing host callbacks
...
This will let us more or less gracefully handle failing host callbacks
during initialization. We cannot catch this from anywhere else since
this these functions get called from unmanaged code.
2021-05-01 17:57:41 +02:00
832089d4d1
Implement the Wine host process watchdog
...
This will shut down a bridge's sockets when the connected native host
process exits, to prevent dangling Wine processes.
2021-05-01 17:54:22 +02:00
e912bdd302
Also make sure that all group host threads exit
...
I actually don't know if this is necessary, but it was necessary for
individually hosted plugins, so presumably it's also needed here.
2021-05-01 17:20:13 +02:00
757fb6d372
Pass the PID of the native host to the Wine hosts
...
We need this for our watchdog.
2021-05-01 17:13:16 +02:00
071bb157ad
Move the pid_running() function to common utils
...
We're going to need this for our watchdog.
2021-05-01 17:08:47 +02:00
131d749ba6
Register the Wine host bridge watchdog
...
It doesn't do anything yet.
2021-05-01 16:52:05 +02:00
d8ca4022d1
Add universal way to shutdown sockets on Wine side
2021-05-01 16:51:55 +02:00
0f2db64aeb
Add the pieces for a watchdog for dangling plugins
...
Once we register our plugins here, we should be able to gracefully shut
down any plugins whose remote process got killed.
2021-05-01 16:51:55 +02:00
167784f93b
Prevent unnecessary copies in ScopedValueCache
2021-05-01 16:28:14 +02:00
7881e0116e
Rename host_guard_handler to host_watchdog_handler
...
To keep the terminology consistent.
2021-05-01 15:46:27 +02:00
aeca37d795
Rearrange src/wine-host/utils.h
...
We're going to need to spawn a Win32Thread in the MainContext to act as
a watchdog.
2021-05-01 15:46:27 +02:00
b22f207aee
Move the MainContext to HostBridge
...
We'll use this to create a watchdog timer that shuts down the sockets
when the native host gets terminated.
2021-05-01 15:06:13 +02:00
564e047b4b
Remove the Ardour warning for VST3 plugins
2021-05-01 12:53:38 +02:00
5b28d44797
Define a static library for the Wine hosts
...
This means we only have to compile everything two times, instead of four
times. We sadly still need to compile lots of files at least three
times. (once for native, once for 64-bit Wine, and once for 32-bit Wine)
2021-05-01 01:09:58 +02:00
177431e202
Fix performEdit() deadlock in Ardour/Mixbus
...
Those DAWs would immediately call `IEditController::performEdit()` with
the same parameter change the plugin has just announced, which would
result in a deadlock. Hopefully this helps with #100 .
2021-04-30 20:50:54 +02:00
2fe6b1c48d
Clean up the main VST3 mutual recursion function
2021-04-30 20:44:50 +02:00
89d6c1b2e0
Fix assertion failure when reading end of stream
...
Getting the address of the end of a vector apparently raises an
assertion failure now, even if we didn't do anything with it.
2021-04-30 17:00:04 +02:00
588a8fd590
Better structure the meson.build file
...
It's been growing a bit over the last year.
2021-04-30 16:16:35 +02:00
5680ed5035
Enable additional assertions in debug builds
2021-04-30 16:06:50 +02:00
66be22b902
Clarify the REAPER context menu regression
2021-04-30 12:47:16 +02:00
36b3636072
Handle mutual recursion in context menus
...
REAPER will call `getState()` while the context menu is open, and that
also has to be handled from the GUI thread.
2021-04-30 02:49:54 +02:00
52428c8749
Also fix mutual recursion across both components
...
When setting the state on the audio processor, it can happen that plugin
triggers a resize from the edit controller. We should also be able to
handle that situation.
2021-04-30 02:11:24 +02:00
949ddaf673
Use mutual recursion for IEditController::setState
...
With this we should be able to handle `setState()`s that try to resize
the currently open editor. This could pop up when using the preset
browser in REAPER with plugins that recall their old size when loading a
preset.
2021-04-30 01:15:18 +02:00
0536256217
Add clamping to IBStream implementation seek
2021-04-29 23:40:42 +02:00
cb07fd07a2
Handle mutual recursion while loading presets
2021-04-29 23:40:42 +02:00
74dd7f61a2
Fix ISizeableStream::getStreamSize implementation
2021-04-29 22:49:16 +02:00
4a92034620
Remove the recommendation for preferring VST3
...
At least for performance reasons, since with the changes in yabridge
3.2.0 they VST2 plugins perform as good or better depending on the
plugin groups settings. Using plugin groups is still less preferable
than using the VST3 version of course.
2021-04-29 19:21:19 +02:00
0f506f75e4
Bypass connection point proxies when possible
...
This greatly improves compatibility with VST3 plugins in Ardour and
Mixbus. Most notably the FabFilter plugins would previously freeze when
having the GUI open while duplicating or inserting new instances.
2021-04-29 15:36:28 +02:00
e4177f2856
Avoid IAudioProcessor::isActive() on GUI thread
...
I haven't seen this cause any issues this way, and I could imagine that
this could cause some hangs when initializing a second instance of a
plugin while you're interacting with the GUI of the first instance.
2021-04-29 14:29:15 +02:00
a56e4b337f
Rename the VST3 mutual recursion functions
2021-04-29 14:03:15 +02:00
45d83ad9a1
Revert "Separate mutual recursion on GUI and other threads"
...
This reverts commit a495f1a67f
2021-04-29 13:54:48 +02:00
a2e3e691d2
Change wording in the changelog
2021-04-29 12:30:19 +02:00
a0abfed689
Merge the two prefetching changelog entries
2021-04-29 12:25:54 +02:00
a495f1a67f
Separate mutual recursion on GUI and other threads
...
I wasn't able to get this to clash, but this way we can be 100% sure
that there aren't any weird issues.
2021-04-29 12:07:11 +02:00
8b168b310c
Fix mutual recursion with latency in Ardour/Mixbus
...
This would cause Ardour and Mixbus to freeze when inserting a latency
introducing (JUCE based) VST3 plugin. As mentioned in #98 .
2021-04-29 03:13:53 +02:00
22f94dd22f
Prevent unnecessary copies in ScopedValueCache
2021-04-29 02:58:05 +02:00
ee24969539
Mention SWAM Cello among the fixed plugins
...
Since this now no longer requires an option to get good performance.
2021-04-29 02:19:17 +02:00
fb04524b1a
Update the tested Mixbus version
2021-04-29 01:22:50 +02:00
6f6e6c5b94
Reword caching to prefetching
...
Since that makes it much clearer what we're actually doing. With old
`cache_time_info` was actually caching the response, but now we're
querying it before the plugin has even requested the information.
2021-04-29 01:07:14 +02:00
c82eb35243
Also cache audioMasterGetCurrentProcessLevel()
...
Melda plugins seem to call this during every processing cycle.
2021-04-29 01:02:10 +02:00
383b0c3f10
Remove the cache_time_info option
...
The new time info caching behaviour supersedes this by getting rid of
callbacks altogether.
2021-04-29 00:46:43 +02:00
2a9840828f
Add a todo for also caching the process level
2021-04-29 00:42:44 +02:00
b8f8dd7236
Log cached audioMasterGetTime() calls
...
Just like we log cached VST3 function calls.
2021-04-29 00:41:56 +02:00
1deb4cf664
Send the VST2 transport info along with processing
...
And cache it during the processing cycle. This greatly reduces the
overhead of bridging VST2 plugins.
2021-04-29 00:32:25 +02:00
Robbert van der Helm