aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim (unfollow)
Commit message (Collapse)AuthorFilesLines
2013-07-17This is a completely unreasonable thing to do, effectively defying the ↵Diva Canto1-40/+52
purpose of BlockingQueues. Trying this, to see the effect on CPU.
2013-07-17Now trying DoubleQueue instead of BlockingQueue for the ↵Diva Canto1-2/+2
PollServiceRequestManager.
2013-07-17I confuse myself. Let's try this variable name instead.Diva Canto1-3/+3
2013-07-17Do the same trick that dahlia did for Dequeue(timeout)Diva Canto1-2/+3
2013-07-17Putting the requests back in the queue while testing for count >0 is not the ↵Diva Canto1-3/+7
smartest move...
2013-07-17Didn't mean to commit this change in BlockingQueue.csDiva Canto1-1/+1
2013-07-17Cleared up much confusion in PollServiceRequestManager. Here's the history:Diva Canto4-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.
2013-07-17BulletSim: fix small bug where everything looked like it was collidingRobert Adams2-6/+10
before the first simulator step.
2013-07-16Removed the MapItems thread. Redirected the map items requests to the ↵Diva Canto2-9/+45
services throttle thread. Didn't change anything in how that processor is implemented, for better or for worse.
2013-07-16Improvements to the ServiceThrottleModule: added a category and an itemid to ↵Diva Canto3-11/+36
the interface, so that duplicate requests aren't enqueued more than once.
2013-07-16UserManagementModule: in the continuation, call the method that also looks ↵Diva Canto1-2/+2
up the cache, because the resource may be here in the meantime
2013-07-16Added config var to fiddle with the Interval for the service throttle threadDiva Canto1-1/+4
2013-07-16Attachments module only registers when enabled. This enables alternative ↵Dan Lake3-6/+12
attachments module implementations. All calls to Scene.AttachmentsModule are checking for null. Ideally, if we support disabling attachments then we need a null attachments module to register with the scene.
2013-07-16Deleted a couple of verbose messagesDiva Canto2-4/+2
2013-07-16Added IServiceThrottleModule.csDiva Canto1-0/+11
2013-07-16Changed the name to ServiceThrottle/ServiceThrottleModule in order to ↵Diva Canto1-0/+0
reflect its more generic nature.
2013-07-16Eliminated the UserManagement/UserManagementModule throttle thread. Made the ↵Diva Canto2-189/+148
other one generic, taking any continuation.
2013-07-16minor: Add warning method doc about possibly inconsistent results returned ↵Justin Clark-Casey (justincc)1-1/+19
from BlockingQueue.Contains(), Count() and GetQueueArray()
2013-07-16Revert "Revert "MSDN documentation is unclear about whether exiting a lock() ↵Justin Clark-Casey (justincc)1-5/+8
block will trigger a Monitor.Wait() to exit, so avoid some locks that don't actually affect the state of the internal queues in the BlockingQueue class."" This reverts commit 21a09ad3ad42b24bce4fc04c6bcd6f7d9a80af08. After more analysis and discussion, it is apparant that the Count(), Contains() and GetQueueArray() cannot be made thread-safe anyway without external locking And this change appears to have a positive impact on performance. I still believe that Monitor.Exit() will not release any thread for Monitor.Wait(), as per http://msdn.microsoft.com/en-gb/library/vstudio/system.threading.monitor.exit%28v=vs.100%29.aspx so this should in theory make no difference, though mono implementation issues could possibly be coming into play.
2013-07-16Revert "MSDN documentation is unclear about whether exiting a lock() block ↵Justin Clark-Casey (justincc)1-8/+5
will trigger a Monitor.Wait() to exit, so avoid some locks that don't actually affect the state of the internal queues in the BlockingQueue class." This reverts commit 42e2a0d66eaa7e322bce817e9e2cc9a288de167b Reverting because unfortunately this introduces race conditions because Contains(), Count() and GetQueueArray() may now end up returning the wrong result if another thread performs a simultaneous update on m_queue. Code such as PollServiceRequestManager.Stop() relies on the count being correct otherwise a request may be lost. Also, though some of the internal queue methods do not affect state, they are not thread-safe and could return the wrong result generating the same problem lock() generates Monitor.Enter() and Monitor.Exit() under the covers. Monitor.Exit() does not cause Monitor.Wait() to exist, only Pulse() and PulseAll() will do this Reverted with agreement.
2013-07-16In the pursuit of using less CPU: now trying to avoid blocking queues ↵Diva Canto2-15/+120
altogether. Instead, this uses a timer. No sure if it's better or worse, but worth the try.
2013-07-16Actually use DoubleQueue in UserManagement/UserManagementModuleDiva Canto1-1/+1
2013-07-16High CPU hunt: try a different blocking queue, DoubleQueueDiva Canto2-1/+3
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-16MSDN documentation is unclear about whether exiting a lock() block will ↵dahlia1-5/+8
trigger a Monitor.Wait() to exit, so avoid some locks that don't actually affect the state of the internal queues in the BlockingQueue class.
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)11-126/+267
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-15Protect against null requestsDiva Canto2-12/+16
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-15This might address the following observed exception:Diva Canto1-1/+1
17:14:28 - [APPLICATION]: APPLICATION EXCEPTION DETECTED: System.UnhandledExceptionEventArgs Exception: System.InvalidOperationException: Operation is not valid due to the current state of the object at System.Collections.Generic.Queue`1[OpenSim.Region.ClientStack.Linden.WebFetchInvDescModule+aPollRequest].Peek () [0x00011] in /root/install/mono-3.1.0/mono/mcs/class/System/System.Collections.Generic/Queue.cs:158 at System.Collections.Generic.Queue`1[OpenSim.Region.ClientStack.Linden.WebFetchInvDescModule+aPollRequest].Dequeue () [0x00000] in /root/install/mono-3.1.0/mono/mcs/class/System/System.Collections.Generic/Queue.cs:140 at OpenSim.Framework.DoubleQueue`1[OpenSim.Region.ClientStack.Linden.WebFetchInvDescModule+aPollRequest].Dequeue (TimeSpan wait, OpenSim.Region.ClientStack.Linden.aPollRequest& res) [0x0004e] in /home/avacon/opensim_2013-07-14/OpenSim/Framework/Util.cs:2297
2013-07-14Added SQLite version of hg travel data store. UNTESTED. Hope it works!Diva Canto2-0/+100
2013-07-14Cleanup old hg sessions (older than 2 days)Diva Canto5-2/+20
2013-07-14HG UAS: Moved hg-session data from memory to DB storage. This makes it so ↵Diva Canto5-90/+344
that traveling info survives Robust resets. It should also eliminate the cause of empty IP addresses in agent circuit data that we saw in CC grid. MySQL only.
2013-07-14Minor typo in log messageDiva Canto1-1/+1
2013-07-14Let's go easy on authenticating ChildAgentUpdates, otherwise this will be ↵Diva Canto1-7/+5
chaotic while ppl are using different versions of opensim. Warning only, but no enforcement.
2013-07-14Fix broken tests -- the test setup was wrong... sigh.Diva Canto2-1/+2
2013-07-14Authenticate ChildAgentUpdate too.Diva Canto4-18/+32
2013-07-14Change the auth token to be the user's sessionid.Diva Canto4-7/+10
2013-07-13And this fixes the other failing tests. Justin, the thread pool is not being ↵Diva Canto1-2/+0
initialized in the tests!
2013-07-13This should fix the failing test.Diva Canto1-2/+1
2013-07-13Deleted a line too manyDiva Canto1-0/+3
2013-07-13Guard against unauthorized agent deletes.Diva Canto9-32/+57
2013-07-13Deleted GET agent all around. Not used.Diva Canto5-134/+2
2013-07-13Gatekeeper: stop bogus agents earlier, here at the Gatekeeper. No need to ↵Diva Canto1-0/+6
bother the sim.
2013-07-13Some more debug to see how many threads are available.Diva Canto1-0/+2
generated by cgit v1.1 at 2024-12-30 11:41:10 +0000