changes for OS X support: change waf config define to COREAUDIO_SUPPORT, remove Plugi...
[ardour.git] / libs / ardour / plugin_insert.cc
index 41e1376b9a33a267944dd421f25d271a07f0f618..78f65963302cd77ac90a0d756075385d95560343 100644 (file)
@@ -23,7 +23,6 @@
 
 #include <string>
 
-
 #include "pbd/failed_constructor.h"
 #include "pbd/xml++.h"
 #include "pbd/convert.h"
@@ -31,6 +30,7 @@
 #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"
@@ -38,7 +38,7 @@
 #include "ardour/port.h"
 #include "ardour/route.h"
 
-#ifdef HAVE_SLV2
+#ifdef LV2_SUPPORT
 #include "ardour/lv2_plugin.h"
 #endif
 
 #include "ardour/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
 
@@ -66,16 +70,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) {
-                _plugins.push_back (plug);
-                set_automatable ();
-
-               Glib::Mutex::Lock em (_session.engine().process_lock());
-               IO::PortCountChanged (max(input_streams(), output_streams()));
+       if (plug) {
+               add_plugin (plug);
+               create_automatable_parameters ();
        }
 }
 
@@ -134,6 +134,8 @@ PluginInsert::output_streams() const
 {
        ChanCount out = _plugins.front()->get_info()->n_outputs;
 
+       DEBUG_TRACE (DEBUG::Processors, string_compose ("Plugin insert, static output streams = %1\n", out));
+
        if (out == ChanCount::INFINITE) {
                return _plugins.front()->output_streams ();
        } else {
@@ -148,7 +150,9 @@ PluginInsert::input_streams() const
 {
        ChanCount in = _plugins[0]->get_info()->n_inputs;
 
-       if (_splitting) {
+       DEBUG_TRACE (DEBUG::Processors, string_compose ("Plugin insert, static 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.
@@ -160,12 +164,24 @@ PluginInsert::input_streams() const
                }
                return in;
 
+       } 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 if (in == ChanCount::INFINITE) {
+               
                return _plugins[0]->input_streams ();
+
        } 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;
+               
        }
 }
 
@@ -182,18 +198,34 @@ 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
-PluginInsert::set_automatable ()
+PluginInsert::create_automatable_parameters ()
 {
+       assert (!_plugins.empty());
+
        set<Evoral::Parameter> a = _plugins.front()->automatable ();
 
        Plugin::ParameterDescriptor desc;
@@ -204,7 +236,7 @@ PluginInsert::set_automatable ()
                        Evoral::Parameter param(*i);
 
                        _plugins.front()->get_parameter_descriptor(i->id(), desc);
-                        
+
                        /* the Parameter belonging to the actual plugin doesn't have its range set
                           but we want the Controllable related to this Parameter to have those limits.
                        */
@@ -212,7 +244,7 @@ PluginInsert::set_automatable ()
                        param.set_range (desc.lower, desc.upper, _plugins.front()->default_value(i->id()), desc.toggled);
                        can_automate (param);
                        boost::shared_ptr<AutomationList> list(new AutomationList(param));
-                        add_control (boost::shared_ptr<AutomationControl> (new PluginControl(this, param, list)));
+                       add_control (boost::shared_ptr<AutomationControl> (new PluginControl(this, param, list)));
                }
        }
 }
@@ -238,13 +270,13 @@ PluginInsert::parameter_changed (Evoral::Parameter which, float val)
 int
 PluginInsert::set_block_size (pframes_t nframes)
 {
-        int ret = 0;
+       int ret = 0;
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                if ((*i)->set_block_size (nframes) != 0) {
-                        ret = -1;
-                }
+                       ret = -1;
+               }
        }
-        return ret;
+       return ret;
 }
 
 void
@@ -253,7 +285,7 @@ PluginInsert::activate ()
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                (*i)->activate ();
        }
-       
+
        Processor::activate ();
 }
 
@@ -270,9 +302,9 @@ PluginInsert::deactivate ()
 void
 PluginInsert::flush ()
 {
-        for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-                (*i)->flush ();
-        }
+       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->flush ();
+       }
 }
 
 void
@@ -285,18 +317,30 @@ 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 ();
+
+       ChanMapping in_map (in_streams);
+       ChanMapping out_map (out_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 ());
 
                /* 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));
+               for (uint32_t i = in_streams.n_audio(); i < natural_input_streams().n_audio(); ++i) {
+                       memcpy (bufs.get_audio (in_map.get (DataType::AUDIO, i)).data (offset), mono, sizeof (Sample) * nframes);
+               }
+       }
+
+       if (_match.method == Hide) {
+               /* Silence the hidden input buffers */
+               for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+                       for (uint32_t i = in_streams.get(*t); i < (in_streams.get(*t) + _match.hide.get(*t)); ++i) {
+                               bufs.get(*t, i).silence (nframes);
+                       }
                }
        }
 
@@ -385,15 +429,15 @@ PluginInsert::silence (framecnt_t nframes)
        if (!active ()) {
                return;
        }
-       
+
        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);
        }
@@ -413,21 +457,38 @@ PluginInsert::run (BufferSet& bufs, framepos_t /*start_frame*/, framepos_t /*end
 
        } else {
 
-               /* FIXME: type, audio only */
+               if (has_no_audio_inputs()) {
 
-               uint32_t in = _plugins[0]->get_info()->n_inputs.n_audio();
-               uint32_t out = _plugins[0]->get_info()->n_outputs.n_audio();
+                       /* silence all (audio) outputs. Should really declick
+                        * at the transitions of "active"
+                        */
 
-               if (out > in) {
+                       uint32_t out = _plugins[0]->get_info()->n_outputs.n_audio();
 
-                       /* not active, but something has make up for any channel count increase */
+                       for (uint32_t n = 0; n < out; ++n) {
+                               bufs.get_audio (n).silence (nframes);
+                       }
+
+                       bufs.count().set_audio (out);
+
+               } 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();
 
-                       for (uint32_t n = out - in; n < out; ++n) {
-                               memcpy (bufs.get_audio(n).data(), bufs.get_audio(in - 1).data(), sizeof (Sample) * nframes);
+                       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);
+                               }
                        }
-               }
 
-               bufs.count().set_audio(out);
+                       bufs.count().set_audio (out);
+               }
        }
 
        _active = _pending_active;
@@ -438,7 +499,7 @@ PluginInsert::set_parameter (Evoral::Parameter param, float val)
 {
        if (param.type() != PluginAutomation) {
                return;
-        }
+       }
 
        /* the others will be set from the event triggered by this */
 
@@ -532,19 +593,22 @@ boost::shared_ptr<Plugin>
 PluginInsert::plugin_factory (boost::shared_ptr<Plugin> other)
 {
        boost::shared_ptr<LadspaPlugin> lp;
-#ifdef HAVE_SLV2
+#ifdef LV2_SUPPORT
        boost::shared_ptr<LV2Plugin> lv2p;
 #endif
 #ifdef VST_SUPPORT
        boost::shared_ptr<VSTPlugin> vp;
 #endif
-#ifdef HAVE_AUDIOUNITS
+#ifdef LXVST_SUPPORT
+       boost::shared_ptr<LXVSTPlugin> lxvp;
+#endif
+#ifdef AUDIOUNIT_SUPPORT
        boost::shared_ptr<AUPlugin> ap;
 #endif
 
        if ((lp = boost::dynamic_pointer_cast<LadspaPlugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new LadspaPlugin (*lp));
-#ifdef HAVE_SLV2
+#ifdef LV2_SUPPORT
        } else if ((lv2p = boost::dynamic_pointer_cast<LV2Plugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new LV2Plugin (*lv2p));
 #endif
@@ -552,7 +616,11 @@ PluginInsert::plugin_factory (boost::shared_ptr<Plugin> other)
        } else if ((vp = boost::dynamic_pointer_cast<VSTPlugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new VSTPlugin (*vp));
 #endif
-#ifdef HAVE_AUDIOUNITS
+#ifdef LXVST_SUPPORT
+       } else if ((lxvp = boost::dynamic_pointer_cast<LXVSTPlugin> (other)) != 0) {
+               return boost::shared_ptr<Plugin> (new LXVSTPlugin (*lxvp));
+#endif
+#ifdef AUDIOUNIT_SUPPORT
        } else if ((ap = boost::dynamic_pointer_cast<AUPlugin> (other)) != 0) {
                return boost::shared_ptr<Plugin> (new AUPlugin (*ap));
 #endif
@@ -568,22 +636,33 @@ 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;
+
+       /* 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);
+       /* a signal needs emitting if we start or stop splitting */
+       if (old_match.method != _match.method && (old_match.method == Split || _match.method == Split)) {
+               SplittingChanged (); /* 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
@@ -601,74 +680,99 @@ 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
 {
-       // Plugin has flexible I/O, so delegate to it
+       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 & in, ChanCount& out) const
+{
        if (_plugins.front()->reconfigurable_io()) {
-               return _plugins.front()->can_support_io_configuration (in, out);
+               /* 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  = _plugins[0]->get_info()->n_inputs;
        ChanCount outputs = _plugins[0]->get_info()->n_outputs;
 
-        bool no_inputs = true;
+       bool no_inputs = true;
        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
-                if (inputs.get (*t) != 0) {
-                        no_inputs = false;
-                        break;
-                }
-        }
-
-        if (no_inputs) {
-                /* no inputs so we can take any input configuration since we throw it away */
-                out = outputs;
-                return true;
-        }
-
-       // Plugin inputs match requested inputs exactly
+               if (inputs.get (*t) != 0) {
+                       no_inputs = false;
+                       break;
+               }
+       }
+
+       if (no_inputs) {
+               /* no inputs so we can take any input configuration since we throw it away */
+               out = outputs;
+               return Match (NoInputs, 1);
+       }
+
+       /* Plugin inputs match requested inputs exactly */
        if (inputs == in) {
                out = outputs;
-               return true;
+               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) {
 
-                uint32_t nin = inputs.get (*t);
+               uint32_t nin = inputs.get (*t);
 
                // No inputs of this type
                if (nin == 0 && in.get(*t) == 0) {
                        continue;
-                }
-
-                if (nin != 1 || outputs.get (*t) != 1) {
-                        can_replicate = false;
-                        break;
-                }
+               }
 
-                // Potential factor not set yet
+               if (nin != 1 || outputs.get (*t) != 1) {
+                       can_replicate = false;
+                       break;
+               }
 
-                if (f == 0) {
-                        f = in.get(*t) / nin;
-                }
+               // Potential factor not set yet
+               if (f == 0) {
+                       f = in.get(*t) / nin;
+               }
 
-                // Factor for this type does not match another type, can not replicate
-                if (f != (in.get(*t) / nin)) {
-                        can_replicate = false;
-                        break;
-                }
+               // Factor for this type does not match another type, can not replicate
+               if (f != (in.get(*t) / nin)) {
+                       can_replicate = false;
+                       break;
+               }
        }
-        
+
        if (can_replicate) {
                for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
                        out.set (*t, outputs.get(*t) * f);
                }
-               return true;
+               return Match (Replicate, f);
        }
 
        /* If the processor has exactly one input of a given type, and
@@ -692,64 +796,33 @@ PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out)
 
        if (can_split) {
                out = outputs;
-               return true;
-       }
-
-       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;
+               return Match (Split, 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 can_hide = false;
+       ChanCount hide_channels;
+       
+       for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+               if (inputs.get(*t) > in.get(*t)) {
+                       hide_channels.set (*t, inputs.get(*t) - in.get(*t));
+                       can_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 (can_hide) {
+               out = outputs;
+               return Match (Hide, 1, hide_channels);
        }
 
-       /* err... */
-       return 0;
+       return Match (Impossible, 0);
 }
 
 XMLNode&
-PluginInsert::get_state(void)
+PluginInsert::get_state ()
 {
        return state (true);
 }
@@ -765,11 +838,11 @@ PluginInsert::state (bool full)
        node.add_child_nocopy (_plugins[0]->get_state());
 
        for (Controls::iterator c = controls().begin(); c != controls().end(); ++c) {
-                boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> ((*c).second);
-                if (ac) {
-                        node.add_child_nocopy (ac->get_state());
-                }
-        }
+               boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> ((*c).second);
+               if (ac) {
+                       node.add_child_nocopy (ac->get_state());
+               }
+       }
 
        return node;
 }
@@ -777,28 +850,29 @@ PluginInsert::state (bool full)
 void
 PluginInsert::set_control_ids (const XMLNode& node, int version)
 {
-        const XMLNodeList& nlist = node.children();
-        XMLNodeConstIterator iter;
-        set<Evoral::Parameter>::const_iterator p;
-
-        for (iter = nlist.begin(); iter != nlist.end(); ++iter) {
-                if ((*iter)->name() == Controllable::xml_node_name) {
-                        const XMLProperty* prop;
-
-                        if ((prop = (*iter)->property (X_("parameter"))) != 0) {
-                                uint32_t p = atoi (prop->value());
-                                boost::shared_ptr<Evoral::Control> c = control (Evoral::Parameter (PluginAutomation, 0, p));
-                                if (!c) {
-                                        continue;
-                                }
-                                boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> (c);
-                                if (ac) {
-                                        ac->set_state (**iter, version);
-                                }
-                        }
-                }
-        }
+       const XMLNodeList& nlist = node.children();
+       XMLNodeConstIterator iter;
+       set<Evoral::Parameter>::const_iterator p;
+
+       for (iter = nlist.begin(); iter != nlist.end(); ++iter) {
+               if ((*iter)->name() == Controllable::xml_node_name) {
+                       const XMLProperty* prop;
+
+                       if ((prop = (*iter)->property (X_("parameter"))) != 0) {
+                               uint32_t p = atoi (prop->value());
+                               boost::shared_ptr<Evoral::Control> c = control (Evoral::Parameter (PluginAutomation, 0, p));
+                               if (!c) {
+                                       continue;
+                               }
+                               boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> (c);
+                               if (ac) {
+                                       ac->set_state (**iter, version);
+                               }
+                       }
+               }
+       }
 }
+
 int
 PluginInsert::set_state(const XMLNode& node, int version)
 {
@@ -819,6 +893,8 @@ PluginInsert::set_state(const XMLNode& node, int version)
                type = ARDOUR::LV2;
        } else if (prop->value() == X_("vst")) {
                type = ARDOUR::VST;
+       } else if (prop->value() == X_("lxvst")) {
+               type = ARDOUR::LXVST;
        } else if (prop->value() == X_("audiounit")) {
                type = ARDOUR::AudioUnit;
        } else {
@@ -834,11 +910,19 @@ PluginInsert::set_state(const XMLNode& node, int version)
 #ifdef VST_SUPPORT
                /* older sessions contain VST plugins with only an "id" field.
                 */
-               
+
                if (type == ARDOUR::VST) {
                        prop = node.property ("id");
                }
-#endif         
+#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
                /* recheck  */
 
                if (prop == 0) {
@@ -847,25 +931,36 @@ PluginInsert::set_state(const XMLNode& node, int version)
                }
        }
 
-       boost::shared_ptr<Plugin> plugin;
-
-       plugin = find_plugin (_session, prop->value(), type);
+       boost::shared_ptr<Plugin> plugin = find_plugin (_session, prop->value(), type);
 
        if (plugin == 0) {
-               error << string_compose(_("Found a reference to a plugin (\"%1\") that is unknown.\n"
-                                  "Perhaps it was removed or moved since it was last used."), prop->value())
+               error << string_compose(
+                       _("Found a reference to a plugin (\"%1\") that is unknown.\n"
+                         "Perhaps it was removed or moved since it was last used."),
+                       prop->value())
                      << endmsg;
                return -1;
        }
 
+       // The name of the PluginInsert comes from the plugin, nothing else
+       _name = plugin->get_info()->name;
+
        uint32_t count = 1;
-       bool need_automatables = true;
+
+       // 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.
                */
-               need_automatables = true;
+               add_plugin (plugin);
+               create_automatable_parameters ();
+               set_control_ids (node, version);
        }
 
        if ((prop = node.property ("count")) != 0) {
@@ -873,38 +968,36 @@ PluginInsert::set_state(const XMLNode& node, int version)
        }
 
        if (_plugins.size() != count) {
-
-               add_plugin_with_activation (plugin);
-
                for (uint32_t n = 1; n < count; ++n) {
-                       add_plugin_with_activation (plugin_factory (plugin));
+                       add_plugin (plugin_factory (plugin));
                }
        }
 
-       if (need_automatables) {
-               set_automatable ();
-                set_control_ids (node, version);
-       }
+       Processor::set_state (node, version);
 
-       /* Handle the node list for this Processor (or Insert if an A2 session) */
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
 
+               /* find the node with the type-specific node name ("lv2", "ladspa", etc)
+                  and set all plugins to the same state.
+               */
+
                if ((*niter)->name() == plugin->state_node_name()) {
 
+                       plugin->set_state (**niter, version);
+
                        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                                (*i)->set_state (**niter, version);
                        }
+
                        break;
                }
        }
 
-        Processor::set_state (node, version);
-
        if (version < 3000) {
 
-                /* Only 2.X sessions need a call to set_parameter_state() - in 3.X and above
-                   this is all handled by Automatable
-                */
+               /* Only 2.X sessions need a call to set_parameter_state() - in 3.X and above
+                  this is all handled by Automatable
+               */
 
                for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
                        if ((*niter)->name() == "Redirect") {
@@ -913,18 +1006,16 @@ PluginInsert::set_state(const XMLNode& node, int version)
                                break;
                        }
                }
-               
+
                set_parameter_state_2X (node, version);
        }
 
-       // The name of the PluginInsert comes from the plugin, nothing else
-       _name = plugin->get_info()->name;
-
-        /* catch up on I/O */
-
-       {
-               Glib::Mutex::Lock em (_session.engine().process_lock());
-               IO::PortCountChanged (max(input_streams(), output_streams()));
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               if (active()) {
+                       (*i)->activate ();
+               } else {
+                       (*i)->deactivate ();
+               }
        }
 
        return 0;
@@ -937,7 +1028,7 @@ PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
        XMLNodeIterator niter;
 
        /* look for port automation node */
-       
+
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
 
                if ((*niter)->name() != port_automation_node_name) {
@@ -954,18 +1045,18 @@ PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
                cnodes = (*niter)->children ("port");
 
                for (iter = cnodes.begin(); iter != cnodes.end(); ++iter){
-                       
+
                        child = *iter;
-                       
+
                        if ((cprop = child->property("number")) != 0) {
                                port = cprop->value().c_str();
                        } else {
                                warning << _("PluginInsert: Auto: no ladspa port number") << endmsg;
                                continue;
                        }
-                       
+
                        sscanf (port, "%" PRIu32, &port_id);
-                       
+
                        if (port_id >= _plugins[0]->parameter_count()) {
                                warning << _("PluginInsert: Auto: port id out of range") << endmsg;
                                continue;
@@ -995,7 +1086,7 @@ PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
                                        if (min_y == FLT_MIN) {
                                                min_y = desc.lower;
                                        }
-                                       
+
                                        if (max_y == FLT_MAX) {
                                                max_y = desc.upper;
                                        }
@@ -1006,11 +1097,11 @@ PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
                                error << string_compose (_("PluginInsert: automatable control %1 not found - ignored"), port_id) << endmsg;
                        }
                }
-               
+
                /* done */
-               
+
                break;
-       } 
+       }
 }
 
 
@@ -1019,7 +1110,7 @@ PluginInsert::describe_parameter (Evoral::Parameter param)
 {
        if (param.type() != PluginAutomation) {
                return Automatable::describe_parameter(param);
-        }
+       }
 
        return _plugins[0]->describe_parameter (param);
 }
@@ -1074,10 +1165,7 @@ PluginInsert::PluginControl::set_value (double user_val)
 double
 PluginInsert::PluginControl::user_to_plugin (double val) const
 {
-       if (_sr_dependent) {
-               val /= _session.frame_rate ();
-       }
-       
+       /* no known transformations at this time */
        return val;
 }
 
@@ -1115,23 +1203,20 @@ PluginInsert::PluginControl::plugin_to_ui (double val) const
 double
 PluginInsert::PluginControl::plugin_to_user (double val) const
 {
-       if (_sr_dependent) {
-               val = val * _session.frame_rate ();
-       }
-
+       /* no known transformations at this time */
        return val;
 }
 
 XMLNode&
 PluginInsert::PluginControl::get_state ()
 {
-        stringstream ss;
+       stringstream ss;
 
-        XMLNode& node (AutomationControl::get_state());
-        ss << parameter().id();
-        node.add_property (X_("parameter"), ss.str());
+       XMLNode& node (AutomationControl::get_state());
+       ss << parameter().id();
+       node.add_property (X_("parameter"), ss.str());
 
-        return node;
+       return node;
 }
 
 /** @return `user' val */
@@ -1173,27 +1258,25 @@ PluginInsert::collect_signal_for_analysis (framecnt_t nframes)
 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::realtime_handle_transport_stopped ()
+PluginInsert::add_plugin (boost::shared_ptr<Plugin> plugin)
 {
-       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-               (*i)->realtime_handle_transport_stopped ();
-       }
+       plugin->set_insert_info (this);
+       _plugins.push_back (plugin);
 }
 
 void
-PluginInsert::set_splitting (bool s)
+PluginInsert::realtime_handle_transport_stopped ()
 {
-       if (_splitting == s) {
-               return;
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->realtime_handle_transport_stopped ();
        }
-                       
-       _splitting = s;
-       SplittingChanged (); /* EMIT SIGNAL */
 }