aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Region/ClientStack (follow)
Commit message (Collapse)AuthorAgeFilesLines
* minor: Comment out log line for the SimulatorFeatures cap request for now to ↵Justin Clark-Casey (justincc)2015-01-211-1/+1
| | | | reduce log verbiage. Please revert if still required.
* And finally renamed the classes themselves.Diva Canto2015-01-131-2/+2
|
* Fix bug where gathering the clientstack..OutgoingPacketsQueuedCount stat ↵Justin Clark-Casey (justincc)2015-01-131-2/+6
| | | | | | would fail with a casting exception for scenes with NPCs Present since 51eb8fa (Oct 2 2014)
* Make the performance controlling job processing threads introduced in ↵Justin Clark-Casey (justincc)2015-01-126-629/+92
| | | | conference code use a generic JobEngine class rather than 4 slightly different copy/pasted versions.
* SimulatorFeatures: the viewer also takes GridName in OpenSim extras. Added ↵Diva Canto2015-01-101-1/+4
| | | | that (plus GridURL, in case viewers want to use it too) to the GridService that gives out that info to simulators.
* Added GetDisplayNames capability. For now, we don't actually use display ↵Diva Canto2015-01-091-0/+144
| | | | names, and this cap returns the regular name. But this moves the server side into the newer, preferred, protocol used by the viewer for fetching the names of agents in the scene given their UUIDs. (the old protocol is via UDP). This works fine in my limited tests, but could use further testing by others.
* Merge branch 'mb-throttle-test'Mic Bowman2014-12-306-37/+149
|\ | | | | | | Merge in the new throttle code.
| * Comment out the debugging statements added in the last commit. KeepingMic Bowman2014-12-302-13/+13
| | | | | | | | | | | | | | them in the code for later use rather than just reverting them. Fixed the throttle tests for the new algorithm used when packets are marked as expired.
| * Turn on logging of old acks, this is for debugging only, stats version ↵Mic Bowman2014-12-301-1/+12
| | | | | | | | coming later
| * Another technique inspired by some of the newer flow control algorithms... ↵Mic Bowman2014-12-301-11/+17
| | | | | | | | | | | | | | | | rather than drop exponentially to 0 (and then adjust up for the minimum flow), drop on the delta between current rate and the minimum rate. This should smooth the fallback to minimum.
| * Fix the throttle tests. Remove the hardcoded constant multipliers andMic Bowman2014-12-301-9/+11
| | | | | | | | compute the expected values without depending on the token bucket code.
| * Change the effect of successfully acknowledged packets to bump theMic Bowman2014-12-293-7/+7
| | | | | | | | | | | | | | adaptive throttle by a full MTU. This is consistent with some implementations of congestion control algorithms and certainly has the effect of opening the throttle window more quickly after errors. This is especially important after initial scene load when the number and size of packets is small.
| * Enable runtime configuration of the minimum rate for adaptiveMic Bowman2014-12-293-19/+46
| | | | | | | | | | | | | | | | throttles. Setting adaptive_throttle_min_bps will change the minimum rate that the adapative throttles will drop to in case of network packet loss. The current rate default rate is 256kbps. The viewer can throttle rates under that amount, but the dynamic adaptation will not.
| * Add debugging code for simulating packet loss in outgoing UDP packets. TheMic Bowman2014-12-291-0/+66
| | | | | | | | | | | | | | algorithm for dropping packets is a modified two state algorithm for creating bursts of dropped packets. As configured there is about a 1.5% drop rate. Invocation of the packet loss code is commented out by default.
* | Renamed VERSION_NUMBER to VersionNumberDiva Canto2014-12-301-2/+2
| |
* | This cleans up versioning. Specifically:Diva Canto2014-12-301-2/+2
| | | | | | | | | | | | | | - It establishes 4 digits for opensim versions - It uses the same number between opensim releases and mono addins versions It also eliminates the last addin.xml files that were still there, for consistency.
* | Merge branch 'master' of ssh://opensimulator.org/var/git/opensimDiva Canto2014-12-301-1/+1
|\ \
| * | minor: correct sent -> set in TokenBucket.RequestedDripRate method docJustin Clark-Casey (justincc)2014-12-301-1/+1
| |/
* | WARNING: BREAKING CHANGES FOR REGION MODULE DEVELOPMENT.Diva Canto2014-12-301-1/+1
| | | | | | | | | | | | | | This cleans up Opensim's use of mono addins. In particular, the extension points /OpenSim/RegionModules and /OpenSim/WindModule moved from OpenSim.exe to OpenSim.Region.Framework.dll. From here on, developers of region modules should declare their dlls to be dependent on OpenSim.Region.Framework, starting with version 0.8.1 Additional changes: - Addins version uniformly updated to 0.8.1. These numbers should be compatible with the release numbers or else it becomes very confusing. - Mono addins directives moved from files addins.xml to embedded directives in the class and assembly declarations, to make it all consistent
* | Updated OpenSim-as-addin version from 0.5 to 0.8.1. The addin version number ↵Diva Canto2014-12-291-1/+1
|/ | | | doesn't need to match the release version number, but I think it's a very good idea that they do.
* Somewhat improve avatar region crossings by properly preserving velocity ↵Justin Clark-Casey (justincc)2014-11-291-4/+5
| | | | | | | | | when avatar enters the new region. This commit addresses the following issues were causing velocity to be set to 0 on the new region, disrupting flight in particular * Full avatar updates contained no velocity information, which does appear to have some effect in testing. * BulletSim was always setting the velocity to 0 for the new BSCharacter. Now, physics engines take a velocity parameter when setting up characters so we can avoid this. This patch applies to both Bullet and ODE.
* refactor: Move methods to start a monitored thread, start work in its own ↵Justin Clark-Casey (justincc)2014-11-254-5/+5
| | | | | | | | thread and run work in the jobengine from Watchdog to a WorkManager class. This is to achieve a clean separation of concerns - the watchdog is an inappropriate place for work management. Also adds a WorkManager.RunInThreadPool() class which feeds through to Util.FireAndForget. Also switches around the name and obj arguments to the new RunInThread() and RunJob() methods so that the callback obj comes after the callback as seen in the SDK and elsewhere
* Fix ghosts copy of SimulatorFeatureModule to enable logging.Justin Clark-Casey (justincc)2014-11-251-8/+13
|
* Add incoming packet async handling engine to queue some inbound udp async ↵Justin Clark-Casey (justincc)2014-11-253-11/+393
| | | | | | | requests. This is to reduce the potential for overload of the threadpool if there are many simultaneous requets in high concurrency situations. Currently only applied to AvatarProperties and GenericMessage requests.
* Label all threadpool calls being made in core OpenSimulator. This is to add ↵Justin Clark-Casey (justincc)2014-11-253-8/+11
| | | | | | problem diagnosis. "show threadpool calls" now also returns named (labelled), anonymous (unlabelled) and total call stats.
* Stop Mono 3.2.8 from binding a UDP socket to a port already in use.Justin Clark-Casey (justincc)2014-11-251-0/+6
| | | | | | At least on Mono 3.2.8 (but not under Windows), one can bind multiple UDP sockets to the same port by default. Different simulators cannot demultiplex each other's messages, so a set of confusing non-obvious errors arise if this occurs. This change prevents such multiple binding.
* Fix recent regression where adaptive throttles stopped adjusting.Justin Clark-Casey (justincc)2014-11-253-49/+61
| | | | Extends regression tests to test response of adaptive throttles to ack'ed and expired packets.
* Fix setting of max scene throttle so that setting it restricts the child ↵Justin Clark-Casey (justincc)2014-11-254-51/+78
| | | | | | | client throttles properly. In "show throttles", also renames 'total' column to 'actual' to reflect that it is not necessarily the throttles requested for/by the client. Also fills out 'target' in non-adapative mode to the actual throttle requested for/by the client.
* Add request drip rate to assertions for token bucket regression testsJustin Clark-Casey (justincc)2014-11-251-12/+14
|
* Add regression tests for token buckets on their ownJustin Clark-Casey (justincc)2014-11-251-1/+49
|
* Make regression throttle tests consistently test target and max throttle ↵Justin Clark-Casey (justincc)2014-11-251-136/+62
| | | | | | settings. As part of this also refactors code to put all throttle asserts in a single regression test method
* For now, send all non-full terse updates for ones own avatar directly to the ↵Justin Clark-Casey (justincc)2014-11-252-4/+27
| | | | | | | | | LLUDP client stack rather than queueing internally within LLClientView. When an HG avatar enters a scene, it delays processing of entity updates. Could be crowding out by other updates or something else. This delay in ones own av mvmt updates results in mvmt lag experienced on the client. Avoiding the internal LLClientView for these packets appears to resolve this issue. Appears most noticeably for avatars with attachments, though has also been seen on those without sometimes. Hasn't been observed for non-HG avatars in general. Will be investigating exactly what the problem is, at which point there will be a more permanent solution.
* Add "debug lludp throttles get/set request" and get currentJustin Clark-Casey (justincc)2014-11-251-0/+26
| | | | This allows one to set the requested throttle (which normally comes from the client) as opposed to the max.
* Change the word order of some debug lludp settings for readabilityJustin Clark-Casey (justincc)2014-11-251-15/+15
| | | | | On server, scene-throttle-max becomes max-scene-throttle and likewise max-new-client-throttle On clients, throttle-max becomes max
* Add "debug lludp get/set new-client-throttle-max" to allow default new ↵Justin Clark-Casey (justincc)2014-11-251-8/+25
| | | | | | client throttle to be set separately from existing clients. "debug lludp throttles get/set throttle-max" now only gets and sets current max client throttles
* Add "debug lludp get" command which currently just shows scene-throttle-max ↵Justin Clark-Casey (justincc)2014-11-251-0/+25
| | | | | | to mirror "debug lludp set" Information is also available in "show server throttles" but that's more for non-debug info rather than attempting to get and set parameters on the fly for debug purposes.
* Allow GetMesh capability to be served directly by a server like GetTextureJustin Clark-Casey (justincc)2014-11-251-18/+15
| | | | | To do this required GetMesh to be converted to a BaseStreamHandler Unlike GetTexture connector, no redirect URL functionality yet (this wasn't present in the first place).
* Fix an issue where specifying both max client and server outgoing UDP ↵Justin Clark-Casey (justincc)2014-11-256-66/+275
| | | | | | | | | | throttles would cause client throttles to be lower than expected when total requests exceeded the scene limit. This was because specifying a max client throttle would always request the max from the parent server throttle, no matter the actual total requests on the client throttle. This would lead to a lower server multiplier than expected. This change also adds a 'target' column to the "show throttles" output that shows the target rate (as set by client) if adaptive throttles is active. This commit also re-adds the functionality lost in recent 5c1a1458 to set a max client throttle when adaptive is active. This commit also adds TestClientThrottlePerClientAndRegionLimited and TestClientThrottleAdaptiveNoLimit regression tests
* Use automatic properties for Parent and TotalDripRequest in TokenBucket to ↵Justin Clark-Casey (justincc)2014-11-251-32/+21
| | | | make code analysis easier. No functional change.
* Remove the unnecessary intermediate total token bucket.Justin Clark-Casey (justincc)2014-11-252-15/+3
| | | | | This only had one child, which is the 'adaptive' token bucket. So from testing and currently analysis, we can use that bucket directly which simplifies the code.
* minor: remove compiler warning from unused field in SimulatorFeaturesModuleJustin Clark-Casey (justincc)2014-11-251-2/+2
|
* extend TestClientThrottleRegionLimited for 2 clients after testing 1. ↵Justin Clark-Casey (justincc)2014-11-251-25/+71
| | | | Renames to TestSingleClientThrottleRegionLimited()
* Add regression test TestClientThrottleRegionLimited() for testing simple ↵Justin Clark-Casey (justincc)2014-11-251-2/+50
| | | | behaviour of throttles where a region-wide total outbound limit is in place.
* factor out common throttle setting byte[] array construction in ThrottleTests.Justin Clark-Casey (justincc)2014-11-251-44/+43
|
* Add regression test TestClientThrottleLimited() for throttle behaviour when ↵Justin Clark-Casey (justincc)2014-11-252-1/+64
| | | | a max client total limit is enforced server-side
* refactor: consistently put all test classes in the OpenSim.Tests.Common ↵Justin Clark-Casey (justincc)2014-11-256-84/+0
| | | | | | package rather than some in OpenSim.Tests.Common.Mock the separate mock package was not useful and was just another using line to always add
* minor: disable logging in regression test TestClientThrottleSetNoLimitJustin Clark-Casey (justincc)2014-11-251-1/+1
|
* minor: comment out unused method in regression BasicCircuitTestsJustin Clark-Casey (justincc)2014-11-251-17/+17
|
* Add basic regression test ThrottleTests.TestClientThrottleSetNoLimitJustin Clark-Casey (justincc)2014-11-252-0/+114
|
* refactor: Move test clientstack setup code out of BasicCircuitTests into ↵Justin Clark-Casey (justincc)2014-11-252-229/+12
| | | | OpenSim.Tests.Common.ClientStackHelpers