aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim
diff options
context:
space:
mode:
authorJustin Clarke Casey2008-11-05 19:23:59 +0000
committerJustin Clarke Casey2008-11-05 19:23:59 +0000
commitfc36d7fdf2bf012e6ac60f62875a4a5973af4970 (patch)
tree460b98adcd43e3983ab32f82518a2b5ee1d9577c /OpenSim
parentMoved a couple of more configuration fields to ConfigSettings (diff)
downloadopensim-SC_OLD-fc36d7fdf2bf012e6ac60f62875a4a5973af4970.zip
opensim-SC_OLD-fc36d7fdf2bf012e6ac60f62875a4a5973af4970.tar.gz
opensim-SC_OLD-fc36d7fdf2bf012e6ac60f62875a4a5973af4970.tar.bz2
opensim-SC_OLD-fc36d7fdf2bf012e6ac60f62875a4a5973af4970.tar.xz
* Set default client throttle multiplier to 2 (old value was effectively 8). See OpenSim.ini.example for details as to what this means
* Really this should be 1, but I think that this would be too slow compared to a Second Life server until we improve our ability to send textures of variable quality * This may improve one aspect of sim performance where there are many avatars. However, there are still other performance problems that are unrelated to this change * Value may be further tuned * Removed temporary decals since the multipler setting will stick around now
Diffstat (limited to 'OpenSim')
-rw-r--r--OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs3
-rw-r--r--OpenSim/Region/ClientStack/LindenUDP/LLPacketThrottle.cs9
-rw-r--r--OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs2
3 files changed, 8 insertions, 6 deletions
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs b/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
index 317d852..9aa27ec 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
@@ -127,7 +127,8 @@ namespace OpenSim.Region.ClientStack.LindenUDP
127 127
128 TotalThrottle 128 TotalThrottle
129 = new LLPacketThrottle( 129 = new LLPacketThrottle(
130 totalThrottleSettings.Min, totalThrottleSettings.Max, totalThrottleSettings.Current, userSettings.ClientThrottleMultipler); 130 totalThrottleSettings.Min, totalThrottleSettings.Max, totalThrottleSettings.Current,
131 userSettings.ClientThrottleMultipler);
131 132
132 throttleTimer = new Timer((int) (throttletimems/throttleTimeDivisor)); 133 throttleTimer = new Timer((int) (throttletimems/throttleTimeDivisor));
133 throttleTimer.Elapsed += new ElapsedEventHandler(ThrottleTimerElapsed); 134 throttleTimer.Elapsed += new ElapsedEventHandler(ThrottleTimerElapsed);
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLPacketThrottle.cs b/OpenSim/Region/ClientStack/LindenUDP/LLPacketThrottle.cs
index 2c67d63..305a984 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLPacketThrottle.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLPacketThrottle.cs
@@ -36,7 +36,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
36 private int m_currentBitsSent; 36 private int m_currentBitsSent;
37 37
38 /// <value> 38 /// <value>
39 /// Temporary field 39 /// Value with which to multiply all the throttle fields
40 /// </value> 40 /// </value>
41 private float m_throttleMultiplier; 41 private float m_throttleMultiplier;
42 42
@@ -47,8 +47,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP
47 /// <param name="max"></param> 47 /// <param name="max"></param>
48 /// <param name="throttle"></param> 48 /// <param name="throttle"></param>
49 /// <param name="throttleMultiplier"> 49 /// <param name="throttleMultiplier">
50 /// A temporary parameter that's ends up multiplying all throttle settings. This only exists as a path to 50 /// A parameter that's ends up multiplying all throttle settings. An alternative solution would have been
51 /// using real throttle values instead of the *8 multipler that we had been using (bytes instead of btis) 51 /// to multiply all the parameters by this before giving them to the constructor. But doing it this way
52 /// represents the fact that the multiplier is a hack that pumps data to clients much faster than the actual
53 /// settings that we are given.
52 /// </param> 54 /// </param>
53 public LLPacketThrottle(int min, int max, int throttle, float throttleMultiplier) 55 public LLPacketThrottle(int min, int max, int throttle, float throttleMultiplier)
54 { 56 {
@@ -81,7 +83,6 @@ namespace OpenSim.Region.ClientStack.LindenUDP
81 return m_currentBitsSent; 83 return m_currentBitsSent;
82 } 84 }
83 85
84 // Properties
85 public int Max 86 public int Max
86 { 87 {
87 get { return m_maxAllowableThrottle; } 88 get { return m_maxAllowableThrottle; }
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
index 1779d54..f777e0e 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
@@ -149,7 +149,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
149 AssetCache assetCache, AgentCircuitManager circuitManager) 149 AssetCache assetCache, AgentCircuitManager circuitManager)
150 { 150 {
151 ClientStackUserSettings userSettings = new ClientStackUserSettings(); 151 ClientStackUserSettings userSettings = new ClientStackUserSettings();
152 userSettings.ClientThrottleMultipler = 8; 152 userSettings.ClientThrottleMultipler = 2; // default. TODO: Should be a constant somewhere
153 153
154 IConfig config = configSource.Configs["ClientStack.LindenUDP"]; 154 IConfig config = configSource.Configs["ClientStack.LindenUDP"];
155 155