(1) silence non-process FIFO message (2) move MIDI state tracking down to the MidiPor...

[ardour.git] / libs / ardour / meter.cc

diff --git a/libs/ardour/meter.cc b/libs/ardour/meter.cc
index d8fee1ac475e2e663aa792be17b6acdfe9c3d59e..a5e82c3bf3411131dd47d21903bbd371e79071ed 100644 (file)
--- a/libs/ardour/meter.cc
+++ b/libs/ardour/meter.cc
@@ -1,16 +1,16 @@
 /*
-    Copyright (C) 2006 Paul Davis 
-    
+    Copyright (C) 2006 Paul Davis
+
     This program is free software; you can redistribute it and/or modify it
     under the terms of the GNU General Public License as published by the Free
     Software Foundation; either version 2 of the License, or (at your option)
     any later version.
-    
+
     This program is distributed in the hope that it will be useful, but WITHOUT
     ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
     FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
     for more details.
-    
+
     You should have received a copy of the GNU General Public License along
     with this program; if not, write to the Free Software Foundation, Inc.,
     675 Mass Ave, Cambridge, MA 02139, USA.
@@ -29,8 +29,41 @@
 
 using namespace std;
 
-namespace ARDOUR {
+using namespace ARDOUR;
+
+sigc::signal<void> Metering::Meter;
+Glib::StaticMutex  Metering::m_meter_signal_lock;
+
+sigc::connection
+Metering::connect (sigc::slot<void> the_slot)
+{
+       // SignalProcessor::Meter is emitted from another thread so the
+       // Meter signal must be protected.
+       Glib::Mutex::Lock guard (m_meter_signal_lock);
+       return Meter.connect (the_slot);
+}
+
+void
+Metering::disconnect (sigc::connection& c)
+{
+       Glib::Mutex::Lock guard (m_meter_signal_lock);
+       c.disconnect ();
+}
+
+/**
+    Update the meters.
 
+    The meter signal lock is taken to prevent modification of the
+    Meter signal while updating the meters, taking the meter signal
+    lock prior to taking the io_lock ensures that all IO will remain
+    valid while metering.
+*/
+void
+Metering::update_meters()
+{
+       Glib::Mutex::Lock guard (m_meter_signal_lock);
+       Meter(); /* EMIT SIGNAL */
+}
 
 /** Get peaks from @a bufs
  * Input acceptance is lenient - the first n buffers from @a bufs will
@@ -38,13 +71,17 @@ namespace ARDOUR {
  * be set to 0.
  */
 void
-PeakMeter::run_in_place (BufferSet& bufs, nframes_t start_frame, nframes_t end_frame, nframes_t nframes)
+PeakMeter::run (BufferSet& bufs, sframes_t /*start_frame*/, sframes_t /*end_frame*/, nframes_t nframes)
 {
+       if (!_active && !_pending_active) {
+               return;
+       }
+
        const uint32_t n_audio = min(_configured_input.n_audio(), bufs.count().n_audio());
        const uint32_t n_midi  = min(_configured_input.n_midi(), bufs.count().n_midi());
-       
+
        uint32_t n = 0;
-       
+
        // Meter MIDI in to the first n_midi peaks
        for (uint32_t i = 0; i < n_midi; ++i, ++n) {
                float val = 0.0f;
@@ -67,13 +104,20 @@ PeakMeter::run_in_place (BufferSet& bufs, nframes_t start_frame, nframes_t end_f
 
        // Meter audio in to the rest of the peaks
        for (uint32_t i = 0; i < n_audio; ++i, ++n) {
-               _peak_power[n] = compute_peak (bufs.get_audio(i).data(), nframes, _peak_power[n]); 
+               _peak_power[n] = compute_peak (bufs.get_audio(i).data(), nframes, _peak_power[n]);
        }
 
        // Zero any excess peaks
        for (uint32_t i = n; i < _peak_power.size(); ++i) {
                _peak_power[i] = 0.0f;
        }
+
+       _active = _pending_active;
+}
+
+PeakMeter::PeakMeter (Session& s, const XMLNode& node)
+       : Processor (s, node)
+{
 }
 
 void
@@ -105,9 +149,9 @@ PeakMeter::configure_io (ChanCount in, ChanCount out)
        if (out != in) { // always 1:1
                return false;
        }
-       
+
        uint32_t limit = in.n_total();
-       
+
        while (_peak_power.size() > limit) {
                _peak_power.pop_back();
                _visible_peak_power.pop_back();
@@ -128,36 +172,40 @@ PeakMeter::configure_io (ChanCount in, ChanCount out)
 }
 
 /** To be driven by the Meter signal from IO.
- * Caller MUST hold io_lock!
+ * Caller MUST hold its own processor_lock to prevent reconfiguration
+ * of meter size during this call.
  */
+
 void
 PeakMeter::meter ()
 {
+       if (!_active) {
+               return;
+       }
+
        assert(_visible_peak_power.size() == _peak_power.size());
 
        const size_t limit = _peak_power.size();
 
        for (size_t n = 0; n < limit; ++n) {
 
-               /* XXX we should use atomic exchange here */
-
                /* grab peak since last read */
 
-               float new_peak = _peak_power[n];
-               _peak_power[n] = 0;
-               
+               float new_peak = _peak_power[n]; /* XXX we should use atomic exchange from here ... */
+               _peak_power[n] = 0;              /* ... to here */
+
                /* compute new visible value using falloff */
 
                if (new_peak > 0.0) {
-                       new_peak = coefficient_to_dB (new_peak);
+                       new_peak = fast_coefficient_to_dB (new_peak);
                } else {
                        new_peak = minus_infinity();
                }
-               
+
                /* update max peak */
-               
+
                _max_peak_power[n] = std::max (new_peak, _max_peak_power[n]);
-               
+
                if (Config->get_meter_falloff() == 0.0f || new_peak > _visible_peak_power[n]) {
                        _visible_peak_power[n] = new_peak;
                } else {
@@ -171,15 +219,14 @@ PeakMeter::meter ()
 XMLNode&
 PeakMeter::state (bool full_state)
 {
-       return get_state();
+       XMLNode& node (Processor::state (full_state));
+       node.add_property("type", "meter");
+       return node;
 }
 
-XMLNode&
-PeakMeter::get_state()
+
+bool
+PeakMeter::visible() const
 {
-       XMLNode* node = new XMLNode(state_node_name);
-       node->add_property("type", "meter");
-       return *node;
+       return true;
 }
-
-} // namespace ARDOUR