midi-controller, support log parameters - fixes #5890
[ardour.git] / libs / ardour / plugin_insert.cc
index fadf588ffead12fcc1775a21ae7061b6671ea4ff..ae018fb2a3c53544c5e20034e4b13455b87d16ef 100644 (file)
 #include "ardour/audio_buffer.h"
 #include "ardour/automation_list.h"
 #include "ardour/buffer_set.h"
+#include "ardour/debug.h"
 #include "ardour/event_type_map.h"
 #include "ardour/ladspa_plugin.h"
 #include "ardour/plugin.h"
 #include "ardour/plugin_insert.h"
-#include "ardour/port.h"
-#include "ardour/route.h"
 
 #ifdef LV2_SUPPORT
 #include "ardour/lv2_plugin.h"
 #endif
 
-#ifdef VST_SUPPORT
-#include "ardour/vst_plugin.h"
+#ifdef WINDOWS_VST_SUPPORT
+#include "ardour/windows_vst_plugin.h"
 #endif
 
-#ifdef HAVE_AUDIOUNITS
+#ifdef LXVST_SUPPORT
+#include "ardour/lxvst_plugin.h"
+#endif
+
+#ifdef AUDIOUNIT_SUPPORT
 #include "ardour/audio_unit.h"
 #endif
 
-#include "ardour/audioengine.h"
 #include "ardour/session.h"
 #include "ardour/types.h"
 
@@ -65,18 +67,12 @@ PluginInsert::PluginInsert (Session& s, boost::shared_ptr<Plugin> plug)
        : Processor (s, (plug ? plug->name() : string ("toBeRenamed")))
        , _signal_analysis_collected_nframes(0)
        , _signal_analysis_collect_nframes_max(0)
-       , _splitting (false)
 {
        /* the first is the master */
 
        if (plug) {
-               plug->set_insert_info (this);
-               _plugins.push_back (plug);
-
+               add_plugin (plug);
                create_automatable_parameters ();
-
-               Glib::Mutex::Lock em (_session.engine().process_lock());
-               IO::PortCountChanged (max(input_streams(), output_streams()));
        }
 }
 
@@ -95,7 +91,11 @@ PluginInsert::set_count (uint32_t num)
                uint32_t diff = num - _plugins.size();
 
                for (uint32_t n = 0; n < diff; ++n) {
-                       add_plugin_with_activation (plugin_factory (_plugins[0]));
+                       boost::shared_ptr<Plugin> p = plugin_factory (_plugins[0]);
+                       add_plugin (p);
+                       if (active ()) {
+                               p->activate ();
+                       }
 
                        if (require_state) {
                                /* XXX do something */
@@ -133,13 +133,19 @@ PluginInsert::control_list_automation_state_changed (Evoral::Parameter which, Au
 ChanCount
 PluginInsert::output_streams() const
 {
-       ChanCount out = _plugins.front()->get_info()->n_outputs;
+       assert (!_plugins.empty());
 
-       if (out == ChanCount::INFINITE) {
-               return _plugins.front()->output_streams ();
+       PluginInfoPtr info = _plugins.front()->get_info();
+
+       if (info->reconfigurable_io()) {
+               ChanCount out = _plugins.front()->output_streams ();
+               // DEBUG_TRACE (DEBUG::Processors, string_compose ("Plugin insert, reconfigur(able) output streams = %1\n", out));
+               return out;
        } else {
+               ChanCount out = info->n_outputs;
+               // DEBUG_TRACE (DEBUG::Processors, string_compose ("Plugin insert, static output streams = %1 for %2 plugins\n", out, _plugins.size()));
                out.set_audio (out.n_audio() * _plugins.size());
-               out.set_midi (out.n_midi() * _plugins.size());
+               out.set_midi (out.n_midi() * _plugins.size() + midi_bypass.n_midi());
                return out;
        }
 }
@@ -147,9 +153,22 @@ PluginInsert::output_streams() const
 ChanCount
 PluginInsert::input_streams() const
 {
-       ChanCount in = _plugins[0]->get_info()->n_inputs;
+       assert (!_plugins.empty());
+
+       ChanCount in;
+
+       PluginInfoPtr info = _plugins.front()->get_info();
 
-       if (_splitting) {
+       if (info->reconfigurable_io()) {
+               assert (_plugins.size() == 1);
+               in = _plugins.front()->input_streams();
+       } else {
+               in = info->n_inputs;
+       }
+
+       DEBUG_TRACE (DEBUG::Processors, string_compose ("Plugin insert, input streams = %1, match using %2\n", in, _match.method));
+       
+       if (_match.method == Split) {
 
                /* we are splitting 1 processor input to multiple plugin inputs,
                   so we have a maximum of 1 stream of each type.
@@ -161,11 +180,19 @@ PluginInsert::input_streams() const
                }
                return in;
 
-       } else if (in == ChanCount::INFINITE) {
-               return _plugins[0]->input_streams ();
+       } else if (_match.method == Hide) {
+
+               for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+                       in.set (*t, in.get (*t) - _match.hide.get (*t));
+               }
+               return in;
+
        } else {
-               in.set_audio (in.n_audio() * _plugins.size());
-               in.set_midi (in.n_midi() * _plugins.size());
+               
+               for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+                       in.set (*t, in.get (*t) * _plugins.size ());
+               }
+
                return in;
        }
 }
@@ -183,13 +210,27 @@ PluginInsert::natural_input_streams() const
 }
 
 bool
-PluginInsert::is_generator() const
+PluginInsert::has_no_inputs() const
+{
+       return _plugins[0]->get_info()->n_inputs == ChanCount::ZERO;
+}
+
+bool
+PluginInsert::has_no_audio_inputs() const
+{
+       return _plugins[0]->get_info()->n_inputs.n_audio() == 0;
+}
+
+bool
+PluginInsert::is_midi_instrument() const
 {
        /* XXX more finesse is possible here. VST plugins have a
           a specific "instrument" flag, for example.
         */
+       PluginInfoPtr pi = _plugins[0]->get_info();
 
-       return _plugins[0]->get_info()->n_inputs.n_audio() == 0;
+       return pi->n_inputs.n_midi() != 0 &&
+               pi->n_outputs.n_audio() > 0;
 }
 
 void
@@ -221,21 +262,24 @@ PluginInsert::create_automatable_parameters ()
 }
 
 void
-PluginInsert::parameter_changed (Evoral::Parameter which, float val)
+PluginInsert::parameter_changed (uint32_t which, float val)
 {
-       if (which.type() != PluginAutomation)
-               return;
-
-       Plugins::iterator i = _plugins.begin();
-
-       /* don't set the first plugin, just all the slaves */
+       boost::shared_ptr<AutomationControl> ac = automation_control (Evoral::Parameter (PluginAutomation, 0, which));
 
-       if (i != _plugins.end()) {
-               ++i;
-               for (; i != _plugins.end(); ++i) {
-                       (*i)->set_parameter (which, val);
-               }
-       }
+       if (ac) {
+               ac->set_value (val);
+                
+                Plugins::iterator i = _plugins.begin();
+                
+                /* don't set the first plugin, just all the slaves */
+                
+                if (i != _plugins.end()) {
+                        ++i;
+                        for (; i != _plugins.end(); ++i) {
+                                (*i)->set_parameter (which, val);
+                        }
+                }
+        }
 }
 
 int
@@ -288,18 +332,23 @@ PluginInsert::connect_and_run (BufferSet& bufs, pframes_t nframes, framecnt_t of
                collect_signal_nframes = nframes;
        }
 
-       ChanMapping in_map(input_streams());
-       ChanMapping out_map(output_streams());
+       ChanCount const in_streams = input_streams ();
+       ChanCount const out_streams = output_streams ();
 
-       if (_splitting) {
+       ChanMapping in_map (in_streams);
+       ChanMapping out_map (out_streams);
+       bool valid;
+       if (_match.method == Split) {
                /* fix the input mapping so that we have maps for each of the plugin's inputs */
                in_map = ChanMapping (natural_input_streams ());
 
                /* copy the first stream's buffer contents to the others */
                /* XXX: audio only */
-               Sample const * mono = bufs.get_audio (in_map.get (DataType::AUDIO, 0)).data (offset);
-               for (uint32_t i = input_streams().n_audio(); i < natural_input_streams().n_audio(); ++i) {
-                       memcpy (bufs.get_audio (in_map.get (DataType::AUDIO, i)).data() + offset, mono + offset, sizeof (Sample) * (nframes - offset));
+               uint32_t first_idx = in_map.get (DataType::AUDIO, 0, &valid);
+               if (valid) {
+                       for (uint32_t i = in_streams.n_audio(); i < natural_input_streams().n_audio(); ++i) {
+                               bufs.get_audio(in_map.get (DataType::AUDIO, i, &valid)).read_from(bufs.get_audio(first_idx), nframes, offset, offset);
+                       }
                }
        }
 
@@ -392,13 +441,13 @@ PluginInsert::silence (framecnt_t nframes)
        ChanMapping in_map(input_streams());
        ChanMapping out_map(output_streams());
 
-       if (_splitting) {
+       if (_match.method == Split) {
                /* fix the input mapping so that we have maps for each of the plugin's inputs */
                in_map = ChanMapping (natural_input_streams ());
        }
 
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-               (*i)->connect_and_run (_session.get_silent_buffers ((*i)->get_info()->n_inputs), in_map, out_map, nframes, 0);
+               (*i)->connect_and_run (_session.get_scratch_buffers ((*i)->get_info()->n_inputs, true), in_map, out_map, nframes, 0);
        }
 }
 
@@ -415,42 +464,39 @@ PluginInsert::run (BufferSet& bufs, framepos_t /*start_frame*/, framepos_t /*end
                }
 
        } else {
+               uint32_t in = input_streams ().n_audio ();
+               uint32_t out = output_streams().n_audio ();
 
-               if (is_generator()) {
+               if (has_no_audio_inputs() || in == 0) {
 
                        /* silence all (audio) outputs. Should really declick
                         * at the transitions of "active"
                         */
 
-                       uint32_t out = _plugins[0]->get_info()->n_outputs.n_audio();
-
                        for (uint32_t n = 0; n < out; ++n) {
                                bufs.get_audio (n).silence (nframes);
                        }
 
-                       bufs.count().set_audio (out);
+               } else if (out > in) {
 
-               } else {
-
-                       /* does this need to be done with MIDI? it appears not */
-
-                       uint32_t in = _plugins[0]->get_info()->n_inputs.n_audio();
-                       uint32_t out = _plugins[0]->get_info()->n_outputs.n_audio();
+                       /* not active, but something has make up for any channel count increase */
 
-                       if (out > in) {
-
-                               /* not active, but something has make up for any channel count increase */
-
-                               for (uint32_t n = out - in; n < out; ++n) {
-                                       memcpy (bufs.get_audio(n).data(), bufs.get_audio(in - 1).data(), sizeof (Sample) * nframes);
-                               }
+                       // TODO: option round-robin (n % in) or silence additional buffers ??
+                       // for now , simply replicate last buffer
+                       for (uint32_t n = in; n < out; ++n) {
+                               bufs.get_audio(n).read_from(bufs.get_audio(in - 1), nframes);
                        }
-
-                       bufs.count().set_audio (out);
                }
+
+               bufs.count().set_audio (out);
        }
 
        _active = _pending_active;
+
+       /* we have no idea whether the plugin generated silence or not, so mark
+        * all buffers appropriately.
+        */
+
 }
 
 void
@@ -496,7 +542,7 @@ PluginInsert::automation_run (BufferSet& bufs, pframes_t nframes)
        framepos_t end = now + nframes;
        framecnt_t offset = 0;
 
-       Glib::Mutex::Lock lm (control_lock(), Glib::TRY_LOCK);
+       Glib::Threads::Mutex::Lock lm (control_lock(), Glib::Threads::TRY_LOCK);
 
        if (!lm.locked()) {
                connect_and_run (bufs, nframes, offset, false);
@@ -555,10 +601,13 @@ PluginInsert::plugin_factory (boost::shared_ptr<Plugin> other)
 #ifdef LV2_SUPPORT
        boost::shared_ptr<LV2Plugin> lv2p;
 #endif
-#ifdef VST_SUPPORT
-       boost::shared_ptr<VSTPlugin> vp;
+#ifdef WINDOWS_VST_SUPPORT
+       boost::shared_ptr<WindowsVSTPlugin> vp;
 #endif
-#ifdef HAVE_AUDIOUNITS
+#ifdef LXVST_SUPPORT
+       boost::shared_ptr<LXVSTPlugin> lxvp;
+#endif
+#ifdef AUDIOUNIT_SUPPORT
        boost::shared_ptr<AUPlugin> ap;
 #endif
 
@@ -568,11 +617,15 @@ PluginInsert::plugin_factory (boost::shared_ptr<Plugin> other)
        } else if ((lv2p = boost::dynamic_pointer_cast<LV2Plugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new LV2Plugin (*lv2p));
 #endif
-#ifdef VST_SUPPORT
-       } else if ((vp = boost::dynamic_pointer_cast<VSTPlugin> (other)) != 0) {
-               return boost::shared_ptr<Plugin> (new VSTPlugin (*vp));
+#ifdef WINDOWS_VST_SUPPORT
+       } else if ((vp = boost::dynamic_pointer_cast<WindowsVSTPlugin> (other)) != 0) {
+               return boost::shared_ptr<Plugin> (new WindowsVSTPlugin (*vp));
+#endif
+#ifdef LXVST_SUPPORT
+       } else if ((lxvp = boost::dynamic_pointer_cast<LXVSTPlugin> (other)) != 0) {
+               return boost::shared_ptr<Plugin> (new LXVSTPlugin (*lxvp));
 #endif
-#ifdef HAVE_AUDIOUNITS
+#ifdef AUDIOUNIT_SUPPORT
        } else if ((ap = boost::dynamic_pointer_cast<AUPlugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new AUPlugin (*ap));
 #endif
@@ -588,22 +641,38 @@ PluginInsert::plugin_factory (boost::shared_ptr<Plugin> other)
 bool
 PluginInsert::configure_io (ChanCount in, ChanCount out)
 {
-       if (set_count (count_for_configuration (in, out)) == false) {
-               set_splitting (false);
+       Match old_match = _match;
+       ChanCount old_in = input_streams ();
+       ChanCount old_out = output_streams ();
+
+       /* set the matching method and number of plugins that we will use to meet this configuration */
+       _match = private_can_support_io_configuration (in, out);
+       if (set_count (_match.plugins) == false) {
                return false;
        }
 
-       if (_plugins.front()->get_info()->n_inputs <= in) {
-               set_splitting (false);
+       if (  (old_match.method != _match.method && (old_match.method == Split || _match.method == Split))
+                       || old_in != in
+                       || old_out != out
+                       )
+       {
+               PluginIoReConfigure (); /* EMIT SIGNAL */
+       }
+
+       /* configure plugins */
+       switch (_match.method) {
+       case Split:
+       case Hide:
+               if (_plugins.front()->configure_io (_plugins.front()->get_info()->n_inputs, out)) {
+                       return false;
+               }
+               break;
+
+       default:
                if (_plugins.front()->configure_io (in, out) == false) {
                        return false;
                }
-       } else {
-               /* we must be splitting a single processor input to
-                  multiple plugin inputs
-               */
-               set_splitting (true);
-               _plugins.front()->configure_io (_plugins.front()->get_info()->n_inputs, out);
+               break;
        }
 
        // we don't know the analysis window size, so we must work with the
@@ -621,16 +690,56 @@ PluginInsert::configure_io (ChanCount in, ChanCount out)
        return Processor::configure_io (in, out);
 }
 
+/** Decide whether this PluginInsert can support a given IO configuration.
+ *  To do this, we run through a set of possible solutions in rough order of
+ *  preference.
+ *
+ *  @param in Required input channel count.
+ *  @param out Filled in with the output channel count if we return true.
+ *  @return true if the given IO configuration can be supported.
+ */
 bool
-PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out) const
+PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
 {
-       // Plugin has flexible I/O, so delegate to it
-       if (_plugins.front()->reconfigurable_io()) {
-               return _plugins.front()->can_support_io_configuration (in, out);
+       return private_can_support_io_configuration (in, out).method != Impossible;
+}
+
+/** A private version of can_support_io_configuration which returns the method
+ *  by which the configuration can be matched, rather than just whether or not
+ *  it can be.
+ */
+PluginInsert::Match
+PluginInsert::private_can_support_io_configuration (ChanCount const & inx, ChanCount& out)
+{
+       if (_plugins.empty()) {
+               return Match();
        }
 
-       ChanCount inputs  = _plugins[0]->get_info()->n_inputs;
-       ChanCount outputs = _plugins[0]->get_info()->n_outputs;
+       PluginInfoPtr info = _plugins.front()->get_info();
+       ChanCount in; in += inx;
+       midi_bypass.reset();
+
+       if (info->reconfigurable_io()) {
+               /* Plugin has flexible I/O, so delegate to it */
+               bool const r = _plugins.front()->can_support_io_configuration (in, out);
+               if (!r) {
+                       return Match (Impossible, 0);
+               }
+
+               return Match (Delegate, 1);
+       }
+
+       ChanCount inputs  = info->n_inputs;
+       ChanCount outputs = info->n_outputs;
+
+       if (in.get(DataType::MIDI) == 1 && outputs.get(DataType::MIDI) == 0) {
+               DEBUG_TRACE ( DEBUG::Processors, string_compose ("bypassing midi-data around %1\n", name()));
+               midi_bypass.set(DataType::MIDI, 1);
+       }
+       if (in.get(DataType::MIDI) == 1 && inputs.get(DataType::MIDI) == 0) {
+               DEBUG_TRACE ( DEBUG::Processors, string_compose ("hiding midi-port from plugin %1\n", name()));
+               in.set(DataType::MIDI, 0);
+       }
 
        bool no_inputs = true;
        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
@@ -642,19 +751,22 @@ PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
 
        if (no_inputs) {
                /* no inputs so we can take any input configuration since we throw it away */
-               out = outputs;
-               return true;
+               out = outputs + midi_bypass;
+               return Match (NoInputs, 1);
        }
 
-       // Plugin inputs match requested inputs exactly
+       /* Plugin inputs match requested inputs exactly */
        if (inputs == in) {
-               out = outputs;
-               return true;
+               out = outputs + midi_bypass;
+               return Match (ExactMatch, 1);
        }
 
-       // See if replication is possible
-       // We allow replication only for plugins with either zero or 1 inputs and outputs
-       // for every valid data type.
+       /* We may be able to run more than one copy of the plugin within this insert
+          to cope with the insert having more inputs than the plugin.
+          We allow replication only for plugins with either zero or 1 inputs and outputs
+          for every valid data type.
+       */
+       
        uint32_t f             = 0;
        bool     can_replicate = true;
        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
@@ -672,7 +784,6 @@ PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
                }
 
                // Potential factor not set yet
-
                if (f == 0) {
                        f = in.get(*t) / nin;
                }
@@ -688,7 +799,8 @@ PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
                for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
                        out.set (*t, outputs.get(*t) * f);
                }
-               return true;
+               out += midi_bypass;
+               return Match (Replicate, f);
        }
 
        /* If the processor has exactly one input of a given type, and
@@ -711,65 +823,40 @@ PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
        }
 
        if (can_split) {
-               out = outputs;
-               return true;
+               out = outputs + midi_bypass;
+               return Match (Split, 1);
        }
 
-       return false;
-}
-
-/* Number of plugin instances required to support a given channel configuration.
- * (private helper)
- */
-int32_t
-PluginInsert::count_for_configuration (ChanCount in, ChanCount /*out*/) const
-{
-       if (_plugins.front()->reconfigurable_io()) {
-               /* plugin has flexible I/O, so the answer is always 1 */
-               /* this could change if we ever decide to replicate AU's */
-               return 1;
-       }
-
-       // FIXME: take 'out' into consideration
-
-       ChanCount outputs = _plugins[0]->get_info()->n_outputs;
-       ChanCount inputs = _plugins[0]->get_info()->n_inputs;
-
-       if (inputs.n_total() == 0) {
-               /* instrument plugin, always legal, but throws away any existing streams */
-               return 1;
-       }
-
-       if (inputs.n_total() == 1 && outputs == inputs
-                       && ((inputs.n_audio() == 0 && in.n_audio() == 0)
-                               || (inputs.n_midi() == 0 && in.n_midi() == 0))) {
-               /* mono plugin, replicate as needed to match in */
-               return in.n_total();
-       }
-
-       if (inputs == in) {
-               /* exact match */
-               return 1;
-       }
+       /* If the plugin has more inputs than we want, we can `hide' some of them
+          by feeding them silence.
+       */
 
-       if (inputs > in) {
-               /* more plugin inputs than processor inputs, so we are splitting */
-               return 1;
+       bool could_hide = false;
+       bool cannot_hide = false;
+       ChanCount hide_channels;
+       
+       for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+               if (inputs.get(*t) > in.get(*t)) {
+                       /* there is potential to hide, since the plugin has more inputs of type t than the insert */
+                       hide_channels.set (*t, inputs.get(*t) - in.get(*t));
+                       could_hide = true;
+               } else if (inputs.get(*t) < in.get(*t)) {
+                       /* we definitely cannot hide, since the plugin has fewer inputs of type t than the insert */
+                       cannot_hide = true;
+               }
        }
 
-       // assumes in is valid, so we must be replicating
-       if (inputs.n_total() < in.n_total()
-                       && (in.n_total() % inputs.n_total() == 0)) {
-
-               return in.n_total() / inputs.n_total();
+       if (could_hide && !cannot_hide) {
+               out = outputs + midi_bypass;
+               return Match (Hide, 1, hide_channels);
        }
 
-       /* err... */
-       return 0;
+       midi_bypass.reset();
+       return Match (Impossible, 0);
 }
 
 XMLNode&
-PluginInsert::get_state(void)
+PluginInsert::get_state ()
 {
        return state (true);
 }
@@ -807,7 +894,12 @@ PluginInsert::set_control_ids (const XMLNode& node, int version)
 
                        if ((prop = (*iter)->property (X_("parameter"))) != 0) {
                                uint32_t p = atoi (prop->value());
+
+                               /* this may create the new controllable */
+
                                boost::shared_ptr<Evoral::Control> c = control (Evoral::Parameter (PluginAutomation, 0, p));
+
+#ifndef NO_PLUGIN_STATE
                                if (!c) {
                                        continue;
                                }
@@ -815,6 +907,7 @@ PluginInsert::set_control_ids (const XMLNode& node, int version)
                                if (ac) {
                                        ac->set_state (**iter, version);
                                }
+#endif
                        }
                }
        }
@@ -838,8 +931,10 @@ PluginInsert::set_state(const XMLNode& node, int version)
                type = ARDOUR::LADSPA;
        } else if (prop->value() == X_("lv2")) {
                type = ARDOUR::LV2;
-       } else if (prop->value() == X_("vst")) {
-               type = ARDOUR::VST;
+       } else if (prop->value() == X_("windows-vst")) {
+               type = ARDOUR::Windows_VST;
+       } else if (prop->value() == X_("lxvst")) {
+               type = ARDOUR::LXVST;
        } else if (prop->value() == X_("audiounit")) {
                type = ARDOUR::AudioUnit;
        } else {
@@ -852,11 +947,19 @@ PluginInsert::set_state(const XMLNode& node, int version)
        prop = node.property ("unique-id");
 
        if (prop == 0) {
-#ifdef VST_SUPPORT
+#ifdef WINDOWS_VST_SUPPORT
                /* older sessions contain VST plugins with only an "id" field.
                 */
 
-               if (type == ARDOUR::VST) {
+               if (type == ARDOUR::Windows_VST) {
+                       prop = node.property ("id");
+               }
+#endif
+
+#ifdef LXVST_SUPPORT
+               /*There shouldn't be any older sessions with linuxVST support.. but anyway..*/
+
+               if (type == ARDOUR::LXVST) {
                        prop = node.property ("id");
                }
 #endif
@@ -884,6 +987,13 @@ PluginInsert::set_state(const XMLNode& node, int version)
 
        uint32_t count = 1;
 
+       // Processor::set_state() will set this, but too late
+       // for it to be available when setting up plugin
+       // state. We can't call Processor::set_state() until
+       // the plugins themselves are created and added.
+
+       set_id (node);
+
        if (_plugins.empty()) {
                /* if we are adding the first plugin, we will need to set
                   up automatable controls.
@@ -903,6 +1013,8 @@ PluginInsert::set_state(const XMLNode& node, int version)
                }
        }
 
+       Processor::set_state (node, version);
+
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
 
                /* find the node with the type-specific node name ("lv2", "ladspa", etc)
@@ -921,8 +1033,6 @@ PluginInsert::set_state(const XMLNode& node, int version)
                }
        }
 
-       Processor::set_state (node, version);
-
        if (version < 3000) {
 
                /* Only 2.X sessions need a call to set_parameter_state() - in 3.X and above
@@ -948,13 +1058,6 @@ PluginInsert::set_state(const XMLNode& node, int version)
                }
        }
 
-       /* catch up on I/O */
-
-       {
-               Glib::Mutex::Lock em (_session.engine().process_lock());
-               IO::PortCountChanged (max(input_streams(), output_streams()));
-       }
-
        return 0;
 }
 
@@ -1073,10 +1176,18 @@ PluginInsert::PluginControl::PluginControl (PluginInsert* p, const Evoral::Param
        , _plugin (p)
 {
        Plugin::ParameterDescriptor desc;
-       p->plugin(0)->get_parameter_descriptor (param.id(), desc);
+       boost::shared_ptr<Plugin> plugin = p->plugin (0);
+       
+       alist()->reset_default (plugin->default_value (param.id()));
+
+       plugin->get_parameter_descriptor (param.id(), desc);
        _logarithmic = desc.logarithmic;
        _sr_dependent = desc.sr_dependent;
        _toggled = desc.toggled;
+
+       if (desc.toggled) {
+               set_flags(Controllable::Toggle);
+       }
 }
 
 /** @param val `user' value */
@@ -1085,35 +1196,39 @@ PluginInsert::PluginControl::set_value (double user_val)
 {
        /* FIXME: probably should be taking out some lock here.. */
 
-       double const plugin_val = user_to_plugin (user_val);
-
        for (Plugins::iterator i = _plugin->_plugins.begin(); i != _plugin->_plugins.end(); ++i) {
-               (*i)->set_parameter (_list->parameter().id(), plugin_val);
+               (*i)->set_parameter (_list->parameter().id(), user_val);
        }
 
        boost::shared_ptr<Plugin> iasp = _plugin->_impulseAnalysisPlugin.lock();
        if (iasp) {
-               iasp->set_parameter (_list->parameter().id(), plugin_val);
+               iasp->set_parameter (_list->parameter().id(), user_val);
        }
 
        AutomationControl::set_value (user_val);
 }
 
 double
-PluginInsert::PluginControl::user_to_plugin (double val) const
-{
-       /* no known transformations at this time */
-       return val;
-}
-
-double
-PluginInsert::PluginControl::user_to_ui (double val) const
+PluginInsert::PluginControl::internal_to_interface (double val) const
 {
        if (_logarithmic) {
+               /* some plugins have a log-scale range "0.."
+                * ideally we'd map the range down to infinity somehow :)
+                *
+                * one solution could be to use
+                *   val = exp(lower + log(range) * value);
+                *   (log(val) - lower) / range)
+                * This approach would require access to the actual range (ie
+                * Plugin::ParameterDescriptor) and also require handling
+                * of unbound ranges..
+                *
+                * currently an arbitrarly low number is assumed to represnt
+                * log(0) as hot-fix solution.
+                */
                if (val > 0) {
                        val = log (val);
                } else {
-                       val = 0;
+                       val = -8; // ~ -70dB = 20 * log10(exp(-8))
                }
        }
 
@@ -1121,29 +1236,20 @@ PluginInsert::PluginControl::user_to_ui (double val) const
 }
 
 double
-PluginInsert::PluginControl::ui_to_user (double val) const
+PluginInsert::PluginControl::interface_to_internal (double val) const
 {
        if (_logarithmic) {
-               val = exp (val);
+               if (val <= -8) {
+                       /* see note in PluginInsert::PluginControl::internal_to_interface() */
+                       val= 0;
+               } else {
+                       val = exp (val);
+               }
        }
 
        return val;
 }
 
-/** Convert plugin values to UI values.  See pbd/controllable.h */
-double
-PluginInsert::PluginControl::plugin_to_ui (double val) const
-{
-       return user_to_ui (plugin_to_user (val));
-}
-
-double
-PluginInsert::PluginControl::plugin_to_user (double val) const
-{
-       /* no known transformations at this time */
-       return val;
-}
-
 XMLNode&
 PluginInsert::PluginControl::get_state ()
 {
@@ -1161,8 +1267,7 @@ double
 PluginInsert::PluginControl::get_value () const
 {
        /* FIXME: probably should be taking out some lock here.. */
-
-       return plugin_to_user (_plugin->get_parameter (_list->parameter()));
+       return _plugin->get_parameter (_list->parameter());
 }
 
 boost::shared_ptr<Plugin>
@@ -1191,22 +1296,21 @@ PluginInsert::collect_signal_for_analysis (framecnt_t nframes)
        _signal_analysis_collect_nframes_max = nframes;
 }
 
-/** Add a plugin to our list and activate it if we have already been activated */
-void
-PluginInsert::add_plugin_with_activation (boost::shared_ptr<Plugin> plugin)
-{
-       plugin->set_insert_info (this);
-       _plugins.push_back (plugin);
-       if (active()) {
-               plugin->activate ();
-       }
-}
-
 /** Add a plugin to our list */
 void
 PluginInsert::add_plugin (boost::shared_ptr<Plugin> plugin)
 {
        plugin->set_insert_info (this);
+       
+       if (_plugins.empty()) {
+                /* first (and probably only) plugin instance - connect to relevant signals 
+                 */
+
+               plugin->ParameterChanged.connect_same_thread (*this, boost::bind (&PluginInsert::parameter_changed, this, _1, _2));
+                plugin->StartTouch.connect_same_thread (*this, boost::bind (&PluginInsert::start_touch, this, _1));
+                plugin->EndTouch.connect_same_thread (*this, boost::bind (&PluginInsert::end_touch, this, _1));
+       }
+
        _plugins.push_back (plugin);
 }
 
@@ -1219,12 +1323,35 @@ PluginInsert::realtime_handle_transport_stopped ()
 }
 
 void
-PluginInsert::set_splitting (bool s)
+PluginInsert::realtime_locate ()
 {
-       if (_splitting == s) {
-               return;
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->realtime_locate ();
        }
+}
 
-       _splitting = s;
-       SplittingChanged (); /* EMIT SIGNAL */
+void
+PluginInsert::monitoring_changed ()
+{
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->monitoring_changed ();
+       }
+}
+
+void
+PluginInsert::start_touch (uint32_t param_id)
+{
+        boost::shared_ptr<AutomationControl> ac = automation_control (Evoral::Parameter (PluginAutomation, 0, param_id));
+        if (ac) {
+                ac->start_touch (session().audible_frame());
+        }
+}
+
+void
+PluginInsert::end_touch (uint32_t param_id)
+{
+        boost::shared_ptr<AutomationControl> ac = automation_control (Evoral::Parameter (PluginAutomation, 0, param_id));
+        if (ac) {
+                ac->stop_touch (true, session().audible_frame());
+        }
 }