diff options
Diffstat (limited to 'OpenSim/Region/ClientStack/LindenUDP')
6 files changed, 406 insertions, 270 deletions
diff --git a/OpenSim/Region/ClientStack/LindenUDP/J2KImage.cs b/OpenSim/Region/ClientStack/LindenUDP/J2KImage.cs index e9e2dca..9dd6663 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/J2KImage.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/J2KImage.cs | |||
@@ -202,6 +202,9 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
202 | m_stopPacket = TexturePacketCount(); | 202 | m_stopPacket = TexturePacketCount(); |
203 | } | 203 | } |
204 | 204 | ||
205 | //Give them at least two packets, to play nice with some broken viewers (SL also behaves this way) | ||
206 | if (m_stopPacket == 1 && Layers[0].End > FIRST_PACKET_SIZE) m_stopPacket++; | ||
207 | |||
205 | m_currentPacket = StartPacket; | 208 | m_currentPacket = StartPacket; |
206 | } | 209 | } |
207 | } | 210 | } |
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs index 320a2fa..87ed90f 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs | |||
@@ -154,6 +154,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
154 | public event RequestTaskInventory OnRequestTaskInventory; | 154 | public event RequestTaskInventory OnRequestTaskInventory; |
155 | public event UpdateInventoryItem OnUpdateInventoryItem; | 155 | public event UpdateInventoryItem OnUpdateInventoryItem; |
156 | public event CopyInventoryItem OnCopyInventoryItem; | 156 | public event CopyInventoryItem OnCopyInventoryItem; |
157 | public event MoveItemsAndLeaveCopy OnMoveItemsAndLeaveCopy; | ||
157 | public event MoveInventoryItem OnMoveInventoryItem; | 158 | public event MoveInventoryItem OnMoveInventoryItem; |
158 | public event RemoveInventoryItem OnRemoveInventoryItem; | 159 | public event RemoveInventoryItem OnRemoveInventoryItem; |
159 | public event RemoveInventoryFolder OnRemoveInventoryFolder; | 160 | public event RemoveInventoryFolder OnRemoveInventoryFolder; |
@@ -335,11 +336,13 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
335 | // protected HashSet<uint> m_attachmentsSent; | 336 | // protected HashSet<uint> m_attachmentsSent; |
336 | 337 | ||
337 | private int m_moneyBalance; | 338 | private int m_moneyBalance; |
339 | private bool m_deliverPackets = true; | ||
338 | private int m_animationSequenceNumber = 1; | 340 | private int m_animationSequenceNumber = 1; |
339 | private bool m_SendLogoutPacketWhenClosing = true; | 341 | private bool m_SendLogoutPacketWhenClosing = true; |
340 | private AgentUpdateArgs lastarg; | 342 | private AgentUpdateArgs lastarg; |
341 | private bool m_IsActive = true; | 343 | private bool m_IsActive = true; |
342 | private bool m_IsLoggingOut = false; | 344 | private bool m_IsLoggingOut = false; |
345 | private bool m_IsPresenceReady = false; | ||
343 | 346 | ||
344 | protected Dictionary<PacketType, PacketProcessor> m_packetHandlers = new Dictionary<PacketType, PacketProcessor>(); | 347 | protected Dictionary<PacketType, PacketProcessor> m_packetHandlers = new Dictionary<PacketType, PacketProcessor>(); |
345 | protected Dictionary<string, GenericMessage> m_genericPacketHandlers = new Dictionary<string, GenericMessage>(); //PauPaw:Local Generic Message handlers | 348 | protected Dictionary<string, GenericMessage> m_genericPacketHandlers = new Dictionary<string, GenericMessage>(); //PauPaw:Local Generic Message handlers |
@@ -362,6 +365,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
362 | 365 | ||
363 | private Timer m_propertiesPacketTimer; | 366 | private Timer m_propertiesPacketTimer; |
364 | private List<ObjectPropertiesPacket.ObjectDataBlock> m_propertiesBlocks = new List<ObjectPropertiesPacket.ObjectDataBlock>(); | 367 | private List<ObjectPropertiesPacket.ObjectDataBlock> m_propertiesBlocks = new List<ObjectPropertiesPacket.ObjectDataBlock>(); |
368 | private List<Packet> m_pendingPackets; | ||
365 | 369 | ||
366 | #endregion Class Members | 370 | #endregion Class Members |
367 | 371 | ||
@@ -377,6 +381,14 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
377 | get { return m_startpos; } | 381 | get { return m_startpos; } |
378 | set { m_startpos = value; } | 382 | set { m_startpos = value; } |
379 | } | 383 | } |
384 | public bool DeliverPackets | ||
385 | { | ||
386 | get { return m_deliverPackets; } | ||
387 | set { | ||
388 | m_deliverPackets = value; | ||
389 | m_udpClient.m_deliverPackets = value; | ||
390 | } | ||
391 | } | ||
380 | public UUID AgentId { get { return m_agentId; } } | 392 | public UUID AgentId { get { return m_agentId; } } |
381 | public UUID ActiveGroupId { get { return m_activeGroupID; } } | 393 | public UUID ActiveGroupId { get { return m_activeGroupID; } } |
382 | public string ActiveGroupName { get { return m_activeGroupName; } } | 394 | public string ActiveGroupName { get { return m_activeGroupName; } } |
@@ -402,6 +414,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
402 | get { return m_IsActive; } | 414 | get { return m_IsActive; } |
403 | set { m_IsActive = value; } | 415 | set { m_IsActive = value; } |
404 | } | 416 | } |
417 | |||
405 | public bool IsLoggingOut | 418 | public bool IsLoggingOut |
406 | { | 419 | { |
407 | get { return m_IsLoggingOut; } | 420 | get { return m_IsLoggingOut; } |
@@ -471,18 +484,30 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
471 | 484 | ||
472 | #region Client Methods | 485 | #region Client Methods |
473 | 486 | ||
487 | |||
474 | /// <summary> | 488 | /// <summary> |
475 | /// Shut down the client view | 489 | /// Shut down the client view |
476 | /// </summary> | 490 | /// </summary> |
477 | public void Close() | 491 | public void Close() |
478 | { | 492 | { |
493 | Close(true); | ||
494 | } | ||
495 | |||
496 | /// <summary> | ||
497 | /// Shut down the client view | ||
498 | /// </summary> | ||
499 | public void Close(bool sendStop) | ||
500 | { | ||
479 | m_log.DebugFormat( | 501 | m_log.DebugFormat( |
480 | "[CLIENT]: Close has been called for {0} attached to scene {1}", | 502 | "[CLIENT]: Close has been called for {0} attached to scene {1}", |
481 | Name, m_scene.RegionInfo.RegionName); | 503 | Name, m_scene.RegionInfo.RegionName); |
482 | 504 | ||
483 | // Send the STOP packet | 505 | if (sendStop) |
484 | DisableSimulatorPacket disable = (DisableSimulatorPacket)PacketPool.Instance.GetPacket(PacketType.DisableSimulator); | 506 | { |
485 | OutPacket(disable, ThrottleOutPacketType.Unknown); | 507 | // Send the STOP packet |
508 | DisableSimulatorPacket disable = (DisableSimulatorPacket)PacketPool.Instance.GetPacket(PacketType.DisableSimulator); | ||
509 | OutPacket(disable, ThrottleOutPacketType.Unknown); | ||
510 | } | ||
486 | 511 | ||
487 | IsActive = false; | 512 | IsActive = false; |
488 | 513 | ||
@@ -762,7 +787,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
762 | reply.ChatData.OwnerID = fromAgentID; | 787 | reply.ChatData.OwnerID = fromAgentID; |
763 | reply.ChatData.SourceID = fromAgentID; | 788 | reply.ChatData.SourceID = fromAgentID; |
764 | 789 | ||
765 | OutPacket(reply, ThrottleOutPacketType.Task); | 790 | OutPacket(reply, ThrottleOutPacketType.Unknown); |
766 | } | 791 | } |
767 | 792 | ||
768 | /// <summary> | 793 | /// <summary> |
@@ -1048,6 +1073,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
1048 | public virtual void SendLayerData(float[] map) | 1073 | public virtual void SendLayerData(float[] map) |
1049 | { | 1074 | { |
1050 | Util.FireAndForget(DoSendLayerData, map); | 1075 | Util.FireAndForget(DoSendLayerData, map); |
1076 | |||
1077 | // Send it sync, and async. It's not that much data | ||
1078 | // and it improves user experience just so much! | ||
1079 | DoSendLayerData(map); | ||
1051 | } | 1080 | } |
1052 | 1081 | ||
1053 | /// <summary> | 1082 | /// <summary> |
@@ -1060,16 +1089,13 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
1060 | 1089 | ||
1061 | try | 1090 | try |
1062 | { | 1091 | { |
1063 | //for (int y = 0; y < 16; y++) | 1092 | for (int y = 0; y < 16; y++) |
1064 | //{ | 1093 | { |
1065 | // for (int x = 0; x < 16; x++) | 1094 | for (int x = 0; x < 16; x+=4) |
1066 | // { | 1095 | { |
1067 | // SendLayerData(x, y, map); | 1096 | SendLayerPacket(x, y, map); |
1068 | // } | 1097 | } |
1069 | //} | 1098 | } |
1070 | |||
1071 | // Send LayerData in a spiral pattern. Fun! | ||
1072 | SendLayerTopRight(map, 0, 0, 15, 15); | ||
1073 | } | 1099 | } |
1074 | catch (Exception e) | 1100 | catch (Exception e) |
1075 | { | 1101 | { |
@@ -1077,51 +1103,35 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
1077 | } | 1103 | } |
1078 | } | 1104 | } |
1079 | 1105 | ||
1080 | private void SendLayerTopRight(float[] map, int x1, int y1, int x2, int y2) | ||
1081 | { | ||
1082 | // Row | ||
1083 | for (int i = x1; i <= x2; i++) | ||
1084 | SendLayerData(i, y1, map); | ||
1085 | |||
1086 | // Column | ||
1087 | for (int j = y1 + 1; j <= y2; j++) | ||
1088 | SendLayerData(x2, j, map); | ||
1089 | |||
1090 | if (x2 - x1 > 0) | ||
1091 | SendLayerBottomLeft(map, x1, y1 + 1, x2 - 1, y2); | ||
1092 | } | ||
1093 | |||
1094 | void SendLayerBottomLeft(float[] map, int x1, int y1, int x2, int y2) | ||
1095 | { | ||
1096 | // Row in reverse | ||
1097 | for (int i = x2; i >= x1; i--) | ||
1098 | SendLayerData(i, y2, map); | ||
1099 | |||
1100 | // Column in reverse | ||
1101 | for (int j = y2 - 1; j >= y1; j--) | ||
1102 | SendLayerData(x1, j, map); | ||
1103 | |||
1104 | if (x2 - x1 > 0) | ||
1105 | SendLayerTopRight(map, x1 + 1, y1, x2, y2 - 1); | ||
1106 | } | ||
1107 | |||
1108 | /// <summary> | 1106 | /// <summary> |
1109 | /// Sends a set of four patches (x, x+1, ..., x+3) to the client | 1107 | /// Sends a set of four patches (x, x+1, ..., x+3) to the client |
1110 | /// </summary> | 1108 | /// </summary> |
1111 | /// <param name="map">heightmap</param> | 1109 | /// <param name="map">heightmap</param> |
1112 | /// <param name="px">X coordinate for patches 0..12</param> | 1110 | /// <param name="px">X coordinate for patches 0..12</param> |
1113 | /// <param name="py">Y coordinate for patches 0..15</param> | 1111 | /// <param name="py">Y coordinate for patches 0..15</param> |
1114 | // private void SendLayerPacket(float[] map, int y, int x) | 1112 | private void SendLayerPacket(int x, int y, float[] map) |
1115 | // { | 1113 | { |
1116 | // int[] patches = new int[4]; | 1114 | int[] patches = new int[4]; |
1117 | // patches[0] = x + 0 + y * 16; | 1115 | patches[0] = x + 0 + y * 16; |
1118 | // patches[1] = x + 1 + y * 16; | 1116 | patches[1] = x + 1 + y * 16; |
1119 | // patches[2] = x + 2 + y * 16; | 1117 | patches[2] = x + 2 + y * 16; |
1120 | // patches[3] = x + 3 + y * 16; | 1118 | patches[3] = x + 3 + y * 16; |
1121 | 1119 | ||
1122 | // Packet layerpack = LLClientView.TerrainManager.CreateLandPacket(map, patches); | 1120 | float[] heightmap = (map.Length == 65536) ? |
1123 | // OutPacket(layerpack, ThrottleOutPacketType.Land); | 1121 | map : |
1124 | // } | 1122 | LLHeightFieldMoronize(map); |
1123 | |||
1124 | try | ||
1125 | { | ||
1126 | Packet layerpack = TerrainCompressor.CreateLandPacket(heightmap, patches); | ||
1127 | OutPacket(layerpack, ThrottleOutPacketType.Land); | ||
1128 | } | ||
1129 | catch | ||
1130 | { | ||
1131 | for (int px = x ; px < x + 4 ; px++) | ||
1132 | SendLayerData(px, y, map); | ||
1133 | } | ||
1134 | } | ||
1125 | 1135 | ||
1126 | /// <summary> | 1136 | /// <summary> |
1127 | /// Sends a specified patch to a client | 1137 | /// Sends a specified patch to a client |
@@ -1141,7 +1151,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
1141 | LayerDataPacket layerpack = TerrainCompressor.CreateLandPacket(heightmap, patches); | 1151 | LayerDataPacket layerpack = TerrainCompressor.CreateLandPacket(heightmap, patches); |
1142 | layerpack.Header.Reliable = true; | 1152 | layerpack.Header.Reliable = true; |
1143 | 1153 | ||
1144 | OutPacket(layerpack, ThrottleOutPacketType.Land); | 1154 | OutPacket(layerpack, ThrottleOutPacketType.Task); |
1145 | } | 1155 | } |
1146 | catch (Exception e) | 1156 | catch (Exception e) |
1147 | { | 1157 | { |
@@ -2224,6 +2234,15 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
2224 | OutPacket(sound, ThrottleOutPacketType.Task); | 2234 | OutPacket(sound, ThrottleOutPacketType.Task); |
2225 | } | 2235 | } |
2226 | 2236 | ||
2237 | public void SendTransferAbort(TransferRequestPacket transferRequest) | ||
2238 | { | ||
2239 | TransferAbortPacket abort = (TransferAbortPacket)PacketPool.Instance.GetPacket(PacketType.TransferAbort); | ||
2240 | abort.TransferInfo.TransferID = transferRequest.TransferInfo.TransferID; | ||
2241 | abort.TransferInfo.ChannelType = transferRequest.TransferInfo.ChannelType; | ||
2242 | m_log.Debug("[Assets] Aborting transfer; asset request failed"); | ||
2243 | OutPacket(abort, ThrottleOutPacketType.Task); | ||
2244 | } | ||
2245 | |||
2227 | public void SendTriggeredSound(UUID soundID, UUID ownerID, UUID objectID, UUID parentID, ulong handle, Vector3 position, float gain) | 2246 | public void SendTriggeredSound(UUID soundID, UUID ownerID, UUID objectID, UUID parentID, ulong handle, Vector3 position, float gain) |
2228 | { | 2247 | { |
2229 | SoundTriggerPacket sound = (SoundTriggerPacket)PacketPool.Instance.GetPacket(PacketType.SoundTrigger); | 2248 | SoundTriggerPacket sound = (SoundTriggerPacket)PacketPool.Instance.GetPacket(PacketType.SoundTrigger); |
@@ -3508,6 +3527,15 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
3508 | /// </summary> | 3527 | /// </summary> |
3509 | public void SendPrimUpdate(ISceneEntity entity, PrimUpdateFlags updateFlags) | 3528 | public void SendPrimUpdate(ISceneEntity entity, PrimUpdateFlags updateFlags) |
3510 | { | 3529 | { |
3530 | if (entity is SceneObjectPart) | ||
3531 | { | ||
3532 | SceneObjectPart e = (SceneObjectPart)entity; | ||
3533 | SceneObjectGroup g = e.ParentGroup; | ||
3534 | if (g.RootPart.Shape.State > 30) // HUD | ||
3535 | if (g.OwnerID != AgentId) | ||
3536 | return; // Don't send updates for other people's HUDs | ||
3537 | } | ||
3538 | |||
3511 | double priority = m_prioritizer.GetUpdatePriority(this, entity); | 3539 | double priority = m_prioritizer.GetUpdatePriority(this, entity); |
3512 | 3540 | ||
3513 | lock (m_entityUpdates.SyncRoot) | 3541 | lock (m_entityUpdates.SyncRoot) |
@@ -3528,9 +3556,13 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
3528 | EntityUpdate update; | 3556 | EntityUpdate update; |
3529 | while (updatesThisCall < maxUpdates && m_entityUpdates.TryDequeue(out update)) | 3557 | while (updatesThisCall < maxUpdates && m_entityUpdates.TryDequeue(out update)) |
3530 | { | 3558 | { |
3559 | // If we have sent a kill packet for this object | ||
3560 | // drop any updates on the floor | ||
3531 | if (update.Entity is SceneObjectPart) | 3561 | if (update.Entity is SceneObjectPart) |
3532 | { | 3562 | { |
3533 | SceneObjectPart part = (SceneObjectPart)update.Entity; | 3563 | SceneObjectPart part = (SceneObjectPart)update.Entity; |
3564 | if (m_killRecord.Contains(part.ParentGroup.RootPart.LocalId)) | ||
3565 | continue; | ||
3534 | 3566 | ||
3535 | if (part.ParentGroup.IsAttachment && m_disableFacelights) | 3567 | if (part.ParentGroup.IsAttachment && m_disableFacelights) |
3536 | { | 3568 | { |
@@ -3964,6 +3996,9 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
3964 | { | 3996 | { |
3965 | m_propertiesPacketTimer.Stop(); | 3997 | m_propertiesPacketTimer.Stop(); |
3966 | 3998 | ||
3999 | if (m_propertiesBlocks.Count == 0) | ||
4000 | return; | ||
4001 | |||
3967 | proper.ObjectData = new ObjectPropertiesPacket.ObjectDataBlock[m_propertiesBlocks.Count]; | 4002 | proper.ObjectData = new ObjectPropertiesPacket.ObjectDataBlock[m_propertiesBlocks.Count]; |
3968 | 4003 | ||
3969 | int index = 0; | 4004 | int index = 0; |
@@ -4863,6 +4898,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
4863 | AddLocalPacketHandler(PacketType.TransferAbort, HandleTransferAbort, false); | 4898 | AddLocalPacketHandler(PacketType.TransferAbort, HandleTransferAbort, false); |
4864 | AddLocalPacketHandler(PacketType.MuteListRequest, HandleMuteListRequest, false); | 4899 | AddLocalPacketHandler(PacketType.MuteListRequest, HandleMuteListRequest, false); |
4865 | AddLocalPacketHandler(PacketType.UseCircuitCode, HandleUseCircuitCode); | 4900 | AddLocalPacketHandler(PacketType.UseCircuitCode, HandleUseCircuitCode); |
4901 | AddLocalPacketHandler(PacketType.CreateNewOutfitAttachments, HandleCreateNewOutfitAttachments); | ||
4866 | AddLocalPacketHandler(PacketType.AgentHeightWidth, HandleAgentHeightWidth, false); | 4902 | AddLocalPacketHandler(PacketType.AgentHeightWidth, HandleAgentHeightWidth, false); |
4867 | AddLocalPacketHandler(PacketType.InventoryDescendents, HandleInventoryDescendents); | 4903 | AddLocalPacketHandler(PacketType.InventoryDescendents, HandleInventoryDescendents); |
4868 | AddLocalPacketHandler(PacketType.DirPlacesQuery, HandleDirPlacesQuery); | 4904 | AddLocalPacketHandler(PacketType.DirPlacesQuery, HandleDirPlacesQuery); |
@@ -4962,6 +4998,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
4962 | (x.CameraLeftAxis != lastarg.CameraLeftAxis) || | 4998 | (x.CameraLeftAxis != lastarg.CameraLeftAxis) || |
4963 | (x.CameraUpAxis != lastarg.CameraUpAxis) || | 4999 | (x.CameraUpAxis != lastarg.CameraUpAxis) || |
4964 | (x.ControlFlags != lastarg.ControlFlags) || | 5000 | (x.ControlFlags != lastarg.ControlFlags) || |
5001 | (x.ControlFlags != 0) || | ||
4965 | (x.Far != lastarg.Far) || | 5002 | (x.Far != lastarg.Far) || |
4966 | (x.Flags != lastarg.Flags) || | 5003 | (x.Flags != lastarg.Flags) || |
4967 | (x.State != lastarg.State) || | 5004 | (x.State != lastarg.State) || |
@@ -5333,7 +5370,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
5333 | args.Channel = ch; | 5370 | args.Channel = ch; |
5334 | args.From = String.Empty; | 5371 | args.From = String.Empty; |
5335 | args.Message = Utils.BytesToString(msg); | 5372 | args.Message = Utils.BytesToString(msg); |
5336 | args.Type = ChatTypeEnum.Shout; | 5373 | args.Type = ChatTypeEnum.Region; //Behaviour in SL is that the response can be heard from any distance |
5337 | args.Position = new Vector3(); | 5374 | args.Position = new Vector3(); |
5338 | args.Scene = Scene; | 5375 | args.Scene = Scene; |
5339 | args.Sender = this; | 5376 | args.Sender = this; |
@@ -9371,6 +9408,37 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
9371 | { | 9408 | { |
9372 | return true; | 9409 | return true; |
9373 | } | 9410 | } |
9411 | |||
9412 | private bool HandleCreateNewOutfitAttachments(IClientAPI sender, Packet Pack) | ||
9413 | { | ||
9414 | CreateNewOutfitAttachmentsPacket packet = (CreateNewOutfitAttachmentsPacket)Pack; | ||
9415 | |||
9416 | #region Packet Session and User Check | ||
9417 | if (m_checkPackets) | ||
9418 | { | ||
9419 | if (packet.AgentData.SessionID != SessionId || | ||
9420 | packet.AgentData.AgentID != AgentId) | ||
9421 | return true; | ||
9422 | } | ||
9423 | #endregion | ||
9424 | MoveItemsAndLeaveCopy handlerMoveItemsAndLeaveCopy = null; | ||
9425 | List<InventoryItemBase> items = new List<InventoryItemBase>(); | ||
9426 | foreach (CreateNewOutfitAttachmentsPacket.ObjectDataBlock n in packet.ObjectData) | ||
9427 | { | ||
9428 | InventoryItemBase b = new InventoryItemBase(); | ||
9429 | b.ID = n.OldItemID; | ||
9430 | b.Folder = n.OldFolderID; | ||
9431 | items.Add(b); | ||
9432 | } | ||
9433 | |||
9434 | handlerMoveItemsAndLeaveCopy = OnMoveItemsAndLeaveCopy; | ||
9435 | if (handlerMoveItemsAndLeaveCopy != null) | ||
9436 | { | ||
9437 | handlerMoveItemsAndLeaveCopy(this, items, packet.HeaderData.NewFolderID); | ||
9438 | } | ||
9439 | |||
9440 | return true; | ||
9441 | } | ||
9374 | 9442 | ||
9375 | private bool HandleAgentHeightWidth(IClientAPI sender, Packet Pack) | 9443 | private bool HandleAgentHeightWidth(IClientAPI sender, Packet Pack) |
9376 | { | 9444 | { |
@@ -11214,18 +11282,44 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
11214 | } | 11282 | } |
11215 | 11283 | ||
11216 | /// <summary> | 11284 | /// <summary> |
11285 | /// This processes packets which have accumulated while the presence was still in the process of initialising. | ||
11286 | /// </summary> | ||
11287 | public void ProcessPendingPackets() | ||
11288 | { | ||
11289 | m_IsPresenceReady = true; | ||
11290 | if (m_pendingPackets == null) | ||
11291 | return; | ||
11292 | foreach (Packet p in m_pendingPackets) | ||
11293 | { | ||
11294 | ProcessInPacket(p); | ||
11295 | } | ||
11296 | m_pendingPackets.Clear(); | ||
11297 | } | ||
11298 | |||
11299 | /// <summary> | ||
11217 | /// Entryway from the client to the simulator. All UDP packets from the client will end up here | 11300 | /// Entryway from the client to the simulator. All UDP packets from the client will end up here |
11218 | /// </summary> | 11301 | /// </summary> |
11219 | /// <param name="Pack">OpenMetaverse.packet</param> | 11302 | /// <param name="Pack">OpenMetaverse.packet</param> |
11220 | public void ProcessInPacket(Packet Pack) | 11303 | public void ProcessInPacket(Packet Pack) |
11221 | { | 11304 | { |
11222 | if (m_debugPacketLevel >= 255) | 11305 | if (!m_IsPresenceReady) |
11223 | m_log.DebugFormat("[CLIENT]: Packet IN {0}", Pack.Type); | 11306 | { |
11307 | if (m_pendingPackets == null) | ||
11308 | { | ||
11309 | m_pendingPackets = new List<Packet>(); | ||
11310 | } | ||
11311 | m_pendingPackets.Add(Pack); | ||
11312 | } | ||
11313 | else | ||
11314 | { | ||
11315 | if (m_debugPacketLevel >= 255) | ||
11316 | m_log.DebugFormat("[CLIENT]: Packet IN {0}", Pack.Type); | ||
11224 | 11317 | ||
11225 | if (!ProcessPacketMethod(Pack)) | 11318 | if (!ProcessPacketMethod(Pack)) |
11226 | m_log.Warn("[CLIENT]: unhandled packet " + Pack.Type); | 11319 | m_log.Warn("[CLIENT]: unhandled packet " + Pack.Type); |
11227 | 11320 | ||
11228 | PacketPool.Instance.ReturnPacket(Pack); | 11321 | PacketPool.Instance.ReturnPacket(Pack); |
11322 | } | ||
11229 | } | 11323 | } |
11230 | 11324 | ||
11231 | private static PrimitiveBaseShape GetShapeFromAddPacket(ObjectAddPacket addPacket) | 11325 | private static PrimitiveBaseShape GetShapeFromAddPacket(ObjectAddPacket addPacket) |
@@ -11462,7 +11556,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
11462 | 11556 | ||
11463 | // m_log.DebugFormat("[CLIENT]: {0} requesting asset {1}", Name, requestID); | 11557 | // m_log.DebugFormat("[CLIENT]: {0} requesting asset {1}", Name, requestID); |
11464 | 11558 | ||
11559 | |||
11560 | //Note, the bool returned from the below function is useless since it is always false. | ||
11465 | m_assetService.Get(requestID.ToString(), transferRequest, AssetReceived); | 11561 | m_assetService.Get(requestID.ToString(), transferRequest, AssetReceived); |
11562 | |||
11466 | } | 11563 | } |
11467 | 11564 | ||
11468 | /// <summary> | 11565 | /// <summary> |
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPClient.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPClient.cs index 6232c48..eebbfa5 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPClient.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPClient.cs | |||
@@ -149,6 +149,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
149 | 149 | ||
150 | private int m_defaultRTO = 3000; | 150 | private int m_defaultRTO = 3000; |
151 | private int m_maxRTO = 60000; | 151 | private int m_maxRTO = 60000; |
152 | public bool m_deliverPackets = true; | ||
152 | 153 | ||
153 | /// <summary> | 154 | /// <summary> |
154 | /// Default constructor | 155 | /// Default constructor |
@@ -389,6 +390,13 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
389 | if (category >= 0 && category < m_packetOutboxes.Length) | 390 | if (category >= 0 && category < m_packetOutboxes.Length) |
390 | { | 391 | { |
391 | OpenSim.Framework.LocklessQueue<OutgoingPacket> queue = m_packetOutboxes[category]; | 392 | OpenSim.Framework.LocklessQueue<OutgoingPacket> queue = m_packetOutboxes[category]; |
393 | |||
394 | if (m_deliverPackets == false) | ||
395 | { | ||
396 | queue.Enqueue(packet); | ||
397 | return true; | ||
398 | } | ||
399 | |||
392 | TokenBucket bucket = m_throttleCategories[category]; | 400 | TokenBucket bucket = m_throttleCategories[category]; |
393 | 401 | ||
394 | if (bucket.RemoveTokens(packet.Buffer.DataLength)) | 402 | if (bucket.RemoveTokens(packet.Buffer.DataLength)) |
@@ -419,6 +427,8 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
419 | /// <returns>True if any packets were sent, otherwise false</returns> | 427 | /// <returns>True if any packets were sent, otherwise false</returns> |
420 | public bool DequeueOutgoing() | 428 | public bool DequeueOutgoing() |
421 | { | 429 | { |
430 | if (m_deliverPackets == false) return false; | ||
431 | |||
422 | OutgoingPacket packet; | 432 | OutgoingPacket packet; |
423 | OpenSim.Framework.LocklessQueue<OutgoingPacket> queue; | 433 | OpenSim.Framework.LocklessQueue<OutgoingPacket> queue; |
424 | TokenBucket bucket; | 434 | TokenBucket bucket; |
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs index ca5a297..209e35c 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs | |||
@@ -904,7 +904,30 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
904 | client.DisableFacelights = m_disableFacelights; | 904 | client.DisableFacelights = m_disableFacelights; |
905 | 905 | ||
906 | // Start the IClientAPI | 906 | // Start the IClientAPI |
907 | client.Start(); | 907 | // Spin it off so that it doesn't clog up the LLUDPServer |
908 | |||
909 | //First, and very importantly: | ||
910 | // | ||
911 | //Set our DeliverPackets flag in the client to *false* | ||
912 | //this will prevent us from missing important messages | ||
913 | //before the modules are bound | ||
914 | client.DeliverPackets = false; | ||
915 | client.DisableFacelights = m_disableFacelights; | ||
916 | |||
917 | Util.FireAndForget( | ||
918 | delegate | ||
919 | { | ||
920 | try | ||
921 | { | ||
922 | client.Start(); | ||
923 | } | ||
924 | finally | ||
925 | { | ||
926 | //Now, release the hounds. er, packets. | ||
927 | client.DeliverPackets = true; | ||
928 | } | ||
929 | } | ||
930 | ); | ||
908 | } | 931 | } |
909 | else | 932 | else |
910 | { | 933 | { |
@@ -920,7 +943,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
920 | if (m_scene.TryGetClient(udpClient.AgentID, out client)) | 943 | if (m_scene.TryGetClient(udpClient.AgentID, out client)) |
921 | { | 944 | { |
922 | client.IsLoggingOut = true; | 945 | client.IsLoggingOut = true; |
923 | client.Close(); | 946 | client.Close(false); |
924 | } | 947 | } |
925 | } | 948 | } |
926 | 949 | ||
diff --git a/OpenSim/Region/ClientStack/LindenUDP/TokenBucket.cs b/OpenSim/Region/ClientStack/LindenUDP/TokenBucket.cs index bdbd284..91e3d20 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/TokenBucket.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/TokenBucket.cs | |||
@@ -133,7 +133,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
133 | this.parent = parent; | 133 | this.parent = parent; |
134 | MaxBurst = maxBurst; | 134 | MaxBurst = maxBurst; |
135 | DripRate = dripRate; | 135 | DripRate = dripRate; |
136 | lastDrip = Environment.TickCount & Int32.MaxValue; | 136 | lastDrip = Environment.TickCount; |
137 | } | 137 | } |
138 | 138 | ||
139 | /// <summary> | 139 | /// <summary> |
@@ -144,40 +144,30 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
144 | /// the bucket, otherwise false</returns> | 144 | /// the bucket, otherwise false</returns> |
145 | public bool RemoveTokens(int amount) | 145 | public bool RemoveTokens(int amount) |
146 | { | 146 | { |
147 | bool dummy; | ||
148 | return RemoveTokens(amount, out dummy); | ||
149 | } | ||
150 | |||
151 | /// <summary> | ||
152 | /// Remove a given number of tokens from the bucket | ||
153 | /// </summary> | ||
154 | /// <param name="amount">Number of tokens to remove from the bucket</param> | ||
155 | /// <param name="dripSucceeded">True if tokens were added to the bucket | ||
156 | /// during this call, otherwise false</param> | ||
157 | /// <returns>True if the requested number of tokens were removed from | ||
158 | /// the bucket, otherwise false</returns> | ||
159 | public bool RemoveTokens(int amount, out bool dripSucceeded) | ||
160 | { | ||
161 | if (maxBurst == 0) | 147 | if (maxBurst == 0) |
162 | { | 148 | { |
163 | dripSucceeded = true; | ||
164 | return true; | 149 | return true; |
165 | } | 150 | } |
166 | 151 | ||
167 | dripSucceeded = Drip(); | 152 | if (amount > maxBurst) |
168 | |||
169 | if (content - amount >= 0) | ||
170 | { | 153 | { |
171 | if (parent != null && !parent.RemoveTokens(amount)) | 154 | throw new Exception("amount " + amount + " exceeds maxBurst " + maxBurst); |
172 | return false; | 155 | } |
173 | 156 | ||
174 | content -= amount; | 157 | Drip(); |
175 | return true; | 158 | |
159 | if (content < amount) | ||
160 | { | ||
161 | return false; | ||
176 | } | 162 | } |
177 | else | 163 | |
164 | if (parent != null && !parent.RemoveTokens(amount)) | ||
178 | { | 165 | { |
179 | return false; | 166 | return false; |
180 | } | 167 | } |
168 | |||
169 | content -= amount; | ||
170 | return true; | ||
181 | } | 171 | } |
182 | 172 | ||
183 | /// <summary> | 173 | /// <summary> |
@@ -193,25 +183,23 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
193 | content = maxBurst; | 183 | content = maxBurst; |
194 | return true; | 184 | return true; |
195 | } | 185 | } |
196 | else | ||
197 | { | ||
198 | int now = Environment.TickCount & Int32.MaxValue; | ||
199 | int deltaMS = now - lastDrip; | ||
200 | 186 | ||
201 | if (deltaMS <= 0) | 187 | int now = Environment.TickCount; |
202 | { | 188 | int deltaMS = now - lastDrip; |
203 | if (deltaMS < 0) | 189 | lastDrip = now; |
204 | lastDrip = now; | ||
205 | return false; | ||
206 | } | ||
207 | 190 | ||
208 | int dripAmount = deltaMS * tokensPerMS; | 191 | if (deltaMS <= 0) |
209 | 192 | { | |
210 | content = Math.Min(content + dripAmount, maxBurst); | 193 | return false; |
211 | lastDrip = now; | 194 | } |
212 | 195 | ||
213 | return true; | 196 | long dripAmount = (long)deltaMS * (long)tokensPerMS + (long)content; |
197 | if (dripAmount > maxBurst) | ||
198 | { | ||
199 | dripAmount = maxBurst; | ||
214 | } | 200 | } |
201 | content = (int)dripAmount; | ||
202 | return true; | ||
215 | } | 203 | } |
216 | } | 204 | } |
217 | } | 205 | } |
diff --git a/OpenSim/Region/ClientStack/LindenUDP/UnackedPacketCollection.cs b/OpenSim/Region/ClientStack/LindenUDP/UnackedPacketCollection.cs index e43f7cf..c120a12 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/UnackedPacketCollection.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/UnackedPacketCollection.cs | |||
@@ -1,169 +1,184 @@ | |||
1 | /* | 1 | /* |
2 | * Copyright (c) Contributors, http://opensimulator.org/ | 2 | * Copyright (c) Contributors, http://opensimulator.org/ |
3 | * See CONTRIBUTORS.TXT for a full list of copyright holders. | 3 | * See CONTRIBUTORS.TXT for a full list of copyright holders. |
4 | * | 4 | * |
5 | * Redistribution and use in source and binary forms, with or without | 5 | * Redistribution and use in source and binary forms, with or without |
6 | * modification, are permitted provided that the following conditions are met: | 6 | * modification, are permitted provided that the following conditions are met: |
7 | * * Redistributions of source code must retain the above copyright | 7 | * * Redistributions of source code must retain the above copyright |
8 | * notice, this list of conditions and the following disclaimer. | 8 | * notice, this list of conditions and the following disclaimer. |
9 | * * Redistributions in binary form must reproduce the above copyright | 9 | * * Redistributions in binary form must reproduce the above copyright |
10 | * notice, this list of conditions and the following disclaimer in the | 10 | * notice, this list of conditions and the following disclaimer in the |
11 | * documentation and/or other materials provided with the distribution. | 11 | * documentation and/or other materials provided with the distribution. |
12 | * * Neither the name of the OpenSimulator Project nor the | 12 | * * Neither the name of the OpenSimulator Project nor the |
13 | * names of its contributors may be used to endorse or promote products | 13 | * names of its contributors may be used to endorse or promote products |
14 | * derived from this software without specific prior written permission. | 14 | * derived from this software without specific prior written permission. |
15 | * | 15 | * |
16 | * THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY | 16 | * THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY |
17 | * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | 17 | * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED |
18 | * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | 18 | * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE |
19 | * DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY | 19 | * DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY |
20 | * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES | 20 | * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES |
21 | * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; | 21 | * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; |
22 | * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND | 22 | * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND |
23 | * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 23 | * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
24 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | 24 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS |
25 | * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 25 | * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
26 | */ | 26 | */ |
27 | 27 | ||
28 | using System; | 28 | using System; |
29 | using System.Collections.Generic; | 29 | using System.Collections.Generic; |
30 | using System.Net; | 30 | using System.Net; |
31 | using OpenMetaverse; | 31 | using OpenMetaverse; |
32 | 32 | ||
33 | namespace OpenSim.Region.ClientStack.LindenUDP | 33 | namespace OpenSim.Region.ClientStack.LindenUDP |
34 | { | 34 | { |
35 | /// <summary> | 35 | /// <summary> |
36 | /// Special collection that is optimized for tracking unacknowledged packets | 36 | /// Special collection that is optimized for tracking unacknowledged packets |
37 | /// </summary> | 37 | /// </summary> |
38 | public sealed class UnackedPacketCollection | 38 | public sealed class UnackedPacketCollection |
39 | { | 39 | { |
40 | /// <summary> | 40 | /// <summary> |
41 | /// Holds information about a pending acknowledgement | 41 | /// Holds information about a pending acknowledgement |
42 | /// </summary> | 42 | /// </summary> |
43 | private struct PendingAck | 43 | private struct PendingAck |
44 | { | 44 | { |
45 | /// <summary>Sequence number of the packet to remove</summary> | 45 | /// <summary>Sequence number of the packet to remove</summary> |
46 | public uint SequenceNumber; | 46 | public uint SequenceNumber; |
47 | /// <summary>Environment.TickCount value when the remove was queued. | 47 | /// <summary>Environment.TickCount value when the remove was queued. |
48 | /// This is used to update round-trip times for packets</summary> | 48 | /// This is used to update round-trip times for packets</summary> |
49 | public int RemoveTime; | 49 | public int RemoveTime; |
50 | /// <summary>Whether or not this acknowledgement was attached to a | 50 | /// <summary>Whether or not this acknowledgement was attached to a |
51 | /// resent packet. If so, round-trip time will not be calculated</summary> | 51 | /// resent packet. If so, round-trip time will not be calculated</summary> |
52 | public bool FromResend; | 52 | public bool FromResend; |
53 | 53 | ||
54 | public PendingAck(uint sequenceNumber, int currentTime, bool fromResend) | 54 | public PendingAck(uint sequenceNumber, int currentTime, bool fromResend) |
55 | { | 55 | { |
56 | SequenceNumber = sequenceNumber; | 56 | SequenceNumber = sequenceNumber; |
57 | RemoveTime = currentTime; | 57 | RemoveTime = currentTime; |
58 | FromResend = fromResend; | 58 | FromResend = fromResend; |
59 | } | 59 | } |
60 | } | 60 | } |
61 | 61 | ||
62 | /// <summary>Holds the actual unacked packet data, sorted by sequence number</summary> | 62 | /// <summary>Holds the actual unacked packet data, sorted by sequence number</summary> |
63 | private Dictionary<uint, OutgoingPacket> m_packets = new Dictionary<uint, OutgoingPacket>(); | 63 | private Dictionary<uint, OutgoingPacket> m_packets = new Dictionary<uint, OutgoingPacket>(); |
64 | /// <summary>Holds packets that need to be added to the unacknowledged list</summary> | 64 | /// <summary>Holds packets that need to be added to the unacknowledged list</summary> |
65 | private LocklessQueue<OutgoingPacket> m_pendingAdds = new LocklessQueue<OutgoingPacket>(); | 65 | private LocklessQueue<OutgoingPacket> m_pendingAdds = new LocklessQueue<OutgoingPacket>(); |
66 | /// <summary>Holds information about pending acknowledgements</summary> | 66 | /// <summary>Holds information about pending acknowledgements</summary> |
67 | private LocklessQueue<PendingAck> m_pendingRemoves = new LocklessQueue<PendingAck>(); | 67 | private LocklessQueue<PendingAck> m_pendingRemoves = new LocklessQueue<PendingAck>(); |
68 | 68 | ||
69 | /// <summary> | 69 | /// <summary> |
70 | /// Add an unacked packet to the collection | 70 | /// Add an unacked packet to the collection |
71 | /// </summary> | 71 | /// </summary> |
72 | /// <param name="packet">Packet that is awaiting acknowledgement</param> | 72 | /// <param name="packet">Packet that is awaiting acknowledgement</param> |
73 | /// <returns>True if the packet was successfully added, false if the | 73 | /// <returns>True if the packet was successfully added, false if the |
74 | /// packet already existed in the collection</returns> | 74 | /// packet already existed in the collection</returns> |
75 | /// <remarks>This does not immediately add the ACK to the collection, | 75 | /// <remarks>This does not immediately add the ACK to the collection, |
76 | /// it only queues it so it can be added in a thread-safe way later</remarks> | 76 | /// it only queues it so it can be added in a thread-safe way later</remarks> |
77 | public void Add(OutgoingPacket packet) | 77 | public void Add(OutgoingPacket packet) |
78 | { | 78 | { |
79 | m_pendingAdds.Enqueue(packet); | 79 | m_pendingAdds.Enqueue(packet); |
80 | } | 80 | } |
81 | 81 | ||
82 | /// <summary> | 82 | /// <summary> |
83 | /// Marks a packet as acknowledged | 83 | /// Marks a packet as acknowledged |
84 | /// </summary> | 84 | /// </summary> |
85 | /// <param name="sequenceNumber">Sequence number of the packet to | 85 | /// <param name="sequenceNumber">Sequence number of the packet to |
86 | /// acknowledge</param> | 86 | /// acknowledge</param> |
87 | /// <param name="currentTime">Current value of Environment.TickCount</param> | 87 | /// <param name="currentTime">Current value of Environment.TickCount</param> |
88 | /// <remarks>This does not immediately acknowledge the packet, it only | 88 | /// <remarks>This does not immediately acknowledge the packet, it only |
89 | /// queues the ack so it can be handled in a thread-safe way later</remarks> | 89 | /// queues the ack so it can be handled in a thread-safe way later</remarks> |
90 | public void Remove(uint sequenceNumber, int currentTime, bool fromResend) | 90 | public void Remove(uint sequenceNumber, int currentTime, bool fromResend) |
91 | { | 91 | { |
92 | m_pendingRemoves.Enqueue(new PendingAck(sequenceNumber, currentTime, fromResend)); | 92 | m_pendingRemoves.Enqueue(new PendingAck(sequenceNumber, currentTime, fromResend)); |
93 | } | 93 | } |
94 | 94 | ||
95 | /// <summary> | 95 | /// <summary> |
96 | /// Returns a list of all of the packets with a TickCount older than | 96 | /// Returns a list of all of the packets with a TickCount older than |
97 | /// the specified timeout | 97 | /// the specified timeout |
98 | /// </summary> | 98 | /// </summary> |
99 | /// <param name="timeoutMS">Number of ticks (milliseconds) before a | 99 | /// <param name="timeoutMS">Number of ticks (milliseconds) before a |
100 | /// packet is considered expired</param> | 100 | /// packet is considered expired</param> |
101 | /// <returns>A list of all expired packets according to the given | 101 | /// <returns>A list of all expired packets according to the given |
102 | /// expiration timeout</returns> | 102 | /// expiration timeout</returns> |
103 | /// <remarks>This function is not thread safe, and cannot be called | 103 | /// <remarks>This function is not thread safe, and cannot be called |
104 | /// multiple times concurrently</remarks> | 104 | /// multiple times concurrently</remarks> |
105 | public List<OutgoingPacket> GetExpiredPackets(int timeoutMS) | 105 | public List<OutgoingPacket> GetExpiredPackets(int timeoutMS) |
106 | { | 106 | { |
107 | ProcessQueues(); | 107 | ProcessQueues(); |
108 | 108 | ||
109 | List<OutgoingPacket> expiredPackets = null; | 109 | List<OutgoingPacket> expiredPackets = null; |
110 | 110 | ||
111 | if (m_packets.Count > 0) | 111 | if (m_packets.Count > 0) |
112 | { | 112 | { |
113 | int now = Environment.TickCount & Int32.MaxValue; | 113 | int now = Environment.TickCount & Int32.MaxValue; |
114 | 114 | ||
115 | foreach (OutgoingPacket packet in m_packets.Values) | 115 | foreach (OutgoingPacket packet in m_packets.Values) |
116 | { | 116 | { |
117 | // TickCount of zero means a packet is in the resend queue | 117 | // TickCount of zero means a packet is in the resend queue |
118 | // but hasn't actually been sent over the wire yet | 118 | // but hasn't actually been sent over the wire yet |
119 | if (packet.TickCount == 0) | 119 | if (packet.TickCount == 0) |
120 | continue; | 120 | continue; |
121 | 121 | ||
122 | if (now - packet.TickCount >= timeoutMS) | 122 | if (now - packet.TickCount >= timeoutMS) |
123 | { | 123 | { |
124 | if (expiredPackets == null) | 124 | if (expiredPackets == null) |
125 | expiredPackets = new List<OutgoingPacket>(); | 125 | expiredPackets = new List<OutgoingPacket>(); |
126 | 126 | ||
127 | // The TickCount will be set to the current time when the packet | 127 | // The TickCount will be set to the current time when the packet |
128 | // is actually sent out again | 128 | // is actually sent out again |
129 | packet.TickCount = 0; | 129 | packet.TickCount = 0; |
130 | 130 | ||
131 | expiredPackets.Add(packet); | 131 | expiredPackets.Add(packet); |
132 | } | 132 | } |
133 | } | 133 | } |
134 | } | 134 | } |
135 | 135 | ||
136 | return expiredPackets; | 136 | return expiredPackets; |
137 | } | 137 | } |
138 | 138 | ||
139 | private void ProcessQueues() | 139 | private void ProcessQueues() |
140 | { | 140 | { |
141 | // Process all the pending adds | 141 | // Process all the pending adds |
142 | OutgoingPacket pendingAdd; | 142 | |
143 | while (m_pendingAdds.Dequeue(out pendingAdd)) | 143 | OutgoingPacket pendingAdd; |
144 | m_packets[pendingAdd.SequenceNumber] = pendingAdd; | 144 | if (m_pendingAdds != null) |
145 | 145 | { | |
146 | // Process all the pending removes, including updating statistics and round-trip times | 146 | while (m_pendingAdds.Dequeue(out pendingAdd)) |
147 | PendingAck pendingRemove; | 147 | { |
148 | OutgoingPacket ackedPacket; | 148 | if (pendingAdd != null && m_packets != null) |
149 | while (m_pendingRemoves.Dequeue(out pendingRemove)) | 149 | { |
150 | { | 150 | m_packets[pendingAdd.SequenceNumber] = pendingAdd; |
151 | if (m_packets.TryGetValue(pendingRemove.SequenceNumber, out ackedPacket)) | 151 | } |
152 | { | 152 | } |
153 | m_packets.Remove(pendingRemove.SequenceNumber); | 153 | } |
154 | 154 | ||
155 | // Update stats | 155 | // Process all the pending removes, including updating statistics and round-trip times |
156 | System.Threading.Interlocked.Add(ref ackedPacket.Client.UnackedBytes, -ackedPacket.Buffer.DataLength); | 156 | PendingAck pendingRemove; |
157 | 157 | OutgoingPacket ackedPacket; | |
158 | if (!pendingRemove.FromResend) | 158 | if (m_pendingRemoves != null) |
159 | { | 159 | { |
160 | // Calculate the round-trip time for this packet and its ACK | 160 | while (m_pendingRemoves.Dequeue(out pendingRemove)) |
161 | int rtt = pendingRemove.RemoveTime - ackedPacket.TickCount; | 161 | { |
162 | if (rtt > 0) | 162 | if (m_pendingRemoves != null && m_packets != null) |
163 | ackedPacket.Client.UpdateRoundTrip(rtt); | 163 | { |
164 | } | 164 | if (m_packets.TryGetValue(pendingRemove.SequenceNumber, out ackedPacket)) |
165 | } | 165 | { |
166 | } | 166 | m_packets.Remove(pendingRemove.SequenceNumber); |
167 | } | 167 | |
168 | } | 168 | // Update stats |
169 | } | 169 | System.Threading.Interlocked.Add(ref ackedPacket.Client.UnackedBytes, -ackedPacket.Buffer.DataLength); |
170 | |||
171 | if (!pendingRemove.FromResend) | ||
172 | { | ||
173 | // Calculate the round-trip time for this packet and its ACK | ||
174 | int rtt = pendingRemove.RemoveTime - ackedPacket.TickCount; | ||
175 | if (rtt > 0) | ||
176 | ackedPacket.Client.UpdateRoundTrip(rtt); | ||
177 | } | ||
178 | } | ||
179 | } | ||
180 | } | ||
181 | } | ||
182 | } | ||
183 | } | ||
184 | } | ||