Revert "Don't override existing host thread priorities"

This reverts commit 9b0324f4a7.

Since the plugin is initialized from the GUI thread, this doesn't make
any sense.
This commit is contained in:
Robbert van der Helm
2021-01-22 00:38:44 +01:00
parent e84f7425cf
commit 58c687f9d7
2 changed files with 1 additions and 13 deletions
-2
View File
@@ -75,8 +75,6 @@ TODO: Add an updated screenshot with some fancy VST3-only plugins to the readme
priority. This should get rid of any latency spikes during those operations,
as this could otherwise steal resources away from the threads that are
processing audio.
- Yabridge no longer overrides the host's scheduling priorities if the thread
yabridge is being hosted from is already using realtime scheduling.
- When `YABRIDGE_DEBUG_LEVEL` is set to 2 or higher and a plugin asks the host
for the current position in the song, yabridge will now also print the current
tempo to help debugging host bugs.
+1 -11
View File
@@ -32,17 +32,7 @@ fs::path get_temporary_directory() {
}
bool set_realtime_priority(bool sched_fifo) {
// If we're already using SCHED_FIFO then this is likely set by the host,
// and we should not make any additional changes
if (sched_fifo) {
sched_param current_params{};
if (sched_getparam(0, &current_params) == 0 &&
current_params.sched_priority > 0) {
return true;
}
}
sched_param params{.sched_priority = (sched_fifo ? 5 : 0)};
sched_param params{.sched_priority = 5};
return sched_setscheduler(0, sched_fifo ? SCHED_FIFO : SCHED_OTHER,
&params) == 0;
}