more MTC debugging
[ardour.git] / libs / ardour / session.cc
index c8fb202fd4b1c1529fe91bd2fd6aa023d6351160..31693daff39d715ae5384e123f1d7cfcf7f6b3ce 100644 (file)
 #include <unistd.h>
 #include <limits.h>
 
-#include <sigc++/bind.h>
-#include <sigc++/retype.h>
 
 #include <glibmm/thread.h>
 #include <glibmm/miscutils.h>
 #include <glibmm/fileutils.h>
+#include <glibmm/thread.h>
 
 #include "pbd/error.h"
-#include <glibmm/thread.h>
+#include "pbd/boost_debug.h"
 #include "pbd/pathscanner.h"
 #include "pbd/stl_delete.h"
 #include "pbd/basename.h"
@@ -62,6 +61,7 @@
 #include "ardour/crossfade.h"
 #include "ardour/cycle_timer.h"
 #include "ardour/data_type.h"
+#include "ardour/debug.h"
 #include "ardour/filename_extensions.h"
 #include "ardour/internal_send.h"
 #include "ardour/io_processor.h"
@@ -69,6 +69,7 @@
 #include "ardour/midi_playlist.h"
 #include "ardour/midi_region.h"
 #include "ardour/midi_track.h"
+#include "ardour/midi_ui.h"
 #include "ardour/named_selection.h"
 #include "ardour/playlist.h"
 #include "ardour/plugin_insert.h"
@@ -84,6 +85,7 @@
 #include "ardour/session_directory.h"
 #include "ardour/session_directory.h"
 #include "ardour/session_metadata.h"
+#include "ardour/session_playlists.h"
 #include "ardour/slave.h"
 #include "ardour/smf_source.h"
 #include "ardour/source_factory.h"
 #include "ardour/tempo.h"
 #include "ardour/utils.h"
 
+#include "midi++/jack.h"
+
 #include "i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
 using boost::shared_ptr;
+using boost::weak_ptr;
 
 bool Session::_disable_all_loaded_plugins = false;
 
-sigc::signal<void,std::string> Session::Dialog;
-sigc::signal<int> Session::AskAboutPendingState;
-sigc::signal<int,nframes_t,nframes_t> Session::AskAboutSampleRateMismatch;
-sigc::signal<void> Session::SendFeedback;
+PBD::Signal1<void,std::string> Session::Dialog;
+PBD::Signal0<int> Session::AskAboutPendingState;
+PBD::Signal2<int,nframes_t,nframes_t> Session::AskAboutSampleRateMismatch;
+PBD::Signal0<void> Session::SendFeedback;
 
-sigc::signal<void> Session::TimecodeOffsetChanged;
-sigc::signal<void> Session::StartTimeChanged;
-sigc::signal<void> Session::EndTimeChanged;
-sigc::signal<void> Session::AutoBindingOn;
-sigc::signal<void> Session::AutoBindingOff;
-sigc::signal<void, std::string, std::string> Session::Exported;
+PBD::Signal0<void> Session::TimecodeOffsetChanged;
+PBD::Signal0<void> Session::StartTimeChanged;
+PBD::Signal0<void> Session::EndTimeChanged;
+PBD::Signal0<void> Session::AutoBindingOn;
+PBD::Signal0<void> Session::AutoBindingOff;
+PBD::Signal2<void,std::string, std::string> Session::Exported;
+PBD::Signal1<int,boost::shared_ptr<Playlist> > Session::AskAboutPlaylistDeletion;
+
+static void clean_up_session_event (SessionEvent* ev) { delete ev; }
+const SessionEvent::RTeventCallback Session::rt_cleanup (clean_up_session_event);
 
 Session::Session (AudioEngine &eng,
                  const string& fullpath,
@@ -129,13 +138,10 @@ Session::Session (AudioEngine &eng,
          _midi_port (default_midi_port),
          _midi_clock_port (default_midi_clock_port),
          _session_dir (new SessionDirectory(fullpath)),
-         pending_events (2048),
          state_tree (0),
-         _butler (new Butler (this)),
+         _butler (new Butler (*this)),
          _post_transport_work (0),
          _send_timecode_update (false),
-         midi_thread (pthread_t (0)),
-         midi_requests (128), // the size of this should match the midi request pool size
          diskstreams (new DiskstreamList),
          routes (new RouteList),
          _total_free_4k_blocks (0),
@@ -149,6 +155,8 @@ Session::Session (AudioEngine &eng,
          _have_rec_enabled_diskstream (false)
 
 {
+       playlists.reset (new SessionPlaylists);
+       
        bool new_session;
 
        interpolation.add_channel_to (0, 0);
@@ -184,8 +192,8 @@ Session::Session (AudioEngine &eng,
 
        _state_of_the_state = StateOfTheState (_state_of_the_state & ~Dirty);
 
-       Config->ParameterChanged.connect (bind (mem_fun (*this, &Session::config_changed), false));
-       config.ParameterChanged.connect (bind (mem_fun (*this, &Session::config_changed), true));
+       Config->ParameterChanged.connect_same_thread (*this, boost::bind (&Session::config_changed, this, _1, false));
+       config.ParameterChanged.connect_same_thread (*this, boost::bind (&Session::config_changed, this, _1, true));
 
        if (was_dirty) {
                DirtyChanged (); /* EMIT SIGNAL */
@@ -215,13 +223,10 @@ Session::Session (AudioEngine &eng,
          _midi_port (default_midi_port),
          _midi_clock_port (default_midi_clock_port),
          _session_dir ( new SessionDirectory(fullpath)),
-         pending_events (2048),
          state_tree (0),
-         _butler (new Butler (this)),
+         _butler (new Butler (*this)),
          _post_transport_work (0),
          _send_timecode_update (false),
-         midi_thread (pthread_t (0)),
-         midi_requests (16),
          diskstreams (new DiskstreamList),
          routes (new RouteList),
          _total_free_4k_blocks (0),
@@ -234,6 +239,8 @@ Session::Session (AudioEngine &eng,
          _metadata (new SessionMetadata()),
          _have_rec_enabled_diskstream (false)
 {
+       playlists.reset (new SessionPlaylists);
+
        bool new_session;
 
        interpolation.add_channel_to (0, 0);
@@ -274,7 +281,9 @@ Session::Session (AudioEngine &eng,
 
                if (master_out_channels) {
                        ChanCount count(DataType::AUDIO, master_out_channels);
-                       shared_ptr<Route> r (new Route (*this, _("master"), Route::MasterOut, DataType::AUDIO));
+                       Route* rt = new Route (*this, _("master"), Route::MasterOut, DataType::AUDIO);
+                       boost_debug_shared_ptr_mark_interesting (rt, "Route");
+                       boost::shared_ptr<Route> r (rt);
                        r->input()->ensure_io (count, false, this);
                        r->output()->ensure_io (count, false, this);
                        r->set_remote_control_id (control_id);
@@ -287,7 +296,9 @@ Session::Session (AudioEngine &eng,
 
                if (control_out_channels) {
                        ChanCount count(DataType::AUDIO, control_out_channels);
-                       shared_ptr<Route> r (new Route (*this, _("monitor"), Route::ControlOut, DataType::AUDIO));
+                       Route* rt = new Route (*this, _("monitor"), Route::ControlOut, DataType::AUDIO);
+                       boost_debug_shared_ptr_mark_interesting (rt, "Route");
+                       shared_ptr<Route> r (rt);
                        r->input()->ensure_io (count, false, this);
                        r->output()->ensure_io (count, false, this);
                        r->set_remote_control_id (control_id++);
@@ -318,7 +329,7 @@ Session::Session (AudioEngine &eng,
 
        _state_of_the_state = StateOfTheState (_state_of_the_state & ~Dirty);
 
-       Config->ParameterChanged.connect (bind (mem_fun (*this, &Session::config_changed), false));
+       Config->ParameterChanged.connect_same_thread (*this, boost::bind (&Session::config_changed, this, _1, false));
 }
 
 Session::~Session ()
@@ -329,6 +340,8 @@ Session::~Session ()
 void
 Session::destroy ()
 {
+       vector<void*> debug_pointers;
+
        /* if we got to here, leaving pending capture state around
           is a mistake.
        */
@@ -339,12 +352,6 @@ Session::destroy ()
 
        _engine.remove_session ();
 
-       GoingAway (); /* EMIT SIGNAL */
-
-       /* do this */
-
-       notify_callbacks ();
-
        /* clear history so that no references to objects are held any more */
 
        _history.clear ();
@@ -357,8 +364,8 @@ Session::destroy ()
 
        Stateful::loading_state_version = 0;
 
-       _butler->terminate_thread ();
-       //terminate_midi_thread ();
+       delete _butler;
+       delete midi_control_ui;
 
        if (click_data != default_click) {
                delete [] click_data;
@@ -374,123 +381,90 @@ Session::destroy ()
        delete _silent_buffers;
        delete _mix_buffers;
 
-       AudioDiskstream::free_working_buffers();
-
-       Route::SyncOrderKeys.clear();
-
-#undef TRACK_DESTRUCTION
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete named selections\n";
-#endif /* TRACK_DESTRUCTION */
-       for (NamedSelectionList::iterator i = named_selections.begin(); i != named_selections.end(); ) {
-               NamedSelectionList::iterator tmp;
-
-               tmp = i;
-               ++tmp;
-
-               delete *i;
-               i = tmp;
-       }
-
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete playlists\n";
-#endif /* TRACK_DESTRUCTION */
-       for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ) {
-               PlaylistList::iterator tmp;
-
-               tmp = i;
-               ++tmp;
-
-               (*i)->drop_references ();
-
-               i = tmp;
-       }
+       /* clear out any pending dead wood from RCU managed objects */
 
-       for (PlaylistList::iterator i = unused_playlists.begin(); i != unused_playlists.end(); ) {
-               PlaylistList::iterator tmp;
-
-               tmp = i;
-               ++tmp;
-
-               (*i)->drop_references ();
-
-               i = tmp;
-       }
-
-       playlists.clear ();
-       unused_playlists.clear ();
+       routes.flush ();
+       diskstreams.flush ();
+       _bundles.flush ();
+       
+       AudioDiskstream::free_working_buffers();
 
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete regions\n";
-#endif /* TRACK_DESTRUCTION */
+       /* tell everyone who is still standing that we're about to die */
+       drop_references ();
 
-       for (RegionList::iterator i = regions.begin(); i != regions.end(); ) {
-               RegionList::iterator tmp;
+       /* tell everyone to drop references and delete objects as we go */
 
-               tmp = i;
-               ++tmp;
+       DEBUG_TRACE (DEBUG::Destruction, "delete named selections\n");
+       named_selections.clear ();
 
+       DEBUG_TRACE (DEBUG::Destruction, "delete regions\n");
+       for (RegionList::iterator i = regions.begin(); i != regions.end(); ++i) {
+               DEBUG_TRACE(DEBUG::Destruction, string_compose ("Dropping for region %1 ; pre-ref = %2\n", i->second->name(), i->second.use_count()));
                i->second->drop_references ();
-
-               i = tmp;
        }
-
        regions.clear ();
 
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete routes\n";
-#endif /* TRACK_DESTRUCTION */
+       DEBUG_TRACE (DEBUG::Destruction, "delete routes\n");
+       
+       /* reset these three references to special routes before we do the usual route delete thing */
+
+       auditioner.reset ();
+       _master_out.reset ();
+       _control_out.reset ();
+
        {
                RCUWriter<RouteList> writer (routes);
                boost::shared_ptr<RouteList> r = writer.get_copy ();
+
                for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
+                       DEBUG_TRACE(DEBUG::Destruction, string_compose ("Dropping for route %1 ; pre-ref = %2\n", (*i)->name(), (*i).use_count()));
                        (*i)->drop_references ();
                }
+
                r->clear ();
                /* writer goes out of scope and updates master */
        }
-
        routes.flush ();
 
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete diskstreams\n";
-#endif /* TRACK_DESTRUCTION */
-       {
-              RCUWriter<DiskstreamList> dwriter (diskstreams);
-              boost::shared_ptr<DiskstreamList> dsl = dwriter.get_copy();
-              for (DiskstreamList::iterator i = dsl->begin(); i != dsl->end(); ++i) {
-                      (*i)->drop_references ();
-              }
-              dsl->clear ();
-       }
-       diskstreams.flush ();
-
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete audio sources\n";
-#endif /* TRACK_DESTRUCTION */
-       for (SourceMap::iterator i = sources.begin(); i != sources.end(); ) {
-               SourceMap::iterator tmp;
-
-               tmp = i;
-               ++tmp;
+       boost::shared_ptr<RouteList> r = routes.reader ();
 
-               i->second->drop_references ();
+       DEBUG_TRACE (DEBUG::Destruction, "delete diskstreams\n");
+       {
+               RCUWriter<DiskstreamList> dwriter (diskstreams);
+               boost::shared_ptr<DiskstreamList> dsl = dwriter.get_copy();
+               for (DiskstreamList::iterator i = dsl->begin(); i != dsl->end(); ++i) {
+                       DEBUG_TRACE(DEBUG::Destruction, string_compose ("Dropping for diskstream %1 ; pre-ref = %2\n", (*i)->name(), (*i).use_count()));
+                       (*i)->drop_references ();
+               }
 
-               i = tmp;
+               dsl->clear ();
+       }
+       diskstreams.flush ();
+
+       DEBUG_TRACE (DEBUG::Destruction, "delete sources\n");
+       for (SourceMap::iterator i = sources.begin(); i != sources.end(); ++i) {
+               DEBUG_TRACE(DEBUG::Destruction, string_compose ("Dropping for source %1 ; pre-ref = %2\n", i->second->path(), i->second.use_count()));
+               i->second->drop_references ();
        }
-       sources.clear ();
 
+       sources.clear ();
 
-#ifdef TRACK_DESTRUCTION
-       cerr << "delete route groups\n";
-#endif /* TRACK_DESTRUCTION */
+       DEBUG_TRACE (DEBUG::Destruction, "delete route groups\n");
        for (list<RouteGroup *>::iterator i = _route_groups.begin(); i != _route_groups.end(); ++i) {
+               
                delete *i;
        }
 
        Crossfade::set_buffer_size (0);
 
        delete mmc;
+
+       /* not strictly necessary, but doing it here allows the shared_ptr debugging to work */
+       playlists.reset ();
+
+       boost_debug_list_ptrs ();
+
+       DEBUG_TRACE (DEBUG::Destruction, "Session::destroy() done\n");
 }
 
 void
@@ -523,11 +497,15 @@ Session::when_engine_running ()
 
        BootMessage (_("Using configuration"));
 
-       Config->map_parameters (bind (mem_fun (*this, &Session::config_changed), false));
+       boost::function<void (std::string)> ff (boost::bind (&Session::config_changed, this, _1, false));
+       boost::function<void (std::string)> ft (boost::bind (&Session::config_changed, this, _1, true));
+
+       Config->map_parameters (ff);
+       config.map_parameters (ft);
 
        /* every time we reconnect, recompute worst case output latencies */
 
-       _engine.Running.connect (mem_fun (*this, &Session::set_worst_io_latencies));
+       _engine.Running.connect_same_thread (*this, boost::bind (&Session::set_worst_io_latencies, this));
 
        if (synced_to_jack()) {
                _engine.transport_stop ();
@@ -814,6 +792,7 @@ Session::hookup_io ()
        /* Tell all IO objects to connect themselves together */
 
        IO::enable_connecting ();
+       MIDI::JACK_MidiPort::MakeConnections ();
 
        /* Now reset all panners */
 
@@ -881,7 +860,7 @@ Session::diskstream_playlist_changed (boost::weak_ptr<Diskstream> wp)
        boost::shared_ptr<Playlist> playlist;
 
        if ((playlist = dstream->playlist()) != 0) {
-               playlist->LengthChanged.connect (mem_fun (this, &Session::playlist_length_changed));
+               playlist->LengthChanged.connect_same_thread (*this, boost::bind (&Session::playlist_length_changed, this));
        }
 
        /* see comment in playlist_length_changed () */
@@ -930,7 +909,7 @@ Session::reset_input_monitor_state ()
 void
 Session::auto_punch_start_changed (Location* location)
 {
-       replace_event (Event::PunchIn, location->start());
+       replace_event (SessionEvent::PunchIn, location->start());
 
        if (get_record_enabled() && config.get_punch_in()) {
                /* capture start has been changed, so save new pending state */
@@ -943,7 +922,7 @@ Session::auto_punch_end_changed (Location* location)
 {
        nframes_t when_to_stop = location->end();
        // when_to_stop += _worst_output_latency + _worst_input_latency;
-       replace_event (Event::PunchOut, when_to_stop);
+       replace_event (SessionEvent::PunchOut, when_to_stop);
 }
 
 void
@@ -951,15 +930,15 @@ Session::auto_punch_changed (Location* location)
 {
        nframes_t when_to_stop = location->end();
 
-       replace_event (Event::PunchIn, location->start());
+       replace_event (SessionEvent::PunchIn, location->start());
        //when_to_stop += _worst_output_latency + _worst_input_latency;
-       replace_event (Event::PunchOut, when_to_stop);
+       replace_event (SessionEvent::PunchOut, when_to_stop);
 }
 
 void
 Session::auto_loop_changed (Location* location)
 {
-       replace_event (Event::AutoLoop, location->end(), location->start());
+       replace_event (SessionEvent::AutoLoop, location->end(), location->start());
 
        if (transport_rolling() && play_loop) {
 
@@ -968,7 +947,7 @@ Session::auto_loop_changed (Location* location)
 
                if (_transport_frame < location->start() || _transport_frame > location->end()) {
                        // relocate to beginning of loop
-                       clear_events (Event::LocateRoll);
+                       clear_events (SessionEvent::LocateRoll);
 
                        request_locate (location->start(), true);
 
@@ -980,8 +959,8 @@ Session::auto_loop_changed (Location* location)
                        loop_changing = true;
 
                        if (location->end() > last_loopend) {
-                               clear_events (Event::LocateRoll);
-                               Event *ev = new Event (Event::LocateRoll, Event::Add, last_loopend, last_loopend, 0, true);
+                               clear_events (SessionEvent::LocateRoll);
+                               SessionEvent *ev = new SessionEvent (SessionEvent::LocateRoll, SessionEvent::Add, last_loopend, last_loopend, 0, true);
                                queue_event (ev);
                        }
 
@@ -997,12 +976,10 @@ Session::set_auto_punch_location (Location* location)
        Location* existing;
 
        if ((existing = _locations.auto_punch_location()) != 0 && existing != location) {
-               auto_punch_start_changed_connection.disconnect();
-               auto_punch_end_changed_connection.disconnect();
-               auto_punch_changed_connection.disconnect();
+               punch_connections.drop_connections();
                existing->set_auto_punch (false, this);
-               remove_event (existing->start(), Event::PunchIn);
-               clear_events (Event::PunchOut);
+               remove_event (existing->start(), SessionEvent::PunchIn);
+               clear_events (SessionEvent::PunchOut);
                auto_punch_location_changed (0);
        }
 
@@ -1017,17 +994,14 @@ Session::set_auto_punch_location (Location* location)
                return;
        }
 
-       auto_punch_start_changed_connection.disconnect();
-       auto_punch_end_changed_connection.disconnect();
-       auto_punch_changed_connection.disconnect();
+       punch_connections.drop_connections ();
 
-       auto_punch_start_changed_connection = location->start_changed.connect (mem_fun (this, &Session::auto_punch_start_changed));
-       auto_punch_end_changed_connection = location->end_changed.connect (mem_fun (this, &Session::auto_punch_end_changed));
-       auto_punch_changed_connection = location->changed.connect (mem_fun (this, &Session::auto_punch_changed));
+       location->start_changed.connect_same_thread (punch_connections, boost::bind (&Session::auto_punch_start_changed, this, _1));
+       location->end_changed.connect_same_thread (punch_connections, boost::bind (&Session::auto_punch_end_changed, this, _1));
+       location->changed.connect_same_thread (punch_connections, boost::bind (&Session::auto_punch_changed, this, _1));
 
        location->set_auto_punch (true, this);
 
-
        auto_punch_changed (location);
 
        auto_punch_location_changed (location);
@@ -1039,11 +1013,9 @@ Session::set_auto_loop_location (Location* location)
        Location* existing;
 
        if ((existing = _locations.auto_loop_location()) != 0 && existing != location) {
-               auto_loop_start_changed_connection.disconnect();
-               auto_loop_end_changed_connection.disconnect();
-               auto_loop_changed_connection.disconnect();
+               loop_connections.drop_connections ();
                existing->set_auto_loop (false, this);
-               remove_event (existing->end(), Event::AutoLoop);
+               remove_event (existing->end(), SessionEvent::AutoLoop);
                auto_loop_location_changed (0);
        }
 
@@ -1060,16 +1032,11 @@ Session::set_auto_loop_location (Location* location)
 
        last_loopend = location->end();
 
-       auto_loop_start_changed_connection.disconnect();
-       auto_loop_end_changed_connection.disconnect();
-       auto_loop_changed_connection.disconnect();
+       loop_connections.drop_connections ();
 
-       auto_loop_start_changed_connection = location->start_changed.connect (
-                       mem_fun (this, &Session::auto_loop_changed));
-       auto_loop_end_changed_connection = location->end_changed.connect (
-                       mem_fun (this, &Session::auto_loop_changed));
-       auto_loop_changed_connection = location->changed.connect (
-                       mem_fun (this, &Session::auto_loop_changed));
+       location->start_changed.connect_same_thread (loop_connections, boost::bind (&Session::auto_loop_changed, this, _1));
+       location->end_changed.connect_same_thread (loop_connections, boost::bind (&Session::auto_loop_changed, this, _1));
+       location->changed.connect_same_thread (loop_connections, boost::bind (&Session::auto_loop_changed, this, _1));
 
        location->set_auto_loop (true, this);
 
@@ -1173,8 +1140,9 @@ Session::disable_record (bool rt_context, bool force)
                // FIXME: timestamp correct? [DR]
                // FIXME FIXME FIXME: rt_context?  this must be called in the process thread.
                // does this /need/ to be sent in all cases?
-               if (rt_context)
+               if (rt_context) {
                        deliver_mmc (MIDI::MachineControl::cmdRecordExit, _transport_frame);
+               }
 
                if (Config->get_monitoring_model() == HardwareMonitoring && config.get_auto_input()) {
                        boost::shared_ptr<DiskstreamList> dsl = diskstreams.reader();
@@ -1444,15 +1412,18 @@ trace_terminal (shared_ptr<Route> r1, shared_ptr<Route> rbase)
 
        /* make a copy of the existing list of routes that feed r1 */
 
-       set<shared_ptr<Route> > existing = r1->fed_by;
+       set<weak_ptr<Route> > existing = r1->fed_by;
 
        /* for each route that feeds r1, recurse, marking it as feeding
           rbase as well.
        */
 
-       for (set<shared_ptr<Route> >::iterator i = existing.begin(); i != existing.end(); ++i) {
-               r2 =* i;
-
+       for (set<weak_ptr<Route> >::iterator i = existing.begin(); i != existing.end(); ++i) {
+               if (!(r2 = (*i).lock ())) {
+                       /* (*i) went away, ignore it */
+                       continue;
+               }
+               
                /* r2 is a route that feeds r1 which somehow feeds base. mark
                   base as being fed by r2
                */
@@ -1463,7 +1434,7 @@ trace_terminal (shared_ptr<Route> r1, shared_ptr<Route> rbase)
 
                        /* 2nd level feedback loop detection. if r1 feeds or is fed by r2,
                           stop here.
-                        */
+                       */
 
                        if ((r1->fed_by.find (r2) != r1->fed_by.end()) && (r2->fed_by.find (r1) != r2->fed_by.end())) {
                                continue;
@@ -1655,9 +1626,11 @@ Session::new_midi_track (TrackMode mode, RouteGroup* route_group, uint32_t how_m
                        */
 
                        track->midi_diskstream()->non_realtime_input_change();
-                       track->set_route_group (route_group, 0);
+                       if (route_group) {
+                               route_group->add (track);
+                       }
 
-                       track->DiskstreamChanged.connect (mem_fun (this, &Session::resort_routes));
+                       track->DiskstreamChanged.connect_same_thread (*this, boost::bind (&Session::resort_routes, this));
                        //track->set_remote_control_id (control_id);
 
                        new_routes.push_back (track);
@@ -1769,7 +1742,9 @@ Session::new_audio_track (int input_channels, int output_channels, TrackMode mod
                shared_ptr<AudioTrack> track;
 
                try {
-                       track = boost::shared_ptr<AudioTrack>((new AudioTrack (*this, track_name, Route::Flag (0), mode)));
+                       AudioTrack* at = new AudioTrack (*this, track_name, Route::Flag (0), mode);
+                       boost_debug_shared_ptr_mark_interesting (at, "Track");
+                       track = boost::shared_ptr<AudioTrack>(at);
 
                        if (track->input()->ensure_io (ChanCount(DataType::AUDIO, input_channels), false, this)) {
                                error << string_compose (_("cannot configure %1 in/%2 out configuration for new audio track"),
@@ -1824,11 +1799,13 @@ Session::new_audio_track (int input_channels, int output_channels, TrackMode mod
 
                        channels_used += track->n_inputs ().n_audio();
 
-                       track->set_route_group (route_group, 0);
+                       if (route_group) {
+                               route_group->add (track);
+                       }
 
                        track->audio_diskstream()->non_realtime_input_change();
 
-                       track->DiskstreamChanged.connect (mem_fun (this, &Session::resort_routes));
+                       track->DiskstreamChanged.connect_same_thread (*this, boost::bind (&Session::resort_routes, this));
                        track->set_remote_control_id (control_id);
                        ++control_id;
 
@@ -1886,25 +1863,32 @@ void
 Session::set_remote_control_ids ()
 {
        RemoteModel m = Config->get_remote_model();
+       bool emit_signal = false;
 
        shared_ptr<RouteList> r = routes.reader ();
 
        for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               if ( MixerOrdered == m) {
+               if (MixerOrdered == m) {
                        long order = (*i)->order_key(N_("signal"));
-                       (*i)->set_remote_control_id( order+1 );
-               } else if ( EditorOrdered == m) {
+                       (*i)->set_remote_control_id (order+1, false);
+                       emit_signal = true;
+               } else if (EditorOrdered == m) {
                        long order = (*i)->order_key(N_("editor"));
-                       (*i)->set_remote_control_id( order+1 );
-               } else if ( UserOrdered == m) {
+                       (*i)->set_remote_control_id (order+1, false);
+                       emit_signal = true;
+               } else if (UserOrdered == m) {
                        //do nothing ... only changes to remote id's are initiated by user
                }
        }
+
+       if (emit_signal) {
+               Route::RemoteControlIDChange();
+       }
 }
 
 
 RouteList
-Session::new_audio_route (int input_channels, int output_channels, RouteGroup* route_group, uint32_t how_many)
+Session::new_audio_route (bool aux, int input_channels, int output_channels, RouteGroup* route_group, uint32_t how_many)
 {
        char bus_name[32];
        uint32_t bus_id = 1;
@@ -1956,7 +1940,9 @@ Session::new_audio_route (int input_channels, int output_channels, RouteGroup* r
                } while (bus_id < (UINT_MAX-1));
 
                try {
-                       shared_ptr<Route> bus (new Route (*this, bus_name, Route::Flag(0), DataType::AUDIO));
+                       Route* rt = new Route (*this, bus_name, Route::Flag(0), DataType::AUDIO);
+                       boost_debug_shared_ptr_mark_interesting (rt, "Route");
+                       shared_ptr<Route> bus (rt);
 
                        if (bus->input()->ensure_io (ChanCount(DataType::AUDIO, input_channels), false, this)) {
                                error << string_compose (_("cannot configure %1 in/%2 out configuration for new audio track"),
@@ -2003,10 +1989,16 @@ Session::new_audio_route (int input_channels, int output_channels, RouteGroup* r
 
                        channels_used += bus->n_inputs ().n_audio();
 
-                       bus->set_route_group (route_group, 0);
+                       if (route_group) {
+                               route_group->add (bus);
+                       }
                        bus->set_remote_control_id (control_id);
                        ++control_id;
 
+                       if (aux) {
+                               bus->add_internal_return ();
+                       }
+
                        ret.push_back (bus);
                }
 
@@ -2146,20 +2138,21 @@ Session::add_routes (RouteList& new_routes, bool save)
        for (RouteList::iterator x = new_routes.begin(); x != new_routes.end(); ++x) {
 
                boost::weak_ptr<Route> wpr (*x);
+               boost::shared_ptr<Route> r (*x);
 
-               (*x)->listen_changed.connect (sigc::bind (mem_fun (*this, &Session::route_listen_changed), wpr));
-               (*x)->solo_changed.connect (sigc::bind (mem_fun (*this, &Session::route_solo_changed), wpr));
-               (*x)->mute_changed.connect (mem_fun (*this, &Session::route_mute_changed));
-               (*x)->output()->changed.connect (mem_fun (*this, &Session::set_worst_io_latencies_x));
-               (*x)->processors_changed.connect (bind (mem_fun (*this, &Session::update_latency_compensation), false, false));
-               (*x)->route_group_changed.connect (hide (mem_fun (*this, &Session::route_group_changed)));
+               r->listen_changed.connect_same_thread (*this, boost::bind (&Session::route_listen_changed, this, _1, wpr));
+               r->solo_changed.connect_same_thread (*this, boost::bind (&Session::route_solo_changed, this, _1, wpr));
+               r->mute_changed.connect_same_thread (*this, boost::bind (&Session::route_mute_changed, this, _1));
+               r->output()->changed.connect_same_thread (*this, boost::bind (&Session::set_worst_io_latencies_x, this, _1, _2));
+               r->processors_changed.connect_same_thread (*this, boost::bind (&Session::route_processors_changed, this, _1));
+               r->route_group_changed.connect_same_thread (*this, boost::bind (&Session::route_group_changed, this));
 
-               if ((*x)->is_master()) {
-                       _master_out = (*x);
+               if (r->is_master()) {
+                       _master_out = r;
                }
 
-               if ((*x)->is_control()) {
-                       _control_out = (*x);
+               if (r->is_control()) {
+                       _control_out = r;
                }
        }
 
@@ -2184,6 +2177,7 @@ Session::add_routes (RouteList& new_routes, bool save)
        }
 
        RouteAdded (new_routes); /* EMIT SIGNAL */
+       Route::RemoteControlIDChange (); /* EMIT SIGNAL */
 }
 
 void
@@ -2254,7 +2248,6 @@ Session::globally_add_internal_sends (boost::shared_ptr<Route> dest, Placement p
        add_internal_sends (dest, p, t);
 }
 
-
 void
 Session::add_internal_sends (boost::shared_ptr<Route> dest, Placement p, boost::shared_ptr<RouteList> senders)
 {
@@ -2274,6 +2267,8 @@ Session::add_internal_sends (boost::shared_ptr<Route> dest, Placement p, boost::
 
                (*i)->listen_via (dest, p, true, true);
        }
+
+       graph_reordered ();
 }
 
 void
@@ -2291,11 +2286,11 @@ Session::add_diskstream (boost::shared_ptr<Diskstream> dstream)
                /* writer goes out of scope, copies ds back to main */
        }
 
-       dstream->PlaylistChanged.connect (sigc::bind (mem_fun (*this, &Session::diskstream_playlist_changed), boost::weak_ptr<Diskstream> (dstream)));
+       dstream->PlaylistChanged.connect_same_thread (*this, boost::bind (&Session::diskstream_playlist_changed, this, boost::weak_ptr<Diskstream> (dstream)));
        /* this will connect to future changes, and check the current length */
        diskstream_playlist_changed (boost::weak_ptr<Diskstream> (dstream));
 
-       dstream->RecordEnableChanged.connect (mem_fun (*this, &Session::update_have_rec_enabled_diskstream));
+       dstream->RecordEnableChanged.connect_same_thread (*this, boost::bind (&Session::update_have_rec_enabled_diskstream, this));
 
        dstream->prepare ();
 
@@ -2358,6 +2353,18 @@ Session::remove_route (shared_ptr<Route> route)
        route->input()->disconnect (0);
        route->output()->disconnect (0);
 
+       /* if the route had internal sends sending to it, remove them */
+       if (route->internal_return()) {
+
+               boost::shared_ptr<RouteList> r = routes.reader ();
+               for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
+                       boost::shared_ptr<Send> s = (*i)->internal_send_for (route);
+                       if (s) {
+                               (*i)->remove_processor (s);
+                       }
+               }
+       }       
+
        update_latency_compensation (false, false);
        set_dirty();
 
@@ -2373,6 +2380,8 @@ Session::remove_route (shared_ptr<Route> route)
 
        sync_order_keys (N_("session"));
 
+       Route::RemoteControlIDChange(); /* EMIT SIGNAL */
+
        /* save the new state of the world */
 
        if (save_state (_current_snapshot_name)) {
@@ -2421,7 +2430,7 @@ Session::route_solo_changed (void* /*src*/, boost::weak_ptr<Route> wpr)
        shared_ptr<RouteList> r = routes.reader ();
        int32_t delta;
 
-       if (route->soloed()) {
+       if (route->self_soloed()) {
                delta = 1;
        } else {
                delta = -1;
@@ -2432,24 +2441,30 @@ Session::route_solo_changed (void* /*src*/, boost::weak_ptr<Route> wpr)
        */
 
        solo_update_disabled = true;
+
        for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
+               bool via_sends_only;
 
-               if ((*i)->feeds (route) && !(*i)->is_hidden() && !(*i)->is_master() && !(*i)->is_control()) {
-                       /* do it */
-                       (*i)->mod_solo_level (delta);
-               }
+
+               if ((*i) == route || !(*i)->solo_isolated() || !(*i)->is_master() || !(*i)->is_control() || (*i)->is_hidden()) {
+                       continue;
+               } else if ((*i)->feeds (route, &via_sends_only)) {
+                       if (!via_sends_only) {
+                               (*i)->mod_solo_by_others (delta);
+                       }
+               } 
        }
 
        /* make sure master is never muted by solo */
 
-       if (_master_out && route != _master_out && _master_out->solo_level() == 0 && !_master_out->soloed()) {
-               _master_out->mod_solo_level (1);
-       }
-
+       if (_master_out && route != _master_out && _master_out->soloed_by_others() == 0 && !_master_out->soloed()) {
+               _master_out->mod_solo_by_others (1);
+       }
        /* ditto for control outs make sure master is never muted by solo */
 
-       if (_control_out && route != _control_out && _control_out && _control_out->solo_level() == 0) {
-               _control_out->mod_solo_level (1);
+       if (_control_out && route != _control_out && _control_out && _control_out->soloed_by_others() == 0) {
+               _control_out->mod_solo_by_others (1);
        }
 
        solo_update_disabled = false;
@@ -2470,7 +2485,7 @@ Session::update_route_solo_state (boost::shared_ptr<RouteList> r)
        }
 
        for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               if (!(*i)->is_master() && !(*i)->is_control() && !(*i)->is_hidden() && (*i)->soloed()) {
+               if (!(*i)->is_master() && !(*i)->is_control() && !(*i)->is_hidden() && (*i)->self_soloed()) {
                        something_soloed = true;
                        break;
                }
@@ -2482,6 +2497,20 @@ Session::update_route_solo_state (boost::shared_ptr<RouteList> r)
        }
 }
 
+boost::shared_ptr<RouteList> 
+Session::get_routes_with_internal_returns() const
+{
+       shared_ptr<RouteList> r = routes.reader ();
+       boost::shared_ptr<RouteList> rl (new RouteList);
+
+       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
+               if ((*i)->internal_return ()) {
+                       rl->push_back (*i);
+               }
+       }
+       return rl;
+}
+
 shared_ptr<Route>
 Session::route_by_name (string name)
 {
@@ -2777,9 +2806,7 @@ Session::add_regions (vector<boost::shared_ptr<Region> >& new_regions)
                                }
                        }
 
-                       region->StateChanged.connect (sigc::bind (mem_fun (*this, &Session::region_changed), boost::weak_ptr<Region>(region)));
-                       region->GoingAway.connect (sigc::bind (mem_fun (*this, &Session::remove_region), boost::weak_ptr<Region>(region)));
-
+                       region->StateChanged.connect_same_thread (*this, boost::bind (&Session::region_changed, this, _1, boost::weak_ptr<Region>(region)));
                        update_region_name_map (region);
                }
 
@@ -2882,13 +2909,6 @@ Session::find_whole_file_parent (boost::shared_ptr<Region const> child)
        return boost::shared_ptr<Region> ();
 }
 
-void
-Session::find_equivalent_playlist_regions (boost::shared_ptr<Region> region, vector<boost::shared_ptr<Region> >& result)
-{
-       for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ++i)
-               (*i)->get_region_list_equivalent_regions (region, result);
-}
-
 int
 Session::destroy_region (boost::shared_ptr<Region> region)
 {
@@ -2942,6 +2962,10 @@ Session::remove_last_capture ()
                }
        }
 
+       for (list<boost::shared_ptr<Region> >::iterator i = r.begin(); i != r.end(); ++i) {
+               remove_region (*i);
+       }
+       
        destroy_regions (r);
 
        save_state (_current_snapshot_name);
@@ -2973,7 +2997,6 @@ Session::add_source (boost::shared_ptr<Source> source)
        }
 
        if (result.second) {
-               source->GoingAway.connect (sigc::bind (mem_fun (this, &Session::remove_source), boost::weak_ptr<Source> (source)));
                set_dirty();
        }
 
@@ -3014,23 +3037,6 @@ Session::remove_source (boost::weak_ptr<Source> src)
        }
 }
 
-/** Return the number of playlists (not regions) that contain @a src */
-uint32_t
-Session::source_use_count (boost::shared_ptr<const Source> src) const
-{
-       uint32_t count = 0;
-       for (PlaylistList::const_iterator p = playlists.begin(); p != playlists.end(); ++p) {
-               for (Playlist::RegionList::const_iterator r = (*p)->region_list().begin();
-                               r != (*p)->region_list().end(); ++r) {
-                       if ((*r)->uses_source(src)) {
-                               ++count;
-                               break;
-                       }
-               }
-       }
-       return count;
-}
-
 boost::shared_ptr<Source>
 Session::source_by_id (const PBD::ID& id)
 {
@@ -3177,7 +3183,7 @@ Session::change_source_path_by_name (string path, string oldname, string newname
        return path;
 }
 
-/** Return the full path (in some session directory) for a new embedded source.
+/** Return the full path (in some session directory) for a new within-session source.
  * \a name must be a session-unique name that does not contain slashes
  *         (e.g. as returned by new_*_source_name)
  */
@@ -3298,16 +3304,16 @@ Session::new_audio_source_name (const string& base, uint32_t nchan, uint32_t cha
        return Glib::path_get_basename(buf);
 }
 
-/** Create a new embedded audio source */
+/** Create a new within-session audio source */
 boost::shared_ptr<AudioFileSource>
 Session::create_audio_source_for_session (AudioDiskstream& ds, uint32_t chan, bool destructive)
 {
        const size_t n_chans = ds.n_channels().n_audio();
        const string name    = new_audio_source_name (ds.name(), n_chans, chan, destructive);
        const string path    = new_source_path_from_name(DataType::AUDIO, name);
+
        return boost::dynamic_pointer_cast<AudioFileSource> (
-                       SourceFactory::createWritable (
-                                       DataType::AUDIO, *this, path, true, destructive, frame_rate()));
+               SourceFactory::createWritable (DataType::AUDIO, *this, path, destructive, frame_rate()));
 }
 
 /** Return a unique name based on \a base for a new internal MIDI source */
@@ -3359,7 +3365,7 @@ Session::new_midi_source_name (const string& base)
 }
 
 
-/** Create a new embedded MIDI source */
+/** Create a new within-session MIDI source */
 boost::shared_ptr<MidiSource>
 Session::create_midi_source_for_session (MidiDiskstream& ds)
 {
@@ -3368,46 +3374,10 @@ Session::create_midi_source_for_session (MidiDiskstream& ds)
 
        return boost::dynamic_pointer_cast<SMFSource> (
                        SourceFactory::createWritable (
-                                       DataType::MIDI, *this, path, true, false, frame_rate()));
+                                       DataType::MIDI, *this, path, false, frame_rate()));
 }
 
 
-/* Playlist management */
-
-boost::shared_ptr<Playlist>
-Session::playlist_by_name (string name)
-{
-       Glib::Mutex::Lock lm (playlist_lock);
-       for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-               if ((*i)->name() == name) {
-                       return* i;
-               }
-       }
-       for (PlaylistList::iterator i = unused_playlists.begin(); i != unused_playlists.end(); ++i) {
-               if ((*i)->name() == name) {
-                       return* i;
-               }
-       }
-
-       return boost::shared_ptr<Playlist>();
-}
-
-void
-Session::unassigned_playlists (std::list<boost::shared_ptr<Playlist> > & list)
-{
-       Glib::Mutex::Lock lm (playlist_lock);
-       for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-               if (!(*i)->get_orig_diskstream_id().to_s().compare ("0")) {
-                       list.push_back (*i);
-               }
-       }
-       for (PlaylistList::iterator i = unused_playlists.begin(); i != unused_playlists.end(); ++i) {
-               if (!(*i)->get_orig_diskstream_id().to_s().compare ("0")) {
-                       list.push_back (*i);
-               }
-       }
-}
-
 void
 Session::add_playlist (boost::shared_ptr<Playlist> playlist, bool unused)
 {
@@ -3415,75 +3385,13 @@ Session::add_playlist (boost::shared_ptr<Playlist> playlist, bool unused)
                return;
        }
 
-       {
-               Glib::Mutex::Lock lm (playlist_lock);
-               if (find (playlists.begin(), playlists.end(), playlist) == playlists.end()) {
-                       playlists.insert (playlists.begin(), playlist);
-                       playlist->InUse.connect (sigc::bind (mem_fun (*this, &Session::track_playlist), boost::weak_ptr<Playlist>(playlist)));
-                       playlist->GoingAway.connect (sigc::bind (mem_fun (*this, &Session::remove_playlist), boost::weak_ptr<Playlist>(playlist)));
-               }
-       }
+       playlists->add (playlist);
 
        if (unused) {
                playlist->release();
        }
 
        set_dirty();
-
-       PlaylistAdded (playlist); /* EMIT SIGNAL */
-}
-
-void
-Session::get_playlists (vector<boost::shared_ptr<Playlist> >& s)
-{
-       {
-               Glib::Mutex::Lock lm (playlist_lock);
-               for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-                       s.push_back (*i);
-               }
-               for (PlaylistList::iterator i = unused_playlists.begin(); i != unused_playlists.end(); ++i) {
-                       s.push_back (*i);
-               }
-       }
-}
-
-void
-Session::track_playlist (bool inuse, boost::weak_ptr<Playlist> wpl)
-{
-       boost::shared_ptr<Playlist> pl(wpl.lock());
-
-       if (!pl) {
-               return;
-       }
-
-       PlaylistList::iterator x;
-
-       if (pl->hidden()) {
-               /* its not supposed to be visible */
-               return;
-       }
-
-       {
-               Glib::Mutex::Lock lm (playlist_lock);
-
-               if (!inuse) {
-
-                       unused_playlists.insert (pl);
-
-                       if ((x = playlists.find (pl)) != playlists.end()) {
-                               playlists.erase (x);
-                       }
-
-
-               } else {
-
-                       playlists.insert (pl);
-
-                       if ((x = unused_playlists.find (pl)) != unused_playlists.end()) {
-                               unused_playlists.erase (x);
-                       }
-               }
-       }
 }
 
 void
@@ -3499,26 +3407,9 @@ Session::remove_playlist (boost::weak_ptr<Playlist> weak_playlist)
                return;
        }
 
-       {
-               Glib::Mutex::Lock lm (playlist_lock);
-
-               PlaylistList::iterator i;
-
-               i = find (playlists.begin(), playlists.end(), playlist);
-               if (i != playlists.end()) {
-                       playlists.erase (i);
-               }
-
-               i = find (unused_playlists.begin(), unused_playlists.end(), playlist);
-               if (i != unused_playlists.end()) {
-                       unused_playlists.erase (i);
-               }
-
-       }
+       playlists->remove (playlist);
 
        set_dirty();
-
-       PlaylistRemoved (playlist); /* EMIT SIGNAL */
 }
 
 void
@@ -3532,7 +3423,7 @@ Session::set_audition (boost::shared_ptr<Region> r)
 void
 Session::audition_playlist ()
 {
-       Event* ev = new Event (Event::Audition, Event::Add, Event::Immediate, 0, 0.0);
+       SessionEvent* ev = new SessionEvent (SessionEvent::Audition, SessionEvent::Add, SessionEvent::Immediate, 0, 0.0);
        ev->region.reset ();
        queue_event (ev);
 }
@@ -3552,7 +3443,7 @@ Session::non_realtime_set_audition ()
 void
 Session::audition_region (boost::shared_ptr<Region> r)
 {
-       Event* ev = new Event (Event::Audition, Event::Add, Event::Immediate, 0, 0.0);
+       SessionEvent* ev = new SessionEvent (SessionEvent::Audition, SessionEvent::Add, SessionEvent::Immediate, 0, 0.0);
        ev->region = r;
        queue_event (ev);
 }
@@ -3584,8 +3475,8 @@ Session::remove_empty_sounds ()
        TapeFileMatcher tape_file_matcher;
 
        remove_if (audio_filenames.begin(), audio_filenames.end(),
-                       sigc::mem_fun (tape_file_matcher, &TapeFileMatcher::matches));
-
+                  boost::bind (&TapeFileMatcher::matches, &tape_file_matcher, _1));
+       
        for (vector<string>::iterator i = audio_filenames.begin(); i != audio_filenames.end(); ++i) {
 
                sys::path audio_file_path (_session_dir->sound_path());
@@ -3619,48 +3510,6 @@ Session::is_auditioning () const
        }
 }
 
-void
-Session::set_all_solo (bool yn)
-{
-       shared_ptr<RouteList> r = routes.reader ();
-
-       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               if (!(*i)->is_hidden()) {
-                       (*i)->set_solo (yn, this);
-               }
-       }
-
-       set_dirty();
-}
-
-void
-Session::set_all_listen (bool yn)
-{
-       shared_ptr<RouteList> r = routes.reader ();
-
-       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               if (!(*i)->is_hidden()) {
-                       (*i)->set_listen (yn, this);
-               }
-       }
-
-       set_dirty();
-}
-
-void
-Session::set_all_mute (bool yn)
-{
-       shared_ptr<RouteList> r = routes.reader ();
-
-       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               if (!(*i)->is_hidden()) {
-                       (*i)->set_mute (yn, this);
-               }
-       }
-
-       set_dirty();
-}
-
 uint32_t
 Session::n_diskstreams () const
 {
@@ -3680,10 +3529,10 @@ void
 Session::graph_reordered ()
 {
        /* don't do this stuff if we are setting up connections
-          from a set_state() call or creating new tracks.
+          from a set_state() call or creating new tracks. Ditto for deletion.
        */
 
-       if (_state_of_the_state & InitialConnecting) {
+       if (_state_of_the_state & (InitialConnecting|Deletion)) {
                return;
        }
 
@@ -3706,38 +3555,14 @@ Session::graph_reordered ()
        }
 }
 
-void
-Session::record_disenable_all ()
-{
-       record_enable_change_all (false);
-}
-
-void
-Session::record_enable_all ()
-{
-       record_enable_change_all (true);
-}
-
-void
-Session::record_enable_change_all (bool yn)
-{
-       shared_ptr<RouteList> r = routes.reader ();
-
-       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               boost::shared_ptr<Track> t;
-
-               if ((t = boost::dynamic_pointer_cast<Track>(*i)) != 0) {
-                       t->set_record_enable (yn, this);
-               }
-       }
-
-       /* since we don't keep rec-enable state, don't mark session dirty */
-}
-
 void
 Session::add_processor (Processor* processor)
 {
-       processor->GoingAway.connect (sigc::bind (mem_fun (*this, &Session::remove_processor), processor));
+       /* Session does not own Processors (they belong to a Route) but we do want to track
+          the arrival and departure of port inserts, sends and returns for naming
+          purposes.
+       */
+       processor->DropReferences.connect_same_thread (*this, boost::bind (&Session::remove_processor, this, processor));
        set_dirty();
 }
 
@@ -3753,7 +3578,7 @@ Session::remove_processor (Processor* processor)
        } else if ((send = dynamic_cast<Send *> (processor)) != 0) {
                send_bitset[send->bit_slot()] = false;
        } else if ((retrn = dynamic_cast<Return *> (processor)) != 0) {
-               return_bitset[send->bit_slot()] = false;
+               return_bitset[retrn->bit_slot()] = false;
        }
 
        set_dirty();
@@ -3850,13 +3675,7 @@ Session::tempo_map_changed (Change)
 {
        clear_clicks ();
 
-       for (PlaylistList::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-               (*i)->update_after_tempo_map_change ();
-       }
-
-       for (PlaylistList::iterator i = unused_playlists.begin(); i != unused_playlists.end(); ++i) {
-               (*i)->update_after_tempo_map_change ();
-       }
+       playlists->update_after_tempo_map_change ();
 
        set_dirty ();
 }
@@ -3987,37 +3806,33 @@ Session::mark_insert_id (uint32_t id)
 
 /* Named Selection management */
 
-NamedSelection *
+boost::shared_ptr<NamedSelection>
 Session::named_selection_by_name (string name)
 {
        Glib::Mutex::Lock lm (named_selection_lock);
        for (NamedSelectionList::iterator i = named_selections.begin(); i != named_selections.end(); ++i) {
                if ((*i)->name == name) {
-                       returni;
+                       return *i;
                }
        }
-       return 0;
+       return boost::shared_ptr<NamedSelection>();
 }
 
 void
-Session::add_named_selection (NamedSelection* named_selection)
+Session::add_named_selection (boost::shared_ptr<NamedSelection> named_selection)
 {
        {
                Glib::Mutex::Lock lm (named_selection_lock);
                named_selections.insert (named_selections.begin(), named_selection);
        }
 
-       for (list<boost::shared_ptr<Playlist> >::iterator i = named_selection->playlists.begin(); i != named_selection->playlists.end(); ++i) {
-               add_playlist (*i);
-       }
-
        set_dirty();
 
        NamedSelectionAdded (); /* EMIT SIGNAL */
 }
 
 void
-Session::remove_named_selection (NamedSelection* named_selection)
+Session::remove_named_selection (boost::shared_ptr<NamedSelection> named_selection)
 {
        bool removed = false;
 
@@ -4027,7 +3842,6 @@ Session::remove_named_selection (NamedSelection* named_selection)
                NamedSelectionList::iterator i = find (named_selections.begin(), named_selections.end(), named_selection);
 
                if (i != named_selections.end()) {
-                       delete (*i);
                        named_selections.erase (i);
                        set_dirty();
                        removed = true;
@@ -4077,13 +3891,6 @@ Session::route_name_internal (string n) const
        return false;
 }
 
-uint32_t
-Session::n_playlists () const
-{
-       Glib::Mutex::Lock lm (playlist_lock);
-       return playlists.size();
-}
-
 void
 Session::allocate_pan_automation_buffers (nframes_t nframes, uint32_t howmany, bool force)
 {
@@ -4154,8 +3961,7 @@ Session::write_one_track (AudioTrack& track, nframes_t start, nframes_t end,
                return result;
        }
 
-       // any bigger than this seems to cause stack overflows in called functions
-       const nframes_t chunk_size = (128 * 1024)/4;
+       const nframes_t chunk_size = (256 * 1024)/4;
 
        // block all process callback handling
 
@@ -4189,7 +3995,7 @@ Session::write_one_track (AudioTrack& track, nframes_t start, nframes_t end,
 
                try {
                        fsource = boost::dynamic_pointer_cast<AudioFileSource> (
-                               SourceFactory::createWritable (DataType::AUDIO, *this, buf, true, false, frame_rate()));
+                               SourceFactory::createWritable (DataType::AUDIO, *this, buf, false, frame_rate()));
                }
 
                catch (failed_constructor& err) {
@@ -4371,6 +4177,10 @@ Session::compute_initial_length ()
 void
 Session::sync_order_keys (std::string const & base)
 {
+       if (deletion_in_progress()) {
+               return;
+       }
+
        if (!Config->get_sync_all_route_ordering()) {
                /* leave order keys as they are */
                return;
@@ -4383,8 +4193,11 @@ Session::sync_order_keys (std::string const & base)
        }
 
        Route::SyncOrderKeys (base); // EMIT SIGNAL
-}
 
+       /* this might not do anything */
+
+       set_remote_control_ids ();
+}
 
 /** @return true if there is at least one record-enabled diskstream, otherwise false */
 bool
@@ -4440,9 +4253,9 @@ Session::solo_control_mode_changed ()
        /* cancel all solo or all listen when solo control mode changes */
 
        if (Config->get_solo_control_is_listen_control()) {
-               set_all_solo (false);
+               set_solo (routes.reader(), false);
        } else {
-               set_all_listen (false);
+               set_listen (routes.reader(), false);
        }
 }
 
@@ -4469,3 +4282,33 @@ Session::get_available_sync_options () const
 
        return ret;
 }
+
+boost::shared_ptr<RouteList>
+Session::get_routes_with_regions_at (nframes64_t const p) const
+{
+       shared_ptr<RouteList> r = routes.reader ();
+       shared_ptr<RouteList> rl (new RouteList);
+
+       for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
+               boost::shared_ptr<Track> tr = boost::dynamic_pointer_cast<Track> (*i);
+               if (!tr) {
+                       continue;
+               }
+               
+               boost::shared_ptr<Diskstream> ds = tr->diskstream ();
+               if (!ds) {
+                       continue;
+               }
+
+               boost::shared_ptr<Playlist> pl = ds->playlist ();
+               if (!pl) {
+                       continue;
+               }
+               
+               if (pl->has_region_at (p)) {
+                       rl->push_back (*i);
+               }
+       }
+
+       return rl;
+}