replace fixed-point linear interpolation with double-based version, thereby removing...
[ardour.git] / libs / ardour / audioengine.cc
index 4552de11866c01ebc512928a54cea868bb2264db..7533a1ed8f71ce9ad9f500363655b76d772b626c 100644 (file)
 #include <vector>
 #include <exception>
 #include <stdexcept>
+#include <sstream>
 
 #include <glibmm/timer.h>
-#include <pbd/pthread_utils.h>
-#include <pbd/stacktrace.h>
-#include <pbd/unknown_type.h>
-
-#include <ardour/audioengine.h>
-#include <ardour/buffer.h>
-#include <ardour/port.h>
-#include <ardour/jack_audio_port.h>
-#include <ardour/internal_audio_port.h>
-#include <ardour/jack_midi_port.h>
-#include <ardour/session.h>
-#include <ardour/cycle_timer.h>
-#include <ardour/utils.h>
-#ifdef VST_SUPPORT
-#include <fst.h>
-#endif
-
-#include <ardour/timestamps.h>
+#include "pbd/pthread_utils.h"
+#include "pbd/stacktrace.h"
+#include "pbd/unknown_type.h"
+
+#include "midi++/jack.h"
+
+#include "ardour/amp.h"
+#include "ardour/audio_port.h"
+#include "ardour/audioengine.h"
+#include "ardour/buffer.h"
+#include "ardour/buffer_set.h"
+#include "ardour/cycle_timer.h"
+#include "ardour/delivery.h"
+#include "ardour/event_type_map.h"
+#include "ardour/internal_return.h"
+#include "ardour/io.h"
+#include "ardour/meter.h"
+#include "ardour/midi_port.h"
+#include "ardour/port.h"
+#include "ardour/port_set.h"
+#include "ardour/session.h"
+#include "ardour/timestamps.h"
+#include "ardour/utils.h"
 
 #include "i18n.h"
 
@@ -51,12 +57,6 @@ using namespace PBD;
 
 gint AudioEngine::m_meter_exit;
 
-static void 
-ardour_jack_error (const char* msg) 
-{
-       error << "JACK: " << msg << endmsg;
-}
-
 AudioEngine::AudioEngine (string client_name) 
        : ports (new Ports)
 {
@@ -67,13 +67,12 @@ AudioEngine::AudioEngine (string client_name)
        last_monitor_check = 0;
        monitor_check_interval = max_frames;
        _processed_frames = 0;
-       _freewheeling = false;
        _usecs_per_cycle = 0;
        _jack = 0;
        _frame_rate = 0;
        _buffer_size = 0;
-       _freewheeling = false;
        _freewheel_thread_registered = false;
+       _freewheeling = false;
 
        m_meter_thread = 0;
        g_atomic_int_set (&m_meter_exit, 0);
@@ -82,9 +81,24 @@ AudioEngine::AudioEngine (string client_name)
                throw NoBackendAvailable ();
        }
 
-       start_metering_thread();
-
-       JackPort::set_engine (this);
+       Port::set_engine (this);
+       
+       // Initialize parameter metadata (e.g. ranges)
+       Evoral::Parameter p(NullAutomation);
+       p = EventTypeMap::instance().new_parameter(NullAutomation);
+       p = EventTypeMap::instance().new_parameter(GainAutomation);
+       p = EventTypeMap::instance().new_parameter(PanAutomation);
+       p = EventTypeMap::instance().new_parameter(PluginAutomation);
+       p = EventTypeMap::instance().new_parameter(SoloAutomation);
+       p = EventTypeMap::instance().new_parameter(MuteAutomation);
+       p = EventTypeMap::instance().new_parameter(MidiCCAutomation);
+       p = EventTypeMap::instance().new_parameter(MidiPgmChangeAutomation);
+       p = EventTypeMap::instance().new_parameter(MidiPitchBenderAutomation);
+       p = EventTypeMap::instance().new_parameter(MidiChannelPressureAutomation);
+       p = EventTypeMap::instance().new_parameter(FadeInAutomation);
+       p = EventTypeMap::instance().new_parameter(FadeOutAutomation);
+       p = EventTypeMap::instance().new_parameter(EnvelopeAutomation);
+       p = EventTypeMap::instance().new_parameter(MidiCCAutomation);
 }
 
 AudioEngine::~AudioEngine ()
@@ -95,6 +109,7 @@ AudioEngine::~AudioEngine ()
                
                if (_running) {
                        jack_client_close (_jack);
+                       _jack = 0;
                }
                
                stop_metering_thread ();
@@ -114,20 +129,30 @@ _thread_init_callback (void *arg)
           knows about it.
        */
 
-       PBD::ThreadCreatedWithRequestSize (pthread_self(), X_("Audioengine"), 4096);
+       PBD::notify_gui_about_thread_creation (pthread_self(), X_("Audioengine"), 4096);
+#ifdef WITH_JACK_MIDI
+       MIDI::JACK_MidiPort::set_process_thread (pthread_self());
+#endif // WITH_JACK_MIDI
 }
 
 int
 AudioEngine::start ()
 {
+       if (!_jack) {
+               error << _("AudioEngine::start() called while disconnected from JACK") << endmsg;
+               return -1;
+       }
+
        if (!_running) {
 
+               nframes_t blocksize = jack_get_buffer_size (_jack);
+               
                if (session) {
-                       nframes_t blocksize = jack_get_buffer_size (_jack);
+                       BootMessage (_("Connect session to engine"));
 
                        session->set_block_size (blocksize);
                        session->set_frame_rate (jack_get_sample_rate (_jack));
-
+                       
                        /* page in as much of the session process code as we
                           can before we really start running.
                        */
@@ -155,7 +180,7 @@ AudioEngine::start ()
                jack_set_sync_callback (_jack, _jack_sync_callback, this);
                jack_set_freewheel_callback (_jack, _freewheel_callback, this);
 
-               if (Config->get_jack_time_master()) {
+               if (session && session->config.get_jack_time_master()) {
                        jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
                }
 
@@ -166,6 +191,10 @@ AudioEngine::start ()
                } else {
                        // error << _("cannot activate JACK client") << endmsg;
                }
+
+               start_metering_thread();
+
+               _raw_buffer_sizes[DataType::AUDIO] = blocksize * sizeof(float);
        }
 
        return _running ? 0 : -1;
@@ -174,17 +203,13 @@ AudioEngine::start ()
 int
 AudioEngine::stop (bool forever)
 {
-       if (_running) {
-               _running = false;
+       if (_jack) {
                if (forever) {
-                       jack_client_t* foo = _jack;
-                       _jack = 0;
-                       jack_client_close (foo);
-                       stop_metering_thread ();
+                       disconnect_from_jack ();
                } else {
                        jack_deactivate (_jack);
+                       Stopped(); /* EMIT SIGNAL */
                }
-               Stopped(); /* EMIT SIGNAL */
        }
 
        return _running ? -1 : 0;
@@ -199,11 +224,13 @@ AudioEngine::get_sync_offset (nframes_t& offset) const
 
        jack_position_t pos;
        
-       (void) jack_transport_query (_jack, &pos);
-
-       if (pos.valid & JackVideoFrameOffset) {
-               offset = pos.video_offset;
-               return true;
+       if (_jack) {
+               (void) jack_transport_query (_jack, &pos);
+               
+               if (pos.valid & JackVideoFrameOffset) {
+                       offset = pos.video_offset;
+                       return true;
+               }
        }
 
 #endif
@@ -247,7 +274,7 @@ int
 AudioEngine::_xrun_callback (void *arg)
 {
        AudioEngine* ae = static_cast<AudioEngine*> (arg);
-       if (ae->jack()) {
+       if (ae->connected()) {
                ae->Xrun (); /* EMIT SIGNAL */
        }
        return 0;
@@ -257,7 +284,7 @@ int
 AudioEngine::_graph_order_callback (void *arg)
 {
        AudioEngine* ae = static_cast<AudioEngine*> (arg);
-       if (ae->jack()) {
+       if (ae->connected()) {
                ae->GraphReordered (); /* EMIT SIGNAL */
        }
        return 0;
@@ -280,6 +307,22 @@ AudioEngine::_freewheel_callback (int onoff, void *arg)
        static_cast<AudioEngine*>(arg)->_freewheeling = onoff;
 }
 
+void
+AudioEngine::split_cycle (nframes_t offset)
+{
+       /* caller must hold process lock */
+
+       Port::increment_port_offset (offset);
+
+       /* tell all Ports that we're going to start a new (split) cycle */
+
+       boost::shared_ptr<Ports> p = ports.reader();
+
+       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
+               (*i)->cycle_split ();
+       }
+}
+
 /** Method called by JACK (via _process_callback) which says that there
  * is work to be done.
  * @param nframes Number of frames to process.
@@ -288,6 +331,7 @@ int
 AudioEngine::process_callback (nframes_t nframes)
 {
        // 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
@@ -316,36 +360,40 @@ AudioEngine::process_callback (nframes_t nframes)
                return 0;
        }
 
+       /* tell all relevant objects that we're starting a new cycle */
+
+       Delivery::CycleStart (nframes);
+       Port::set_port_offset (0);
+       InternalReturn::CycleStart (nframes);
+
+       /* tell all Ports that we're starting a new cycle */
+
+       boost::shared_ptr<Ports> p = ports.reader();
+
+       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
+               (*i)->cycle_start (nframes);
+       }
+
        if (_freewheeling) {
                /* emit the Freewheel signal and stop freewheeling in the event of trouble */
                if (Freewheel (nframes)) {
-                       cerr << "Freewheeling returned non-zero!\n";
-                       _freewheeling = false;
                        jack_set_freewheel (_jack, false);
                }
-               return 0;
-       }
-
-       boost::shared_ptr<Ports> p = ports.reader();
 
-       // Prepare ports (ie read data if necessary)
-       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
-               (*i)->cycle_start (nframes);
+       } else {
+               if (session) {
+                       session->process (nframes);
+               }
        }
        
-       if (session) {
-               session->process (nframes);
+       if (_freewheeling) {
+               return 0;
        }
 
        if (!_running) {
                _processed_frames = next_processed_frames;
                return 0;
        }
-       
-       // Finalize ports (ie write data if necessary)
-       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
-               (*i)->cycle_end ();
-       }
 
        if (last_monitor_check + monitor_check_interval < next_processed_frames) {
 
@@ -367,6 +415,26 @@ AudioEngine::process_callback (nframes_t nframes)
                last_monitor_check = next_processed_frames;
        }
 
+       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
+
+       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
+               (*i)->cycle_end (nframes);
+       }
+
        _processed_frames = next_processed_frames;
        return 0;
 }
@@ -407,6 +475,9 @@ int
 AudioEngine::jack_bufsize_callback (nframes_t nframes)
 {
        _buffer_size = nframes;
+       _raw_buffer_sizes[DataType::AUDIO] = nframes * sizeof(float);
+       cout << "FIXME: Assuming maximum MIDI buffer size " << nframes * 4 << "bytes" << endl;
+       _raw_buffer_sizes[DataType::MIDI] = nframes * 4;
        _usecs_per_cycle = (int) floor ((((double) nframes / frame_rate())) * 1000000.0);
        last_monitor_check = 0;
 
@@ -437,8 +508,9 @@ void
 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),
-                               500000, true, true, Glib::THREAD_PRIORITY_NORMAL);
+                                                      500000, true, true, Glib::THREAD_PRIORITY_NORMAL);
        }
 }
 
@@ -450,7 +522,7 @@ AudioEngine::meter_thread ()
                if (g_atomic_int_get(&m_meter_exit)) {
                        break;
                }
-               IO::update_meters ();
+               Metering::update_meters ();
        }
 }
 
@@ -471,8 +543,9 @@ AudioEngine::set_session (Session *s)
                
                boost::shared_ptr<Ports> p = ports.reader();
 
-               for (Ports::iterator i = p->begin(); i != p->end(); ++i)
+               for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
                        (*i)->cycle_start (blocksize);
+               }
 
                s->process (blocksize);
                s->process (blocksize);
@@ -483,9 +556,9 @@ AudioEngine::set_session (Session *s)
                s->process (blocksize);
                s->process (blocksize);
 
-               for (Ports::iterator i = p->begin(); i != p->end(); ++i)
-                       (*i)->cycle_end ();
-
+               for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
+                       (*i)->cycle_end (blocksize);
+               }
        }
 }
 
@@ -505,82 +578,86 @@ AudioEngine::remove_session ()
                session = 0;
        }
        
-       remove_all_ports ();
+       //FIXME: Preliminary bugfix for  http://tracker.ardour.org/view.php?id=1985
+       //remove_all_ports ();
 }
 
+void
+AudioEngine::port_registration_failure (const std::string& portname)
+{
+       string full_portname = jack_client_name;
+       full_portname += ':';
+       full_portname += portname;
+       
+       
+       jack_port_t* p = jack_port_by_name (_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 (PortType ptype, DataType dtype, const string& portname, bool input)
+AudioEngine::register_port (DataType dtype, const string& portname, bool input)
 {
        Port* newport = 0;
 
        try {
-               switch (ptype) {
-               case Jack:
-                       if (dtype == DataType::AUDIO) {
-                               newport = new JackAudioPort (portname, (input ? Port::IsInput : Port::IsOutput));
-                       } else if (dtype == DataType::MIDI) {
-                               newport = new JackMidiPort (portname, (input ? Port::IsInput : Port::IsOutput));
-                       } else {
-                               throw unknown_type();
-                       }
-                       break;
-
-               case Internal:
-                       if (dtype == DataType::AUDIO) {
-                               newport = new InternalAudioPort (portname, (input ? Port::IsInput : Port::IsOutput));
-                       } else if (dtype == DataType::MIDI) {
-                               error << _("Internal MIDI ports are not implemented yet!") << endmsg;
-                               throw unknown_type();
-                       } else {
-                               throw unknown_type();
-                       }
-                       break;
+               if (dtype == DataType::AUDIO) {
+                       newport = new AudioPort (portname, (input ? Port::IsInput : Port::IsOutput));
+               } else if (dtype == DataType::MIDI) {
+                       newport = new MidiPort (portname, (input ? Port::IsInput : Port::IsOutput));
+               } else {
+                       throw PortRegistrationFailure("unable to create port (unknown type)");
                }
-       
-               if (newport != 0) {
-                       RCUWriter<Ports> writer (ports);
-                       boost::shared_ptr<Ports> ps = writer.get_copy ();
-                       ps->insert (ps->begin(), newport);
-                       /* writer goes out of scope, forces update */
+               
+               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) {
+                       old_buffer_size = port_buffer_size;
                }
-       
-               return newport;
-       }
 
-       catch (...) {
-               throw PortRegistrationFailure();
+               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;
        }
-}
 
-InternalPort*
-AudioEngine::get_internal_port (const std::string& short_name)
-{
-       boost::shared_ptr<Ports> p = ports.reader();
-       
-       for (Ports::iterator i = p->begin(); i != p->end(); ++i) {
-               if ((*i)->short_name() == short_name) {
-                       return dynamic_cast<InternalPort*> (*i);
-               }
+       catch (PortRegistrationFailure& err) {
+               throw err;
+       } catch (std::exception& e) {
+               throw PortRegistrationFailure(string_compose(
+                               _("unable to create port: %1"), e.what()).c_str());
+       } catch (...) {
+               throw PortRegistrationFailure("unable to create port (unknown error)");
        }
-       return 0;
 }
 
-
 Port *
-AudioEngine::register_input_port (PortType ptype, DataType type, const string& portname)
+AudioEngine::register_input_port (DataType type, const string& portname)
 {
-       return register_port (ptype, type, portname, true);
+       return register_port (type, portname, true);
 }
 
 Port *
-AudioEngine::register_output_port (PortType ptype, DataType type, const string& portname)
+AudioEngine::register_output_port (DataType type, const string& portname)
 {
-       return register_port (ptype, type, portname, false);
+       return register_port (type, portname, false);
 }
 
-int          
+int
 AudioEngine::unregister_port (Port& port)
 {
+       /* caller must hold process lock */
+
        if (!_running) { 
                /* probably happening when the engine has been halted by JACK,
                   in which case, there is nothing we can do here.
@@ -589,13 +666,11 @@ 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) {
-                               remove_connections_for (port);
                                delete *i;
                                ps->erase (i);
                                break;
@@ -604,6 +679,8 @@ AudioEngine::unregister_port (Port& port)
                
                /* writer goes out of scope, forces update */
        }
+               
+       remove_connections_for (port);
 
        return 0;
 }
@@ -611,6 +688,8 @@ AudioEngine::unregister_port (Port& port)
 int 
 AudioEngine::connect (const string& source, const string& destination)
 {
+       /* caller must hold process lock */
+       
        int ret;
 
        if (!_running) {
@@ -624,34 +703,24 @@ AudioEngine::connect (const string& source, const string& destination)
 
        string s = make_port_name_non_relative (source);
        string d = make_port_name_non_relative (destination);
-               
-       if (source.substr (0, 9) == "internal:") {
-               if (destination.substr (0, 9) == "internal:") {
-                       InternalPort* src = get_internal_port (source);
-                       InternalPort* dst = get_internal_port (destination);
 
-                       InternalPort::connect (*src, *dst);
-                       ret = 0;
-
-               } else {
-                       ret = -1;
-               }
+       Port* src = get_port_by_name_locked (s);
+       Port* dst = get_port_by_name_locked (d);
 
+               if (src) {
+                       ret = src->connect (d);
+       } else if (dst) {
+                       ret = dst->connect (s);
        } else {
-       
-               ret = jack_connect (_jack, s.c_str(), d.c_str());
+               /* neither port is known to us, and this API isn't intended for use as a general patch bay */
+               ret = -1;
        }
-
-       if (ret == 0) {
-               pair<string,string> c (s, d);
-               port_connections.push_back (c);
-       } else if (ret == EEXIST) {
-               error << string_compose(_("AudioEngine: connection already exists: %1 (%2) to %3 (%4)"), 
-                                source, s, destination, d) 
-                     << endmsg;
-       } else {
+       
+       if (ret > 0) {
+               /* 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) 
+                                       source, s, destination, d) 
                      << endmsg;
        }
 
@@ -661,6 +730,8 @@ AudioEngine::connect (const string& source, const string& destination)
 int 
 AudioEngine::disconnect (const string& source, const string& destination)
 {
+       /* caller must hold process lock */
+       
        int ret;
 
        if (!_running) {
@@ -675,31 +746,17 @@ AudioEngine::disconnect (const string& source, const string& destination)
        string s = make_port_name_non_relative (source);
        string d = make_port_name_non_relative (destination);
 
-       if (source.substr (0, 9) == "internal:") {
-               if (destination.substr (0, 9) == "internal:") {
-                       InternalPort* src = get_internal_port (source);
-                       InternalPort* dst = get_internal_port (destination);
-
-                       InternalPort::disconnect (*src, *dst);
-                       ret = 0;
-               } else {
-                       ret = -1;
-               }
+       Port* src = get_port_by_name_locked (s);
+       Port* dst = get_port_by_name_locked (d);
 
+       if (src) {
+                       ret = src->disconnect (d);
+       } else if (dst) {
+                       ret = dst->disconnect (s);
        } else {
-       
-               ret = jack_disconnect (_jack, s.c_str(), d.c_str());
-       }
-
-       if (ret == 0) {
-               pair<string,string> c (s, d);
-               PortConnections::iterator i;
-               
-               if ((i = find (port_connections.begin(), port_connections.end(), c)) != port_connections.end()) {
-                       port_connections.erase (i);
-               }
+               /* neither port is known to us, and this API isn't intended for use as a general patch bay */
+               ret = -1;
        }
-        
        return ret;
 }
 
@@ -715,14 +772,7 @@ AudioEngine::disconnect (Port& port)
                }
        }
 
-       int ret = port.disconnect ();
-
-       if (ret == 0) {
-               remove_connections_for (port);
-       }
-
-       return ret;
-
+       return port.disconnect_all ();
 }
 
 ARDOUR::nframes_t
@@ -742,6 +792,13 @@ AudioEngine::frame_rate ()
        }
 }
 
+size_t
+AudioEngine::raw_buffer_size (DataType t)
+{
+       std::map<DataType,size_t>::const_iterator s = _raw_buffer_sizes.find(t);
+       return (s != _raw_buffer_sizes.end()) ? s->second : 0;
+}
+
 ARDOUR::nframes_t
 AudioEngine::frames_per_cycle ()
 {
@@ -759,34 +816,60 @@ AudioEngine::frames_per_cycle ()
        }
 }
 
-/** Get a port by name.
- * Note this can return NULL, it will NOT create a port if it is not found (any more).
+/** @param name Full name of port (including prefix:)
+ *  @return Corresponding Port*, or 0.  This object remains the property of the AudioEngine
+ *  so must not be deleted.
  */
 Port *
-AudioEngine::get_port_by_name (const string& portname, bool keep)
+AudioEngine::get_port_by_name (const string& portname)
 {
+       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 (s);
+}
+
+Port *
+AudioEngine::get_port_by_name_locked (const string& portname)
+{
+       /* caller must hold process lock */
 
        if (!_running) {
                if (!_has_run) {
-                       fatal << _("get_port_by_name() called before engine was started") << endmsg;
+                       fatal << _("get_port_by_name_locked() called before engine was started") << endmsg;
                        /*NOTREACHED*/
                } else {
                        return 0;
                }
        }
-       
+
+       if (portname.substr (0, jack_client_name.length ()) != jack_client_name) {
+               /* not an ardour: port */
+               return 0;
+       }
+
+       std::string const rel = make_port_name_relative (portname);
+
        boost::shared_ptr<Ports> pr = ports.reader();
-       
+
        for (Ports::iterator i = pr->begin(); i != pr->end(); ++i) {
-               if (portname == (*i)->name()) {
-                       return (*i);
+               if (rel == (*i)->name()) {
+                       return *i;
                }
        }
 
        return 0;
 }
 
+
+
+
+
 const char **
 AudioEngine::get_ports (const string& port_name_pattern, const string& type_name_pattern, uint32_t flags)
 {
@@ -805,17 +888,40 @@ void
 AudioEngine::halted (void *arg)
 {
        AudioEngine* ae = static_cast<AudioEngine *> (arg);
+       bool was_running = ae->_running;
+
+       ae->stop_metering_thread ();
 
        ae->_running = false;
        ae->_buffer_size = 0;
        ae->_frame_rate = 0;
-       ae->_jack = 0;
 
-       ae->Halted(); /* EMIT SIGNAL */
+       if (was_running) {
+               ae->Halted(); /* EMIT SIGNAL */
+       }
+}
+
+bool
+AudioEngine::can_request_hardware_monitoring () 
+{
+       const char ** ports;
+
+       if (!_jack) {
+               return 0;
+       }
+
+       if ((ports = jack_get_ports (_jack, NULL, JACK_DEFAULT_AUDIO_TYPE, JackPortCanMonitor)) == 0) {
+               return false;
+       }
+
+       free (ports);
+
+       return true;
 }
 
+
 uint32_t
-AudioEngine::n_physical_outputs () const
+AudioEngine::n_physical_outputs (DataType type) const
 {
        const char ** ports;
        uint32_t i = 0;
@@ -824,19 +930,18 @@ AudioEngine::n_physical_outputs () const
                return 0;
        }
 
-       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsInput)) == 0) {
+       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
                return 0;
        }
 
-       if (ports) {
-               for (i = 0; ports[i]; ++i);
-               free (ports);
-       }
+       for (i = 0; ports[i]; ++i) {}
+       free (ports);
+
        return i;
 }
 
 uint32_t
-AudioEngine::n_physical_inputs () const
+AudioEngine::n_physical_inputs (DataType type) const
 {
        const char ** ports;
        uint32_t i = 0;
@@ -845,19 +950,19 @@ AudioEngine::n_physical_inputs () const
                return 0;
        }
        
-       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsOutput)) == 0) {
+       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
                return 0;
        }
 
        if (ports) {
-               for (i = 0; ports[i]; ++i);
+               for (i = 0; ports[i]; ++i) {}
                free (ports);
        }
        return i;
 }
 
 void
-AudioEngine::get_physical_inputs (vector<string>& ins)
+AudioEngine::get_physical_inputs (DataType type, vector<string>& ins)
 {
        const char ** ports;
        uint32_t i = 0;
@@ -866,7 +971,7 @@ AudioEngine::get_physical_inputs (vector<string>& ins)
                return;
        }
        
-       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsOutput)) == 0) {
+       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsOutput)) == 0) {
                return;
        }
 
@@ -879,7 +984,7 @@ AudioEngine::get_physical_inputs (vector<string>& ins)
 }
 
 void
-AudioEngine::get_physical_outputs (vector<string>& outs)
+AudioEngine::get_physical_outputs (DataType type, vector<string>& outs)
 {
        const char ** ports;
        uint32_t i = 0;
@@ -888,7 +993,7 @@ AudioEngine::get_physical_outputs (vector<string>& outs)
                return;
        }
        
-       if ((ports = jack_get_ports (_jack, NULL, NULL, JackPortIsPhysical|JackPortIsInput)) == 0) {
+       if ((ports = jack_get_ports (_jack, NULL, type.to_jack_type(), JackPortIsPhysical|JackPortIsInput)) == 0) {
                return;
        }
 
@@ -924,7 +1029,7 @@ AudioEngine::get_nth_physical (DataType type, uint32_t n, int flag)
                return "";
        }
 
-       for (i = 0; i < n && ports[i]; ++i);
+       for (i = 0; i < n && ports[i]; ++i) {}
 
        if (ports[i]) {
                ret = ports[i];
@@ -935,12 +1040,6 @@ AudioEngine::get_nth_physical (DataType type, uint32_t n, int flag)
        return ret;
 }
 
-ARDOUR::nframes_t
-AudioEngine::get_port_total_latency (const Port& port)
-{
-       return port.total_latency ();
-}
-
 void
 AudioEngine::update_total_latency (const Port& port)
 {
@@ -962,7 +1061,6 @@ AudioEngine::update_total_latency (const Port& port)
 void
 AudioEngine::transport_stop ()
 {
-       // cerr << "tell JACK to stop\n";
        if (_jack) {
                jack_transport_stop (_jack);
        }
@@ -971,7 +1069,6 @@ AudioEngine::transport_stop ()
 void
 AudioEngine::transport_start ()
 {
-       // cerr << "tell JACK to start\n";
        if (_jack) {
                jack_transport_start (_jack);
        }
@@ -1000,8 +1097,8 @@ AudioEngine::transport_state ()
 int
 AudioEngine::reset_timebase ()
 {
-       if (_jack) {
-               if (Config->get_jack_time_master()) {
+       if (_jack && session) {
+               if (session->config.get_jack_time_master()) {
                        return jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
                } else {
                        return jack_release_timebase (_jack);
@@ -1016,11 +1113,18 @@ AudioEngine::freewheel (bool onoff)
 {
        if (_jack) {
 
-               if (onoff) {
-                       _freewheel_thread_registered = false;
-               }
+               if (onoff != _freewheeling) {
+
+                       if (onoff) {
+                               _freewheel_thread_registered = false;
+                       }
 
-               return jack_set_freewheel (_jack, onoff);
+                       return jack_set_freewheel (_jack, onoff);
+
+               } else {
+                       /* already doing what has been asked for */
+                       return 0;
+               }
 
        } else {
                return -1;
@@ -1037,8 +1141,6 @@ AudioEngine::remove_all_ports ()
                boost::shared_ptr<Ports> ps = writer.get_copy ();
                ps->clear ();
        }
-
-       port_connections.clear ();
 }
 
 void
@@ -1058,7 +1160,12 @@ AudioEngine::remove_connections_for (Port& port)
        }
 }
 
-#ifdef HAVE_JACK_CLIENT_OPEN
+
+static void 
+ardour_jack_error (const char* msg) 
+{
+       error << "JACK: " << msg << endmsg;
+}
 
 int
 AudioEngine::connect_to_jack (string client_name)
@@ -1068,23 +1175,13 @@ AudioEngine::connect_to_jack (string client_name)
        const char *server_name = NULL;
 
        jack_client_name = client_name; /* might be reset below */
-
        _jack = jack_client_open (jack_client_name.c_str(), options, &status, server_name);
-       
-       if (_jack == NULL) {
 
-               if (status & JackServerFailed) {
-                       error << _("Unable to connect to JACK server") << endmsg;
-               }
-               
+       if (_jack == NULL) {
                // error message is not useful here
                return -1;
        }
 
-       if (status & JackServerStarted) {
-               info << _("JACK server started") << endmsg;
-       }
-
        if (status & JackNameNotUnique) {
                jack_client_name = jack_get_client_name (_jack);
        }
@@ -1094,47 +1191,39 @@ AudioEngine::connect_to_jack (string client_name)
        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())) == NULL) {
-               return -1;
-       }
-
-       return 0;
-}
-
-#endif /* HAVE_JACK_CLIENT_OPEN */
-
 int 
 AudioEngine::disconnect_from_jack ()
 {
-       if (_jack == 0) {
+       if (!_jack) {
                return 0;
        }
 
-       jack_client_close (_jack);
+       if (_running) {
+               stop_metering_thread ();
+       }
+
+       { 
+               Glib::Mutex::Lock lm (_process_lock);
+               jack_client_close (_jack);
+               _jack = 0;
+       }
 
        _buffer_size = 0;
        _frame_rate = 0;
+       _raw_buffer_sizes.clear();
 
        if (_running) {
                _running = false;
                Stopped(); /* EMIT SIGNAL */
        }
 
-       _jack = 0;
        return 0;
 }
 
 int
 AudioEngine::reconnect_to_jack ()
 {
-       if (_jack) {
+       if (_running) {
                disconnect_from_jack ();
                /* XXX give jackd a chance */
                Glib::usleep (250000);
@@ -1150,7 +1239,6 @@ AudioEngine::reconnect_to_jack ()
        boost::shared_ptr<Ports> p = ports.reader ();
 
        for (i = p->begin(); i != p->end(); ++i) {
-
                if ((*i)->reestablish ()) {
                        break;
                } 
@@ -1168,6 +1256,10 @@ AudioEngine::reconnect_to_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;
@@ -1182,9 +1274,9 @@ AudioEngine::reconnect_to_jack ()
        jack_set_sync_callback (_jack, _jack_sync_callback, this);
        jack_set_freewheel_callback (_jack, _freewheel_callback, this);
        
-       if (Config->get_jack_time_master()) {
+       if (session && session->config.get_jack_time_master()) {
                jack_set_timebase_callback (_jack, 0, _jack_timebase_callback, this);
-       }
+       } 
        
        if (jack_activate (_jack) == 0) {
                _running = true;
@@ -1195,21 +1287,14 @@ AudioEngine::reconnect_to_jack ()
 
        /* re-establish connections */
        
-       for (PortConnections::iterator i = port_connections.begin(); i != port_connections.end(); ++i) {
-               
-               int err;
-               
-               if ((err = jack_connect (_jack, (*i).first.c_str(), (*i).second.c_str())) != 0) {
-                       if (err != EEXIST) {
-                               error << string_compose (_("could not reconnect %1 and %2 (err = %3)"),
-                                                 (*i).first, (*i).second, err)
-                                     << endmsg;
-                       }
-               }
+       for (i = p->begin(); i != p->end(); ++i) {
+               (*i)->reconnect ();
        }
 
        Running (); /* EMIT SIGNAL*/
 
+       start_metering_thread ();
+
        return 0;
 }
 
@@ -1233,7 +1318,9 @@ void
 AudioEngine::update_total_latencies ()
 {
 #ifdef HAVE_JACK_RECOMPUTE_LATENCIES
-       jack_recompute_total_latencies (_jack);
+       if (_jack) {
+               jack_recompute_total_latencies (_jack);
+       }
 #endif
 }