On entity transfer of scene presence, replace polling sleep in 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.
bullet-2.82
Justin Clark-Casey (justincc) 2014-08-13 23:45:51 +01:00
parent 0d2a25b477
commit bc0895c758
1 changed files with 13 additions and 14 deletions

View File

@ -343,6 +343,12 @@ namespace OpenSim.Region.Framework.Scenes
/// </summary>
private object m_originRegionIDAccessLock = new object();
/// <summary>
/// Triggered on entity transfer after to allow CompleteMovement() to proceed after we have received an
/// UpdateAgent from the originating region.ddkjjkj
/// </summary>
private AutoResetEvent m_updateAgentReceivedAfterTransferEvent = new AutoResetEvent(false);
/// <summary>
/// Used by the entity transfer module to signal when the presence should not be closed because a subsequent
/// teleport is reusing the connection.
@ -1649,7 +1655,6 @@ namespace OpenSim.Region.Framework.Scenes
PhysicsActor.Size = size;
// PhysicsActor.setAvatarSize(size, feetoffset);
}
}
private bool WaitForUpdateAgent(IClientAPI client)
@ -1658,21 +1663,13 @@ namespace OpenSim.Region.Framework.Scenes
// (which triggers Scene.IncomingUpdateChildAgent(AgentData cAgentData) here in the destination,
// m_originRegionID is UUID.Zero; after, it's non-Zero. The CompleteMovement sequence initiated from the
// viewer (in turn triggered by the source region sending it a TeleportFinish event) waits until it's non-zero
int count = 50;
UUID originID;
m_updateAgentReceivedAfterTransferEvent.WaitOne(10000);
UUID originID = UUID.Zero;
lock (m_originRegionIDAccessLock)
originID = m_originRegionID;
while (originID.Equals(UUID.Zero) && count-- > 0)
{
lock (m_originRegionIDAccessLock)
originID = m_originRegionID;
m_log.DebugFormat("[SCENE PRESENCE]: Agent {0} waiting for update in {1}", client.Name, Scene.Name);
Thread.Sleep(200);
}
if (originID.Equals(UUID.Zero))
{
// Movement into region will fail
@ -3820,6 +3817,8 @@ namespace OpenSim.Region.Framework.Scenes
return;
CopyFrom(cAgentData);
m_updateAgentReceivedAfterTransferEvent.Set();
}
private static Vector3 marker = new Vector3(-1f, -1f, -1f);