Rename internal sends when their send-to bus changes.
[ardour.git] / libs / ardour / session_process.cc
index dfb070b61c63a1ee459a7fc2e7ab950ec792b474..a79a2e4cd8744460c1f85dab6f4a5e9a62bb9101 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 1999-2002 Paul Davis 
+    Copyright (C) 1999-2002 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include <algorithm>
 #include <unistd.h>
 
-#include <pbd/error.h>
+#include "pbd/error.h"
 
 #include <glibmm/thread.h>
 
-#include <ardour/ardour.h>
-#include <ardour/session.h>
-#include <ardour/timestamps.h>
-#include <ardour/audio_diskstream.h>
-#include <ardour/audioengine.h>
-#include <ardour/slave.h>
-#include <ardour/auditioner.h>
-#include <ardour/cycles.h>
-#include <ardour/cycle_timer.h>
+#include "ardour/ardour.h"
+#include "ardour/audio_diskstream.h"
+#include "ardour/audioengine.h"
+#include "ardour/auditioner.h"
+#include "ardour/butler.h"
+#include "ardour/cycle_timer.h"
+#include "ardour/cycles.h"
+#include "ardour/session.h"
+#include "ardour/slave.h"
+#include "ardour/timestamps.h"
 
-#include <midi++/manager.h>
+#include "midi++/manager.h"
 
 #include "i18n.h"
 
@@ -58,24 +59,29 @@ Session::process (nframes_t nframes)
 
        _silent = false;
 
+       if (processing_blocked()) {
+               _silent = true;
+               return;
+       }
+
        if (non_realtime_work_pending()) {
-               if (!transport_work_requested ()) {
+               if (!_butler->transport_work_requested ()) {
                        post_transport ();
-               } 
-       } 
-       
+               }
+       }
+
        (this->*process_function) (nframes);
-       
+
        // the ticker is for sending time information like MidiClock
        nframes_t transport_frames = transport_frame();
        BBT_Time  transport_bbt;
        bbt_time(transport_frames, transport_bbt);
-       SMPTE::Time transport_smpte;
-       smpte_time(transport_frames, transport_smpte);
-       tick (transport_frames, transport_bbt, transport_smpte); /* EMIT SIGNAL */
-       
+       Timecode::Time transport_timecode;
+       timecode_time(transport_frames, transport_timecode);
+       tick (transport_frames, transport_bbt, transport_timecode); /* EMIT SIGNAL */
+
        SendFeedback (); /* EMIT SIGNAL */
-       
+
        MIDI::Manager::instance()->cycle_end();
 }
 
@@ -89,7 +95,13 @@ Session::prepare_diskstreams ()
 }
 
 int
-Session::no_roll (nframes_t nframes, nframes_t offset)
+Session::fail_roll (nframes_t nframes)
+{
+       return no_roll (nframes);
+}
+
+int
+Session::no_roll (nframes_t nframes)
 {
        nframes_t end_frame = _transport_frame + nframes; // FIXME: varispeed + no_roll ??
        int ret = 0;
@@ -97,25 +109,18 @@ Session::no_roll (nframes_t nframes, nframes_t offset)
        boost::shared_ptr<RouteList> r = routes.reader ();
 
        if (_click_io) {
-               _click_io->silence (nframes, offset);
-       }
-
-       if (g_atomic_int_get (&processing_prohibited)) {
-               for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-                       (*i)->silence (nframes, offset);
-               }
-               return 0;
+               _click_io->silence (nframes);
        }
 
        for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
-               
+
                if ((*i)->is_hidden()) {
                        continue;
                }
-               
+
                (*i)->set_pending_declick (declick);
-               
-               if ((*i)->no_roll (nframes, _transport_frame, end_frame, offset, non_realtime_work_pending(), 
+
+               if ((*i)->no_roll (nframes, _transport_frame, end_frame, non_realtime_work_pending(),
                                   actively_recording(), declick)) {
                        error << string_compose(_("Session: error in no roll for %1"), (*i)->name()) << endmsg;
                        ret = -1;
@@ -127,7 +132,7 @@ Session::no_roll (nframes_t nframes, nframes_t offset)
 }
 
 int
-Session::process_routes (nframes_t nframes, nframes_t offset)
+Session::process_routes (nframes_t nframes)
 {
        bool record_active;
        int  declick = get_transport_declick_required();
@@ -154,7 +159,7 @@ Session::process_routes (nframes_t nframes, nframes_t offset)
 
                (*i)->set_pending_declick (declick);
 
-               if ((ret = (*i)->roll (nframes, start_frame, end_frame, offset, declick, record_active, rec_monitors)) < 0) {
+               if ((ret = (*i)->roll (nframes, start_frame, end_frame, declick, record_active, rec_monitors)) < 0) {
 
                        /* we have to do this here. Route::roll() for an AudioTrack will have called AudioDiskstream::process(),
                           and the DS will expect AudioDiskstream::commit() to be called. but we're aborting from that
@@ -168,14 +173,14 @@ Session::process_routes (nframes_t nframes, nframes_t offset)
 
                        stop_transport ();
                        return -1;
-               } 
+               }
        }
 
        return 0;
 }
 
 int
-Session::silent_process_routes (nframes_t nframes, nframes_t offset)
+Session::silent_process_routes (nframes_t nframes)
 {
        bool record_active = actively_recording();
        int  declick = get_transport_declick_required();
@@ -186,7 +191,7 @@ Session::silent_process_routes (nframes_t nframes, nframes_t offset)
                /* force a declick out */
                declick = -1;
        }
-       
+
        const nframes_t start_frame = _transport_frame;
        const nframes_t end_frame = _transport_frame + lrintf(nframes * _transport_speed);
 
@@ -198,8 +203,8 @@ Session::silent_process_routes (nframes_t nframes, nframes_t offset)
                        continue;
                }
 
-               if ((ret = (*i)->silent_roll (nframes, start_frame, end_frame, offset, record_active, rec_monitors)) < 0) {
-                       
+               if ((ret = (*i)->silent_roll (nframes, start_frame, end_frame, record_active, rec_monitors)) < 0) {
+
                        /* we have to do this here. Route::roll() for an AudioTrack will have called AudioDiskstream::process(),
                           and the DS will expect AudioDiskstream::commit() to be called. but we're aborting from that
                           call path, so make sure we release any outstanding locks here before we return failure.
@@ -212,7 +217,7 @@ Session::silent_process_routes (nframes_t nframes, nframes_t offset)
 
                        stop_transport ();
                        return -1;
-               } 
+               }
        }
 
        return 0;
@@ -231,14 +236,14 @@ Session::commit_diskstreams (nframes_t nframes, bool &needs_butler)
                if ((*i)->hidden()) {
                        continue;
                }
-               
+
                /* force all diskstreams not handled by a Route to call do their stuff.
                   Note: the diskstreams that were handled by a route will just return zero
                   from this call, because they know they were processed. So in fact, this
                   also runs commit() for every diskstream.
                 */
 
-               if ((dret = (*i)->process (_transport_frame, nframes, 0, actively_recording(), get_rec_monitors_input())) == 0) {
+               if ((dret = (*i)->process (_transport_frame, nframes, actively_recording(), get_rec_monitors_input())) == 0) {
                        if ((*i)->commit (nframes)) {
                                needs_butler = true;
                        }
@@ -246,7 +251,7 @@ Session::commit_diskstreams (nframes_t nframes, bool &needs_butler)
                } else if (dret < 0) {
                        (*i)->recover();
                }
-               
+
                pworst = min (pworst, (*i)->playback_buffer_load());
                cworst = min (cworst, (*i)->capture_buffer_load());
        }
@@ -273,15 +278,14 @@ Session::process_with_events (nframes_t nframes)
        Event*         ev;
        nframes_t      this_nframes;
        nframes_t      end_frame;
-       nframes_t      offset;
        bool           session_needs_butler = false;
        nframes_t      stop_limit;
        long           frames_moved;
-       
+
        /* make sure the auditioner is silent */
 
        if (auditioner) {
-               auditioner->silence (nframes, 0);
+               auditioner->silence (nframes);
        }
 
        /* handle any pending events */
@@ -292,7 +296,7 @@ Session::process_with_events (nframes_t nframes)
 
        /* if we are not in the middle of a state change,
           and there are immediate events queued up,
-          process them. 
+          process them.
        */
 
        while (!non_realtime_work_pending() && !immediate_events.empty()) {
@@ -301,11 +305,11 @@ Session::process_with_events (nframes_t nframes)
                process_event (ev);
        }
 
-       /* Events caused a transport change, send an MTC Full Frame (SMPTE) message.
+       /* Events caused a transport change, send an MTC Full Frame (Timecode) message.
         * This is sent whether rolling or not, to give slaves an idea of ardour time
         * on locates (and allow slow slaves to position and prepare for rolling)
         */
-       if (_send_smpte_update) {
+       if (_send_timecode_update) {
                send_full_time_code(nframes);
        }
 
@@ -319,14 +323,12 @@ Session::process_with_events (nframes_t nframes)
                return;
        }
 
-       /// TODO: Figure out what happens to phi and phase, if transport speed momentarily becomes
-       /// 1.0 eg. during the adjustments of a slave. If that is a bug, then AudioDiskstream::process
-       /// is very likely broken too
        if (_transport_speed == 1.0) {
                frames_moved = (long) nframes;
-       } else {                
-               frames_moved = (long) AudioDiskstream::
-                       calculate_varispeed_playback_distance(nframes, phase, phi, target_phi); 
+       } else {
+               interpolation.set_target_speed (fabs(_target_transport_speed));
+               interpolation.set_speed (fabs(_transport_speed));
+               frames_moved = (long) interpolation.interpolate (0, nframes, 0, 0);
        }
 
        end_frame = _transport_frame + (nframes_t)frames_moved;
@@ -334,23 +336,23 @@ Session::process_with_events (nframes_t nframes)
        {
                Event* this_event;
                Events::iterator the_next_one;
-               
+
                if (!process_can_proceed()) {
                        _silent = true;
                        return;
                }
-               
+
                if (!_exporting && _slave) {
-                       if (!follow_slave (nframes, 0)) {
+                       if (!follow_slave (nframes)) {
                                return;
                        }
-               } 
+               }
 
                if (_transport_speed == 0) {
-                       no_roll (nframes, 0);
+                       no_roll (nframes);
                        return;
                }
-       
+
                if (!_exporting) {
                        send_midi_time_code_for_cycle (nframes);
                }
@@ -367,46 +369,46 @@ Session::process_with_events (nframes_t nframes)
                }
 
                if (maybe_stop (stop_limit)) {
-                       no_roll (nframes, 0);
+                       no_roll (nframes);
                        return;
-               } 
+               }
 
                this_event = *next_event;
                the_next_one = next_event;
                ++the_next_one;
 
-               offset = 0;
-
                /* yes folks, here it is, the actual loop where we really truly
-                  process some audio */
+                  process some audio
+               */
+
                while (nframes) {
 
                        this_nframes = nframes; /* real (jack) time relative */
+                       frames_moved = (long) floor (_transport_speed * nframes); /* transport relative */
 
                        /* running an event, position transport precisely to its time */
                        if (this_event && this_event->action_frame <= end_frame && this_event->action_frame >= _transport_frame) {
                                /* this isn't quite right for reverse play */
                                frames_moved = (long) (this_event->action_frame - _transport_frame);
                                this_nframes = (nframes_t) abs( floor(frames_moved / _transport_speed) );
-                       } 
+                       }
 
                        if (this_nframes) {
-                               
-                               click (_transport_frame, nframes, offset);
-                               
+
+                               click (_transport_frame, this_nframes);
+
                                /* now process frames between now and the first event in this block */
                                prepare_diskstreams ();
 
-                               if (process_routes (this_nframes, offset)) {
-                                       no_roll (nframes, 0);
+                               if (process_routes (this_nframes)) {
+                                       fail_roll (nframes);
                                        return;
                                }
-                               
+
                                commit_diskstreams (this_nframes, session_needs_butler);
 
                                nframes -= this_nframes;
-                               offset += this_nframes;
-                               
+
                                if (frames_moved < 0) {
                                        decrement_transport_position (-frames_moved);
                                } else {
@@ -417,8 +419,10 @@ Session::process_with_events (nframes_t nframes)
                                check_declick_out ();
                        }
 
+                       _engine.split_cycle (this_nframes);
+
                        /* now handle this event and all others scheduled for the same time */
-                       
+
                        while (this_event && this_event->action_frame == _transport_frame) {
                                process_event (this_event);
 
@@ -428,26 +432,27 @@ Session::process_with_events (nframes_t nframes)
                                        this_event = *the_next_one;
                                        ++the_next_one;
                                }
-                       } 
+                       }
 
                        /* if an event left our state changing, do the right thing */
 
-                       if (non_realtime_work_pending()) {
-                               no_roll (nframes, offset);
+                       if (nframes && non_realtime_work_pending()) {
+                               no_roll (nframes);
                                break;
                        }
 
                        /* this is necessary to handle the case of seamless looping */
                        end_frame = _transport_frame + (nframes_t) floor (nframes * _transport_speed);
-                       
+
                }
 
                set_next_event ();
 
        } /* implicit release of route lock */
 
-       if (session_needs_butler)
-               summon_butler ();
+       if (session_needs_butler) {
+               _butler->summon ();
+       }
 }
 
 void
@@ -464,7 +469,7 @@ Session::transport_locked () const
 {
        Slave* sl = _slave;
 
-       if (!locate_pending() && ((Config->get_slave_source() == None) || (sl && sl->ok() && sl->locked()))) {
+       if (!locate_pending() && (!config.get_external_sync() || (sl && sl->ok() && sl->locked()))) {
                return true;
        }
 
@@ -472,20 +477,20 @@ Session::transport_locked () const
 }
 
 bool
-Session::follow_slave (nframes_t nframes, nframes_t offset)
+Session::follow_slave (nframes_t nframes)
 {
        double slave_speed;
-       nframes_t slave_transport_frame;
+       nframes64_t slave_transport_frame;
        nframes_t this_delta;
        int dir;
        bool starting;
 
        if (!_slave->ok()) {
                stop_transport ();
-               Config->set_slave_source (None);
+               config.set_external_sync (false);
                goto noroll;
        }
-       
+
        _slave->speed_and_position (slave_speed, slave_transport_frame);
 
        if (!_slave->locked()) {
@@ -504,15 +509,15 @@ Session::follow_slave (nframes_t nframes, nframes_t offset)
                slave_speed = 0.0f;
        }
 
-       if (_slave->is_always_synced() || Config->get_timecode_source_is_synced()) {
+       if (_slave->is_always_synced() || config.get_timecode_source_is_synced()) {
 
-               /* if the TC source is synced, then we assume that its 
+               /* if the TC source is synced, then we assume that its
                   speed is binary: 0.0 or 1.0
                */
 
                if (slave_speed != 0.0f) {
                        slave_speed = 1.0f;
-               } 
+               }
 
        } else {
 
@@ -523,17 +528,17 @@ Session::follow_slave (nframes_t nframes, nframes_t offset)
 
                calculate_moving_average_of_slave_delta(dir, this_delta);
        }
-       
+
        track_slave_state(slave_speed, slave_transport_frame, this_delta, starting);
 
-       if (slave_state == Running && !_slave->is_always_synced() && !Config->get_timecode_source_is_synced()) {
+       if (slave_state == Running && !_slave->is_always_synced() && !config.get_timecode_source_is_synced()) {
 
                if (_transport_speed != 0.0f) {
-                       
-                       /* 
-                          note that average_dir is +1 or -1 
+
+                       /*
+                          note that average_dir is +1 or -1
                        */
-                       
+
                        float delta;
 
                        #ifdef USE_MOVING_AVERAGE_OF_SLAVE
@@ -550,7 +555,7 @@ Session::follow_slave (nframes_t nframes, nframes_t offset)
                        #endif
 
                        float adjusted_speed = slave_speed + (delta /  float(_current_frame_rate));
-                       
+
                        if (_slave->give_slave_full_control_over_transport_speed()) {
                                request_transport_speed(slave_speed);
                        } else {
@@ -564,23 +569,23 @@ Session::follow_slave (nframes_t nframes, nframes_t offset)
                                         << endl;
                                #endif
                        }
-                       
+
                        if (abs(average_slave_delta) > (long) _slave->resolution()) {
                                cerr << "average slave delta greater than slave resolution, going to silent motion\n";
                                goto silent_motion;
                        }
                }
-       } 
-       
+       }
+
        #ifdef DEBUG_SLAVES
        if (slave_speed != 0.0)
        cerr << "delta = " << (int) (dir * this_delta)
-                << " speed = " << slave_speed 
-                << " ts = " << _transport_speed 
-                << " M@ "<< slave_transport_frame << " S@ " << _transport_frame 
-                << " avgdelta = " << average_slave_delta 
+                << " speed = " << slave_speed
+                << " ts = " << _transport_speed
+                << " M@ "<< slave_transport_frame << " S@ " << _transport_frame
+                << " avgdelta = " << average_slave_delta
                 << endl;
-       #endif  
+       #endif
 
        if (!starting && !non_realtime_work_pending()) {
                /* speed is set, we're locked, and good to go */
@@ -588,18 +593,18 @@ Session::follow_slave (nframes_t nframes, nframes_t offset)
        }
 
   silent_motion:
-       #ifdef DEBUG_SLAVES     
+       #ifdef DEBUG_SLAVES
        cerr << "reached silent_motion:" <<endl;
        #endif
-       
-       follow_slave_silently(nframes, offset, slave_speed);
-       
+
+       follow_slave_silently (nframes, slave_speed);
+
   noroll:
        /* don't move at all */
-       #ifdef DEBUG_SLAVES     
+       #ifdef DEBUG_SLAVES
        cerr << "reached no_roll:" <<endl;
        #endif
-       no_roll (nframes, 0);
+       no_roll (nframes);
        return false;
 }
 
@@ -614,7 +619,7 @@ Session::calculate_moving_average_of_slave_delta(int dir, nframes_t this_delta)
        if (delta_accumulator_cnt != 0 || this_delta < _current_frame_rate) {
                delta_accumulator[delta_accumulator_cnt++] = long(dir) * long(this_delta);
        }
-       
+
        if (have_first_delta_accumulator) {
                average_slave_delta = 0L;
                for (int i = 0; i < delta_accumulator_size; ++i) {
@@ -632,8 +637,8 @@ Session::calculate_moving_average_of_slave_delta(int dir, nframes_t this_delta)
 
 void
 Session::track_slave_state(
-               float slave_speed, 
-               nframes_t slave_transport_frame, 
+               float slave_speed,
+               nframes_t slave_transport_frame,
                nframes_t this_delta,
                bool starting)
 {
@@ -678,7 +683,7 @@ Session::track_slave_state(
 
                #ifdef DEBUG_SLAVES
                        cerr << "waiting at " << slave_transport_frame << endl;
-               #endif                  
+               #endif
                        if (slave_transport_frame >= slave_wait_end) {
 #ifdef DEBUG_SLAVES
                                cerr << "\tstart at " << _transport_frame << endl;
@@ -689,7 +694,7 @@ Session::track_slave_state(
                                nframes_t frame_delta = slave_transport_frame - _transport_frame;
 
                                boost::shared_ptr<DiskstreamList> dsl = diskstreams.reader();
-                               
+
                                for (DiskstreamList::iterator i = dsl->begin(); i != dsl->end(); ++i) {
                                        if (!(*i)->can_internal_playback_seek (frame_delta)) {
                                                ok = false;
@@ -702,7 +707,7 @@ Session::track_slave_state(
                                                (*i)->internal_playback_seek (frame_delta);
                                        }
                                        _transport_frame += frame_delta;
-                                      
+
                                } else {
                                        cerr << "cannot micro-seek\n";
                                        /* XXX what? */
@@ -713,14 +718,14 @@ Session::track_slave_state(
                                this_delta = 0;
                        }
                }
-               
+
                if (slave_state == Running && _transport_speed == 0.0f) {
-                       
+
         #ifdef DEBUG_SLAVES
                        cerr << "slave starts transport\n";
         #endif
                        start_transport ();
-               } 
+               }
 
        } else { // slave_speed is 0
 
@@ -729,19 +734,15 @@ Session::track_slave_state(
                if (_transport_speed != 0.0f) {
 
          #ifdef DEBUG_SLAVES
-                       cerr << "slave stops transport: " << slave_speed << " frame: " << slave_transport_frame 
+                       cerr << "slave stops transport: " << slave_speed << " frame: " << slave_transport_frame
                             << " tf = " << _transport_frame << endl;
          #endif
-                       
-                       if (Config->get_slave_source() == JACK) {
-                               last_stop_frame = _transport_frame;
-                       }
 
                        stop_transport();
                }
 
                if (slave_transport_frame != _transport_frame) {
-        #ifdef DEBUG_SLAVES                    
+        #ifdef DEBUG_SLAVES
                        cerr << "slave stopped, move to " << slave_transport_frame << endl;
         #endif
                        force_locate (slave_transport_frame, false);
@@ -752,7 +753,7 @@ Session::track_slave_state(
 }
 
 void
-Session::follow_slave_silently(nframes_t nframes, nframes_t offset, float slave_speed)
+Session::follow_slave_silently (nframes_t nframes, float slave_speed)
 {
        if (slave_speed && _transport_speed) {
 
@@ -761,25 +762,25 @@ Session::follow_slave_silently(nframes_t nframes, nframes_t offset, float slave_
                */
 
                bool need_butler;
-               
+
                prepare_diskstreams ();
-               silent_process_routes (nframes, offset);
+               silent_process_routes (nframes);
                commit_diskstreams (nframes, need_butler);
 
                if (need_butler) {
-                       summon_butler ();
+                       _butler->summon ();
                }
-               
+
                int32_t frames_moved = (int32_t) floor (_transport_speed * nframes);
-               
+
                if (frames_moved < 0) {
                        decrement_transport_position (-frames_moved);
                } else {
                        increment_transport_position (frames_moved);
                }
-               
+
                nframes_t stop_limit;
-               
+
                if (actively_recording()) {
                        stop_limit = max_frames;
                } else {
@@ -800,7 +801,6 @@ Session::process_without_events (nframes_t nframes)
        bool session_needs_butler = false;
        nframes_t stop_limit;
        long frames_moved;
-       nframes_t offset = 0;
 
        if (!process_can_proceed()) {
                _silent = true;
@@ -808,16 +808,16 @@ Session::process_without_events (nframes_t nframes)
        }
 
        if (!_exporting && _slave) {
-               if (!follow_slave (nframes, 0)) {
+               if (!follow_slave (nframes)) {
                        return;
                }
-       } 
+       }
 
        if (_transport_speed == 0) {
-               no_roll (nframes, 0);
+               fail_roll (nframes);
                return;
        }
-               
+
        if (!_exporting) {
                send_midi_time_code_for_cycle (nframes);
        }
@@ -831,32 +831,30 @@ Session::process_without_events (nframes_t nframes)
                        stop_limit = max_frames;
                }
        }
-               
+
        if (maybe_stop (stop_limit)) {
-               no_roll (nframes, 0);
+               fail_roll (nframes);
                return;
-       } 
+       }
 
-       if (maybe_sync_start (nframes, offset)) {
+       if (maybe_sync_start (nframes)) {
                return;
        }
 
-       click (_transport_frame, nframes, offset);
+       click (_transport_frame, nframes);
 
        prepare_diskstreams ();
-       
-       /// TODO: Figure out what happens to phi and phase, if transport speed momentarily becomes
-       /// 1.0 eg. during the adjustments of a slave. If that is a bug, then AudioDiskstream::process
-       /// is very likely broken too
+
        if (_transport_speed == 1.0) {
                frames_moved = (long) nframes;
-       } else {                
-               frames_moved = (long) AudioDiskstream::
-                       calculate_varispeed_playback_distance(nframes, phase, phi, target_phi); 
+       } else {
+               interpolation.set_target_speed (fabs(_target_transport_speed));
+               interpolation.set_speed (fabs(_transport_speed));
+               frames_moved = (long) interpolation.interpolate (0, nframes, 0, 0);
        }
 
-       if (process_routes (nframes, offset)) {
-               no_roll (nframes, offset);
+       if (process_routes (nframes)) {
+               fail_roll (nframes);
                return;
        }
 
@@ -871,8 +869,9 @@ Session::process_without_events (nframes_t nframes)
        maybe_stop (stop_limit);
        check_declick_out ();
 
-       if (session_needs_butler)
-               summon_butler ();
+       if (session_needs_butler) {
+               _butler->summon ();
+       }
 }
 
 /** Process callback used when the auditioner is active.
@@ -886,15 +885,15 @@ Session::process_audition (nframes_t nframes)
 
        for (RouteList::iterator i = r->begin(); i != r->end(); ++i) {
                if (!(*i)->is_hidden()) {
-                       (*i)->silence (nframes, 0);
+                       (*i)->silence (nframes);
                }
        }
 
        /* run the auditioner, and if it says we need butler service, ask for it */
-       
+
        if (auditioner->play_audition (nframes) > 0) {
-               summon_butler ();
-       } 
+               _butler->summon ();
+       }
 
        /* handle pending events */
 
@@ -904,7 +903,7 @@ Session::process_audition (nframes_t nframes)
 
        /* if we are not in the middle of a state change,
           and there are immediate events queued up,
-          process them. 
+          process them.
        */
 
        while (!non_realtime_work_pending() && !immediate_events.empty()) {
@@ -920,7 +919,7 @@ Session::process_audition (nframes_t nframes)
 }
 
 bool
-Session::maybe_sync_start (nframes_t& nframes, nframes_t& offset)
+Session::maybe_sync_start (nframes_t& nframes)
 {
        nframes_t sync_offset;
 
@@ -935,25 +934,23 @@ Session::maybe_sync_start (nframes_t& nframes, nframes_t& offset)
                   is left to do.
                */
 
-               no_roll (sync_offset, 0);
+               no_roll (sync_offset);
                nframes -= sync_offset;
-               offset += sync_offset;
+               Port::increment_port_offset (sync_offset);
                waiting_for_sync_offset = false;
-               
+
                if (nframes == 0) {
                        return true; // done, nothing left to process
                }
-               
+
        } else {
 
                /* sync offset point is not within this process()
-                  cycle, so just generate silence. and don't bother 
+                  cycle, so just generate silence. and don't bother
                   with any fancy stuff here, just the minimal silence.
                */
 
-               g_atomic_int_inc (&processing_prohibited);
-               no_roll (nframes, 0);
-               g_atomic_int_dec_and_test (&processing_prohibited);
+               _silent = true;
 
                if (Config->get_locate_while_waiting_for_sync()) {
                        if (micro_locate (nframes)) {