aboutsummaryrefslogtreecommitdiffstatshomepage
Commit message (Collapse)AuthorAgeFilesLines
* Add naive implementation of controlled incoming HG attachments to manage load.Justin Clark-Casey (justincc)2014-11-254-79/+121
| | | | | | | Instead of processing all incoming attachment scene object concurrently, process them consecutively to eliminate potential overload from this source. This is a naive implementation because it does not currently account for slow foreign asset services. Although it may take longer, this approach may also improve attachment visibility for HG avatars since the scene object is now always added to the scene after receiving assets from the foreign service and not before.
* Label all threadpool calls being made in core OpenSimulator. This is to add ↵Justin Clark-Casey (justincc)2014-11-2530-65/+103
| | | | | | problem diagnosis. "show threadpool calls" now also returns named (labelled), anonymous (unlabelled) and total call stats.
* Add "show threadpool calls" command to show count of all labelled ↵Justin Clark-Casey (justincc)2014-11-253-1/+37
| | | | smartthreadpool calls
* Just for now, don't alert the user or log if we couldn't change their ↵Justin Clark-Casey (justincc)2014-11-252-3/+3
| | | | | | | | server-side preferences due to no e-mail address being sent. This is to avoid user confusion in the oscc rehearsal as they are often not aware that this fails because no e-mail is set. Also may be failing in the hypergrid case, though this may also be a config issue. This is meant as a temporary solution.
* Update libomv to cedac55justincc2014-11-251-0/+0
| | | | | | This resolves an issue with pCampbot where some bots would occasionally connect with the same UDP source port. This sometimes led to console messages where bots would report receiving packets multiple times that weren't marked as resends. DLLs built under windows
* 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.
* On pCampbot, if we add the none (n) behaviour then make it actually stop any ↵Justin Clark-Casey (justincc)2014-11-251-0/+3
| | | | | | bots in motion. Previously, adding this behaviour after physics (p) would leave the bot to drift off for ever in its last movement direction.
* Add "wearables check" console commandJustin Clark-Casey (justincc)2014-11-252-73/+245
| | | | This checks that all the wearable assets and any assets for a given logged in avatar exist in the asset service
* Add "wearables show" console command.Justin Clark-Casey (justincc)2014-11-252-3/+97
| | | | | This shows summary wearables information (shape, hair, etc.) for all avatars in the scene or specific information about a given avatar's wearables. Similar to the existing "attachments show" command.
* 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-255-53/+82
| | | | | | | 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
* minor: In "show client stats" command, properly handle the case where a ↵Justin Clark-Casey (justincc)2014-11-251-1/+3
| | | | client has made no AgentUpdate requests (as is the case with agents that have only even been child) rather than throwing an exception
* minor: Correct letter accidentally added to copyright notice on top of ↵Justin Clark-Casey (justincc)2014-11-251-1/+1
| | | | Scene.cs (almost five years ago!)
* In Scene.AddNewAgent(), avoid a situation where an exception can result in a ↵Justin Clark-Casey (justincc)2014-11-251-11/+23
| | | | | | | client being added to the manager without IClientAPI.SceneAgent being set. This is done by adjusting the order of code so that SceneAgent will always be set before adding the client. Various parts of the code (rightly) assume that a a client registered to the manager will always have a SceneAgent set no matter what.
* 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.
* Add some more llGiveInventory() regression testsJustin Clark-Casey (justincc)2014-11-251-48/+1
|
* When inserting missing CreatorData in the HGAssetMapper, do the rewrite on a ↵Justin Clark-Casey (justincc)2014-11-252-31/+185
| | | | | | | | streaming xml basis rather than loading it all into memory via XmlDocument. This is because objects with lots of parts can have a lot of xml to load into memory, and this has been seen to have a noticeable performance impact. Whereas streaming has been seen to reduce the impact in normal serialization. Implmentation is messy but I couldn't see a better way of doing it when you can't assume that you know the exact structure of the input XML.
* Add regression test TestPostAssetRewrite() to check results of ↵Justin Clark-Casey (justincc)2014-11-251-0/+84
| | | | HGAssetMapper.Post() object asset rewriting,
* If an exception makes it to the top of a JobEngine request, catch and log ↵Justin Clark-Casey (justincc)2014-11-251-1/+10
| | | | instead of letting it terminate the simulator...
* Allow GetMesh capability to be served directly by a server like GetTextureJustin Clark-Casey (justincc)2014-11-253-58/+46
| | | | | 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).
* Start JobEngine by default in simulator for now.Justin Clark-Casey (justincc)2014-11-251-0/+2
|
* If JobEngine is active, then use it to copy/rez attachments for an avatar ↵Justin Clark-Casey (justincc)2014-11-251-22/+62
| | | | | | | | | entering the scene that isn't initially logging on. This will execute tasks consecutively rather than concurrently. This has two aims 1) Reduce initial teleport failures when a foreign Hypergrid user enters a region by not holding up the teleport for attachment rez (this can be particularly costly when HG gets all assets in the object graph. 2) Reduce server load that may impact other simulator activities. This complements existing JobEngine options that perform initial login attachment rez and appearance send in consecutive tasks.
* minor: be consistent about reporting errors out of HttpServerBaseJustin Clark-Casey (justincc)2014-11-251-11/+13
|
* If the port entry in [Network] for a server is missing, then actually tell ↵Justin Clark-Casey (justincc)2014-11-251-7/+8
| | | | | | the user that's the problem rather than simply exiting silently. Also exit with Environment.Exit(), not by aborting the thread.
* Add an IConfigSource constructor to HGAssetBroker so that it can be ↵Justin Clark-Casey (justincc)2014-11-251-0/+7
| | | | instantiated directly for potentially handling some capabilities directly in services with HG active
* Fix an issue where specifying both max client and server outgoing UDP ↵Justin Clark-Casey (justincc)2014-11-258-73/+287
| | | | | | | | | | 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 warnings from unused fields in LocalGridServicesConnectorJustin Clark-Casey (justincc)2014-11-251-2/+2
|
* minor: Remove unused field from EntityTransferModuleJustin Clark-Casey (justincc)2014-11-251-2/+0
|
* minor: remove compiler warning from unused field in SimulatorFeaturesModuleJustin Clark-Casey (justincc)2014-11-251-2/+2
|
* minor: resolve warnings from unused fields in XBakesGet and Post handlersJustin Clark-Casey (justincc)2014-11-252-6/+3
|
* Remove compiler warning from UserProfilesHandlersJustin Clark-Casey (justincc)2014-11-251-2/+1
|
* minor: remove warning in BasicHttpAuthentication by commenting out unused ↵Justin Clark-Casey (justincc)2014-11-251-2/+2
| | | | remove_me parameter (which looks like it might still be potentially useful for logging)
* 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-2585-164/+10
| | | | | | 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-254-60/+110
| | | | OpenSim.Tests.Common.ClientStackHelpers