aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-10-10 16:47:57 +0200
committerMax Kellermann <max@duempel.org>2008-10-10 16:47:57 +0200
commit17d9c1708b5afe32c362fdaba114e74f64a7dd06 (patch)
tree530967bf2c6bd2b8e29c89df5fcf93e9dab4d03f /src
parentf41fe1e0b5d86470fab406831c18d032e9835d84 (diff)
downloadmpd-17d9c1708b5afe32c362fdaba114e74f64a7dd06.tar.gz
mpd-17d9c1708b5afe32c362fdaba114e74f64a7dd06.tar.xz
mpd-17d9c1708b5afe32c362fdaba114e74f64a7dd06.zip
player: don't wake up decoder after every frame
The decoder was woken up after each chunk which had been played. That caused a lot of superfluous context switches. Wake up the decoder only when a certain amount of the buffer has been consumed. This formula is somewhat arbitrary, and has to be proven experimentally.
Diffstat (limited to '')
-rw-r--r--src/player_thread.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/src/player_thread.c b/src/player_thread.c
index c5a51a00a..b82005aec 100644
--- a/src/player_thread.c
+++ b/src/player_thread.c
@@ -339,7 +339,13 @@ static void do_play(void)
sizeToTime) < 0)
break;
ob_shift();
- notify_signal(&dc.notify);
+
+ /* this formula should prevent that the
+ decoder gets woken up with each chunk; it
+ is more efficient to make it decode a
+ larger block at a time */
+ if (ob_available() <= (pc.buffered_before_play + ob.size * 3) / 4)
+ notify_signal(&dc.notify);
} else if (!ob_is_empty() && (int)ob.begin == next) {
/* at the beginning of a new song */