fixes for various bugs including dangling ref to route in session, opening sessions...
[ardour.git] / libs / ardour / audioengine.cc
index 982a7c5971cb0fa0c718444fd6e32a97d608aeb1..1b4ca05d389a93448d066d290b828a9b370032df 100644 (file)
@@ -43,10 +43,11 @@ using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 
-jack_nframes_t Port::_short_over_length = 2;
-jack_nframes_t Port::_long_over_length = 10;
+nframes_t Port::_short_over_length = 2;
+nframes_t Port::_long_over_length = 10;
 
 AudioEngine::AudioEngine (string client_name) 
+       : ports (new Ports)
 {
        session = 0;
        session_remove_pending = false;
@@ -62,16 +63,16 @@ AudioEngine::AudioEngine (string client_name)
        _buffer_size = 0;
        _freewheeling = false;
        _freewheel_thread_registered = false;
-    
-    m_meter_thread = 0;
-    m_meter_exit = false;
 
-    start_metering_thread();
-    
+       m_meter_thread = 0;
+       m_meter_exit = false;
+
        if (connect_to_jack (client_name)) {
                throw NoBackendAvailable ();
        }
 
+       start_metering_thread();
+
 }
 
 AudioEngine::~AudioEngine ()
@@ -80,9 +81,9 @@ AudioEngine::~AudioEngine ()
                jack_client_close (_jack);
        }
 
-    if(m_meter_thread) {
-        g_atomic_int_inc(&m_meter_exit);
-    }
+       if(m_meter_thread) {
+               g_atomic_int_inc(&m_meter_exit);
+       }
 }
 
 void
@@ -101,7 +102,7 @@ AudioEngine::start ()
        if (!_running) {
 
                if (session) {
-                       jack_nframes_t blocksize = jack_get_buffer_size (_jack);
+                       nframes_t blocksize = jack_get_buffer_size (_jack);
 
                        session->set_block_size (blocksize);
                        session->set_frame_rate (jack_get_sample_rate (_jack));
@@ -162,18 +163,37 @@ AudioEngine::stop ()
 }
 
 
-void
-AudioEngine::_jack_timebase_callback (jack_transport_state_t state, jack_nframes_t nframes,
+       
+bool
+AudioEngine::get_sync_offset (nframes_t& offset) const
+{
+
+#ifdef HAVE_JACK_VIDEO_SUPPORT
+
+       jack_position_t pos;
+       
+       (void) jack_transport_query (_jack, &pos);
+
+       if (pos.valid & JackVideoFrameOffset) {
+               offset = pos.video_offset;
+               return true;
+       }
+
+#endif
+
+       return false;
+}
 
-                                                                         jack_position_t* pos, int new_position, void *arg)
+void
+AudioEngine::_jack_timebase_callback (jack_transport_state_t state, nframes_t nframes,
+                                     jack_position_t* pos, int new_position, void *arg)
 {
        static_cast<AudioEngine*> (arg)->jack_timebase_callback (state, nframes, pos, new_position);
 }
 
 void
-AudioEngine::jack_timebase_callback (jack_transport_state_t state, jack_nframes_t nframes,
-
-                                                                        jack_position_t* pos, int new_position)
+AudioEngine::jack_timebase_callback (jack_transport_state_t state, nframes_t nframes,
+                                    jack_position_t* pos, int new_position)
 {
        if (session && session->synced_to_jack()) {
                session->jack_timebase_callback (state, nframes, pos, new_position);
@@ -206,12 +226,12 @@ AudioEngine::_xrun_callback (void *arg)
 int
 AudioEngine::_graph_order_callback (void *arg)
 {
-        static_cast<AudioEngine *>(arg)->GraphReordered (); /* EMIT SIGNAL */
+       static_cast<AudioEngine *>(arg)->GraphReordered (); /* EMIT SIGNAL */
        return 0;
 }
 
 int
-AudioEngine::_process_callback (jack_nframes_t nframes, void *arg)
+AudioEngine::_process_callback (nframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->process_callback (nframes);
 }
@@ -223,11 +243,11 @@ AudioEngine::_freewheel_callback (int onoff, void *arg)
 }
 
 int
-AudioEngine::process_callback (jack_nframes_t nframes)
+AudioEngine::process_callback (nframes_t nframes)
 {
        // CycleTimer ct ("AudioEngine::process");
        Glib::Mutex::Lock tm (_process_lock, Glib::TRY_LOCK);
-       jack_nframes_t next_processed_frames;
+       nframes_t next_processed_frames;
        
        /* handle wrap around of total frames counter */
 
@@ -270,7 +290,10 @@ AudioEngine::process_callback (jack_nframes_t nframes)
        }
 
        if (last_monitor_check + monitor_check_interval < next_processed_frames) {
-               for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
+
+               boost::shared_ptr<Ports> p = ports.reader();
+
+               for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        
                        Port *port = (*i);
                        bool x;
@@ -291,13 +314,13 @@ AudioEngine::process_callback (jack_nframes_t nframes)
 }
 
 int
-AudioEngine::_sample_rate_callback (jack_nframes_t nframes, void *arg)
+AudioEngine::_sample_rate_callback (nframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->jack_sample_rate_callback (nframes);
 }
 
 int
-AudioEngine::jack_sample_rate_callback (jack_nframes_t nframes)
+AudioEngine::jack_sample_rate_callback (nframes_t nframes)
 {
        _frame_rate = nframes;
        _usecs_per_cycle = (int) floor ((((double) frames_per_cycle() / nframes)) * 1000000.0);
@@ -317,19 +340,21 @@ AudioEngine::jack_sample_rate_callback (jack_nframes_t nframes)
 }
 
 int
-AudioEngine::_bufsize_callback (jack_nframes_t nframes, void *arg)
+AudioEngine::_bufsize_callback (nframes_t nframes, void *arg)
 {
        return static_cast<AudioEngine *> (arg)->jack_bufsize_callback (nframes);
 }
 
 int
-AudioEngine::jack_bufsize_callback (jack_nframes_t nframes)
+AudioEngine::jack_bufsize_callback (nframes_t nframes)
 {
        _buffer_size = nframes;
        _usecs_per_cycle = (int) floor ((((double) nframes / frame_rate())) * 1000000.0);
        last_monitor_check = 0;
 
-       for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
+       boost::shared_ptr<Ports> p = ports.reader();
+
+       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                (*i)->reset();
        }
 
@@ -343,9 +368,9 @@ AudioEngine::jack_bufsize_callback (jack_nframes_t nframes)
 void
 AudioEngine::start_metering_thread ()
 {
-    if(m_meter_thread == 0) {
-        m_meter_thread = Glib::Thread::create (sigc::mem_fun(this, &AudioEngine::meter_thread), false);
-    }
+       if(m_meter_thread == 0) {
+               m_meter_thread = Glib::Thread::create (sigc::mem_fun(this, &AudioEngine::meter_thread), false);
+       }
 }
 
 void
@@ -400,15 +425,19 @@ AudioEngine::register_input_port (DataType type, const string& portname)
                }
        }
 
-       jack_port_t *p = jack_port_register (_jack, portname.c_str(),
-               type.to_jack_type(), JackPortIsInput, 0);
+       jack_port_t *p = jack_port_register (_jack, portname.c_str(), type.to_jack_type(), JackPortIsInput, 0);
 
        if (p) {
 
                Port *newport;
+
                if ((newport = new Port (p)) != 0) {
-                       ports.insert (ports.begin(), newport);
+                       RCUWriter<Ports> writer (ports);
+                       boost::shared_ptr<Ports> ps = writer.get_copy ();
+                       ps->insert (ps->begin(), newport);
+                       /* writer goes out of scope, forces update */
                }
+
                return newport;
 
        } else {
@@ -436,8 +465,19 @@ AudioEngine::register_output_port (DataType type, const string& portname)
 
        if ((p = jack_port_register (_jack, portname.c_str(),
                type.to_jack_type(), JackPortIsOutput, 0)) != 0) {
-               Port *newport = new Port (p);
-               ports.insert (ports.begin(), newport);
+
+               Port *newport = 0;
+
+               {
+                       RCUWriter<Ports> writer (ports);
+                       boost::shared_ptr<Ports> ps = writer.get_copy ();
+                       
+                       newport = new Port (p);
+                       ps->insert (ps->begin(), newport);
+
+                       /* writer goes out of scope, forces update */
+               }
+
                return newport;
 
        } else {
@@ -465,12 +505,20 @@ AudioEngine::unregister_port (Port *port)
                int ret = jack_port_unregister (_jack, port->_port);
                
                if (ret == 0) {
-
-                       for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
-                               if ((*i) == port) {
-                                       ports.erase (i);
-                                       break;
+                       
+                       {
+
+                               RCUWriter<Ports> writer (ports);
+                               boost::shared_ptr<Ports> ps = writer.get_copy ();
+                               
+                               for (Ports::iterator i = ps->begin(); i != ps->end(); ++i) {
+                                       if ((*i) == port) {
+                                               ps->erase (i);
+                                               break;
+                                       }
                                }
+
+                               /* writer goes out of scope, forces update */
                        }
 
                        remove_connections_for (port);
@@ -563,7 +611,7 @@ AudioEngine::disconnect (Port *port)
 
 }
 
-jack_nframes_t
+nframes_t
 AudioEngine::frame_rate ()
 {
        if (_jack) {
@@ -580,7 +628,7 @@ AudioEngine::frame_rate ()
        }
 }
 
-jack_nframes_t
+nframes_t
 AudioEngine::frames_per_cycle ()
 {
        if (_jack) {
@@ -613,7 +661,9 @@ AudioEngine::get_port_by_name (const string& portname, bool keep)
        
        /* check to see if we have a Port for this name already */
 
-       for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
+       boost::shared_ptr<Ports> pr = ports.reader();
+       
+       for (Ports::iterator i = pr->begin(); i != pr->end(); ++i) {
                if (portname == (*i)->name()) {
                        return (*i);
                }
@@ -623,11 +673,20 @@ AudioEngine::get_port_by_name (const string& portname, bool keep)
 
        if ((p = jack_port_by_name (_jack, portname.c_str())) != 0) {
                Port *newport = new Port (p);
-               if (keep && newport->is_mine (_jack)) {
-                       ports.insert (newport);
+
+               {
+                       if (keep && newport->is_mine (_jack)) {
+                               RCUWriter<Ports> writer (ports);
+                               boost::shared_ptr<Ports> ps = writer.get_copy ();
+                               ps->insert (newport);
+                               /* writer goes out of scope, forces update */
+                       }
                }
+
                return newport;
+
        } else {
+
                return 0;
        }
 }
@@ -702,6 +761,50 @@ AudioEngine::n_physical_inputs () const
        return i;
 }
 
+void
+AudioEngine::get_physical_inputs (vector<string>& ins)
+{
+       const char ** ports;
+       uint32_t i = 0;
+       
+       if (!_jack) {
+               return;
+       }
+       
+       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsOutput)) == 0) {
+               return;
+       }
+
+       if (ports) {
+               for (i = 0; ports[i]; ++i) {
+                       ins.push_back (ports[i]);
+               }
+               free (ports);
+       }
+}
+
+void
+AudioEngine::get_physical_outputs (vector<string>& outs)
+{
+       const char ** ports;
+       uint32_t i = 0;
+       
+       if (!_jack) {
+               return;
+       }
+       
+       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsInput)) == 0) {
+               return;
+       }
+
+       if (ports) {
+               for (i = 0; ports[i]; ++i) {
+                       outs.push_back (ports[i]);
+               }
+               free (ports);
+       }
+}
+
 string
 AudioEngine::get_nth_physical (uint32_t n, int flag)
 {
@@ -735,7 +838,7 @@ AudioEngine::get_nth_physical (uint32_t n, int flag)
        return ret;
 }
 
-jack_nframes_t
+nframes_t
 AudioEngine::get_port_total_latency (const Port& port)
 {
        if (!_jack) {
@@ -772,7 +875,7 @@ AudioEngine::transport_start ()
 }
 
 void
-AudioEngine::transport_locate (jack_nframes_t where)
+AudioEngine::transport_locate (nframes_t where)
 {
        // cerr << "tell JACK to locate to " << where << endl;
        if (_jack) {
@@ -827,12 +930,19 @@ AudioEngine::remove_all_ports ()
        /* process lock MUST be held */
 
        if (_jack) {
-               for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
+               boost::shared_ptr<Ports> p = ports.reader();
+
+               for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        jack_port_unregister (_jack, (*i)->_port);
                }
        }
 
-       ports.clear ();
+       {
+               RCUWriter<Ports> writer (ports);
+               boost::shared_ptr<Ports> ps = writer.get_copy ();
+               ps->clear ();
+       }
+
        port_connections.clear ();
 }
 
@@ -932,7 +1042,7 @@ AudioEngine::reconnect_to_jack ()
        if (_jack) {
                disconnect_from_jack ();
                /* XXX give jackd a chance */
-        Glib::usleep (250000);
+               Glib::usleep (250000);
        }
 
        if (connect_to_jack (jack_client_name)) {
@@ -942,7 +1052,9 @@ AudioEngine::reconnect_to_jack ()
 
        Ports::iterator i;
 
-       for (i = ports.begin(); i != ports.end(); ++i) {
+       boost::shared_ptr<Ports> p = ports.reader ();
+
+       for (i = p->begin(); i != p->end(); ++i) {
 
                /* XXX hack hack hack */
 
@@ -964,8 +1076,9 @@ AudioEngine::reconnect_to_jack ()
                }
        }
 
-       if (i != ports.end()) {
-               for (Ports::iterator i = ports.begin(); i != ports.end(); ++i) {
+       if (i != p->end()) {
+               /* failed */
+               for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        jack_port_unregister (_jack, (*i)->_port);
                }
                return -1;
@@ -973,7 +1086,7 @@ AudioEngine::reconnect_to_jack ()
 
 
        if (session) {
-               jack_nframes_t blocksize = jack_get_buffer_size (_jack);
+               nframes_t blocksize = jack_get_buffer_size (_jack);
                session->set_block_size (blocksize);
                session->set_frame_rate (jack_get_sample_rate (_jack));
        }
@@ -1022,7 +1135,7 @@ AudioEngine::reconnect_to_jack ()
 }
 
 int
-AudioEngine::request_buffer_size (jack_nframes_t nframes)
+AudioEngine::request_buffer_size (nframes_t nframes)
 {
        if (_jack) {
                int ret = jack_set_buffer_size (_jack, nframes);