provide access to Send + Plugins from MIDI binding maps
[ardour.git] / libs / ardour / audioengine.cc
index 3e1ba8b2d9decda1ac82b08f2174b8056658fa85..322365fb9f6783fe5f117fc9287125a7e6be62e6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2002 Paul Davis 
+    Copyright (C) 2002 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
 
 #include "midi++/jack.h"
 
+#include "ardour/amp.h"
+#include "ardour/audio_port.h"
 #include "ardour/audioengine.h"
 #include "ardour/buffer.h"
-#include "ardour/port.h"
-#include "ardour/audio_port.h"
-#include "ardour/midi_port.h"
-#include "ardour/session.h"
+#include "ardour/buffer_set.h"
 #include "ardour/cycle_timer.h"
-#include "ardour/utils.h"
+#include "ardour/delivery.h"
 #include "ardour/event_type_map.h"
+#include "ardour/internal_return.h"
 #include "ardour/io.h"
-#include "ardour/amp.h"
+#include "ardour/meter.h"
+#include "ardour/midi_port.h"
+#include "ardour/port.h"
 #include "ardour/port_set.h"
-#include "ardour/buffer_set.h"
-
+#include "ardour/session.h"
 #include "ardour/timestamps.h"
+#include "ardour/utils.h"
 
 #include "i18n.h"
 
@@ -54,11 +56,16 @@ using namespace ARDOUR;
 using namespace PBD;
 
 gint AudioEngine::m_meter_exit;
+AudioEngine* AudioEngine::_instance = 0;
 
-AudioEngine::AudioEngine (string client_name) 
+#define GET_PRIVATE_JACK_POINTER(j)  jack_client_t* _priv_jack = (jack_client_t*) (j); if (!_priv_jack) { return; }
+#define GET_PRIVATE_JACK_POINTER_RET(j,r) jack_client_t* _priv_jack = (jack_client_t*) (j); if (!_priv_jack) { return r; }
+
+AudioEngine::AudioEngine (string client_name)
        : ports (new Ports)
 {
-       session = 0;
+       _instance = this; /* singleton */
+
        session_remove_pending = false;
        _running = false;
        _has_run = false;
@@ -80,7 +87,7 @@ AudioEngine::AudioEngine (string client_name)
        }
 
        Port::set_engine (this);
-       
+
        // Initialize parameter metadata (e.g. ranges)
        Evoral::Parameter p(NullAutomation);
        p = EventTypeMap::instance().new_parameter(NullAutomation);
@@ -104,12 +111,12 @@ AudioEngine::~AudioEngine ()
        {
                Glib::Mutex::Lock tm (_process_lock);
                session_removed.signal ();
-               
+
                if (_running) {
                        jack_client_close (_jack);
                        _jack = 0;
                }
-               
+
                stop_metering_thread ();
        }
 }
@@ -121,68 +128,80 @@ AudioEngine::jack() const
 }
 
 void
-_thread_init_callback (void *arg)
+_thread_init_callback (void * /*arg*/)
 {
        /* make sure that anybody who needs to know about this thread
           knows about it.
        */
 
-       PBD::notify_gui_about_thread_creation (pthread_self(), X_("Audioengine"), 4096);
-#ifdef WITH_JACK_MIDI
+       char* c = new char[12];
+       strcpy (c, X_("audioengine"));
+       pthread_set_name (c);
+
+       PBD::notify_gui_about_thread_creation ("gui", pthread_self(), X_("Audioengine"), 4096);
+       PBD::notify_gui_about_thread_creation ("midiui", pthread_self(), X_("Audioengine"), 128);
+
+       SessionEvent::create_per_thread_pool (X_("Audioengine"), 512);
+
        MIDI::JACK_MidiPort::set_process_thread (pthread_self());
-#endif // WITH_JACK_MIDI
+}
+
+static void
+ardour_jack_error (const char* msg)
+{
+       error << "JACK: " << msg << endmsg;
 }
 
 int
 AudioEngine::start ()
 {
-       if (!_jack) {
-               error << _("AudioEngine::start() called while disconnected from JACK") << endmsg;
-               return -1;
-       }
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
 
        if (!_running) {
 
-               nframes_t blocksize = jack_get_buffer_size (_jack);
-               
-               if (session) {
+               nframes_t blocksize = jack_get_buffer_size (_priv_jack);
+
+               if (_session) {
                        BootMessage (_("Connect session to engine"));
 
-                       session->set_block_size (blocksize);
-                       session->set_frame_rate (jack_get_sample_rate (_jack));
-                       
+                       _session->set_block_size (blocksize);
+                       _session->set_frame_rate (jack_get_sample_rate (_priv_jack));
+
                        /* page in as much of the session process code as we
                           can before we really start running.
                        */
 
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
-                       session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
+                       _session->process (blocksize);
                }
 
                _processed_frames = 0;
                last_monitor_check = 0;
 
-               jack_on_shutdown (_jack, halted, this);
-               jack_set_graph_order_callback (_jack, _graph_order_callback, this);
-               jack_set_thread_init_callback (_jack, _thread_init_callback, this);
-               jack_set_process_callback (_jack, _process_callback, this);
-               jack_set_sample_rate_callback (_jack, _sample_rate_callback, this);
-               jack_set_buffer_size_callback (_jack, _bufsize_callback, this);
-               jack_set_xrun_callback (_jack, _xrun_callback, this);
-               jack_set_sync_callback (_jack, _jack_sync_callback, this);
-               jack_set_freewheel_callback (_jack, _freewheel_callback, this);
-
-               if (Config->get_jack_time_master()) {
-                       jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
+               jack_on_shutdown (_priv_jack, halted, this);
+               jack_set_graph_order_callback (_priv_jack, _graph_order_callback, this);
+               jack_set_thread_init_callback (_priv_jack, _thread_init_callback, this);
+               jack_set_process_callback (_priv_jack, _process_callback, this);
+               jack_set_sample_rate_callback (_priv_jack, _sample_rate_callback, this);
+               jack_set_buffer_size_callback (_priv_jack, _bufsize_callback, this);
+               jack_set_xrun_callback (_priv_jack, _xrun_callback, this);
+               jack_set_sync_callback (_priv_jack, _jack_sync_callback, this);
+               jack_set_freewheel_callback (_priv_jack, _freewheel_callback, this);
+               jack_set_port_registration_callback (_priv_jack, _registration_callback, this);
+
+               if (_session && _session->config.get_jack_time_master()) {
+                       jack_set_timebase_callback (_priv_jack, 0, _jack_timebase_callback, this);
                }
 
-               if (jack_activate (_jack) == 0) {
+               jack_set_error_function (ardour_jack_error);
+
+               if (jack_activate (_priv_jack) == 0) {
                        _running = true;
                        _has_run = true;
                        Running(); /* EMIT SIGNAL */
@@ -201,12 +220,15 @@ AudioEngine::start ()
 int
 AudioEngine::stop (bool forever)
 {
-       if (_jack) {
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
+
+       if (_priv_jack) {
                if (forever) {
                        disconnect_from_jack ();
                } else {
-                       jack_deactivate (_jack);
+                       jack_deactivate (_priv_jack);
                        Stopped(); /* EMIT SIGNAL */
+                       MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
                }
        }
 
@@ -220,17 +242,21 @@ AudioEngine::get_sync_offset (nframes_t& offset) const
 
 #ifdef HAVE_JACK_VIDEO_SUPPORT
 
+        GET_PRIVATE_JACK_POINTER_RET (_jack, false);
+
        jack_position_t pos;
-       
-       if (_jack) {
-               (void) jack_transport_query (_jack, &pos);
-               
+
+       if (_priv_jack) {
+               (void) jack_transport_query (_priv_jack, &pos);
+
                if (pos.valid & JackVideoFrameOffset) {
                        offset = pos.video_offset;
                        return true;
                }
        }
-
+#else
+       /* keep gcc happy */
+       offset = 0;
 #endif
 
        return false;
@@ -247,8 +273,8 @@ void
 AudioEngine::jack_timebase_callback (jack_transport_state_t state, nframes_t nframes,
                                     jack_position_t* pos, int new_position)
 {
-       if (_jack && session && session->synced_to_jack()) {
-               session->jack_timebase_callback (state, nframes, pos, new_position);
+       if (_jack && _session && _session->synced_to_jack()) {
+               _session->jack_timebase_callback (state, nframes, pos, new_position);
        }
 }
 
@@ -261,8 +287,8 @@ AudioEngine::_jack_sync_callback (jack_transport_state_t state, jack_position_t*
 int
 AudioEngine::jack_sync_callback (jack_transport_state_t state, jack_position_t* pos)
 {
-       if (_jack && session) {
-               return session->jack_sync_callback (state, pos);
+       if (_jack && _session) {
+               return _session->jack_sync_callback (state, pos);
        }
 
        return true;
@@ -305,6 +331,13 @@ AudioEngine::_freewheel_callback (int onoff, void *arg)
        static_cast<AudioEngine*>(arg)->_freewheeling = onoff;
 }
 
+void
+AudioEngine::_registration_callback (jack_port_id_t /*id*/, int /*reg*/, void* arg)
+{
+       AudioEngine* ae = static_cast<AudioEngine*> (arg);
+       ae->PortRegisteredOrUnregistered (); /* EMIT SIGNAL */
+}
+
 void
 AudioEngine::split_cycle (nframes_t offset)
 {
@@ -328,12 +361,13 @@ AudioEngine::split_cycle (nframes_t offset)
 int
 AudioEngine::process_callback (nframes_t nframes)
 {
+       GET_PRIVATE_JACK_POINTER_RET(_jack,0)
        // CycleTimer ct ("AudioEngine::process");
        Glib::Mutex::Lock tm (_process_lock, Glib::TRY_LOCK);
 
        /// The number of frames that will have been processed when we've finished
        nframes_t next_processed_frames;
-       
+
        /* handle wrap around of total frames counter */
 
        if (max_frames - _processed_frames < nframes) {
@@ -342,7 +376,7 @@ AudioEngine::process_callback (nframes_t nframes)
                next_processed_frames = _processed_frames + nframes;
        }
 
-       if (!tm.locked() || session == 0) {
+       if (!tm.locked() || _session == 0) {
                /* return having done nothing */
                _processed_frames = next_processed_frames;
                return 0;
@@ -350,17 +384,18 @@ AudioEngine::process_callback (nframes_t nframes)
 
        if (session_remove_pending) {
                /* perform the actual session removal */
-               session = 0;
+               _session = 0;
                session_remove_pending = false;
                session_removed.signal();
                _processed_frames = next_processed_frames;
                return 0;
        }
 
-       /* tell all IO objects that we're starting a new cycle */
+       /* tell all relevant objects that we're starting a new cycle */
 
-       IO::CycleStart (nframes);
+       Delivery::CycleStart (nframes);
        Port::set_port_offset (0);
+       InternalReturn::CycleStart (nframes);
 
        /* tell all Ports that we're starting a new cycle */
 
@@ -371,17 +406,19 @@ AudioEngine::process_callback (nframes_t nframes)
        }
 
        if (_freewheeling) {
-               /* emit the Freewheel signal and stop freewheeling in the event of trouble */
-               if (Freewheel (nframes)) {
-                       jack_set_freewheel (_jack, false);
+               /* emit the Freewheel signal and stop freewheeling in the event of trouble 
+                * the indirection is to pick up the return value of the signal.
+                */
+               if (*Freewheel (nframes)) {
+                       jack_set_freewheel (_priv_jack, false);
                }
 
        } else {
-               if (session) {
-                       session->process (nframes);
+               if (_session) {
+                       _session->process (nframes);
                }
        }
-       
+
        if (_freewheeling) {
                return 0;
        }
@@ -396,13 +433,13 @@ AudioEngine::process_callback (nframes_t nframes)
                boost::shared_ptr<Ports> p = ports.reader();
 
                for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
-                       
+
                        Port *port = (*i);
                        bool x;
-                       
+
                        if (port->_last_monitor != (x = port->monitoring_input ())) {
                                port->_last_monitor = x;
-                               /* XXX I think this is dangerous, due to 
+                               /* XXX I think this is dangerous, due to
                                   a likely mutex in the signal handlers ...
                                */
                                 port->MonitorInputChanged (x); /* EMIT SIGNAL */
@@ -411,19 +448,19 @@ AudioEngine::process_callback (nframes_t nframes)
                last_monitor_check = next_processed_frames;
        }
 
-       if (session->silent()) {
+       if (_session->silent()) {
 
                boost::shared_ptr<Ports> p = ports.reader();
 
                for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
-                       
+
                        Port *port = (*i);
-                       
+
                        if (port->sends_output()) {
                                port->get_buffer(nframes).silence(nframes);
                        }
                }
-       } 
+       }
 
        // Finalize ports
 
@@ -446,14 +483,14 @@ AudioEngine::jack_sample_rate_callback (nframes_t nframes)
 {
        _frame_rate = nframes;
        _usecs_per_cycle = (int) floor ((((double) frames_per_cycle() / nframes)) * 1000000.0);
-       
+
        /* check for monitor input change every 1/10th of second */
 
        monitor_check_interval = nframes / 10;
        last_monitor_check = 0;
-       
-       if (session) {
-               session->set_frame_rate (nframes);
+
+       if (_session) {
+               _session->set_frame_rate (nframes);
        }
 
        SampleRateChanged (nframes); /* EMIT SIGNAL */
@@ -483,8 +520,8 @@ AudioEngine::jack_bufsize_callback (nframes_t nframes)
                (*i)->reset();
        }
 
-       if (session) {
-               session->set_block_size (_buffer_size);
+       if (_session) {
+               _session->set_block_size (_buffer_size);
        }
 
        return 0;
@@ -505,7 +542,7 @@ AudioEngine::start_metering_thread ()
 {
        if (m_meter_thread == 0) {
                g_atomic_int_set (&m_meter_exit, 0);
-               m_meter_thread = Glib::Thread::create (sigc::mem_fun(this, &AudioEngine::meter_thread),
+               m_meter_thread = Glib::Thread::create (boost::bind (&AudioEngine::meter_thread, this),
                                                       500000, true, true, Glib::THREAD_PRIORITY_NORMAL);
        }
 }
@@ -518,19 +555,19 @@ AudioEngine::meter_thread ()
                if (g_atomic_int_get(&m_meter_exit)) {
                        break;
                }
-               IO::update_meters ();
+               Metering::Meter ();
        }
 }
 
-void 
+void
 AudioEngine::set_session (Session *s)
 {
        Glib::Mutex::Lock pl (_process_lock);
 
-       if (!session) {
-
-               session = s;
+       SessionHandlePtr::set_session (s);
 
+       if (_session) {
+               
                nframes_t blocksize = jack_get_buffer_size (_jack);
                
                /* page in as much of the session process code as we
@@ -538,65 +575,65 @@ AudioEngine::set_session (Session *s)
                */
                
                boost::shared_ptr<Ports> p = ports.reader();
-
+               
                for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        (*i)->cycle_start (blocksize);
                }
-
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-               s->process (blocksize);
-
+               
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               _session->process (blocksize);
+               
                for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        (*i)->cycle_end (blocksize);
                }
        }
 }
 
-void 
+void
 AudioEngine::remove_session ()
 {
        Glib::Mutex::Lock lm (_process_lock);
 
        if (_running) {
 
-               if (session) {
+               if (_session) {
                        session_remove_pending = true;
                        session_removed.wait(_process_lock);
                }
 
        } else {
-               session = 0;
+               SessionHandlePtr::set_session (0);
        }
-       
-       //FIXME: Preliminary bugfix for  http://tracker.ardour.org/view.php?id=1985
-       //remove_all_ports ();
+
+       remove_all_ports ();
 }
 
 void
 AudioEngine::port_registration_failure (const std::string& portname)
 {
+        GET_PRIVATE_JACK_POINTER (_jack);
        string full_portname = jack_client_name;
        full_portname += ':';
        full_portname += portname;
-       
-       
-       jack_port_t* p = jack_port_by_name (_jack, full_portname.c_str());
+
+
+       jack_port_t* p = jack_port_by_name (_priv_jack, full_portname.c_str());
        string reason;
-       
+
        if (p) {
                reason = string_compose (_("a port with the name \"%1\" already exists: check for duplicated track/bus names"), portname);
        } else {
                reason = _("No more JACK ports are available. You will need to stop Ardour and restart JACK with ports if you need this many tracks.");
        }
-       
+
        throw PortRegistrationFailure (string_compose (_("AudioEngine: cannot register port \"%1\": %2"), portname, reason).c_str());
-}      
+}
 
 Port *
 AudioEngine::register_port (DataType dtype, const string& portname, bool input)
@@ -611,7 +648,7 @@ AudioEngine::register_port (DataType dtype, const string& portname, bool input)
                } else {
                        throw PortRegistrationFailure("unable to create port (unknown type)");
                }
-               
+
                size_t& old_buffer_size  = _raw_buffer_sizes[newport->type()];
                size_t  port_buffer_size = newport->raw_buffer_size(0);
                if (port_buffer_size > old_buffer_size) {
@@ -654,7 +691,7 @@ AudioEngine::unregister_port (Port& port)
 {
        /* caller must hold process lock */
 
-       if (!_running) { 
+       if (!_running) {
                /* probably happening when the engine has been halted by JACK,
                   in which case, there is nothing we can do here.
                   */
@@ -664,7 +701,7 @@ AudioEngine::unregister_port (Port& port)
        {
                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) {
                                delete *i;
@@ -672,20 +709,18 @@ AudioEngine::unregister_port (Port& port)
                                break;
                        }
                }
-               
+
                /* writer goes out of scope, forces update */
        }
-               
-       remove_connections_for (port);
 
        return 0;
 }
 
-int 
+int
 AudioEngine::connect (const string& source, const string& destination)
 {
        /* caller must hold process lock */
-       
+
        int ret;
 
        if (!_running) {
@@ -700,36 +735,35 @@ AudioEngine::connect (const string& source, const string& destination)
        string s = make_port_name_non_relative (source);
        string d = make_port_name_non_relative (destination);
 
+
        Port* src = get_port_by_name_locked (s);
        Port* dst = get_port_by_name_locked (d);
 
-               if (src) {
-                       ret = src->connect (d);
+       if (src) {
+               ret = src->connect (d);
        } else if (dst) {
-                       ret = dst->connect (s);
+               ret = dst->connect (s);
        } else {
                /* neither port is known to us, and this API isn't intended for use as a general patch bay */
                ret = -1;
        }
-       
+
        if (ret > 0) {
-               error << string_compose(_("AudioEngine: connection already exists: %1 (%2) to %3 (%4)"), 
-                                       source, s, destination, d) 
-                     << endmsg;
+               /* already exists - no error, no warning */
        } else if (ret < 0) {
-               error << string_compose(_("AudioEngine: cannot connect %1 (%2) to %3 (%4)"), 
-                                       source, s, destination, d) 
+               error << string_compose(_("AudioEngine: cannot connect %1 (%2) to %3 (%4)"),
+                                       source, s, destination, d)
                      << endmsg;
        }
 
        return ret;
 }
 
-int 
+int
 AudioEngine::disconnect (const string& source, const string& destination)
 {
        /* caller must hold process lock */
-       
+
        int ret;
 
        if (!_running) {
@@ -740,7 +774,7 @@ AudioEngine::disconnect (const string& source, const string& destination)
                        return -1;
                }
        }
-       
+
        string s = make_port_name_non_relative (source);
        string d = make_port_name_non_relative (destination);
 
@@ -761,6 +795,8 @@ AudioEngine::disconnect (const string& source, const string& destination)
 int
 AudioEngine::disconnect (Port& port)
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,-1);
+
        if (!_running) {
                if (!_has_run) {
                        fatal << _("disconnect called before engine was started") << endmsg;
@@ -774,19 +810,13 @@ AudioEngine::disconnect (Port& port)
 }
 
 ARDOUR::nframes_t
-AudioEngine::frame_rate ()
+AudioEngine::frame_rate () const
 {
-       if (_jack) {
-               if (_frame_rate == 0) {
-                       return (_frame_rate = jack_get_sample_rate (_jack));
-               } else {
-                       return _frame_rate;
-               }
+        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
+       if (_frame_rate == 0) {
+         return (_frame_rate = jack_get_sample_rate (_priv_jack));
        } else {
-               fatal << X_("programming error: AudioEngine::frame_rate() called while disconnected from JACK")
-                     << endmsg;
-               /*NOTREACHED*/
-               return 0;
+         return _frame_rate;
        }
 }
 
@@ -798,19 +828,13 @@ AudioEngine::raw_buffer_size (DataType t)
 }
 
 ARDOUR::nframes_t
-AudioEngine::frames_per_cycle ()
+AudioEngine::frames_per_cycle () const
 {
-       if (_jack) {
-               if (_buffer_size == 0) {
-                       return (_buffer_size = jack_get_buffer_size (_jack));
-               } else {
-                       return _buffer_size;
-               }
+        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
+       if (_buffer_size == 0) {
+         return (_buffer_size = jack_get_buffer_size (_jack));
        } else {
-               fatal << X_("programming error: AudioEngine::frame_rate() called while disconnected from JACK")
-                     << endmsg;
-               /*NOTREACHED*/
-               return 0;
+         return _buffer_size;
        }
 }
 
@@ -821,10 +845,15 @@ AudioEngine::frames_per_cycle ()
 Port *
 AudioEngine::get_port_by_name (const string& portname)
 {
-       assert (portname.find_first_of (':') != string::npos);
-       
+       string s;
+       if (portname.find_first_of (':') == string::npos) {
+               s = make_port_name_non_relative (portname);
+       } else {
+               s = portname;
+       }
+
        Glib::Mutex::Lock lm (_process_lock);
-       return get_port_by_name_locked (portname);
+       return get_port_by_name_locked (s);
 }
 
 Port *
@@ -859,13 +888,10 @@ AudioEngine::get_port_by_name_locked (const string& portname)
        return 0;
 }
 
-
-
-
-
 const char **
 AudioEngine::get_ports (const string& port_name_pattern, const string& type_name_pattern, uint32_t flags)
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
        if (!_running) {
                if (!_has_run) {
                        fatal << _("get_ports called before engine was started") << endmsg;
@@ -874,12 +900,14 @@ AudioEngine::get_ports (const string& port_name_pattern, const string& type_name
                        return 0;
                }
        }
-       return jack_get_ports (_jack, port_name_pattern.c_str(), type_name_pattern.c_str(), flags);
+       return jack_get_ports (_priv_jack, port_name_pattern.c_str(), type_name_pattern.c_str(), flags);
 }
 
 void
 AudioEngine::halted (void *arg)
 {
+        /* called from jack shutdown handler  */
+
        AudioEngine* ae = static_cast<AudioEngine *> (arg);
        bool was_running = ae->_running;
 
@@ -891,19 +919,30 @@ AudioEngine::halted (void *arg)
 
        if (was_running) {
                ae->Halted(); /* EMIT SIGNAL */
+               MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
        }
 }
 
+void
+AudioEngine::died ()
+{
+        /* called from a signal handler for SIGPIPE */
+
+       stop_metering_thread ();
+
+        _running = false;
+       _buffer_size = 0;
+       _frame_rate = 0;
+       _jack = 0;
+}
+
 bool
-AudioEngine::can_request_hardware_monitoring () 
+AudioEngine::can_request_hardware_monitoring ()
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,false);
        const char ** ports;
 
-       if (!_jack) {
-               return 0;
-       }
-
-       if ((ports = jack_get_ports (_jack, NULL, JACK_DEFAULT_AUDIO_TYPE, JackPortCanMonitor)) == 0) {
+       if ((ports = jack_get_ports (_priv_jack, NULL, JACK_DEFAULT_AUDIO_TYPE, JackPortCanMonitor)) == 0) {
                return false;
        }
 
@@ -916,14 +955,11 @@ AudioEngine::can_request_hardware_monitoring ()
 uint32_t
 AudioEngine::n_physical_outputs (DataType type) const
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
        const char ** ports;
        uint32_t i = 0;
 
-       if (!_jack) {
-               return 0;
-       }
-
-       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
                return 0;
        }
 
@@ -936,35 +972,29 @@ AudioEngine::n_physical_outputs (DataType type) const
 uint32_t
 AudioEngine::n_physical_inputs (DataType type) const
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,0);
        const char ** ports;
        uint32_t i = 0;
-       
-       if (!_jack) {
-               return 0;
-       }
-       
-       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
+
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
                return 0;
        }
 
-       if (ports) {
-               for (i = 0; ports[i]; ++i) {}
-               free (ports);
-       }
+       for (i = 0; ports[i]; ++i) {}
+       free (ports);
+
        return i;
 }
 
 void
 AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
 {
+        GET_PRIVATE_JACK_POINTER (_jack);
        const char ** ports;
        uint32_t i = 0;
-       
-       if (!_jack) {
-               return;
-       }
-       
-       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
+
+
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
                return;
        }
 
@@ -979,47 +1009,32 @@ AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
 void
 AudioEngine::get_physical_outputs (DataType type, vector<string>& outs)
 {
+        GET_PRIVATE_JACK_POINTER (_jack);
        const char ** ports;
        uint32_t i = 0;
-       
-       if (!_jack) {
-               return;
-       }
-       
-       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
+
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
                return;
        }
 
-       if (ports) {
-               for (i = 0; ports[i]; ++i) {
-                       outs.push_back (ports[i]);
-               }
-               free (ports);
+       for (i = 0; ports[i]; ++i) {
+               outs.push_back (ports[i]);
        }
+       free (ports);
 }
 
 string
 AudioEngine::get_nth_physical (DataType type, uint32_t n, int flag)
 {
+        GET_PRIVATE_JACK_POINTER_RET (_jack,"");
        const char ** ports;
        uint32_t i;
        string ret;
 
        assert(type != DataType::NIL);
 
-       if (!_running || !_jack) {
-               if (!_has_run) {
-                       fatal << _("get_nth_physical called before engine was started") << endmsg;
-                       /*NOTREACHED*/
-               } else {
-                       return "";
-               }
-       }
-
-       ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|flag);
-       
-       if (ports == 0) {
-               return "";
+       if ((ports = jack_get_ports (_priv_jack, NULL, type.to_jack_type(), JackPortIsPhysical|flag)) == 0) {
+               return ret;
        }
 
        for (i = 0; i < n && ports[i]; ++i) {}
@@ -1036,91 +1051,69 @@ AudioEngine::get_nth_physical (DataType type, uint32_t n, int flag)
 void
 AudioEngine::update_total_latency (const Port& port)
 {
-       if (!_jack) {
-               fatal << _("update_total_latency() called with no JACK client connection") << endmsg;
-               /*NOTREACHED*/
-       }
-
-       if (!_running) {
-               if (!_has_run) {
-                       fatal << _("update_total_latency() called before engine was started") << endmsg;
-                       /*NOTREACHED*/
-               } 
-       }
-
        port.recompute_total_latency ();
 }
 
 void
 AudioEngine::transport_stop ()
 {
-       if (_jack) {
-               jack_transport_stop (_jack);
-       }
+        GET_PRIVATE_JACK_POINTER (_jack);
+       jack_transport_stop (_priv_jack);
 }
 
 void
 AudioEngine::transport_start ()
 {
-       if (_jack) {
-               jack_transport_start (_jack);
-       }
+        GET_PRIVATE_JACK_POINTER (_jack);
+       jack_transport_start (_priv_jack);
 }
 
 void
 AudioEngine::transport_locate (nframes_t where)
 {
+        GET_PRIVATE_JACK_POINTER (_jack);
        // cerr << "tell JACK to locate to " << where << endl;
-       if (_jack) {
-               jack_transport_locate (_jack, where);
-       }
+       jack_transport_locate (_priv_jack, where);
 }
 
 AudioEngine::TransportState
 AudioEngine::transport_state ()
 {
-       if (_jack) {
-               jack_position_t pos;
-               return (TransportState) jack_transport_query (_jack, &pos);
-       } else {
-               return (TransportState) JackTransportStopped;
-       }
+        GET_PRIVATE_JACK_POINTER_RET (_jack, ((TransportState) JackTransportStopped));
+       jack_position_t pos;
+       return (TransportState) jack_transport_query (_priv_jack, &pos);
 }
 
 int
 AudioEngine::reset_timebase ()
 {
-       if (_jack) {
-               if (Config->get_jack_time_master()) {
-                       return jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
+       if (_session) {
+               if (_session->config.get_jack_time_master()) {
+                       return jack_set_timebase_callback (_priv_jack, 0, _jack_timebase_callback, this);
                } else {
                        return jack_release_timebase (_jack);
                }
-       } else {
-               return -1;
        }
+       return 0;
 }
 
 int
 AudioEngine::freewheel (bool onoff)
 {
-       if (_jack) {
-
-               if (onoff != _freewheeling) {
-
-                       if (onoff) {
-                               _freewheel_thread_registered = false;
-                       }
-
-                       return jack_set_freewheel (_jack, onoff);
-
-               } else {
-                       /* already doing what has been asked for */
-                       return 0;
-               }
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
 
+       if (onoff != _freewheeling) {
+         
+         if (onoff) {
+           _freewheel_thread_registered = false;
+         }
+         
+         return jack_set_freewheel (_priv_jack, onoff);
+         
        } else {
-               return -1;
+         /* already doing what has been asked for */
+         return 0;
        }
 }
 
@@ -1132,34 +1125,17 @@ AudioEngine::remove_all_ports ()
        {
                RCUWriter<Ports> writer (ports);
                boost::shared_ptr<Ports> ps = writer.get_copy ();
-               ps->clear ();
-       }
-}
 
-void
-AudioEngine::remove_connections_for (Port& port)
-{
-       for (PortConnections::iterator i = port_connections.begin(); i != port_connections.end(); ) {
-               PortConnections::iterator tmp;
-               
-               tmp = i;
-               ++tmp;
-               
-               if ((*i).first == port.name()) {
-                       port_connections.erase (i);
+               for (Ports::iterator i = ps->begin(); i != ps->end(); ++i) {
+                       delete *i;
                }
 
-               i = tmp;
+               ps->clear ();
        }
-}
-
 
-#ifdef HAVE_JACK_CLIENT_OPEN
+       /* clear dead wood list too */
 
-static void 
-ardour_jack_error (const char* msg) 
-{
-       error << "JACK: " << msg << endmsg;
+       ports.flush ();
 }
 
 int
@@ -1177,45 +1153,27 @@ AudioEngine::connect_to_jack (string client_name)
                return -1;
        }
 
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
+
        if (status & JackNameNotUnique) {
-               jack_client_name = jack_get_client_name (_jack);
+               jack_client_name = jack_get_client_name (_priv_jack);
        }
 
-       jack_set_error_function (ardour_jack_error);
-       
        return 0;
 }
 
-#else
-
 int
-AudioEngine::connect_to_jack (string client_name)
-{
-       jack_client_name = client_name;
-
-       if ((_jack = jack_client_new (client_name.c_str())) == 0) {
-               return -1;
-       }
-
-       return 0;
-}
-
-#endif /* HAVE_JACK_CLIENT_OPEN */
-
-int 
 AudioEngine::disconnect_from_jack ()
 {
-       if (!_jack) {
-               return 0;
-       }
+        GET_PRIVATE_JACK_POINTER_RET (_jack, 0);
 
        if (_running) {
                stop_metering_thread ();
        }
 
-       { 
+       {
                Glib::Mutex::Lock lm (_process_lock);
-               jack_client_close (_jack);
+               jack_client_close (_priv_jack);
                _jack = 0;
        }
 
@@ -1226,6 +1184,7 @@ AudioEngine::disconnect_from_jack ()
        if (_running) {
                _running = false;
                Stopped(); /* EMIT SIGNAL */
+               MIDI::JACK_MidiPort::JackHalted (); /* EMIT SIGNAL */
        }
 
        return 0;
@@ -1252,44 +1211,45 @@ AudioEngine::reconnect_to_jack ()
        for (i = p->begin(); i != p->end(); ++i) {
                if ((*i)->reestablish ()) {
                        break;
-               } 
+               }
        }
 
        if (i != p->end()) {
                /* failed */
                remove_all_ports ();
                return -1;
-       } 
+       }
 
+        GET_PRIVATE_JACK_POINTER_RET (_jack,-1);
+
+       if (_session) {
+               _session->reset_jack_connection (_priv_jack);
+               nframes_t blocksize = jack_get_buffer_size (_priv_jack);
+               _session->set_block_size (blocksize);
+               _session->set_frame_rate (jack_get_sample_rate (_priv_jack));
 
-       if (session) {
-               session->reset_jack_connection (_jack);
-               nframes_t blocksize = jack_get_buffer_size (_jack);
-               session->set_block_size (blocksize);
-               session->set_frame_rate (jack_get_sample_rate (_jack));
-               
                _raw_buffer_sizes[DataType::AUDIO] = blocksize * sizeof(float);
                cout << "FIXME: Assuming maximum MIDI buffer size " << blocksize * 4 << "bytes" << endl;
                _raw_buffer_sizes[DataType::MIDI] = blocksize * 4;
        }
 
        last_monitor_check = 0;
-       
-       jack_on_shutdown (_jack, halted, this);
-       jack_set_graph_order_callback (_jack, _graph_order_callback, this);
-       jack_set_thread_init_callback (_jack, _thread_init_callback, this);
-       jack_set_process_callback (_jack, _process_callback, this);
-       jack_set_sample_rate_callback (_jack, _sample_rate_callback, this);
-       jack_set_buffer_size_callback (_jack, _bufsize_callback, this);
-       jack_set_xrun_callback (_jack, _xrun_callback, this);
-       jack_set_sync_callback (_jack, _jack_sync_callback, this);
-       jack_set_freewheel_callback (_jack, _freewheel_callback, this);
-       
-       if (Config->get_jack_time_master()) {
-               jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
-       } 
-       
-       if (jack_activate (_jack) == 0) {
+
+       jack_on_shutdown (_priv_jack, halted, this);
+       jack_set_graph_order_callback (_priv_jack, _graph_order_callback, this);
+       jack_set_thread_init_callback (_priv_jack, _thread_init_callback, this);
+       jack_set_process_callback (_priv_jack, _process_callback, this);
+       jack_set_sample_rate_callback (_priv_jack, _sample_rate_callback, this);
+       jack_set_buffer_size_callback (_priv_jack, _bufsize_callback, this);
+       jack_set_xrun_callback (_priv_jack, _xrun_callback, this);
+       jack_set_sync_callback (_priv_jack, _jack_sync_callback, this);
+       jack_set_freewheel_callback (_priv_jack, _freewheel_callback, this);
+
+       if (_session && _session->config.get_jack_time_master()) {
+               jack_set_timebase_callback (_priv_jack, 0, _jack_timebase_callback, this);
+       }
+
+       if (jack_activate (_priv_jack) == 0) {
                _running = true;
                _has_run = true;
        } else {
@@ -1297,7 +1257,7 @@ AudioEngine::reconnect_to_jack ()
        }
 
        /* re-establish connections */
-       
+
        for (i = p->begin(); i != p->end(); ++i) {
                (*i)->reconnect ();
        }
@@ -1312,35 +1272,30 @@ AudioEngine::reconnect_to_jack ()
 int
 AudioEngine::request_buffer_size (nframes_t nframes)
 {
-       if (_jack) {
-
-               if (nframes == jack_get_buffer_size (_jack)) {
-                       return 0;
-               }
+        GET_PRIVATE_JACK_POINTER_RET (_jack, -1);
 
-               return jack_set_buffer_size (_jack, nframes);
-
-       } else {
-               return -1;
+       if (nframes == jack_get_buffer_size (_priv_jack)) {
+         return 0;
        }
+       
+       return jack_set_buffer_size (_priv_jack, nframes);
 }
 
 void
 AudioEngine::update_total_latencies ()
 {
 #ifdef HAVE_JACK_RECOMPUTE_LATENCIES
-       if (_jack) {
-               jack_recompute_total_latencies (_jack);
-       }
+        GET_PRIVATE_JACK_POINTER (_jack);
+       jack_recompute_total_latencies (_priv_jack);
 #endif
 }
-               
+
 string
 AudioEngine::make_port_name_relative (string portname)
 {
        string::size_type len;
        string::size_type n;
-       
+
        len = portname.length();
 
        for (n = 0; n < len; ++n) {
@@ -1348,7 +1303,7 @@ AudioEngine::make_port_name_relative (string portname)
                        break;
                }
        }
-       
+
        if ((n != len) && (portname.substr (0, n) == jack_client_name)) {
                return portname.substr (n+1);
        }
@@ -1368,16 +1323,13 @@ AudioEngine::make_port_name_non_relative (string portname)
        str  = jack_client_name;
        str += ':';
        str += portname;
-       
+
        return str;
 }
 
 bool
 AudioEngine::is_realtime () const
 {
-       if (_jack) {
-               return jack_is_realtime (_jack);
-       } else {
-               return false;
-       }
+        GET_PRIVATE_JACK_POINTER_RET (_jack,false);
+       return jack_is_realtime (_priv_jack);
 }