From f3c498c7f10dea395ee098a4d1d7e2f9602e2a99 Mon Sep 17 00:00:00 2001 From: wm4 Date: Thu, 13 Feb 2020 01:06:02 +0100 Subject: ao: avoid unnecessary wakeups If ao_add_events() is used, but all events flags are already set, then we don't need to wakeup the core again. Also, make the underrun message "exact" by avoiding the race condition mentioned in the comment. Avoiding redundant wakeups is not really worth the trouble, and it's actually just a bonus in the change making the ao_underrun_event() function return whether a new underrun was set, which is needed by the following commit. --- audio/out/ao.c | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) (limited to 'audio/out/ao.c') diff --git a/audio/out/ao.c b/audio/out/ao.c index 73131020c0..fa6cc9e76d 100644 --- a/audio/out/ao.c +++ b/audio/out/ao.c @@ -444,10 +444,14 @@ int ao_query_and_reset_events(struct ao *ao, int events) return atomic_fetch_and(&ao->events_, ~(unsigned)events) & events; } -void ao_add_events(struct ao *ao, int events) +// Returns events that were set by this calls. +int ao_add_events(struct ao *ao, int events) { - atomic_fetch_or(&ao->events_, events); - ao->wakeup_cb(ao->wakeup_ctx); + unsigned prev_events = atomic_fetch_or(&ao->events_, events); + unsigned new = events & ~prev_events; + if (new) + ao->wakeup_cb(ao->wakeup_ctx); + return new; } // Request that the player core destroys and recreates the AO. Fully thread-safe. @@ -462,12 +466,13 @@ void ao_hotplug_event(struct ao *ao) ao_add_events(ao, AO_EVENT_HOTPLUG); } -void ao_underrun_event(struct ao *ao) +// Returns whether this call actually set a new underrun flag. +bool ao_underrun_event(struct ao *ao) { - // Racy check, but it's just for the message. - if (!(atomic_load(&ao->events_) & AO_EVENT_UNDERRUN)) + bool new_underrun = ao_add_events(ao, AO_EVENT_UNDERRUN); + if (new_underrun) MP_WARN(ao, "Device underrun detected.\n"); - ao_add_events(ao, AO_EVENT_UNDERRUN); + return new_underrun; } bool ao_chmap_sel_adjust(struct ao *ao, const struct mp_chmap_sel *s, -- cgit v1.2.3