Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add "debug lludp throttles get/set request" and get current | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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 readability | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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-25 | 1 | -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-25 | 1 | -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 tests | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -48/+1 |
| | |||||
* | When inserting missing CreatorData in the HGAssetMapper, do the rewrite on a ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -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-25 | 1 | -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-25 | 1 | -1/+10 |
| | | | | instead of letting it terminate the simulator... | ||||
* | Allow GetMesh capability to be served directly by a server like GetTexture | Justin Clark-Casey (justincc) | 2014-11-25 | 3 | -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-25 | 1 | -0/+2 |
| | |||||
* | If JobEngine is active, then use it to copy/rez attachments for an avatar ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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 HttpServerBase | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -11/+13 |
| | |||||
* | If the port entry in [Network] for a server is missing, then actually tell ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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-25 | 1 | -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-25 | 8 | -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-25 | 1 | -32/+21 |
| | | | | make code analysis easier. No functional change. | ||||
* | Remove the unnecessary intermediate total token bucket. | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -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 LocalGridServicesConnector | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+2 |
| | |||||
* | minor: Remove unused field from EntityTransferModule | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+0 |
| | |||||
* | minor: remove compiler warning from unused field in SimulatorFeaturesModule | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+2 |
| | |||||
* | minor: resolve warnings from unused fields in XBakesGet and Post handlers | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -6/+3 |
| | |||||
* | Remove compiler warning from UserProfilesHandlers | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+1 |
| | |||||
* | minor: remove warning in BasicHttpAuthentication by commenting out unused ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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-25 | 1 | -25/+71 |
| | | | | Renames to TestSingleClientThrottleRegionLimited() | ||||
* | Add regression test TestClientThrottleRegionLimited() for testing simple ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -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-25 | 1 | -44/+43 |
| | |||||
* | Add regression test TestClientThrottleLimited() for throttle behaviour when ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -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-25 | 85 | -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 TestClientThrottleSetNoLimit | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -1/+1 |
| | |||||
* | minor: comment out unused method in regression BasicCircuitTests | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -17/+17 |
| | |||||
* | Add basic regression test ThrottleTests.TestClientThrottleSetNoLimit | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -0/+114 |
| | |||||
* | refactor: Move test clientstack setup code out of BasicCircuitTests into ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 4 | -60/+110 |
| | | | | OpenSim.Tests.Common.ClientStackHelpers | ||||
* | Fix recent regression in "debug lludp throttles get" command that stopped it ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+4 |
| | | | | printing any information. Also fix max throttle displayed to be properly kbps | ||||
* | minor: in "show server throttles", display unset if new client throttle ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+6 |
| | | | | value is not set rather than 0 | ||||
* | Add throttle-max option to "debug lludp throttles set" to allow runtime ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+33 |
| | | | | | | setting of default and existing client throttles. Doesn't yet adjust until clients submit new throttle settings. | ||||
* | Change "debug lludp throttle *" commands to "debug lludp throttles" for ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -9/+9 |
| | | | | | | consistency (and because I keep typing throttles). This will still work with "debug lludp throttle" anyway. | ||||
* | small adjustment to commented out scene_throttle_max_bps and ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+2 |
| | | | | | | client_throttle_max_bps examples in OpenSimDefaults.ini Make them actually reflect 20 mbit and 1.5 mbit respectively | ||||
* | Change help text for client_throttle_max_bps in OpenSimDefaults.ini to state ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -1/+1 |
| | | | | | | | | it is in bytes, not bits This is the same as already done for scene_throttle_max_bps Internally, the token buckets are in bytes and the other help text makes it clear that the number is bytes per second (though with the wrong assumption that 1 mbit = 1024 * 1024 bits whereas 1 mbit = 1000 kbits = 1000000 bits) | ||||
* | Fix console set and get of max scene rate. Was performing wrong calculation ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -3/+3 |
| | | | | - throttle buckets are set in bytes, not bits | ||||
* | Eliminate 'max' throttle setting from server throttles report since this ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -9/+4 |
| | | | | never applies. | ||||
* | Allow "debug lludp throttle *" commands to work without a user name, in ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -21/+51 |
| | | | | which case they apply to all users in the lludp server | ||||
* | minor: add apparant total to logging when client sets throttles | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -2/+5 |
| | |||||
* | minor: fix bug in throttle logging where arguments were mismatched | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -1/+1 |
| | |||||
* | Add "debug lludp set scene-throttle-max <value>" console command to allow us ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -6/+40 |
| | | | | to potentially set the scene max throttle on the fly. | ||||
* | Move information about "server agent rate" throttles into "show server ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -39/+57 |
| | | | | | | throttles" command rather than "show throttles" THis allows us to see the rates when no client is connected to the region. | ||||
* | Add "show server throttles" command for showing server specific information ↵ | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -0/+26 |
| | | | | | | | about throttles This is separate from the user-oriented "show throttles" command since one will often only want to know about varying client throttle settings. Currently displays max scene throttle and adaptive throttles config if set. | ||||
* | refactor: Use simpler auto-implemented property for HttpPort in GridRegion | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -10/+5 |
| | |||||
* | minor: Remove compiler warning from GridRegion in IGridService | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -1/+1 |
| | |||||
* | minor: remove compiler warning from OpenProfileClient | Justin Clark-Casey (justincc) | 2014-11-25 | 1 | -4/+3 |
| | |||||
* | Add OutgoingPacketsQueuedCount clientstack stat. | Justin Clark-Casey (justincc) | 2014-11-25 | 2 | -0/+53 |
| | | | | | | This is the total of queued outgoing packets across all connections, as also seen in the "show queues" command. Gives some early indication of whether the simulator can't send all outgoing packets fast enough. Though then one would want to check that this isn't due to a few bad client connections. |