* Unregister Mono.Addins event handlers in PluginLoader.Dispose() and always handle PluginLoader with the using pattern. This freed up 121,634,796 bytes on my system

* Avoid allocating an Action<IClientAPI> object every round of the OutgoingPacketHandler
* Removed unnecessary semi-colon endings from OpenSim.ini.example [InterestManagement] section

OpenSim/Data/DataPluginFactory.cs

@@ -119,14 +119,15 @@ namespace OpenSim.Data
 
             PluginLoaderParamFactory<T>(connect, out pluginInitialiser, out extensionPointPath);
 
-            PluginLoader<T> loader = new PluginLoader<T>(pluginInitialiser);
+            using (PluginLoader<T> loader = new PluginLoader<T>(pluginInitialiser))
+            {
+                // loader will try to load all providers (MySQL, MSSQL, etc)
+                // unless it is constrainted to the correct "Provider" entry in the addin.xml
+                loader.Add(extensionPointPath, new PluginProviderFilter(provider));
+                loader.Load();
 
-            // loader will try to load all providers (MySQL, MSSQL, etc)
+                return loader.Plugins;
-            // unless it is constrainted to the correct "Provider" entry in the addin.xml
+            }
-            loader.Add(extensionPointPath, new PluginProviderFilter(provider));
-            loader.Load();
-
-            return loader.Plugins;
         }
 
         /// <summary>

OpenSim/Framework/PluginLoader.cs

@@ -194,10 +194,15 @@ namespace OpenSim.Framework
             }
         }
 
+        /// <summary>
+        /// Unregisters Mono.Addins event handlers, allowing temporary Mono.Addins
+        /// data to be garbage collected. Since the plugins created by this loader
+        /// are meant to outlive the loader itself, they must be disposed separately
+        /// </summary>
         public void Dispose()
         {
-            foreach (T plugin in Plugins)
+            AddinManager.AddinLoadError -= on_addinloaderror_;
-                plugin.Dispose();
+            AddinManager.AddinLoaded -= on_addinloaded_;
         }
 
         private void initialise_plugin_dir_(string dir)

OpenSim/Grid/GridServer/GridServerBase.cs

@@ -115,11 +115,11 @@ namespace OpenSim.Grid.GridServer
 
         protected virtual void LoadPlugins()
         {
-            PluginLoader<IGridPlugin> loader =
+            using (PluginLoader<IGridPlugin> loader = new PluginLoader<IGridPlugin>(new GridPluginInitialiser(this)))
-                new PluginLoader<IGridPlugin>(new GridPluginInitialiser(this));
+            {
-
+                loader.Load("/OpenSim/GridServer");
-            loader.Load("/OpenSim/GridServer");
+                m_plugins = loader.Plugins;
-            m_plugins = loader.Plugins;
+            }
         }
 
         public override void ShutdownSpecific()

OpenSim/Region/Application/OpenSimBase.cs

@@ -162,11 +162,11 @@ namespace OpenSim
 
         protected virtual void LoadPlugins()
         {
-            PluginLoader<IApplicationPlugin> loader =
+            using (PluginLoader<IApplicationPlugin> loader = new PluginLoader<IApplicationPlugin>(new ApplicationPluginInitialiser(this)))
-                new PluginLoader<IApplicationPlugin>(new ApplicationPluginInitialiser(this));
+            {
-
+                loader.Load("/OpenSim/Startup");
-            loader.Load("/OpenSim/Startup");
+                m_plugins = loader.Plugins;
-            m_plugins = loader.Plugins;
+            }
         }
 
         protected override List<string> GetHelpTopics()

OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs

@@ -814,6 +814,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP
             // on to en-US to avoid number parsing issues
             Culture.SetCurrentCulture();
 
+            // Typecast the function to an Action<IClientAPI> once here to avoid allocating a new
+            // Action generic every round
+            Action<IClientAPI> clientPacketHandler = ClientOutgoingPacketHandler;
+
             while (base.IsRunning)
             {
                 try
@@ -862,7 +866,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
                     // Handle outgoing packets, resends, acknowledgements, and pings for each
                     // client. m_packetSent will be set to true if a packet is sent
-                    m_scene.ClientManager.ForEachSync(ClientOutgoingPacketHandler);
+                    m_scene.ClientManager.ForEachSync(clientPacketHandler);
 
                     // If nothing was sent, sleep for the minimum amount of time before a
                     // token bucket could get more tokens

OpenSim/Region/Framework/Scenes/Scene.cs

@@ -1223,10 +1223,7 @@ namespace OpenSim.Region.Framework.Scenes
             if (!m_backingup)
             {
                 m_backingup = true;
-
+                Util.FireAndForget(BackupWaitCallback);
-                System.ComponentModel.BackgroundWorker backupWorker = new System.ComponentModel.BackgroundWorker();
-                backupWorker.DoWork += delegate(object sender, System.ComponentModel.DoWorkEventArgs e) { Backup(); };
-                backupWorker.RunWorkerAsync();
             }
         }
 
@@ -1238,6 +1235,14 @@ namespace OpenSim.Region.Framework.Scenes
             m_eventManager.TriggerOnFrame();
         }
 
+        /// <summary>
+        /// Wrapper for Backup() that can be called with Util.FireAndForget()
+        /// </summary>
+        private void BackupWaitCallback(object o)
+        {
+            Backup();
+        }
+
         /// <summary>
         /// Backup the scene.  This acts as the main method of the backup thread.
         /// </summary>

bin/OpenSim.ini.example

@@ -1396,11 +1396,11 @@
 [InterestManagement]
      ; This section controls how state updates are prioritized for each client
      ; Valid values are Time, Distance, SimpleAngularDistance, and FrontBack
-     UpdatePrioritizationScheme = FrontBack;
+     UpdatePrioritizationScheme = FrontBack
-     ReprioritizationEnabled = true;
+     ReprioritizationEnabled = true
-     ReprioritizationInterval = 2000.0;
+     ReprioritizationInterval = 2000.0
-     RootReprioritizationDistance = 10.0;
+     RootReprioritizationDistance = 10.0
-     ChildReprioritizationDistance = 20.0;
+     ChildReprioritizationDistance = 20.0
 
 ;;
 ;; These are defaults that are overwritten below in [Architecture].