Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | A couple of small optimizations over the previous commit | Diva Canto | 2013-07-21 | 2 | -9/+11 |
| | |||||
* | Manage AgentUpdates more sanely: | Diva Canto | 2013-07-21 | 6 | -146/+183 |
| | | | | | | - 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 | ||||
* | Fixed the stats in show client stats. Also left some comments with ↵ | Diva Canto | 2013-07-21 | 3 | -9/+19 |
| | | | | observations about AgentUpdates. | ||||
* | Removed verbose debug from previous commit | Diva Canto | 2013-07-21 | 1 | -1/+0 |
| | |||||
* | PollServiceRequestManager: changed the long poll from a Queue to a List. No ↵ | Diva Canto | 2013-07-21 | 1 | -13/+17 |
| | | | | need to dequeue and enqueue items every 1sec. | ||||
* | furhter shorten CheckAgentUpdateSignificance(). No real perf impact. | Justin Clark-Casey (justincc) | 2013-07-21 | 1 | -7/+5 |
| | |||||
* | Remove some pointless code in CheckAgentUpdateSignificance() | Justin Clark-Casey (justincc) | 2013-07-21 | 1 | -30/+20 |
| | |||||
* | Make the check as to whether any particular inbound AgentUpdate packet is ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 3 | -60/+131 |
| | | | | | | 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. | ||||
* | Add measure of number of inbound AgentUpdates that were seen as significant ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 2 | -3/+12 |
| | | | | | | | | | 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. | ||||
* | Hack in console command "debug lludp toggle agentupdate" to allow ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 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. | ||||
* | minor: provide user feedback in the log for now when udp in/out bound ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 1 | -4/+8 |
| | | | | threads are started/stopped | ||||
* | Fix up a temporary debugging change from last commit which stopped "lludp ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 1 | -2/+1 |
| | | | | stop out" from actually doing anything | ||||
* | Delay the enqueueing of non-longpoll requests for 100ms. No need to have ↵ | Diva Canto | 2013-07-21 | 1 | -1/+11 |
| | | | | these requests actively on the processing queue if it seems they're not ready. | ||||
* | Do some simple queue empty checks in the main outgoing udp loop instead of ↵ | Justin Clark-Casey (justincc) | 2013-07-21 | 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. | ||||
* | Merge branch 'master' of ssh://opensimulator.org/var/git/opensim | Diva Canto | 2013-07-19 | 2 | -5/+20 |
|\ | |||||
| * | BulletSim: add position resetting for stationary avatars so they don't | Robert Adams | 2013-07-18 | 2 | -5/+20 |
| | | | | | | | | | | | | move around when standing on a stationary object. Create proper linkage between BSCharacter and its actor by generating a UpdatedProperties event the same way BSPrim does. | ||||
* | | Fix HGTravelStore.migrations in SQLite (mantis #6709) | Diva Canto | 2013-07-19 | 1 | -10/+10 |
|/ | |||||
* | Enable storing of environment settings in NullSimulationData | Dan Lake | 2013-07-18 | 1 | -3/+17 |
| | |||||
* | Changed the timoeut of EQ 502s (no events) to 50 secs. The viewer post ↵ | Diva Canto | 2013-07-18 | 2 | -3/+10 |
| | | | | | | requests timeout in 60 secs. There's plenty of room for improvement in handling the EQs. Some other time... | ||||
* | try Hacking in an AutoResetEvent to control the outgoing UDP loop instead of ↵ | Justin Clark-Casey (justincc) | 2013-07-18 | 2 | -2/+28 |
| | | | | | | a continuous loop with sleeps. Does appear to have a cpu impact but may need further tweaking | ||||
* | Revert "Revert "I confuse myself. Let's try this variable name instead."" | Diva Canto | 2013-07-18 | 1 | -3/+3 |
| | | | | This reverts commit 52dc7b2a96a28798d55d07d79d003ce5e3d35216. | ||||
* | Revert "Revert "Do the same trick that dahlia did for Dequeue(timeout)"" | Diva Canto | 2013-07-18 | 1 | -2/+3 |
| | | | | This reverts commit 5495df74436d6c0039a1500d979a964b003abfdf. | ||||
* | Revert "Revert "Putting the requests back in the queue while testing for ↵ | Diva Canto | 2013-07-18 | 1 | -3/+7 |
| | | | | | | count >0 is not the smartest move..."" This reverts commit 71278919575b0e0222cdbe3c0cefa5919f9a75bc. | ||||
* | Revert "Revert "Didn't mean to commit this change in BlockingQueue.cs"" | Diva Canto | 2013-07-18 | 1 | -1/+1 |
| | | | | This reverts commit fda91d93dad1fa6f901e8db5829aa8b70477c97e. | ||||
* | Reverting the reverts I did yesterday. cpu-branch has now been | Diva Canto | 2013-07-18 | 4 | -52/+41 |
| | | | | | | | | 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. | ||||
* | Merge branch 'master' of git://opensimulator.org/git/opensim | Dan Lake | 2013-07-18 | 1 | -1/+1 |
|\ | |||||
| * | Fix what apepars to be a bug in DoubleQueue<T>.Enqueue(Queue<T> q, T data) ↵ | Justin Clark-Casey (justincc) | 2013-07-18 | 1 | -1/+1 |
| | | | | | | | | | | | | where the q parmater is ignored and everyghig is always placed on m_lowQueue. No actual impact presently since nothing ends up calling EnqueueHigh() | ||||
* | | Added MinPoolThreads to ini [Startup] section to control SmartThreadPool. | Dan Lake | 2013-07-18 | 2 | -4/+8 |
|/ | |||||
* | Merge branch 'master' of ssh://opensimulator.org/var/git/opensim | Diva Canto | 2013-07-17 | 1 | -2/+3 |
|\ | |||||
| * | Merge branch 'master' of git://opensimulator.org/git/opensim | Dan Lake | 2013-07-17 | 10 | -114/+188 |
| |\ | |||||
| * | | command line kick user now uses exact name match instead of substring search ↵ | Dan Lake | 2013-07-17 | 1 | -2/+3 |
| | | | | | | | | | | | | to avoid kicking the wrong user or multiple wrong users. | ||||
* | | | Revert "Cleared up much confusion in PollServiceRequestManager. Here's the ↵ | Diva Canto | 2013-07-17 | 4 | -41/+52 |
| | | | | | | | | | | | | | | | | | | history:" This reverts commit e46459ef21e1ee5ceaeca70365a7c881d33b09ce. | ||||
* | | | Revert "Didn't mean to commit this change in BlockingQueue.cs" | Diva Canto | 2013-07-17 | 1 | -1/+1 |
| | | | | | | | | | | | | This reverts commit 0f5b616fb0ebf9207b3cc81771622ed1290ea7d6. | ||||
* | | | Revert "Putting the requests back in the queue while testing for count >0 is ↵ | Diva Canto | 2013-07-17 | 1 | -7/+3 |
| | | | | | | | | | | | | | | | | | | not the smartest move..." This reverts commit f4317dc26d670c853d0ea64b401b00f718f09474. | ||||
* | | | Revert "Do the same trick that dahlia did for Dequeue(timeout)" | Diva Canto | 2013-07-17 | 1 | -3/+2 |
| | | | | | | | | | | | | This reverts commit af792bc7f2504e9ccf1c8ae7568919785dc397c9. | ||||
* | | | Revert "I confuse myself. Let's try this variable name instead." | Diva Canto | 2013-07-17 | 1 | -3/+3 |
| | | | | | | | | | | | | This reverts commit 1d3deda10cf85abd68a5f904d6698ae597a67cc0. | ||||
* | | | Revert "Now trying DoubleQueue instead of BlockingQueue for the ↵ | Diva Canto | 2013-07-17 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | PollServiceRequestManager." This reverts commit 5f95f4d78e8c7d17b8ba866907156fe6d4444c04. | ||||
* | | | Revert "This is a completely unreasonable thing to do, effectively defying ↵ | Diva Canto | 2013-07-17 | 1 | -52/+40 |
| |/ |/| | | | | | | | | | the purpose of BlockingQueues. Trying this, to see the effect on CPU." This reverts commit 5232ab0496eb4fe6903a0fd328974ac69df29ad8. | ||||
* | | This is a completely unreasonable thing to do, effectively defying the ↵ | Diva Canto | 2013-07-17 | 1 | -40/+52 |
| | | | | | | | | purpose of BlockingQueues. Trying this, to see the effect on CPU. | ||||
* | | Now trying DoubleQueue instead of BlockingQueue for the ↵ | Diva Canto | 2013-07-17 | 1 | -2/+2 |
| | | | | | | | | PollServiceRequestManager. | ||||
* | | I confuse myself. Let's try this variable name instead. | Diva Canto | 2013-07-17 | 1 | -3/+3 |
| | | |||||
* | | Do the same trick that dahlia did for Dequeue(timeout) | Diva Canto | 2013-07-17 | 1 | -2/+3 |
| | | |||||
* | | Putting the requests back in the queue while testing for count >0 is not the ↵ | Diva Canto | 2013-07-17 | 1 | -3/+7 |
| | | | | | | | | smartest move... | ||||
* | | Didn't mean to commit this change in BlockingQueue.cs | Diva Canto | 2013-07-17 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'master' of ssh://opensimulator.org/var/git/opensim | Diva Canto | 2013-07-17 | 2 | -6/+10 |
|\ \ | |||||
| * | | BulletSim: fix small bug where everything looked like it was colliding | Robert Adams | 2013-07-17 | 2 | -6/+10 |
| | | | | | | | | | | | | before the first simulator step. | ||||
* | | | Cleared up much confusion in PollServiceRequestManager. Here's the history: | Diva Canto | 2013-07-17 | 4 | -52/+41 |
|/ / | | | | | | | | | | | 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. | ||||
* | | Removed the MapItems thread. Redirected the map items requests to the ↵ | Diva Canto | 2013-07-16 | 2 | -9/+45 |
| | | | | | | | | services throttle thread. Didn't change anything in how that processor is implemented, for better or for worse. | ||||
* | | Improvements to the ServiceThrottleModule: added a category and an itemid to ↵ | Diva Canto | 2013-07-16 | 3 | -11/+36 |
| | | | | | | | | the interface, so that duplicate requests aren't enqueued more than once. | ||||
* | | UserManagementModule: in the continuation, call the method that also looks ↵ | Diva Canto | 2013-07-16 | 1 | -2/+2 |
| | | | | | | | | up the cache, because the resource may be here in the meantime |