aboutsummaryrefslogtreecommitdiffstatshomepage
Commit message (Collapse)AuthorAgeFilesLines
* minor:Give console feedback when we sit or stand pCampbot bots.Justin Clark-Casey (justincc)2014-08-191-2/+16
| | | | Also only write console lines for actually connected bots.
* Add clientstack.<scene>.OQRERequestsWaiting statJustin Clark-Casey (justincc)2014-08-191-0/+19
| | | | Only present if OQRE is enabled
* Add experimental OutgoingQueueRefillEngine to handle queue refill processing ↵Justin Clark-Casey (justincc)2014-08-193-5/+288
| | | | | | | | | | | on a controlled number of threads rather than the threadpool. Disabled by default. Currently can only be enabled with console "debug lludp oqre start" command, though this can be started and stopped whilst simulator is running. When a connection requires packet queue refill processing (used to populate queues with entity updates, entity prop updates and image queue updates), this is done via Threadpool requests. However, with a very high number of connections (e.g. 100 root + 300 child) a very large number of simultaneous requests may be causing performance issues. This commit adds an experimental engine for processing these requests from a queue with a persistent thread instead. Unlike inbound processing, there are no network requests in this processing that might hold the thread up for a long time. Early implementation - currently only one thread which may (or may not) get overloaded with requests. Added for testing purposes.
* Make LLUDPServer.Scene publicly gettable/privately settable instead of ↵Justin Clark-Casey (justincc)2014-08-193-49/+55
| | | | | | | protected so that other logging code in the clientstack can record more useful information Adds some commented out logging for use again in the future. No functional change.
* Prevent exception if inventory item in llGiveInventory() call doesn't exist.Kevin Cozens2014-08-181-0/+1
|
* Don't allow the last behavior to be removed from a pCampbot botJustin Clark-Casey (justincc)2014-08-152-1/+4
| | | | If you want to stop existing behavious, add the None behaviour.
* Resolve a small race condition on removing bot behaviours that might leave ↵Justin Clark-Casey (justincc)2014-08-151-2/+3
| | | | | | | previous behaviour active Also closes behaviours on disconnect instead of interrupt, though this makes no practical difference. If existing behaviour is None, other added behavious will not take affect until None is removed (as this is an infinite wait until interrupted).
* If a user moves back in sight of a child region before the agent has been ↵Justin Clark-Casey (justincc)2014-08-153-7/+9
| | | | closed on teleport, don't unnecessarily resend all avatar and object data about that region.
* On teleport to a region that already has a child agent established (e.g. a ↵Justin Clark-Casey (justincc)2014-08-157-39/+50
| | | | | | neighbour) don't resend all the initial avatar and object data again. This is unnecessary since it has been received (and data continues to be received) in the existing child connection.
* Added RestrictEmail to make llEmail only send to avatars email address if true.Kevin Cozens2014-08-142-0/+39
|
* Make RootTerseUpdatePeriod and ChildTerseUpdatePeriod configurable in ↵Justin Clark-Casey (justincc)2014-08-142-0/+12
| | | | | | | | | | [InterestManagement] in OpenSim.ini for experimental purposes. If n > 1 for RootTerseUpdatePeriod only every n terse update is actually sent to observers on same region, unless velocity is effectively zero (to stop av drift). If n > 1 for ChildTerseUpdatePeriod only every n terse update is sent to observers in other regions, unless velocity is effectively zero. Defaults are same as before (all packets are sent). Tradeoff is reduction of UDP traffic vs fidelity of observed av mvmt. Increasing n > 1 leads to jerky observed mvmt immediateley for root, though not on child, where experimentally have gone to n = 4 before jerkiness is noticeable.
* Make some existing reprioritization values changeable outside the scene for ↵Justin Clark-Casey (justincc)2014-08-141-15/+18
| | | | | | test purposes, and use more consise property syntax. No functional change.
* Update OpenID server handler with proper interface. fixes ↵BlueWall2014-08-131-1/+1
| | | | http://opensimulator.org/mantis/view.php?id=7301
* On entity transfer of scene presence, replace polling sleep in ↵Justin Clark-Casey (justincc)2014-08-131-14/+13
| | | | | | | SP.WaitForUpdateAgent() with a triggered event instead. Rapid polls are more expensive than triggered events (several polls vs one trigger) and may be problematic on heavily loaded simulators where many threads are vying for processor time. A triggered event is also slightly quicker as there is no maximum 200ms wait between polls.
* Remove redundant origin region lock in SP.CompleteMovement()Justin Clark-Casey (justincc)2014-08-131-6/+1
| | | | This is already going to be correctly set by WaitForUpdateAgent() earlier on in that method, which is always called where a callback to the originating region is required.
* Only set up the UnackedMethod for an outgoing message if that message is ↵Justin Clark-Casey (justincc)2014-08-131-1/+3
| | | | actually meant to get an ack (because it's reliable).
* Terminate 'nothing' behaviour (and potentially others) by signalling using ↵Justin Clark-Casey (justincc)2014-08-135-36/+56
| | | | | | | an event rather than polling connection state every 100ms This kind of polling is very expensive with many bots/polling threads and appears to be the primary cause of bot falloff from the client end at higher loads. Where inbound packet threads can't run in time due to contention and simulator disconnect timeout occurs.
* Add 'server' stats information to pCampbot, as used elsewhere in OpenSimulatorJustin Clark-Casey (justincc)2014-08-135-0/+47
| | | | | This adds the "show stats", "stats record", etc. commands and information on available Threadpool threads, etc. It also adds the Watchdog which logs warnings if time between executions is unexpectedly large.
* For pCampbot, set max number of permitted connections to an endpoint to ↵Justin Clark-Casey (justincc)2014-08-121-0/+4
| | | | | | int.MaxValue This is to avoid issues where many bots connect to a single end point with multiple regions, where each region requires a long-lived poll connection for each bot.
* Update libomv libraries with those built from commit 2208379.Justin Clark-Casey (justincc)2014-08-124-0/+0
| | | | This is to resolve some issues for pCampbot, chiefly with capability connection and inbound handling.
* Avination's Dynamic Floater Module. This works with Singularity viewer only.Melanie Thielker2014-08-122-0/+290
| | | | | It's WIP in that a dialog builder is on it's way. For now, the XML needs to be handmade.
* Move the inventory request lock introduced in git master a58152bd to ↵Justin Clark-Casey (justincc)2014-08-092-27/+47
| | | | | | | HGInventoryBroker to preserve that behaviour there but allow 2 simultaneous inv requests (chiefly WebFetch) rather than 1 This lock serialized all requests and made the inventory throttling in WebFetch redundant. By moving this lock, two simultaneous requests may now take place which may help with http://opensimulator.org/mantis/view.php?id=7054
* Change RootRotationUpdateTolerance from 0.01 to 0.1 in code as well.Justin Clark-Casey (justincc)2014-08-081-1/+1
|
* Change MaxPoolThreads default in code as well from 15 to 300Justin Clark-Casey (justincc)2014-08-081-1/+1
|
* Reduce default rotation AgentUpdate output sensitivity to a setting that ↵Justin Clark-Casey (justincc)2014-08-081-1/+1
| | | | | | | | | | cuts down UDP traffic without obvious adverse effects on observed avatar rotations. Experimentally, on the Linden Lab grid the avatar can rotate slightly before triggering AvatarUpdates, whereas this is practically impossible in OpenSimulator. These updates allow other avatars to see rotations, though sensitivity is low since other avatars can only be seen in one of 8 body rotations. This commit changes sensitivity from 0.01 to 0.1, which better matches LL and reduces UDP traffic which has a beneficial impact on network and CPU load. This has no impact on rotations in the simulator itself so simulation fidelity is the same as before. To change this setting back for test/other purposes, edit RootRotationUpdateTolerance in the [InterestManagement] section of OpenSim.ini
* Change default max threads if SmartThreadPool is used as the main thread ↵Justin Clark-Casey (justincc)2014-08-081-1/+1
| | | | | | | | | pool from 15 to 300 Running out of such threads under heavy load causes delayed packet processing which can lead to spurious UDP resends and knock on issues. We already massively boost the min/max builtin pool worker and IOCP threads (which even with STP are still used for inbound network requests) without obvious adverse effects. The threads are only instantiated if they are required. This change does not affect other async_call_method options.
* Fix short help string for show region command.BlueWall2014-08-061-1/+1
|
* OpenSimExtrasBlueWall2014-08-0615-434/+182
| | | | Move the experimental extra features functionality into the GridService. This sends default values for map, search and destination guide, plus ExportSupported control to the region on startup. Please watch http://opensimulator.org/wiki/SimulatorFeatures_Extras for changes and documentation.
* Fixed crash when using Allowed/Denied Viewers, and the viewer's name is ↵Oren Hurvitz2014-08-061-2/+2
| | | | | | shorter than one of the test strings This fixes http://opensimulator.org/mantis/view.php?id=7294
* Add RootPositionUpdateTolerance, RootRotationUpdateTolerance, and ↵Justin Clark-Casey (justincc)2014-08-062-8/+23
| | | | | | | RootVelocityUpdateTolerance parameters to [InterestManagement] in OpenSimDefaults.ini These govern when AgentUpdates are sent to observers on position, rotation and velocity changes to an avatar (including the avatar themselves). Higher values reduce AgentUpdate traffic but at a certain level will degrade smoothness of avatar and perceived avatar movement.
* refactor: Rename recent new Client*UpdateTolerance to Root*UpdateTolerance ↵Justin Clark-Casey (justincc)2014-08-063-19/+19
| | | | for better accuracy and consistency with other similar parameters
* Fixed premature closing of the connection in DataSnapshotManagerOren Hurvitz2014-08-051-14/+14
|
* Go back to disconnecting bots in parallel since serially is too slow.Justin Clark-Casey (justincc)2014-08-051-3/+5
| | | | However, disconnecting now halts any current connection, with the possible exception of the single currently connecting bot.
* Put pCampbot "disconnect" command on separate thread like "connect" so that ↵Justin Clark-Casey (justincc)2014-08-051-27/+35
| | | | we can continue to run status commands whilst bots are disconnecting.
* Allow "show bots" pCampbot console command to quickly report status by not ↵Justin Clark-Casey (justincc)2014-08-051-18/+20
| | | | locking entire bot list for almost 100% of connection time.
* Make currently unfiltered EventQueue log messages only appear now at ↵Justin Clark-Casey (justincc)2014-08-051-16/+24
| | | | | | | | | DebugLevel 1 This covers event queue setup messages and some outgoing messages (e.g. EnableSimulator) In my experience these messages are only useful if you really know what they mean and you're looking for them Otherwise, they're quite spammy. Event queue DebugLevel 1 is enabled with the "debug eq 1" console command
* Fix typo in log messageBlueWall2014-08-021-1/+1
|
* Fix regression in SimulatorFeatures module that would stop OpenSimulator ↵Justin Clark-Casey (justincc)2014-08-011-12/+20
| | | | from starting if no [SimulatorFeatures] section was present in config.
* If REMOVEAGENTFROMGROUP core groups call fails because requesting agent does ↵Justin Clark-Casey (justincc)2014-07-314-10/+22
| | | | | | | not have sufficient permission, return null failure result rather than true. On non-HG this is on the only recognized failure state so we can return more information in the error result. On HG there are multiple failure states which would require more work to distinguish, so currently return the unsatisfying "Internal Error" like some other existing calls.
* Don't overwrite the null result with the true result is groups service ↵Justin Clark-Casey (justincc)2014-07-312-2/+4
| | | | REMOVEAGENTFROMGROUP call has failed because of missing parameters
* Add region-side extra feature setting for destination guideBlueWall2014-07-312-0/+6
|
* BulletSim: thread safe handling of list of avatars.Robert Adams2014-07-301-3/+18
| | | | Fix for 7284 which is an enumeration exception when starting up a region.
* Fix bug where calling PUTGROUP on the core groups service without specifying ↵Justin Clark-Casey (justincc)2014-07-311-1/+1
| | | | | | a ServiceLocation would set the group name to an empty string. This should set the ServiceLocation to an empty string instead.
* Add Chat module extra feature settings to GridExtraFeatures serviceBlueWall2014-07-303-0/+27
|
* Simulator Extra Features ServiceBlueWall2014-07-307-9/+415
| | | | | Provide a means for regions to fetch extra features supported by modern viewers from a central location .
* In TerrainModule, lock m_perClientPatchUpdates when removing entries.Justin Clark-Casey (justincc)2014-07-291-3/+2
| | | | | This may have been the trigger CheckSendingPatchesToClients() dictionary out of sync exceptions in today's load test. Don't need to check ContainsKey() since Remove() returns false on a request to remove a key that it doesn't have
* Implement "scene debug set root-upd-per" for dropping 1 in N root agent ↵Justin Clark-Casey (justincc)2014-07-293-7/+41
| | | | | | | updates except to originator For experimental purposes. Also corrects a previous bug where each terse update sent was counted rather than each set of terse updates to agents.
* Add debug mechanism for only sending 1 in N AgentUpdate packets to child agents.Justin Clark-Casey (justincc)2014-07-293-1/+30
| | | | | | Allows experiments in manually reducing updates under heavy load. Activated by "debug scene set client-upd-per" console command. In a simple test, can send as few as every 4th update before observed movement starts becoming disturbingly rubber-banded.
* Add "debug scene set appear-refresh true|false" to control whether periodic ↵Justin Clark-Casey (justincc)2014-07-291-0/+12
| | | | | | | appearance refresh is active. Corresponds to ResendAppearnceUpdates setting in [Appearance] in OpenSim.ini This was originally implemented to alleviate cloud appearance problems but could be too expensive with large numbers of avatars.
* Make it possible to change avatar position update, rotation and velocity ↵Justin Clark-Casey (justincc)2014-07-293-29/+79
| | | | | | | tolerances on the fly. This is done via "debug scene set client-pos-upd, client-rot-upd, client-vel-upd". For testing purposes.