aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Region/ClientStack/Linden (unfollow)
Commit message (Collapse)AuthorFilesLines
2013-07-21Minor cosmetic changes.Diva Canto1-27/+23
2013-07-21The quaternion delta was a bit to high, now that the head rotation is out of ↵Diva Canto1-1/+1
the equation. (head rotation was the problematic one)
2013-07-21EDIT BEAMS!!! They had been missing from OpenSim since ever. Thanks to ↵Diva Canto1-23/+4
lkalif for telling me how to route the information. The viewer effect is under the distance filter, so only avatars with cameras < 10m away see the beams.
2013-07-21A couple of small optimizations over the previous commitDiva Canto1-8/+10
2013-07-21Manage AgentUpdates more sanely:Diva Canto1-106/+108
- The existing event to scene has been split into 2: OnAgentUpdate and OnAgentCameraUpdate, to better reflect the two types of updates that the viewer sends. We can run one without the other, which is what happens when the avie is still but the user is camming around - Added thresholds (as opposed to equality) to determine whether the update is significant or not. I thin these thresholds are ok, but we can play with them later - Ignore updates of HeadRotation, which were problematic and aren't being used up stream
2013-07-21Fixed the stats in show client stats. Also left some comments with ↵Diva Canto2-5/+15
observations about AgentUpdates.
2013-07-21furhter shorten CheckAgentUpdateSignificance(). No real perf impact.Justin Clark-Casey (justincc)1-7/+5
2013-07-21Remove some pointless code in CheckAgentUpdateSignificance()Justin Clark-Casey (justincc)1-30/+20
2013-07-21Make the check as to whether any particular inbound AgentUpdate packet is ↵Justin Clark-Casey (justincc)2-58/+129
significant much earlier in UDP processing (i.e. before we pointlessly place such packets on internal queues, etc.) Appears to have some impact on cpu but needs testing.
2013-07-21Add measure of number of inbound AgentUpdates that were seen as significant ↵Justin Clark-Casey (justincc)1-0/+3
to "show client stats" (i.e. sent on for further processing instead of being discarded) Added here since it was the most convenient place Number is in the last column, "Sig. AgentUpdates" along with percentage of all AgentUpdates Percentage largely falls over time, most cpu for processing AgentUpdates may be in UDP processing as turning this off even earlier (with "debug lludp toggle agentupdate" results in a big cpu fall Also tidies up display.
2013-07-21Hack in console command "debug lludp toggle agentupdate" to allow ↵Justin Clark-Casey (justincc)1-0/+24
AgentUpdate in packets to be discarded at a very early stage. Enabling this will stop anybody from moving on a sim, though all other updates should be unaffected. Appears to make some cpu difference on very basic testing with a static standing avatar (though not all that much). Need to see the results with much higher av numbers.
2013-07-21minor: provide user feedback in the log for now when udp in/out bound ↵Justin Clark-Casey (justincc)1-4/+8
threads are started/stopped
2013-07-21Fix up a temporary debugging change from last commit which stopped "lludp ↵Justin Clark-Casey (justincc)1-2/+1
stop out" from actually doing anything
2013-07-21Do some simple queue empty checks in the main outgoing udp loop instead of ↵Justin Clark-Casey (justincc)5-22/+92
always performing these on a separate fired thread. This appears to improve cpu usage since launching a new thread is more expensive than performing a small amount of inline logic. However, needs testing at scale.
2013-07-18Changed the timoeut of EQ 502s (no events) to 50 secs. The viewer post ↵Diva Canto1-2/+9
requests timeout in 60 secs. There's plenty of room for improvement in handling the EQs. Some other time...
2013-07-18try Hacking in an AutoResetEvent to control the outgoing UDP loop instead of ↵Justin Clark-Casey (justincc)2-2/+28
a continuous loop with sleeps. Does appear to have a cpu impact but may need further tweaking
2013-07-18Reverting the reverts I did yesterday. cpu-branch has now beenDiva Canto1-3/+5
successfully tested, and I'm merging back those changes, which proved to be good. Revert "Revert "Cleared up much confusion in PollServiceRequestManager. Here's the history:"" This reverts commit fa2370b32ee57a07f27501152c3c705a883b13d8.
2013-07-17Revert "Cleared up much confusion in PollServiceRequestManager. Here's the ↵Diva Canto1-5/+3
history:" This reverts commit e46459ef21e1ee5ceaeca70365a7c881d33b09ce.
2013-07-17Cleared up much confusion in PollServiceRequestManager. Here's the history:Diva Canto1-3/+5
When Melanie added the web fetch inventory throttle to core, she made the long poll requests (EQs) effectively be handled on an active loop. All those requests, if they existed, were being constantly dequeued, checked for events (which most often they didn't have), and requeued again. This was an active loop thread on a 100ms cycle! This fixes the issue. Now the inventory requests, if they aren't ready to be served, are placed directly back in the queue, but the long poll requests aren't placed there until there are events ready to be sent or timeout has been reached. This puts the LongPollServiceWatcherThread back to 1sec cycle, as it was before.
2013-07-16revert last commit which seems to conflict with DoubleQueue internals. The ↵dahlia1-10/+5
random crash might be in DoubleQueue instead. See http://pastebin.com/XhNBNqsc
2013-07-16add locking to internal queue in WebFetchInvDescModule; lack of which caused ↵dahlia1-5/+10
a random crash in a load test yesterday
2013-07-16Simplify EventQueue cap setup so that it is also stat monitored.Justin Clark-Casey (justincc)1-45/+29
Curiously, the number of requests received is always one greater than that shown as handled - needs investigation
2013-07-15Add request received/handling stats for caps which are served by http poll ↵Justin Clark-Casey (justincc)3-48/+33
handlers. This adds explicit cap poll handler supporting to the Caps classes rather than relying on callers to do the complicated coding. Other refactoring was required to get logic into the right places to support this.
2013-07-15Revert "Puts RequestImage (UDP) back to asyn -- CPU spike hunt"Diva Canto1-1/+1
This reverts commit b060ce96d93a33298b59392210af4d336e0d171b.
2013-07-15Guard against null refDiva Canto1-1/+2
2013-07-15Puts RequestImage (UDP) back to asyn -- CPU spike huntDiva Canto1-1/+1
2013-07-15Revert the revertDiva Canto1-0/+2
Revert "Trying to hunt the CPU spikes recently experienced." This reverts commit ac73e702935dd4607c13aaec3095940fba7932ca.
2013-07-15Trying to hunt the CPU spikes recently experienced.Diva Canto1-2/+0
Revert "Comment out old inbound UDP throttling hack. This would cause the UDP" This reverts commit 38e6da5522a53c7f65eac64ae7b0af929afb1ae6.
2013-07-13Moved SendInitialDataToMe to earlier in CompleteMovement. Moved ↵Diva Canto1-1/+1
TriggerOnMakeRootAgent to the end of CompleteMovement. Justin, if you read this, there's a long story here. Some time ago you placed SendInitialDataToMe at the very beginning of client creation (in LLUDPServer). That is problematic, as we discovered relatively recently: on TPs, as soon as the client starts getting data from child agents, it starts requesting resources back *from the simulator where its root agent is*. We found this to be the problem behind meshes missing on HG TPs (because the viewer was requesting the meshes of the receiving sim from the departing grid). But this affects much more than meshes and HG TPs. It may also explain cloud avatars after a local TP: baked textures are only stored in the simulator, so if a child agent receives a UUID of a baked texture in the destination sim and requests that texture from the departing sim where the root agent is, it will fail to get that texture. Bottom line: we need to delay sending the new simulator data to the viewer until we are absolutely sure that the viewer knows that its main agent is in a new sim. Hence, moving it to CompleteMovement. Now I am trying to tune the initial rez delay that we all experience in the CC. I think that when I fixed the issue described above, I may have moved SendInitialDataToMe to much later than it should be, so now I'm moving to earlier in CompleteMovement.
2013-07-12Changed UploadBakedTextureModule so that it uses the same pattern as the ↵Diva Canto1-8/+27
others, in preparation for experiments to direct baked texture uploads to a robust instance. No functional or configuration changes -- should work exactly as before.
2013-07-11Switched UUIDNameRequest and RegionHandleRequest to Sync, because now they ↵Diva Canto1-2/+2
are also non-blocking handlers.
2013-07-11Switched TransferRequest (UDP packet handler) to sync. The permissions ↵Diva Canto1-99/+115
checks may block, so they get a FireAndForget. Everything else is non-blocking.
2013-07-10Switched RegionHandshakeReply to Sync, because it's not doing anything blocking.Diva Canto1-1/+1
2013-07-10EXPERIMENTAL: make RequestImage (UDP packet handler) sync instead of async. ↵Diva Canto1-1/+1
This _shouldn't_ screw things up, given that all this does is to dump the request in a queue.
2013-07-10Added show client-stats [first last] command to expose what viewers are ↵Diva Canto2-15/+30
requesting.
2013-07-09Comment out old inbound UDP throttling hack. This would cause the UDPRobert Adams1-0/+2
reception thread to sleep for 30ms if the number of available user worker threads got low. It doesn't look like any of the UDP packet types are marked async so this check is 1) unnecessary and 2) really crazy since it stops up the reception thread under heavy load without any indication.
2013-07-09Handle UUIDNameRequest UDP packet processing async instead of within the ↵Justin Clark-Casey (justincc)1-1/+1
main inbound UDP processing loop, to avoid any chance that this is delaying the main udp in loop. The potential impact of this should be lower now that these requests are being placed on a queue.
2013-07-08For stat purposes, add names to capability request handlers where these were ↵Justin Clark-Casey (justincc)2-3/+10
not set
2013-07-08Add "show caps stats by user" and "show caps stats by cap" console commands ↵Justin Clark-Casey (justincc)1-1/+1
to print various counts of capability invocation by user and by cap This currently prints caps requests received and handled, so that overload of received compared to handled or deadlock can be detected. This involves making BaseStreamHandler and BaseOutputStream record the ints, which means inheritors should subclass ProcessRequest() instead of Handle() However, existing inheriting classes overriding Handle() will still work, albeit without stats recording. "show caps" becomes "show caps list" to disambiguate between show caps commands
2013-07-06Fix mono warning in LLImageManagerTestsJustin Clark-Casey (justincc)1-1/+1
2013-07-06Get InventoryWorkerThreads to update watchdog on processing requestsJustin Clark-Casey (justincc)1-0/+2
2013-07-04Add --default option to "debug lludp packet" command to allow packet logging ↵Justin Clark-Casey (justincc)1-17/+52
to be performed immediately from client start
2013-07-04change "debug packet" command to "debug lludp packet" to conform with other ↵Justin Clark-Casey (justincc)1-0/+59
"debug lludp" options also moves the implementing code into LLUDPServer.cs along with other debug commands from OpenSim.cs gets all debug lludp commands to only activate for the set scene if not root
2013-07-02HG: close a loophole by which if something was wrong with the ServiceURLs it ↵Diva Canto1-1/+1
resulted in never ending asset requests
2013-06-17minor: remove mono compiler warnings in WebFetchInvDescModuleJustin Clark-Casey (justincc)1-3/+2
2013-06-17Fix test failure in BasicCircuitTests from previous commit 0d2fd0d9Justin Clark-Casey (justincc)1-2/+2
2013-06-14Don't try to abort worker threads in WebFetchInvDescModule if module was not ↵Justin Clark-Casey (justincc)1-6/+4
enabled. This also moves the abort to RemoveRegion() rather than a destructor.
2013-06-07Put the configuration back in FetchInventoryDesc2 cap.Diva Canto1-0/+29
2013-06-07Supply proper type information for the various types of requestsMelanie1-0/+1
2013-06-07Adding Avination's PollService to round out the HTTP inventory changesMelanie2-4/+4