summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibAudio/MP3Loader.cpp
diff options
context:
space:
mode:
authorkleines Filmröllchen <filmroellchen@serenityos.org>2022-02-20 13:01:22 +0100
committerLinus Groh <mail@linusgroh.de>2022-04-21 13:55:00 +0200
commit49b087f3cd49261164bd4556cd6e9e0d95a3afc1 (patch)
tree6e247f1fe819504cfa1ae4a1b9c9b97adaa40be7 /Userland/Libraries/LibAudio/MP3Loader.cpp
parentcb0e95c928e152d39dc60198ab714f437a2347ce (diff)
downloadserenity-49b087f3cd49261164bd4556cd6e9e0d95a3afc1.zip
LibAudio+Userland: Use new audio queue in client-server communication
Previously, we were sending Buffers to the server whenever we had new audio data for it. This meant that for every audio enqueue action, we needed to create a new shared memory anonymous buffer, send that buffer's file descriptor over IPC (+recfd on the other side) and then map the buffer into the audio server's memory to be able to play it. This was fine for sending large chunks of audio data, like when playing existing audio files. However, in the future we want to move to real-time audio in some applications like Piano. This means that the size of buffers that are sent need to be very small, as just the size of a buffer itself is part of the audio latency. If we were to try real-time audio with the existing system, we would run into problems really quickly. Dealing with a continuous stream of new anonymous files like the current audio system is rather expensive, as we need Kernel help in multiple places. Additionally, every enqueue incurs an IPC call, which are not optimized for >1000 calls/second (which would be needed for real-time audio with buffer sizes of ~40 samples). So a fundamental change in how we handle audio sending in userspace is necessary. This commit moves the audio sending system onto a shared single producer circular queue (SSPCQ) (introduced with one of the previous commits). This queue is intended to live in shared memory and be accessed by multiple processes at the same time. It was specifically written to support the audio sending case, so e.g. it only supports a single producer (the audio client). Now, audio sending follows these general steps: - The audio client connects to the audio server. - The audio client creates a SSPCQ in shared memory. - The audio client sends the SSPCQ's file descriptor to the audio server with the set_buffer() IPC call. - The audio server receives the SSPCQ and maps it. - The audio client signals start of playback with start_playback(). - At the same time: - The audio client writes its audio data into the shared-memory queue. - The audio server reads audio data from the shared-memory queue(s). Both sides have additional before-queue/after-queue buffers, depending on the exact application. - Pausing playback is just an IPC call, nothing happens to the buffer except that the server stops reading from it until playback is resumed. - Muting has nothing to do with whether audio data is read or not. - When the connection closes, the queues are unmapped on both sides. This should already improve audio playback performance in a bunch of places. Implementation & commit notes: - Audio loaders don't create LegacyBuffers anymore. LegacyBuffer is kept for WavLoader, see previous commit message. - Most intra-process audio data passing is done with FixedArray<Sample> or Vector<Sample>. - Improvements to most audio-enqueuing applications. (If necessary I can try to extract some of the aplay improvements.) - New APIs on LibAudio/ClientConnection which allows non-realtime applications to enqueue audio in big chunks like before. - Removal of status APIs from the audio server connection for information that can be directly obtained from the shared queue. - Split the pause playback API into two APIs with more intuitive names. I know this is a large commit, and you can kinda tell from the commit message. It's basically impossible to break this up without hacks, so please forgive me. These are some of the best changes to the audio subsystem and I hope that that makes up for this :yaktangle: commit. :yakring:
Diffstat (limited to 'Userland/Libraries/LibAudio/MP3Loader.cpp')
-rw-r--r--Userland/Libraries/LibAudio/MP3Loader.cpp15
1 files changed, 6 insertions, 9 deletions
diff --git a/Userland/Libraries/LibAudio/MP3Loader.cpp b/Userland/Libraries/LibAudio/MP3Loader.cpp
index ca0199f805..b8fc92c949 100644
--- a/Userland/Libraries/LibAudio/MP3Loader.cpp
+++ b/Userland/Libraries/LibAudio/MP3Loader.cpp
@@ -7,6 +7,7 @@
#include "MP3Loader.h"
#include "MP3HuffmanTables.h"
#include "MP3Tables.h"
+#include <AK/FixedArray.h>
#include <LibCore/File.h>
#include <LibCore/FileStream.h>
@@ -115,18 +116,17 @@ MaybeLoaderError MP3LoaderPlugin::seek(int const position)
return {};
}
-LoaderSamples MP3LoaderPlugin::get_more_samples(size_t max_bytes_to_read_from_input)
+LoaderSamples MP3LoaderPlugin::get_more_samples(size_t max_samples_to_read_from_input)
{
- Vector<Sample> samples;
+ FixedArray<Sample> samples = LOADER_TRY(FixedArray<Sample>::try_create(max_samples_to_read_from_input));
- size_t samples_to_read = max_bytes_to_read_from_input;
- samples.resize(samples_to_read);
+ size_t samples_to_read = max_samples_to_read_from_input;
while (samples_to_read > 0) {
if (!m_current_frame.has_value()) {
auto maybe_frame = read_next_frame();
if (maybe_frame.is_error()) {
if (m_input_stream->unreliable_eof()) {
- return LegacyBuffer::create_empty();
+ return FixedArray<Sample> {};
}
return maybe_frame.release_error();
}
@@ -156,10 +156,7 @@ LoaderSamples MP3LoaderPlugin::get_more_samples(size_t max_bytes_to_read_from_in
}
m_loaded_samples += samples.size();
- auto maybe_buffer = LegacyBuffer::create_with_samples(move(samples));
- if (maybe_buffer.is_error())
- return LoaderError { LoaderError::Category::Internal, m_loaded_samples, "Couldn't allocate sample buffer" };
- return maybe_buffer.release_value();
+ return samples;
}
MaybeLoaderError MP3LoaderPlugin::build_seek_table()