From 6c2c3a49c4564aeb4f3db4d4fdfd9503e5c24d62 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Thu, 5 May 2022 00:46:11 +0200 Subject: [PATCH] Update comment about required buffers --- libs/ardour/globals.cc | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/libs/ardour/globals.cc b/libs/ardour/globals.cc index df472cc9af..5a86a2348d 100644 --- a/libs/ardour/globals.cc +++ b/libs/ardour/globals.cc @@ -646,10 +646,14 @@ ARDOUR::init (bool try_optimization, const char* localedir, bool with_gui) */ EventLoop::register_request_buffer_factory (X_("midiUI"), MidiControlUI::request_factory); - /* the + 4 is a bit of a handwave. i don't actually know - how many more per-thread buffer sets we need above - the h/w concurrency, but its definitely > 1 more. - */ + /* Every Process Graph thread (up to hardware_concurrency) keeps a buffer. + * The main engine callback uses one (but returns it after use + * each cycle). Session Export uses one, and the GUI requires + * buffers (for plugin-analysis, auditioner updates) but not + * concurrently. + * + * In theory (hw + 3) should be sufficient, let's add one for luck. + */ BufferManager::init (hardware_concurrency () + 4); PannerManager::instance ().discover_panners ();