gitea/SDL3_fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

audio: Never SDL_PushEvent from anywhere but SDL_UpdateAudio().

Browse Source 
Fixes some corner-case deadlocks.
This commit is contained in:
Ryan C. Gordon
2023-10-30 13:08:10 -04:00
parent 875e45e70b
commit 9cb259e865
1 changed files with 64 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

84
src/audio/SDL_audio.c
View File

@@ -1916,6 +1916,13 @@ void SDL_DefaultAudioDeviceChanged(SDL_AudioDevice *new_default_device)
return; // this is already the default. return; // this is already the default.
} }
// Queue up events to push to the queue next time it pumps (presumably
// in a safer thread).
// !!! FIXME: this duplicates some code we could probably refactor.
SDL_PendingAudioDeviceEvent pending;
pending.next = NULL;
SDL_PendingAudioDeviceEvent *pending_tail = &pending;
SDL_LockMutex(new_default_device->lock); SDL_LockMutex(new_default_device->lock);
SDL_AudioDevice *current_default_device = ObtainPhysicalAudioDevice(current_devid); SDL_AudioDevice *current_default_device = ObtainPhysicalAudioDevice(current_devid);
@@ -1966,7 +1973,6 @@ void SDL_DefaultAudioDeviceChanged(SDL_AudioDevice *new_default_device)
if (needs_migration) { if (needs_migration) {
const SDL_bool spec_changed = !AUDIO_SPECS_EQUAL(current_default_device->spec, new_default_device->spec); const SDL_bool spec_changed = !AUDIO_SPECS_EQUAL(current_default_device->spec, new_default_device->spec);
const SDL_bool post_fmt_event = (spec_changed && SDL_EventEnabled(SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED)) ? SDL_TRUE : SDL_FALSE;
SDL_LogicalAudioDevice *next = NULL; SDL_LogicalAudioDevice *next = NULL;
for (SDL_LogicalAudioDevice *logdev = current_default_device->logical_devices; logdev != NULL; logdev = next) { for (SDL_LogicalAudioDevice *logdev = current_default_device->logical_devices; logdev != NULL; logdev = next) {
next = logdev->next; next = logdev->next;
@@ -1995,15 +2001,18 @@ void SDL_DefaultAudioDeviceChanged(SDL_AudioDevice *new_default_device)
RefPhysicalAudioDevice(new_default_device); RefPhysicalAudioDevice(new_default_device);
UnrefPhysicalAudioDevice(current_default_device); UnrefPhysicalAudioDevice(current_default_device);
// Post an event for each logical device we moved. SDL_PendingAudioDeviceEvent *p;
if (post_fmt_event) {
SDL_Event event; // Queue an event for each logical device we moved.
SDL_zero(event); if (spec_changed) {
event.type = SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED; p = (SDL_PendingAudioDeviceEvent *)SDL_malloc(sizeof(SDL_PendingAudioDeviceEvent));
event.common.timestamp = 0; if (p) { // if this failed, no event for you, but you have deeper problems anyhow.
event.adevice.iscapture = iscapture ? 1 : 0; p->type = SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED;
event.adevice.which = logdev->instance_id; p->devid = logdev->instance_id;
SDL_PushEvent(&event); p->next = NULL;
pending_tail->next = p;
pending_tail = p;
}
} }
} }
@@ -2027,6 +2036,15 @@ void SDL_DefaultAudioDeviceChanged(SDL_AudioDevice *new_default_device)
if (current_default_device && SDL_AtomicGet(&current_default_device->zombie)) { if (current_default_device && SDL_AtomicGet(&current_default_device->zombie)) {
UnrefPhysicalAudioDevice(current_default_device); UnrefPhysicalAudioDevice(current_default_device);
} }
if (pending.next) {
SDL_LockRWLockForWriting(current_audio.device_hash_lock);
SDL_assert(current_audio.pending_events_tail != NULL);
SDL_assert(current_audio.pending_events_tail->next == NULL);
current_audio.pending_events_tail->next = pending.next;
current_audio.pending_events_tail = pending_tail;
SDL_UnlockRWLock(current_audio.device_hash_lock);
}
} }
int SDL_AudioDeviceFormatChangedAlreadyLocked(SDL_AudioDevice *device, const SDL_AudioSpec *newspec, int new_sample_frames) int SDL_AudioDeviceFormatChangedAlreadyLocked(SDL_AudioDevice *device, const SDL_AudioSpec *newspec, int new_sample_frames)
@@ -2070,17 +2088,43 @@ int SDL_AudioDeviceFormatChangedAlreadyLocked(SDL_AudioDevice *device, const SDL
} }
// Post an event for the physical device, and each logical device on this physical device. // Post an event for the physical device, and each logical device on this physical device.
if (!kill_device && SDL_EventEnabled(SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED)) { if (!kill_device) {
SDL_Event event; // Queue up events to push to the queue next time it pumps (presumably
SDL_zero(event); // in a safer thread).
event.type = SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED; // !!! FIXME: this duplicates some code we could probably refactor.
event.common.timestamp = 0; SDL_PendingAudioDeviceEvent pending;
event.adevice.iscapture = device->iscapture ? 1 : 0; pending.next = NULL;
event.adevice.which = device->instance_id; SDL_PendingAudioDeviceEvent *pending_tail = &pending;
SDL_PushEvent(&event);
SDL_PendingAudioDeviceEvent *p;
p = (SDL_PendingAudioDeviceEvent *)SDL_malloc(sizeof(SDL_PendingAudioDeviceEvent));
if (p) { // if this failed, no event for you, but you have deeper problems anyhow.
p->type = SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED;
p->devid = device->instance_id;
p->next = NULL;
pending_tail->next = p;
pending_tail = p;
}
for (SDL_LogicalAudioDevice *logdev = device->logical_devices; logdev != NULL; logdev = logdev->next) { for (SDL_LogicalAudioDevice *logdev = device->logical_devices; logdev != NULL; logdev = logdev->next) {
event.adevice.which = logdev->instance_id; p = (SDL_PendingAudioDeviceEvent *)SDL_malloc(sizeof(SDL_PendingAudioDeviceEvent));
SDL_PushEvent(&event); if (p) { // if this failed, no event for you, but you have deeper problems anyhow.
p->type = SDL_EVENT_AUDIO_DEVICE_FORMAT_CHANGED;
p->devid = logdev->instance_id;
p->next = NULL;
pending_tail->next = p;
pending_tail = p;
}
}
if (pending.next) {
SDL_LockRWLockForWriting(current_audio.device_hash_lock);
SDL_assert(current_audio.pending_events_tail != NULL);
SDL_assert(current_audio.pending_events_tail->next == NULL);
current_audio.pending_events_tail->next = pending.next;
current_audio.pending_events_tail = pending_tail;
SDL_UnlockRWLock(current_audio.device_hash_lock);
} }
} }